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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <887d5cfabeccf5dae0538a9c59ba74e4.sboyd@kernel.org>
Date: Thu, 05 Sep 2024 13:50:52 -0700
From: Stephen Boyd <sboyd@...nel.org>
To: Michael Turquette <mturquette@...libre.com>, Nikita Shubin <nikita.shubin@...uefel.me>, Nikita Shubin via B4 Relay <devnull+nikita.shubin.maquefel.me@...nel.org>
Cc: linux-kernel@...r.kernel.org, linux-clk@...r.kernel.org
Subject: Re: [PATCH v11 03/38] clk: ep93xx: add DT support for Cirrus EP93xx

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.