]> git.kernelconcepts.de Git - karo-tx-linux.git/commit
THERMAL_HWMON is implemented inside the thermal_sys driver and has no
authorJean Delvare <khali@linux-fr.org>
Tue, 26 Jul 2011 10:14:36 +0000 (20:14 +1000)
committerStephen Rothwell <sfr@canb.auug.org.au>
Mon, 1 Aug 2011 03:36:28 +0000 (13:36 +1000)
commitb0b959fc16826d3bf0774fc4c5b876c812f2aa0e
tree48862e5d620b3ead081869767a334786778d4bca
parentf02496bae059611e7992224078f8f1d09b147d04
THERMAL_HWMON is implemented inside the thermal_sys driver and has no
effect on drivers implementing thermal zones, so they shouldn't see
anything related to it in <linux/thermal.h>.  Making the THERMAL_HWMON
implementation fully internal has two advantages beyond the cleaner
design:

* This avoids rebuilding all thermal drivers if the THERMAL_HWMON
  implementation changes, or if CONFIG_THERMAL_HWMON gets enabled or
  disabled.

* This avoids breaking the thermal kABI in these cases too, which should
  make distributions happy.

The only drawback I can see is slightly higher memory fragmentation, as
the number of kzalloc() calls will increase by one per thermal zone.  But
I doubt it will be a problem in practice, as I've never seen a system with
more than two thermal zones.

Signed-off-by: Jean Delvare <khali@linux-fr.org>
Cc: Rene Herman <rene.herman@gmail.com>
Cc: Len Brown <len.brown@intel.com>
Acked-by: Guenter Roeck <guenter.roeck@ericsson.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
drivers/thermal/thermal_sys.c
include/linux/thermal.h