RE: [PATCH v2] clk: imx: don't build i.MX/RTxxxx code for all users of CCF

2026-05-18 Thread Peng Fan
> Subject: [PATCH v2] clk: imx: don't build i.MX/RTxxxx code for all users
> of CCF
> 
> From: Alexander Sverdlin 
> 
> When commit 1d7993d1d0ef
> ("clk: Port Linux common clock framework [CCF] for imx6q to U-boot
> (tag: v5.1.12)") introduced the parts of Linux Common Clock
> Framework, it was done for i.MX6 only and even had "depends on
> SPL_CLK_IMX6Q" conditions. Since commit
> ccab06689aa2 ("clk: imx: expose CCF entry for all") the framework can
> be re-used with SoCs of other vendors (say, TI), but NXP SoC-specific
> code is still being build. Is is especially problematic for size-constrained

Typo: Is->It

> SPL images on TI AM62x.
> 
> Make the build of the i.MX/RT code not only dependent on
> CONFIG_$(PHASE_)CLK_CCF, but also on CONFIG_MACH_IMX options
> which shall cover the i.MX platform users.
> 
> This saves 2264 bytes on 32-bit ARM platforms [using CCF].
> 
> Signed-off-by: Alexander Sverdlin 

LGTM: Reviewed-by: Peng Fan 


[PATCH v2] clk: imx: don't build i.MX/RTxxxx code for all users of CCF

2026-05-18 Thread A. Sverdlin
From: Alexander Sverdlin 

When commit 1d7993d1d0ef
("clk: Port Linux common clock framework [CCF] for imx6q to U-boot (tag: 
v5.1.12)")
introduced the parts of Linux Common Clock Framework, it was done for i.MX6
only and even had "depends on SPL_CLK_IMX6Q" conditions. Since commit
ccab06689aa2 ("clk: imx: expose CCF entry for all") the framework can be
re-used with SoCs of other vendors (say, TI), but NXP SoC-specific code is
still being build. Is is especially problematic for size-constrained SPL
images on TI AM62x.

Make the build of the i.MX/RT code not only dependent on
CONFIG_$(PHASE_)CLK_CCF, but also on CONFIG_MACH_IMX options which shall
cover the i.MX platform users.

This saves 2264 bytes on 32-bit ARM platforms [using CCF].

Signed-off-by: Alexander Sverdlin 
---
Changelog:
v2:
- CONFIG_MACH_IMX instead of (CONFIG_ARCH_MX6 | CONFIG_ARCH_IMXRT)
v1:
- 
https://lore.kernel.org/all/[email protected]/

 drivers/clk/Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/clk/Makefile b/drivers/clk/Makefile
index 5f0c0d8a5c2..c37ef75d420 100644
--- a/drivers/clk/Makefile
+++ b/drivers/clk/Makefile
@@ -16,7 +16,7 @@ obj-$(CONFIG_$(PHASE_)CLK_STUB) += clk-stub.o
 obj-y += adi/
 obj-y += airoha/
 obj-y += analogbits/
-obj-y += imx/
+obj-$(CONFIG_MACH_IMX) += imx/
 obj-$(CONFIG_CLK_JH7110) += starfive/
 obj-y += tegra/
 obj-y += ti/
-- 
2.52.0