Hi On Thu, 2005-03-24 at 06:06, David Brownell wrote: > On Tuesday 22 March 2005 4:52 pm, Patrick Mochel wrote: > > > > > I think the core should always call ->suspend() for a device, regardless > > of whether it thinks it's in a low power state, or inactive. This is > > specifically for the reason that a device could be a low-power runtime > > state when the system is suspended. > > I don't quite see a need for this. If the parent/bridge driver knows > the device is adequately suspended, why kick it again? It's actually > rather annoying -- and error/bug prone! -- to have to code drivers to > detect and cope with superfluous suspend calls. I would think it would be simpler not to have the parent/bridge check whether the device is already suspended, and to just have all the logic that decides what to do in the driver. I'm only thinking intuitively, but it sounds like that should help with avoiding race conditions. Regards, Nigel -- Nigel Cunningham Software Engineer, Canberra, Australia http://www.cyclades.com Bus: +61 (2) 6291 9554; Hme: +61 (2) 6292 8028; Mob: +61 (417) 100 574 Maintainer of Suspend2 Kernel Patches http://suspend2.net
