Messages in this thread | | | From | David Brownell <> | Subject | Re: PATCH/RFC: usbcore wakeup updates (3/4) | Date | Thu, 7 Oct 2004 08:35:53 -0700 |
| |
Hi Pavel,
On Wednesday 06 October 2004 3:51 am, Pavel Machek wrote: > > There were already some hooks in usbcore, but they were only > > configurable for root hubs ... but not keyboards, mice, Ethernet > > adapters, or other devices. > > That "when asked about D1 enter D3" bit worries me a bit, because > it is (ugly) workaround to core problems, but I can survive it.
I'm not sure what a better fix would be though ... I think WIndows doesn't bother entering a low power state at all in such cases. Which seems particularly pointless for the typical USB controller, which is probably idle at that point already, and which can't take all that much longer to resume from D3hot than from D1.
But that part is peripheral to the thrust of this set of patches.
Do you think adding those two bits to per-device PM state is basically a good way to introduce their wakeup capabilities to the PM core? Suggestions on the next step?
> Introducing enums where PCI suspend level is stored in u32 > would be welcome...
I'm not averse to enums, especially once sparse does good things with them, but I still think that sort of change would just be nibbling around the edges of a larger problem. (Which should be addressed by different patches making device power states/policies, like G0/D1 or "idle yourself", be types that are fully distinct from system power states like G1/S3, and for which abuse creates compiler errors.)
- Dave
> > Pavel > -- > 64 bytes from 195.113.31.123: icmp_seq=28 ttl=51 time=448769.1 ms > > - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |