18 Apr
2014
18 Apr
'14
6:44 p.m.
On Fri, Apr 18, 2014 at 08:17:35PM +0200, Andrew Lunn wrote:
The driver already has:
static const struct i2c_device_id alc5623_i2c_table[] = { {"alc5621", 0x21}, {"alc5622", 0x22}, {"alc5623", 0x23}, {} }; MODULE_DEVICE_TABLE(i2c, alc5623_i2c_table);
which is enough for the i2c layer to load the driver when it walks the nodes under the i2c bus driver in the DT.
You still need to add the compatible strings to the driver explicitly, device names can be and are duplicated between manufacturers sometimes and it makes things directly greppable.