Messages in this thread | ![/](/images/icornerl.gif) | | Date | Wed, 26 Mar 2003 13:23:18 -0800 | From | Greg KH <> | Subject | Re: w83781d i2c driver updated for 2.5.66 (without sysfs support) |
| |
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/
| ![\](/images/icornerr.gif) |