* [PATCH] clk: k210: Fix k210_clk_set_parent()
@ 2021-06-22 6:45 Damien Le Moal
2021-06-25 7:19 ` Damien Le Moal
2021-06-30 18:34 ` Stephen Boyd
0 siblings, 2 replies; 4+ messages in thread
From: Damien Le Moal @ 2021-06-22 6:45 UTC (permalink / raw)
To: Stephen Boyd, linux-clk; +Cc: stable
In k210_clk_set_parent(), add missing writel() call to update the mux
register of a clock to change its parent. This also fixes a compilation
warning with clang when compiling with W=1.
Fixes: c6ca7616f7d5 ("clk: Add RISC-V Canaan Kendryte K210 clock driver")
Cc: stable@vger.kernel.org
Signed-off-by: Damien Le Moal <damien.lemoal@wdc.com>
---
drivers/clk/clk-k210.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/clk/clk-k210.c b/drivers/clk/clk-k210.c
index 6c84abf5b2e3..67a7cb3503c3 100644
--- a/drivers/clk/clk-k210.c
+++ b/drivers/clk/clk-k210.c
@@ -722,6 +722,7 @@ static int k210_clk_set_parent(struct clk_hw *hw, u8 index)
reg |= BIT(cfg->mux_bit);
else
reg &= ~BIT(cfg->mux_bit);
+ writel(reg, ksc->regs + cfg->mux_reg);
spin_unlock_irqrestore(&ksc->clk_lock, flags);
return 0;
--
2.31.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] clk: k210: Fix k210_clk_set_parent()
2021-06-22 6:45 [PATCH] clk: k210: Fix k210_clk_set_parent() Damien Le Moal
@ 2021-06-25 7:19 ` Damien Le Moal
2021-06-30 18:34 ` Stephen Boyd
1 sibling, 0 replies; 4+ messages in thread
From: Damien Le Moal @ 2021-06-25 7:19 UTC (permalink / raw)
To: Stephen Boyd, linux-clk
On 2021/06/22 15:45, Damien Le Moal wrote:
> In k210_clk_set_parent(), add missing writel() call to update the mux
> register of a clock to change its parent. This also fixes a compilation
> warning with clang when compiling with W=1.
>
> Fixes: c6ca7616f7d5 ("clk: Add RISC-V Canaan Kendryte K210 clock driver")
> Cc: stable@vger.kernel.org
> Signed-off-by: Damien Le Moal <damien.lemoal@wdc.com>
> ---
> drivers/clk/clk-k210.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/clk/clk-k210.c b/drivers/clk/clk-k210.c
> index 6c84abf5b2e3..67a7cb3503c3 100644
> --- a/drivers/clk/clk-k210.c
> +++ b/drivers/clk/clk-k210.c
> @@ -722,6 +722,7 @@ static int k210_clk_set_parent(struct clk_hw *hw, u8 index)
> reg |= BIT(cfg->mux_bit);
> else
> reg &= ~BIT(cfg->mux_bit);
> + writel(reg, ksc->regs + cfg->mux_reg);
> spin_unlock_irqrestore(&ksc->clk_lock, flags);
>
> return 0;
>
Stephen,
Ping ?
--
Damien Le Moal
Western Digital Research
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] clk: k210: Fix k210_clk_set_parent()
2021-06-22 6:45 [PATCH] clk: k210: Fix k210_clk_set_parent() Damien Le Moal
2021-06-25 7:19 ` Damien Le Moal
@ 2021-06-30 18:34 ` Stephen Boyd
1 sibling, 0 replies; 4+ messages in thread
From: Stephen Boyd @ 2021-06-30 18:34 UTC (permalink / raw)
To: Damien Le Moal, linux-clk; +Cc: stable
Quoting Damien Le Moal (2021-06-21 23:45:02)
> In k210_clk_set_parent(), add missing writel() call to update the mux
> register of a clock to change its parent. This also fixes a compilation
> warning with clang when compiling with W=1.
>
> Fixes: c6ca7616f7d5 ("clk: Add RISC-V Canaan Kendryte K210 clock driver")
> Cc: stable@vger.kernel.org
> Signed-off-by: Damien Le Moal <damien.lemoal@wdc.com>
> ---
Applied to clk-next
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH] clk: k210: Fix k210_clk_set_parent()
@ 2021-07-17 0:28 Damien Le Moal
0 siblings, 0 replies; 4+ messages in thread
From: Damien Le Moal @ 2021-07-17 0:28 UTC (permalink / raw)
To: Damien Le Moal; +Cc: stable
From: Damien Le Moal <damien.lemoal@wdc.com>
In k210_clk_set_parent(), add missing writel() call to update the mux
register of a clock to change its parent. This also fixes a compilation
warning with clang when compiling with W=1.
Fixes: c6ca7616f7d5 ("clk: Add RISC-V Canaan Kendryte K210 clock driver")
Cc: stable@vger.kernel.org
Signed-off-by: Damien Le Moal <damien.lemoal@wdc.com>
---
drivers/clk/clk-k210.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/clk/clk-k210.c b/drivers/clk/clk-k210.c
index 6c84abf5b2e3..67a7cb3503c3 100644
--- a/drivers/clk/clk-k210.c
+++ b/drivers/clk/clk-k210.c
@@ -722,6 +722,7 @@ static int k210_clk_set_parent(struct clk_hw *hw, u8 index)
reg |= BIT(cfg->mux_bit);
else
reg &= ~BIT(cfg->mux_bit);
+ writel(reg, ksc->regs + cfg->mux_reg);
spin_unlock_irqrestore(&ksc->clk_lock, flags);
return 0;
--
2.31.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
end of thread, other threads:[~2021-07-17 0:28 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-06-22 6:45 [PATCH] clk: k210: Fix k210_clk_set_parent() Damien Le Moal
2021-06-25 7:19 ` Damien Le Moal
2021-06-30 18:34 ` Stephen Boyd
2021-07-17 0:28 Damien Le Moal
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.