On Thu, 2008-06-26 at 07:38 +0800, Rene Herman wrote: > On 26-06-08 01:16, Len Brown wrote: > > > >> +config THERMAL_HWMON > >> + bool "Hardware monitoring support" > >> + depends on HWMON=y || HWMON=THERMAL > > > > > > for > > CONFIG_HWMON=m > > CONFIG_THERMAL=y > > CONFIG_THERMAL_HWMON=n, always. > > > > This is not a change to the logic of the existing ifdefs, > > but was wondering if this was your intent, Rui. > > My intentioon in this case. Yes, this should be. THERMAL_HWMON is a > bool > (as in, not a tristate) that just changes THERMAL. > > If THERMAL=y, we need HWMON=y or we'd get a link failure. If > THERMAL=m, > we're okay with either HWMON=y and HWMON=m and if THERMAL=n, we don't > care one hoot about THERMAL_HWMON... Agree. Acked-by: Zhang Rui <rui.zhang at intel.com>
