[PATCH 2.6] Add class definition to the elektor bus driver
Frank Beesley
frank.beesley at aeroflex.com
Fri Feb 25 21:55:48 CET 2005
Jean Delvare wrote:
>>So I believe what you are asking for is the i2c_elektor driver for the
>>PCF8584 ISA to I2C chip.
>>
>>
>
>Correct, I just checked and this one actually lacks its class. Patch
>follows.
>
>Greg, can you please apply this one to your tree?
>Thanks.
>
>
>
The patch works BUT...
The PCF8584 device is a bridge, not a hardware monitor. One of the other
I2C device types could very well be behind it such as a sound chip.
Maybe we could add a BRIDGE type or have it as a I2C_CLASS_ALL device.
This is just a suggestion - like I said, the patch works for me.
Frank
>This patch adds a class definition to the elektor i2c bus driver.
>Without this definition, hardware monitoring chips located on such
>busses cannot possibly be driven.
>
>Signed-off-by: Jean Delvare <khali at linux-fr.org>
>
>--- linux-2.6.11-rc5/drivers/i2c/busses/i2c-elektor.c.orig 2004-12-24 22:33:49.000000000 +0100
>+++ linux-2.6.11-rc5/drivers/i2c/busses/i2c-elektor.c 2005-02-25 20:41:26.000000000 +0100
>@@ -183,6 +183,7 @@
>
> static struct i2c_adapter pcf_isa_ops = {
> .owner = THIS_MODULE,
>+ .class = I2C_CLASS_HWMON,
> .id = I2C_HW_P_ELEK,
> .algo_data = &pcf_isa_data,
> .name = "PCF8584 ISA adapter",
>
>
>
>
More information about the lm-sensors
mailing list