Re: [PATCH] usb: Add support for runtime power management of the hcd

[Oliver Neukum <oliver-GvhC2dPhHPQdnm+yROfE0A@public.gmane.org>]

Am Donnerstag, 12. November 2009 15:51:22 schrieb Matthew Garrett:
> On Thu, Nov 12, 2009 at 08:41:11AM +0100, Oliver Neukum wrote:

> > That I would consider backwards. If the user enables runtime power
> > management of a device, he has to live with remote wakeup being enabled.
> > At most the kernel might resume and switch off remote wakeup on the
> > device, but we might also leave this to user space.
> 
> My point was that if we don't have remote wakeup support at runtime, we
> can't enable runtime PM of the device. So we need to be able to
> distinguish between runtime remote wakeup and system sleep remote
> wakeup.

Exactly, but that isn't the same as needing two attributes exported to user
space. That is an absolute requirement only if we can't tell what a device's
capabilities are.
We could introduce a second attribute if we want to prevent drivers from
using remote wakeup to do runtime power management, but it is not
required. We could simply not give user space that choice.

	Regards
		Oliver
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html