[PATCH][V4] powerpc/fsl-pci: Add pci inbound/outbound PM support

Jia Hongtao-B38951 B38951 at freescale.com
Fri Sep 28 12:57:06 EST 2012



> -----Original Message-----
> From: Kumar Gala [mailto:galak at kernel.crashing.org]
> Sent: Friday, September 28, 2012 5:38 AM
> To: Li Yang-R58472
> Cc: Jia Hongtao-B38951; Wood Scott-B07421; linuxppc-dev at lists.ozlabs.org
> Subject: Re: [PATCH][V4] powerpc/fsl-pci: Add pci inbound/outbound PM
> support
> 
> 
> On Sep 27, 2012, at 11:27 AM, Li Yang-R58472 wrote:
> 
> >
> > 在 Sep 28, 2012,0:07,"Kumar Gala" <galak at kernel.crashing.org> 写道:
> >
> >>>>> Hi Kumar,
> >>>>> I have already sent the log.
> >>>>> Do you have any comment on it?
> >>>>>
> >>>>> Thanks.
> >>>>> - Hongtao.
> >>>>>
> >>>>
> >>>> Hongtao,
> >>>>
> >>>> You mentioned:
> >>>>
> >>>>> I tested the re-parsing way by using setup_pci_atmu() when resume.
> >>>>> And I found out that re-parsing will *change* outbound IO
> >>>>> translation address regitster.
> >>>>
> >>>> What do the values look like in both ATMU registers and io_resource
> if you reparse?
> >>>
> >>> I think Hongtao mentioned in previous email as follows, the ATMU
> >>> registers are inline with the io_resource address.
> >>
> >> I was under that the impression that was the normal boot case, not the
> values from after wakeup.
> >
> > It is for the normal boot.  But re-parse will use the io resource after
> pic scan to initialize atmu.  Instead, the original atmu is initialized
> use the io resource before the scan.
> >
> > Leo
> 
> I think I see, so isn't the mem resources also wrong?
> 
> Can we dump the following:
> 
> 1. enable pr_debug() in pcibios_setup_phb_resources so we get "PHB: "
> prints 2. Can we dump hose->io_resource & hose->mem_resources[] right
> after wakeup?
> 
> I think I see what direction but would be useful to get a few more
> answers.
> 
> - k

I enable pr_debug() on these three files:
arch/powerpc/sysdev/fsl_pci.c
arch/powerpc/kernel/pci-common.c
arch/powerpc/kernel/pci_32.c

Also I print the io_resource and mem_resources after wakeup.
(actually pr_debug already done this)

You can see that io resource is changed after pci scan in normal boot.

To see the log please refer to the attachment.

- Hongtao.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: pci_pm_setup_pci_atmu_debug_2.log
Type: application/octet-stream
Size: 31608 bytes
Desc: pci_pm_setup_pci_atmu_debug_2.log
URL: <http://lists.ozlabs.org/pipermail/linuxppc-dev/attachments/20120928/ce530874/attachment-0001.obj>


More information about the Linuxppc-dev mailing list