lkml.org 
[lkml]   [2008]   [Oct]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
SubjectRe: scx200_i2c driver fails to detect any i2c devices in 2.6.26
Hi Lennart,

On Thu, 23 Oct 2008 12:54:19 -0400, Lennart Sorensen wrote:
> In 2.6.18, the scx200_i2c driver was working just fine. Now while
> moving to 2.6.26 I find that the bus no longer locates any devices when
> drivers are loaded, even though i2cdetect has no problem locating
> devices on the bus.
>
> After some searching, I discovered that the scx200_i2c driver doesn't
> have a .class parameter filled in, even though every other i2c driver
> does. A small patch adding the .class parameter fixes the problem.

Correct. Not sure how we missed this for years.

> Not sure if it should be I2C_CLASS_ALL or I2C_CLASS_HWMON, or even
> I2C_CLASS_HWMON | I2C_CLASS_SPD (in current git tree). Using
> I2C_CLASS_ALL works for me and works with my lm75, m41t81 and spd
> eeprom in 2.6.26.

I2C_CLASS_HWMON | I2C_CLASS_SPD please. I2C_CLASS_ALL is almost never
what people want, and actually I'm thinking of removing it to prevent
developers from getting it wrong.

>
> --- a/drivers/i2c/busses/scx200_i2c.c 2008-07-13 17:51:29.000000000 -0400
> +++ b/drivers/i2c/busses/scx200_i2c.c 2008-10-23 12:41:02.000000000 -0400
> @@ -81,6 +81,7 @@
>
> static struct i2c_adapter scx200_i2c_ops = {
> .owner = THIS_MODULE,
> + .class = I2C_CLASS_ALL,
> .id = I2C_HW_B_SCX200,
> .algo_data = &scx200_i2c_data,
> .name = "NatSemi SCx200 I2C",
>


--
Jean Delvare


\
 
 \ /
  Last update: 2008-10-23 19:23    [W:0.640 / U:0.048 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site