On Wed, 31 Aug 2016, Keerthy wrote: > On Wednesday 31 August 2016 01:26 PM, Lee Jones wrote: > > On Fri, 12 Aug 2016, Keerthy wrote: > > > > > The LP873X chip is a power management IC for Portable Navigation Systems > > > and Tablet Computing devices. It contains the following components: > > > > > > - Regulators. > > > - Configurable General Purpose Output Signals(GPO). [...] > > > +static const struct i2c_device_id lp873x_id_table[] = { > > > + { "lp873x", LP873X }, > > > + { }, > > > +}; > > > +MODULE_DEVICE_TABLE(i2c, lp873x_id_table); > > > > Don't set the ID in this table. > > I did not understand this. Could you please elaborate? > You mean no need of LP873X can just be a '0' as i am seeing in other mfd > drivers? Just leave it blank. No need for the second attribute. -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html