[v2,3/4] clk: rockchip: Remove values for mmask and nmask in struct clk_fractional_divider

Commit Message

Christophe JAILLET April 2, 2023, 9:42 a.m. UTC

Now that fractional_divider clk computes mmask and nmask when needed, there
is no more need to provide them explicitly anymore.

Signed-off-by: Christophe JAILLET <[e-mail]>
---
 drivers/clk/rockchip/clk.c | 2 --
 1 file changed, 2 deletions(-)

Comments

Heiko Stuebner April 2, 2023, 2:35 p.m. UTC | #1

Am Sonntag, 2. April 2023, 11:42:06 CEST schrieb Christophe JAILLET:
> Now that fractional_divider clk computes mmask and nmask when needed, there
> is no more need to provide them explicitly anymore.
> 
> Signed-off-by: Christophe JAILLET <[e-mail]>

Reviewed-by: Heiko Stuebner <[e-mail]>

Stephen Boyd April 5, 2023, 7:12 p.m. UTC | #2

Quoting Christophe JAILLET (2023-04-02 02:42:06)
> Now that fractional_divider clk computes mmask and nmask when needed, there
> is no more need to provide them explicitly anymore.
> 
> Signed-off-by: Christophe JAILLET <[e-mail]>
> ---

Applied to clk-next

Patch

diff --git a/drivers/clk/rockchip/clk.c b/drivers/clk/rockchip/clk.c
index a8646794575a..4059d9365ae6 100644
--- a/drivers/clk/rockchip/clk.c
+++ b/drivers/clk/rockchip/clk.c
@@ -244,10 +244,8 @@  static struct clk *rockchip_clk_register_frac_branch(
 	div->reg = base + muxdiv_offset;
 	div->mshift = 16;
 	div->mwidth = 16;
-	div->mmask = GENMASK(div->mwidth - 1, 0) << div->mshift;
 	div->nshift = 0;
 	div->nwidth = 16;
-	div->nmask = GENMASK(div->nwidth - 1, 0) << div->nshift;
 	div->lock = lock;
 	div->approximation = rockchip_fractional_approximation;
 	div_ops = &clk_fractional_divider_ops;