From mboxrd@z Thu Jan 1 00:00:00 1970 From: Greg KH Subject: Re: PowerOp Design and working patch Date: Sun, 6 Aug 2006 21:17:46 -0700 Message-ID: <20060807041746.GA22493@kroah.com> References: <242d827d33807b6b46608a26ed29c273@mvista.com> <20060728233837.GG2140@kroah.com> <20060729004546.GA3917@kroah.com> <1c70fd37e4eb876295e0194e2b8725d6@mvista.com> <44CBB1E0.3030908@gmail.com> <62e4476a851bbed395b509023edc5c5c@mvista.com> <20060806220519.GA4205@ucw.cz> <4af8b613a706c78ae841d671bd1900cf@mvista.com> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Return-path: Content-Disposition: inline In-Reply-To: <4af8b613a706c78ae841d671bd1900cf@mvista.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: linux-pm-bounces@lists.osdl.org Errors-To: linux-pm-bounces@lists.osdl.org To: david singleton Cc: David Singleton , linux-pm@lists.osdl.org, Pavel Machek List-Id: linux-pm@vger.kernel.org On Sun, Aug 06, 2006 at 08:52:15PM -0700, david singleton wrote: > = > On Aug 6, 2006, at 3:05 PM, Pavel Machek wrote: > = > > Hi! > > > >>> 2) interface (kernel as well as userspace(sysfs)) for the rest of = > >>> power > >>> parameters except cpu voltage and frequency > >> > >> > >> The /sys/power/supported_states file shows the supported operating > >> points > >> and their parameters. > > > > We have one-value-per-file rule for sysfs. > = > Yes, the supported_states file would be better off in /proc. I'll > move it. No, don't put non-process stuff in /proc, that's not where it belongs. > I just put it in /sys because it was so easy. Then use the proper interface with sysfs please. Just listing the supported states is acceptable, there is prior-usages like this. Just not the paramaters, this doesn't need to be self-documented... Or use configfs, that might be good for something like this. thanks, greg k-h