On Mon, Sep 2, 2013 at 11:00 AM, Angelo Compagnucci <angelo.compagnucci@xxxxxxxxx> wrote: > Kawabonga! It works! Really, but really (no joke here!) thank you > for your precious support and sorry for not being so smart to > understand what you were saying me. Sorry again for being so pedantic! > > I'm planning to write a tutorial based on my journey, something like > "Howto write your first IIO LDD", what do you think? > > Attached here, there is the latest version of the patch based on jour suggestions. > I think all of your concerns are solved now. > > Thank you again! > > Signed-off-by: Angelo Compagnucci <angelo.compagnucci@xxxxxxxxx> ... > +static const struct i2c_device_id mcp3422_id[] = { > + { "mcp3422", 2 }, > + { "mcp3423", 3 }, > + { "mcp3424", 4 }, > + { } > +}; > +MODULE_DEVICE_TABLE(i2c, mcp3422_id); > + > +#ifdef CONFIG_OF > +static const struct of_device_id mcp3422_of_match[] = { > + { .compatible = "mcp3422" }, > + { } Shouldn't this be mcp,mcp3422? Another question, are you aware of need changes to use with mcp3428? Otherwise I will look at this. -- Otavio Salvador O.S. Systems http://www.ossystems.com.br http://code.ossystems.com.br Mobile: +55 (53) 9981-7854 Mobile: +1 (347) 903-9750 -- 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