On Thu, Jun 15, 2023 at 1:21 AM Andy Shevchenko <andy.shevchenko@xxxxxxxxx> wrote: > On Wed, Jun 14, 2023 at 11:49 PM Sakari Ailus > <sakari.ailus@xxxxxxxxxxxxxxx> wrote: > > On Wed, Jun 14, 2023 at 09:23:39PM +0200, Hans de Goede wrote: ... > > > +config V4L2_CCI > > > + tristate > > > + depends on I2C > > > > This won't do anything if the dependent driver will select V4L2_CCI, will > > it? > > Actually it will. It may warn the user about missing dependency on the > `make *config` stage. > See, for example, this: > https://lore.kernel.org/lkml/1f6c4559-ee48-ca67-8ff1-f7f9d330bc12@xxxxxxx/. With corrected link https://lore.kernel.org/lkml/20220316183708.1505846-1-arnd@xxxxxxxxxx/ > > I'd let the sensor driver depend on I2C instead. CCI is also supported > > on I3C, for instance. -- With Best Regards, Andy Shevchenko
