diff mbox series

[v2] media: hi846: depend on OF

Message ID 20211018134246.383594-1-martin.kepplinger@puri.sm (mailing list archive)
State New, archived
Headers show
Series [v2] media: hi846: depend on OF | expand

Commit Message

Martin Kepplinger Oct. 18, 2021, 1:42 p.m. UTC
Since other device enumerate mechanisms are currently not implemented,
make the hi846 sensor driver depend on CONFIG_OF.

Signed-off-by: Martin Kepplinger <martin.kepplinger@puri.sm>
Reported-by: kernel test robot <lkp@intel.com>
---

revision history
----------------
v2:
* added the reported-by tag

v1:
https://lore.kernel.org/linux-media/20211018095859.255912-1-martin.kepplinger@puri.sm/



 drivers/media/i2c/Kconfig | 1 +
 1 file changed, 1 insertion(+)

Comments

Sakari Ailus Oct. 19, 2021, 12:27 p.m. UTC | #1
Hi Martin,

On Mon, Oct 18, 2021 at 03:42:46PM +0200, Martin Kepplinger wrote:
> Since other device enumerate mechanisms are currently not implemented,
> make the hi846 sensor driver depend on CONFIG_OF.

As far as I understand, this should work on ACPI based systems with just
of_match_ptr() macro removed.

In any case, linux/of_graph.h header should be replaced with
linux/property.h.

How about that instead?
diff mbox series

Patch

diff --git a/drivers/media/i2c/Kconfig b/drivers/media/i2c/Kconfig
index 0e56489c97fb..53268f1f761f 100644
--- a/drivers/media/i2c/Kconfig
+++ b/drivers/media/i2c/Kconfig
@@ -746,6 +746,7 @@  config VIDEO_HI556
 config VIDEO_HI846
 	tristate "Hynix Hi-846 sensor support"
 	depends on I2C && VIDEO_V4L2
+	depends on OF
 	select MEDIA_CONTROLLER
 	select VIDEO_V4L2_SUBDEV_API
 	select V4L2_FWNODE