[PATCH] clk: spacemit: ccu_mix: fix inverted condition in ccu_mix_trigger_fc()

Shuwei Wu posted 1 patch 1 month ago
drivers/clk/spacemit/ccu_mix.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH] clk: spacemit: ccu_mix: fix inverted condition in ccu_mix_trigger_fc()
Posted by Shuwei Wu 1 month ago
Fix inverted condition that skips frequency change trigger,
causing kernel panics during cpufreq scaling.

Fixes: 1b72c59db0ad ("clk: spacemit: Add clock support for SpacemiT K1 SoC")
Signed-off-by: Shuwei Wu <shuwei.wu@mailbox.org>
---
 drivers/clk/spacemit/ccu_mix.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/clk/spacemit/ccu_mix.c b/drivers/clk/spacemit/ccu_mix.c
index 9578366e9746..a8b407049bf4 100644
--- a/drivers/clk/spacemit/ccu_mix.c
+++ b/drivers/clk/spacemit/ccu_mix.c
@@ -73,7 +73,7 @@ static int ccu_mix_trigger_fc(struct clk_hw *hw)
 	struct ccu_common *common = hw_to_ccu_common(hw);
 	unsigned int val;
 
-	if (common->reg_fc)
+	if (!common->reg_fc)
 		return 0;
 
 	ccu_update(common, fc, common->mask_fc, common->mask_fc);

---
base-commit: c107785c7e8dbabd1c18301a1c362544b5786282
change-id: 20260305-k1-clk-fix-25b101c88459

Best regards,
-- 
Shuwei Wu <shuwei.wu@mailbox.org>
Re: [PATCH] clk: spacemit: ccu_mix: fix inverted condition in ccu_mix_trigger_fc()
Posted by Yixun Lan 2 weeks, 2 days ago
On Thu, 05 Mar 2026 20:46:08 +0800, Shuwei Wu wrote:
> Fix inverted condition that skips frequency change trigger,
> causing kernel panics during cpufreq scaling.
> 
> 

Applied, thanks! Will send to clk maintainer as v7.0 fix

[1/1] clk: spacemit: ccu_mix: fix inverted condition in ccu_mix_trigger_fc()
      https://github.com/spacemit-com/linux/commit/54e97360b44bed6b4399dd3be3d65f392df940fa

Best regards,
-- 
Yixun Lan <dlan@kernel.org>
Re: [PATCH] clk: spacemit: ccu_mix: fix inverted condition in ccu_mix_trigger_fc()
Posted by Yixun Lan 1 month ago
Hi Shuwei,

On 20:46 Thu 05 Mar     , Shuwei Wu wrote:
> Fix inverted condition that skips frequency change trigger,
> causing kernel panics during cpufreq scaling.
> 
> Fixes: 1b72c59db0ad ("clk: spacemit: Add clock support for SpacemiT K1 SoC")
> Signed-off-by: Shuwei Wu <shuwei.wu@mailbox.org>

Thanks for catching this..

Reviewed-by: Yixun Lan <dlan@kernel.org>
> ---
>  drivers/clk/spacemit/ccu_mix.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/clk/spacemit/ccu_mix.c b/drivers/clk/spacemit/ccu_mix.c
> index 9578366e9746..a8b407049bf4 100644
> --- a/drivers/clk/spacemit/ccu_mix.c
> +++ b/drivers/clk/spacemit/ccu_mix.c
> @@ -73,7 +73,7 @@ static int ccu_mix_trigger_fc(struct clk_hw *hw)
>  	struct ccu_common *common = hw_to_ccu_common(hw);
>  	unsigned int val;
>  
> -	if (common->reg_fc)
> +	if (!common->reg_fc)
>  		return 0;
>  
>  	ccu_update(common, fc, common->mask_fc, common->mask_fc);
> 
> ---
> base-commit: c107785c7e8dbabd1c18301a1c362544b5786282
> change-id: 20260305-k1-clk-fix-25b101c88459
> 
> Best regards,
> -- 
> Shuwei Wu <shuwei.wu@mailbox.org>
> 

-- 
Yixun Lan (dlan)