On Thu, Jun 15, 2023 at 11:41:10AM +0000, Sakari Ailus wrote: > On Thu, Jun 15, 2023 at 12:15:47PM +0200, Hans de Goede wrote: > > On 6/15/23 11:54, Sakari Ailus wrote: > > > On Thu, Jun 15, 2023 at 10:45:35AM +0200, Hans de Goede wrote: > > >> On 6/14/23 22:39, Sakari Ailus wrote: ... > > If you agree that because of the symbol dependency that > > the select REGMAP_I2C is necessary then the depends on I2C > > is also necessary because any Kconfig symbol selecting > > another symbol MUST depends on all of the dependencies > > of the selected symbol and REGMAP_I2C has: > > > > config REGMAP_I2C > > tristate > > depends on I2C > > Yes. > > How about putting cci_regmap_init_i2c() behind an #ifdef? Then there > wouldn't be a need for REGMAP_I2C unconditionally, but dependent on I2C. > > I guess right now I2C is more or less given in many systems but binding CCI > to it still seems dubious. Can't we solve the issue when it comes? What you are suggesting seems controversial to me at this stage. -- With Best Regards, Andy Shevchenko
