Re: [lm-sensors] unhide_ICH_SMBus: line 33:

[Jean Delvare <khali@linux-fr.org>]

On Tue, 15 Sep 2009 18:41:41 +0200, Martin MOKREJ¦ wrote:
> Jean Delvare wrote:
> > On Tue, 15 Sep 2009 17:43:19 +0200, Martin MOKREJ¦ wrote:
> >> $ gzip -dc /proc/config.gz | grep THERMAL
> >> CONFIG_ACPI_THERMAL=y
> >> CONFIG_THERMAL=y
> >> CONFIG_THERMAL_HWMON=y
> >> $ gzip -dc /proc/config.gz | grep ATK
> >> CONFIG_KEYBOARD_ATKBD=y
> >> CONFIG_SENSORS_ATK0110=y
> >> $
> >>
> >> These I have already in the kernel, so I can drop the whole SMbus thing?
> > 
> > Yes.
> 
> Ooops, are we talking about I2C_I801=N, I2C=N or SENSORS_DS1621=N? ;-)

ENSORS_DS1621=N and I2C_I801=N, yes. I2C=N, bad idea, you might need
I2C support for something else.

> Actually I had SENSORS_LM90=Y as it was also proposed by sensors-detect.
> That one was also mis-detected?

I can't tell as your sensors-detect report was incomplete (with ds1621
loaded, sensors-detect can't probe the chip). I'm not sure why you are
hard-building all the drivers that way, BTW, it makes investigating
problems much harder. Probably the LM90 detection was correct, as these
chips are much more reliably detected than the DS1621.

Anyway, as further kernels say that ACPI is conflicting with the native
SMBus controller driver, the only safe bet is to drop i2c-i801 and go
with the ACPI thermal driver instead.

-- 
Jean Delvare
http://khali.linux-fr.org/wishlist.html

_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors