Re: w83781d i2c driver updated for 2.5.66 (without sysfs support)

From: Greg KH (greg@kroah.com)
Date: Wed Mar 26 2003 - 16:23:18 EST


On Wed, Mar 26, 2003 at 08:43:43PM +0000, Christoph Hellwig wrote:
> On Wed, Mar 26, 2003 at 12:26:23PM -0800, Greg KH wrote:
> > We should really split these multivalue files up into individual files,
> > as sysfs is for single value files. Makes parsing easier too.
> >
> > Here's a patch for the lm75.c driver that does this. As we are going to
> > need a "generic" read and write for the "real" values that the i2c
> > drivers use, I added these files to the i2c-proc.c file.
>
> i2c-proc.c is the wrong place. Please add a i2c-sensor.c file with
> helper code for hardware sensors driver (i2c_detect should move over to
> there from i2c-proc.c aswell)

Oh, I agree. I just threw it there as I was matching up with other
functions already in that file. Eventually i2c-proc.c should be
deleted, as all of the proc stuff will be gone. I like the idea of
i2c-sensor.c for the remaining i2c_detect() function.

thanks,

greg k-h
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Mon Mar 31 2003 - 22:00:26 EST