[PATCH 1/2] clk: scmi: fix set_parent support when CCF is not being used

Peng Fan peng.fan at oss.nxp.com
Mon Aug 18 06:47:07 CEST 2025


On Wed, Aug 13, 2025 at 03:59:35PM +0530, Kamlesh Gurudasani wrote:
>When not using Common clock framework(CCF), calls to
>scmi_clk_set_parent returns -ENOTSUPP, which should not be the case.
>Fix that.
>
>Fixes: 15fdfef6642c ("clk: scmi: check the clock state/parent/rate
>control permissions)
>
>Signed-off-by: Kamlesh Gurudasani <kamlesh at ti.com>
>---
> drivers/clk/clk_scmi.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/drivers/clk/clk_scmi.c b/drivers/clk/clk_scmi.c
>index 0c9a81cabcc..83ea7bb4286 100644
>--- a/drivers/clk/clk_scmi.c
>+++ b/drivers/clk/clk_scmi.c
>@@ -359,7 +359,7 @@ static int scmi_clk_set_parent(struct clk *clk, struct clk *parent)
> 	int ret;
> 
> 	if (!CONFIG_IS_ENABLED(CLK_CCF))
>-		return -ENOTSUPP;
>+		return __scmi_clk_set_parent(clk, parent);

The CCF check should be dropped per my understanding. The function should
work w/o CCF.

Regards,
Peng

> 
> 	ret = clk_get_by_id(clk->id, &c);
> 	if (ret)
>
>-- 
>2.34.1
>


More information about the U-Boot mailing list