On Wed, Oct 01, 2025 at 05:08:46PM +0530, Chawdhry, Manorit wrote: > Hi Tom, > > On 9/26/2025 9:01 PM, Tom Rini wrote: > > A few of the TI clock drivers cannot link without access to TI platform > > specific drivers and headers. Express those requirements in Kconfig as > > well. > > > > Signed-off-by: Tom Rini <[email protected]> > > --- > > Cc: Lukasz Majewski <[email protected]> > > Cc: Sean Anderson <[email protected]> > > Cc: Manorit Chawdhry <[email protected]> > > Cc: Andrew Davis <[email protected]> > > Cc: Udit Kumar <[email protected]> > > --- > > drivers/clk/ti/Kconfig | 6 +++--- > > 1 file changed, 3 insertions(+), 3 deletions(-) > > > > diff --git a/drivers/clk/ti/Kconfig b/drivers/clk/ti/Kconfig > > index e0f15f57f826..ba658c741c4d 100644 > > --- a/drivers/clk/ti/Kconfig > > +++ b/drivers/clk/ti/Kconfig > > @@ -18,19 +18,19 @@ config CLK_TI_CTRL > > config CLK_TI_DIVIDER > > bool "TI divider clock driver" > > - depends on CLK && OF_CONTROL && CLK_CCF > > + depends on CLK && OF_CONTROL && CLK_CCF && ARCH_OMAP2PLUS > > help > > This enables the divider clock driver support on TI's SoCs. > > config CLK_TI_GATE > > bool "TI gate clock driver" > > - depends on CLK && OF_CONTROL > > + depends on CLK && OF_CONTROL && ARCH_OMAP2PLUS > > help > > This enables the gate clock driver support on TI's SoCs. > > config CLK_TI_MUX > > bool "TI mux clock driver" > > - depends on CLK && OF_CONTROL && CLK_CCF > > + depends on CLK && OF_CONTROL && CLK_CCF && ARCH_OMAP2PLUS > > help > > This enables the mux clock driver support on TI's SoCs. > > Do other Kconfigs' work in this file without access to TI platform specific > drivers and headers? am wondering that every Kconfig in this file would > ideally be TI specific only..
Ideally, long term we could build all of this on sandbox with allyesconfig and get static analysis done to it easily. But also yes, after this second patch here every driver has a ARCH dependency because the ones that wouldn't compile without a header caused others to fail to link. This also tells me that some of these options should be select'd by others, rather than everything being a question. -- Tom
signature.asc
Description: PGP signature
