Messages in this thread | | | Date | Thu, 06 Mar 2014 13:33:49 +0800 | From | "Li, Aubrey" <> | Subject | Re: [PATCH] x86: new Intel Atom SoC power management controller driver |
| |
On 2014/3/6 13:28, H. Peter Anvin wrote: > On 03/05/2014 09:21 PM, Li, Aubrey wrote: >>> >>> What is the reason for putting this in debugfs? >> >> S0ix support on Atom SoC needs all of the devices under south complex to >> be quiesced, as well as the devices under north complex. This interface >> shows all of the devices states under platform control unit. If the >> system is not able to enter S0ix, we have a way to diagnose. Or if the >> system entered S0ix, we have counters to show S0ix entered and its >> residency. >> > > I guess the real question is: are you sure this is only for debugging, > or is this going to be an interface that applications or daemons will > actually use (in which case it should live in sysfs).
These interfaces are uncommitted and don't have consumers now, I'd like to keep them unstable in debugfs.
We can change it if we received a requirement to use them later.
Thanks, -Aubrey
> > -hpa > > > >
| |