On Thu, 24 Apr 2008 11:00:45 +0200, Wolfram Sang wrote: > On Wed, Apr 23, 2008 at 08:30:12PM +0200, Jean Delvare wrote: > > Each i2c driver can support many different devices, which are > > differentiated simply by their name. So the driver can behave > > differently for each device type as needed. > > We exactly threw this out of at24 in the new revision, as name-matching > stored quite some data in the driver which was needed only once during > initialization. Also, it was not very flexible; what if you want to mark > this chip as read-only, your variant has bigger page size and could be > faster... The list of device names would grow too fast IMHO, adding a > new feature which is selectable by a flag could easily double it.
The interest of separate chip names depend on the variety of devices your driver support. I am not suggesting to have one name for every different device the at24 driver will support, that would obviously be way too many. But if you could come up with families (e.g. one name per EEPROM size) with sane default settings, and then you can adjust parameters through platform data, for example to maximize performance if your actual EEPROM can handle larger page, or to enable write access. So you probably have to find the right balance between one name for all devices, and a separate name for each device. > Then again: If Jochen's patches get applied, then I have to use > MODULE_DEVICE_TABLE anyhow, or? If you want your driver to load automatically, yes. But this doesn't have much to do with the problem discussed above. -- Jean Delvare _______________________________________________ i2c mailing list [email protected] http://lists.lm-sensors.org/mailman/listinfo/i2c
