]> git.karo-electronics.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>
Sat, 16 Jul 2011 13:30:47 +0000 (23:30 +1000)
committerStephen Rothwell <sfr@canb.auug.org.au>
Tue, 26 Jul 2011 05:04:42 +0000 (15:04 +1000)
commit1d032db4e141c0221337cab72fcdace6a4131f98
treee3637fabf5bf78ad816a3d846050fcad8418abd6
parent3b9af1e18604a94b5829f9abd8a292f46599b07f
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