Re: [PATCH 08/10] [media] tvp5150: Add OF match table
From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Date: 2016-01-06 10:57:53
Also in:
linux-media, lkml
Hi Javier, Thank you for the patch. On Monday 04 January 2016 09:25:30 Javier Martinez Canillas wrote:
From: Eduard Gavin <redacted> 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 <redacted> Signed-off-by: Javier Martinez Canillas <redacted>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
quoted hunk ↗ jump to hunk
--- 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