[lm-sensors] [PATCH] Support for DS75 thermal sensor
wg at grandegger.com
Wed Jul 16 11:12:50 CEST 2008
Jean Delvare wrote:
> Hi Wolfgang,
> On Fri, 11 Jul 2008 14:40:58 +0200, Wolfgang Grandegger wrote:
>> Jean Delvare wrote:
>>> There's no "detection" involved for the new-style i2c devices.
>>> Presumably you are still using the old binding model, not taking
>>> benefit of the patch at all. Use i2c_register_board_info() to
>>> instantiate your "ds75" i2c device from you board's platform code, and
>>> it should work.
>> I have some general question on I2C device probing. Our board does have
>> two I2C buses. On the first on, there is an RTC at addr 0x32 and a DS75
>> at 0x4c. Both are defined in the Flattened Device Tree and therefore
>> i2c_register_board_info() will be called for them. On the second bus,
>> there are other I2C devices. Nevertheless, probing for RTC and DS75 at
>> addr 0x32 and 0x4c will be performed also on that bus. Is this the
>> normal/intended behavior? Can such probing be suppressed?
> Probing isn't supposed to happen at all for RTC chips. There are a few
> drivers which remain to be converted though (rtc-ds1672, rtc-max6900,
> rtc-pcf8583.) If you use one of these and would like to help converting
> it, please tell me (and Alessandro Zummo.)
OK. We ported an old RTC driver for the Epson RX8025 for our board.
Unfortunately, it did not use the new bindings but I fixed that in the
meantime. It will be posted for kernel inclusion soon.
> For hwmon chips, probing only occurs if the i2c adapter accepts to be
> probed, by setting the I2C_CLASS_HWMON flag in i2c_adapter.class. If
> you do not want a given i2c bus to be probed, then do not set the flag
> for that bus.
I see. My problem is that I2C_CLASS_HWMON is set in the I2C bus driver
for the MPC:
If I disable it, the LM75 driver only probes the addresses of the I2C
nodes defined in the FDT DTS file. I wonder if this should not be the
default behaviour for kernels using OF platform description. For this
reason, I have put Linuxppc-dev ML on CC. I also realized, that there
are some patches for OF I2C pending. I will check.
More information about the lm-sensors