Re: [PATCH] More i2c driver changes for 2.5.65

From: Dave Jones (davej@codemonkey.org.uk)
Date: Mon Mar 24 2003 - 21:04:19 EST


On Mon, Mar 24, 2003 at 05:29:23PM -0800, Greg KH wrote:

> > > + .name = "ADM1021-MAX1617",
> > Why dash here
> > > + .name = "LM75 sensor",
> > And space here? Also you should have
> > either 2x "sensor" or none at all.
> What do you mwan "2x"?

The way I parsed it, either have..
 .name = "ADM1021-MAX1617 sensor",
 .name = "LM75 sensor",

or

 .name = "ADM1021-MAX1617",
 .name = "LM75",

ie, both, or all. Personally the latter looks better to me.
Especially given the 16 char limit. Aren't these going to
be in a sysfs heirarchy where its obvious they are sensors
anyway ? like i2c/sensors/lm75 ?

                Dave

-
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:18 EST