Hi Sakari, On Fri, Nov 10, 2023 at 09:15:52PM +0000, Sakari Ailus wrote: > On Fri, Nov 10, 2023 at 05:39:40PM +0200, Laurent Pinchart wrote: > > On Fri, Nov 10, 2023 at 12:10:49PM +0200, Sakari Ailus wrote: > > > Compile sub-device state information access functions > > > v4l2_subdev_state_get_{format,crop,compose} unconditionally as they are > > > now also used by plain V4L2 drivers. > > > > What do you mean by "plain" V4L2 drivers here ? > > Those that do not need MC: if you now disable MC in kernel configuration, > these functions won't be available. This covers subdev drivers (such as sensor drivers) too, not just host drivers ? If so, shouldn't we also drop the dependency on VIDEO_V4L2_SUBDEV_API from drivers/media/i2c/Kconfig ? -- Regards, Laurent Pinchart