On Wed, Jan 27, 2016 at 12:28 AM, Lucas De Marchi <lucas.de.marchi@xxxxxxxxx> wrote: > Hi, > > On Thu, Jan 14, 2016 at 2:25 PM, Daniel Baluta <daniel.baluta@xxxxxxxxx> wrote: >> The driver has sysfs readings with runtime PM support for power saving. >> It also offers buffer support that can be used together with IIO software >> triggers. >> >> Datasheet can be found here: >> http://www.ti.com.cn/cn/lit/ds/symlink/ads1015.pdf >> >> Signed-off-by: Daniel Baluta <daniel.baluta@xxxxxxxxx> >> --- > > [ ... ] > >> +static const struct i2c_device_id ads1015_id[] = { >> + {"ads1015", 0}, > > shouldn't this be ti-ads1015 so not to clash with the hwmon driver? The plan is to remove hwmon driver and replace it with the a backward compatbile IIO driver. I am now working to address the comments from this thread and send a v2 asap. > >> + {} >> +}; >> +MODULE_DEVICE_TABLE(i2c, ads1015_id); >> + >> +static struct i2c_driver ads1015_driver = { >> + .driver = { >> + .name = ADS1015_DRV_NAME, > > same here, otherwise we will have an i2c:ads1015 alias thanks, Daniel. -- To unsubscribe from this list: send the line "unsubscribe linux-iio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html