Re: Thermal: Add a thermal notifier for user space

From: Zhang Rui
Date: Wed Dec 12 2012 - 19:23:16 EST


On Wed, 2012-12-12 at 13:28 -0500, Dave Jones wrote:
> On Wed, Dec 12, 2012 at 06:05:47PM +0000, Linux Kernel wrote:
> > Gitweb: http://git.kernel.org/linus/;a=commit;h=1cc807a234cb988d69ba18f6a3a1c68d71e54bed
> > Commit: 1cc807a234cb988d69ba18f6a3a1c68d71e54bed
> > Parent: e151a202a084f9f4310d1aa4398325c56ca95fda
> > Author: Durgadoss R <durgadoss.r@xxxxxxxxx>
> > AuthorDate: Tue Sep 18 11:05:03 2012 +0530
> > Committer: Zhang Rui <rui.zhang@xxxxxxxxx>
> > CommitDate: Mon Nov 5 14:00:08 2012 +0800
> >
> > Thermal: Add a thermal notifier for user space
> >
> > This patch registers a governor which will let the
> > user land manage the platform thermals. Whenever a
> > trip happens, this governor just notifies the user
> > space using kobj_uevent().
> >
> > Signed-off-by: Durgadoss R <durgadoss.r@xxxxxxxxx>
> > Signed-off-by: Zhang Rui <rui.zhang@xxxxxxxxx>
>
>
> > +config USER_SPACE
> > + bool "User_space thermal governor"
> > + depends on THERMAL
> > + help
> > + Enable this to let the user space manage the platform thermals.
>
> This is terribly generic sounding. Might I suggest THERMAL_USER_SPACE instead ?
>
Hi, Dave,

You're right. How about the patch attached?

thanks,
rui