Hi Greg, > On Sun, Dec 18, 2005 at 04:29:40PM +0100, Jean Delvare wrote: > > -static int w83792d_read_value(struct i2c_client *client, u8 register); > > -static int w83792d_write_value(struct i2c_client *client, u8 register, > > +static inline int w83792d_read_value(struct i2c_client *client, u8 register); > > +static inline int w83792d_write_value(struct i2c_client *client, u8 register, > > u8 value); > > Will cause compile errors, so I've dropped this portion of the patch > (see the patch I sent back to you for details.) Can you explain please? We have four other hardware monitoring drivers with similar code, and no problem was ever reported. This change to the w83792d driver was compiling just fine for me with both gcc 2.95.3 and gcc 3.3.6. What exactly is supposed to cause problems, and with which compiler? Thanks, -- Jean Delvare