Hi Sakari, Quoting Sakari Ailus (2021-11-26 08:06:23) > Depend on VIDEO_V4L2 for the driver actually depends on it, failing to > compile otherwise. Given that this matches how I added the same for max9286, I think this way is better than select. Thanks, Reviewed-by: Kieran Bingham <kieran.bingham+renesas@xxxxxxxxxxxxxxxx> > Signed-off-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> > --- > drivers/staging/media/max96712/Kconfig | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/staging/media/max96712/Kconfig b/drivers/staging/media/max96712/Kconfig > index 258d47644cbd..acde14fd5c4d 100644 > --- a/drivers/staging/media/max96712/Kconfig > +++ b/drivers/staging/media/max96712/Kconfig > @@ -3,6 +3,7 @@ config VIDEO_MAX96712 > tristate "Maxim MAX96712 Quad GMSL2 Deserializer support" > depends on I2C > depends on OF_GPIO > + depends on VIDEO_V4L2 > select V4L2_FWNODE > select VIDEO_V4L2_SUBDEV_API > select MEDIA_CONTROLLER > -- > 2.30.2 >