On Sun, 13 Jan 2008 13:50:46 -0500, Jon Smirl wrote: > On 1/13/08, Jean Delvare <[EMAIL PROTECTED]> wrote: > > The eeprom case might be a bit confusing because that i2c driver > > supports a single device type, so the driver name is the same as the > > device name. Take a look at the hwmon/lm90 driver for a better example: > > this device supports 7 different devices. The devices are mostly > > compatible so it made sense to have a single driver for them, but they > > all differ in some way. For example, the LM90 doesn't support PEC, > > while the ADM1032 does. User-space needs to be able to distinguish > > between the various types. That's the reason why we export the device > > name through sysfs. > > > > Most i2c sensor drivers support several chip types, and libsensors has > > been relying heavily on the name attribute. Less so with the lm-sensors > > 3.0.0 rewrite, where most things are automatically detected, but if > > nothing else, giving humans a way to distinguish between the different > > sensor chip types is very useful. Also, not everyone will upgrade to > > lm-sensors 3.0.0 so we need to keep supporting the previous versions. > > Another way to handle this is to have the drivers register multiple > times using different names. So LM90 would register as both LM90 and > ADM1032. All the code is shared, you just register it multiple times > under different names.
This won't let us get rid of the name attribute. As I repeatedly explained, removing that file now would instantly break at least all versions of lm-sensors before 3.0.0. This alone is enough to make it unacceptable for the years to come. Not to mention that this seems like a waste of kernel memory. The name attribute certainly takes one order of magnitude less memory than registering the same driver multiple times. I'm not sure why you want these i2c chip names to go way, given that the module aliases you'd like to add are almost the same, just in a different form and using OF names instead of arbitrary names. -- Jean Delvare -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
