Re: [Openipmi-developer] [PATCH 1/8] ipmi: ASPEED_BT_IPMI_BMC: select REGMAP_MMIO instead of depending on it
On 2/26/23 14:59, Corey Minyard wrote: > On Sat, Feb 25, 2023 at 09:39:46PM -0800, Randy Dunlap wrote: >> REGMAP is a hidden (not user visible) symbol. Users cannot set it >> directly thru "make *config", so drivers should select it instead of >> depending on it if they need it. >> >> Consistently using "select" or "depends on" can also help reduce >> Kconfig circular dependency issues. >> >> Therefore, change the use of "depends on REGMAP_MMIO" to >> "select REGMAP_MMIO", which will also set REGMAP. > > This seems reasonable. I can take it into my tree, or.. Just take it, please. I expect that each subsystem maintainer will take their respective patches. > Acked-by: Corey Minyard > >> >> Fixes: eb994594bc22 ("ipmi: bt-bmc: Use a regmap for register access") >> Signed-off-by: Randy Dunlap >> Cc: Andrew Jeffery >> Cc: Corey Minyard >> Cc: openipmi-developer@lists.sourceforge.net >> Cc: Arnd Bergmann >> Cc: Greg Kroah-Hartman >> --- >> drivers/char/ipmi/Kconfig |3 ++- >> 1 file changed, 2 insertions(+), 1 deletion(-) >> >> diff -- a/drivers/char/ipmi/Kconfig b/drivers/char/ipmi/Kconfig >> --- a/drivers/char/ipmi/Kconfig >> +++ b/drivers/char/ipmi/Kconfig >> @@ -162,7 +162,8 @@ config IPMI_KCS_BMC_SERIO >> >> config ASPEED_BT_IPMI_BMC >> depends on ARCH_ASPEED || COMPILE_TEST >> -depends on REGMAP && REGMAP_MMIO && MFD_SYSCON >> +depends on MFD_SYSCON >> +select REGMAP_MMIO >> tristate "BT IPMI bmc driver" >> help >>Provides a driver for the BT (Block Transfer) IPMI interface -- ~Randy ___ Openipmi-developer mailing list Openipmi-developer@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/openipmi-developer
Re: [Openipmi-developer] [PATCH 1/8] ipmi: ASPEED_BT_IPMI_BMC: select REGMAP_MMIO instead of depending on it
On Sat, Feb 25, 2023 at 09:39:46PM -0800, Randy Dunlap wrote: > REGMAP is a hidden (not user visible) symbol. Users cannot set it > directly thru "make *config", so drivers should select it instead of > depending on it if they need it. > > Consistently using "select" or "depends on" can also help reduce > Kconfig circular dependency issues. > > Therefore, change the use of "depends on REGMAP_MMIO" to > "select REGMAP_MMIO", which will also set REGMAP. This seems reasonable. I can take it into my tree, or.. Acked-by: Corey Minyard > > Fixes: eb994594bc22 ("ipmi: bt-bmc: Use a regmap for register access") > Signed-off-by: Randy Dunlap > Cc: Andrew Jeffery > Cc: Corey Minyard > Cc: openipmi-developer@lists.sourceforge.net > Cc: Arnd Bergmann > Cc: Greg Kroah-Hartman > --- > drivers/char/ipmi/Kconfig |3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff -- a/drivers/char/ipmi/Kconfig b/drivers/char/ipmi/Kconfig > --- a/drivers/char/ipmi/Kconfig > +++ b/drivers/char/ipmi/Kconfig > @@ -162,7 +162,8 @@ config IPMI_KCS_BMC_SERIO > > config ASPEED_BT_IPMI_BMC > depends on ARCH_ASPEED || COMPILE_TEST > - depends on REGMAP && REGMAP_MMIO && MFD_SYSCON > + depends on MFD_SYSCON > + select REGMAP_MMIO > tristate "BT IPMI bmc driver" > help > Provides a driver for the BT (Block Transfer) IPMI interface ___ Openipmi-developer mailing list Openipmi-developer@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/openipmi-developer