Hi Lukasz,
Subject: Re: [PATCH V3 1/4] clk: imx: add Kconfig entry for i.MX8MM
Hi Peng,
On Fri, Aug 9, 2019 at 5:02 AM Peng Fan peng.fan@nxp.com wrote:
Add Kconfig entry for i.MX8MM
Signed-off-by: Peng Fan peng.fan@nxp.com
drivers/clk/Kconfig | 4 ++-- drivers/clk/imx/Kconfig | 16 ++++++++++++++++ 2 files changed, 18 insertions(+), 2 deletions(-)
diff --git a/drivers/clk/Kconfig b/drivers/clk/Kconfig index a3f0171b45..fce595b4b3 100644 --- a/drivers/clk/Kconfig +++ b/drivers/clk/Kconfig @@ -48,7 +48,7 @@ config CLK_BOSTON
config SPL_CLK_CCF bool "SPL Common Clock Framework [CCF] support "
depends on SPL_CLK_IMX6Q
depends on SPL_CLK_IMX6Q || ARCH_IMX8MI am wondering why this depends on i.MX. Shouldn't it be SoC agnostic?
Are you ok to expose the Kconfig for all with remove the depends on?
Thanks Peng
help Enable this option if you want to (re-)use the Linux kernel'sCommon
Clock Framework [CCF] code in U-Boot's SPL.@@ -62,7 +62,7 @@ config SPL_CLK_COMPOSITE_CCF
config CLK_CCF bool "Common Clock Framework [CCF] support "
depends on CLK_IMX6Q || SANDBOX_CLK_CCF
depends on CLK_IMX6Q || ARCH_IMX8M || SANDBOX_CLK_CCFSame here.
It doesn't look like a good idea to keep expanding this Kconfig every time a new SoC wants to use the CCF.