Thread (14 messages) 14 messages, 2 authors, 2022-07-21

Re: [PATCH v2 3/6] media: i2c: ov9282: Add ov9281 compatible

From: Sakari Ailus <sakari.ailus@iki.fi>
Date: 2022-07-17 14:23:23
Also in: linux-media

Hi Alexander,

On Fri, Jul 15, 2022 at 09:48:55AM +0200, Alexander Stein wrote:
quoted hunk ↗ jump to hunk
According to product brief they are identical from software point of view.
Differences are a different chief ray angle (CRA) and the package.

Signed-off-by: Alexander Stein <redacted>
Acked-by: Daniele Alessandrelli <redacted>
---
 drivers/media/i2c/ov9282.c | 1 +
 1 file changed, 1 insertion(+)
diff --git a/drivers/media/i2c/ov9282.c b/drivers/media/i2c/ov9282.c
index 3269c1983bd5..04fda8222e07 100644
--- a/drivers/media/i2c/ov9282.c
+++ b/drivers/media/i2c/ov9282.c
@@ -1114,6 +1114,7 @@ static const struct dev_pm_ops ov9282_pm_ops = {
 };
 
 static const struct of_device_id ov9282_of_match[] = {
+	{ .compatible = "ovti,ov9281" },
This should also be reflected in the entity name --- the user space
deserves to know it's a different device.
 	{ .compatible = "ovti,ov9282" },
 	{ }
 };
-- 
Sakari Ailus
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help