Re: [PATCH v11 03/38] clk: ep93xx: add DT support for Cirrus EP93xx

From: Stephen Boyd
Date: Thu Sep 05 2024 - 16:51:02 EST


Quoting Nikita Shubin (2024-09-02 06:31:59)
> On Fri, 2024-08-30 at 15:27 -0700, Stephen Boyd wrote:
> > Quoting Nikita Shubin (2024-08-30 02:23:12)
> > >
> > > Indeed REGMAP is selected by COMMON_CLK, MFD_SYSCON not required,
> > > but
> > > it needs AUXILIARY_BUS.
> >
> > I don't see REGMAP selected by COMMON_CLK. Did I miss something?
>
> Indeed REGMAP is selected by COMMON_CLK_MESON_REGMAP not COMMON_CLK on
> make tinyconfig + COMPILE_TEST.

The meson driver isn't used here?

>
> Then we require REGMAP because we are using regmap_read() in clk
> driver.

I think you're supposed to select REGMAP_<BUS> config option, not the
toplevel REGMAP option. For example, if you're using mmio, then select
REGMAP_MMIO.