RE: [PATCH V2 3/9] clk: imx: Support building SCU clock driver as module

From: Anson Huang
Date: Tue Jun 30 2020 - 00:42:02 EST




> Subject: RE: [PATCH V2 3/9] clk: imx: Support building SCU clock driver as
> module
>
> > From: Arnd Bergmann <arnd@xxxxxxxx>
> > Sent: Monday, June 29, 2020 4:20 PM
> >
> > On Mon, Jun 29, 2020 at 9:18 AM Dong Aisheng <dongas86@xxxxxxxxx>
> > wrote:
> > > On Thu, Jun 25, 2020 at 6:43 AM Stephen Boyd <sboyd@xxxxxxxxxx>
> wrote:
> > > > Quoting Aisheng Dong (2020-06-23 19:59:09) Why aren't there
> > > > options to enable clk-imx6q and clk-imx6sl in the clk/imx/Kconfig file?
> > > > Those can be bool or tristate depending on if the SoC drivers use
> > > > CLK_OF_DECLARE or not and depend on the mxc-clk library and SoC
> > > > config we have in the makefile today.
> > >
> > > Yes, we can do that in clk/imx/Kconfig as follows theoretically.
> > > config CLK_IMX6Q
> > > bool
> > > def_bool ARCH_MXC && ARM
> > > select MXC_CLK
> > >
> > > But we have totally 15 platforms that need to change.
> >
> > I would make that
> >
> > config CLK_IMX6Q
> > bool "Clock driver for NXP i.MX6Q"
> > depends on SOC_IMX6Q || COMPILE_TEST
> > default SOC_IMX6Q
> > select MXC_CLK
> >
>
> Yes, this seems better.
> Anson, pls follow this way.

In V4 patch series, I will add a patch to introduce CLK_IMX6Q and other i.MX6/7
clock config following this way.

Anson