Re: [linux-pm] [patch 2.6.25-rc6 7/7] PCI set up device.power.can_wakeup flags

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thursday 20 March 2008, Alan Stern wrote:
> On Thu, 20 Mar 2008, David Brownell wrote:
> 
> > A previous version of this patch broke on PowerPC platforms after they
> > changed how they do early PCI initialization (following the first version
> > of this patch).  Potentially the real fix involves switching PCI init on
> > all platforms so they all adopt the driver model "init() then add()" model,
> > calling device_initialize() early and pci_setup_device() before device_add().
> 
> Would it help to break this apart into two patches: the first to switch 
> PCI over to the driver model approach, and the second to set up the 
> can_wakeup flags?  Those really are two very different goals.

Presumably, yes.  I'm not sure why there isn't a more
homogeneous approach to PCI init, between arch/* trees,
so I suspect there may be some down'n'dirty voodoo to
avoid tripping over.

That said, I'd sure *hope* that it could be cleaned up
in a way that all platforms can share one chunk of code
doing setup (like this) based on config space access.


> What are the barriers to doing the first part right now?  Is it just 
> PPC?

I have no idea, since I've not really looked at this in
any detail for some time now.  (The relevant PPC change
was more than two years ago...)  Heck, maybe it's even
been done already, and I just didn't notice!

- Dave


--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux IBM ACPI]     [Linux Power Management]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux