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... Rene.