Hi Javier, Thank you for the patch. On Monday 04 January 2016 09:25:30 Javier Martinez Canillas wrote: > From: Eduard Gavin <egavinc@xxxxxxxxx> > > The Documentation/devicetree/bindings/media/i2c/tvp5150.txt DT binding doc > lists "ti,tvp5150" as the device compatible string but the driver does not > have an OF match table. Add the table to the driver so the I2C core can do > an OF style match. > > Signed-off-by: Eduard Gavin <egavinc@xxxxxxxxx> > Signed-off-by: Javier Martinez Canillas <javier@xxxxxxxxxxxxxxx> Reviewed-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> > --- > > drivers/media/i2c/tvp5150.c | 9 +++++++++ > 1 file changed, 9 insertions(+) > > diff --git a/drivers/media/i2c/tvp5150.c b/drivers/media/i2c/tvp5150.c > index 105bd1c6b17f..caac96a577f8 100644 > --- a/drivers/media/i2c/tvp5150.c > +++ b/drivers/media/i2c/tvp5150.c > @@ -1295,8 +1295,17 @@ static const struct i2c_device_id tvp5150_id[] = { > }; > MODULE_DEVICE_TABLE(i2c, tvp5150_id); > > +#if IS_ENABLED(CONFIG_OF) > +static const struct of_device_id tvp5150_of_match[] = { > + { .compatible = "ti,tvp5150", }, > + { /* sentinel */ }, > +}; > +MODULE_DEVICE_TABLE(of, tvp5150_of_match); > +#endif > + > static struct i2c_driver tvp5150_driver = { > .driver = { > + .of_match_table = of_match_ptr(tvp5150_of_match), > .name = "tvp5150", > }, > .probe = tvp5150_probe, -- Regards, Laurent Pinchart -- 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