Re: [PATCH] PNPACPI: Enable Power Support

From: Henrique de Moraes Holschuh
Date: Wed Dec 10 2008 - 18:26:20 EST


On Wed, 10 Dec 2008, Oliver Neukum wrote:
> Am Mittwoch, 10. Dezember 2008 10:53:15 schrieb Henrique de Moraes Holschuh:
> > Although I do wonder WTF we don't do that on our USB UHCI/EHCI kernel
> > drivers, that ACPI needs to step in to fix it.  If it is not a wake
>
> EHCI/UHCI have no way to power down host controllers.
> It needs external means, namely ACPI.

I see. So we can't just set the pci devices to D0 to cut power to the USB
ports and USB controller while in STR? Must be the EC stuff I just noticed
in the ACPI AML, then.

> > device, why are we leaving it powered up?  It is bad enough we don't
> > have any sort of proper USB power control, but to leave the entire USB
> > subsystem powered up without reason?!
>
> We don't. If you unplug all devices, USB will power down as much as is
> compatible with detecting new devices.

Thanks for the update.

> > Vista not only powers down the USB HCIs and ports on STR, it also
> > powers down ports when you "safely remove the USB device", and seems
> > to leave the port powered down until you insert a device (or remove
> > the device already in there, whatever).
>
> If you cut power to a port, you cannot detect a hotplugging. You can
> suspend it, which Linux does do.

Hmm, probably it is just suspend, then (outside of STR. On STR, it *is*
somehow powering down the ports when no wake devices are there, but that
might be done using extra stuff, and not just the PCI functions of the Intel
ICH6 EHCI/UHCI controllers).

I guess our userspace is not doing the suspend on "safely remove" then?
Because at least here, it is not behaving as Windows does when I tell KDE to
"safely eject" (which probably ends up being a HAL request of some sort).

--
"One disk to rule them all, One disk to find them. One disk to bring
them all and in the darkness grind them. In the Land of Redmond
where the shadows lie." -- The Silicon Valley Tarot
Henrique Holschuh
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/