On Wed, May 18, 2011 at 7:41 AM, Rafael J. Wysocki <rjw@xxxxxxx> wrote: > On Tuesday, May 17, 2011, MyungJoo Ham wrote: >> On Thu, May 12, 2011 at 7:55 AM, Greg KH <gregkh@xxxxxxx> wrote: >> > On Wed, May 11, 2011 at 04:58:43PM +0900, MyungJoo Ham wrote: >> >> 1. System-wide sysfs interface >> >> - tickle_all R: number of tickle_all execution >> >> W: tickle all devfreq devices >> >> - min_interval R: devfreq monitoring base interval in ms >> >> - monitoring R: shows whether devfreq monitoring is active or >> >> not. >> >> >> >> 2. Device specific sysfs interface >> >> - tickle R: number of tickle execution for the device >> >> W: tickle the device >> > >> > Any sysfs file change/addition/removal needs to have a >> > Documentation/ABI/ entry as well. Please add that to this patch instead >> > of burying it in the changelog entry, where no one will be able to find >> > it in the future. >> > >> > thanks, >> > >> > greg k-h >> > >> >> Sure, I'll add Documentation/ABI/testing/* entries. >> >> However, would it be appropriate for "1. System-wide sysfs interface" >> to be in "sysfs-class-power" and for "2. Device specific sysfs >> interface" to be in "sysfs-devices-devfreq"? >> >> System-wide sysfs interface is in /sys/class/power/devfreq/* and > > Hmm. Why not to use /sys/power/ that's already there? Ah.. that was my mistake. I'd put it at /sys/power/devfreq/*. Thank you for cleaning that up. > >> Device specific sysfs interface is in /sys/...DEVICE.../devfreq/* . > > Thanks, > Rafael > Cheers! - MyungJoo -- MyungJoo Ham, Ph.D. Mobile Software Platform Lab, Digital Media and Communications (DMC) Business Samsung Electronics cell: 82-10-6714-2858 _______________________________________________ linux-pm mailing list linux-pm@xxxxxxxxxxxxxxxxxxxxxxxxxx https://lists.linux-foundation.org/mailman/listinfo/linux-pm
