Re: [PATCH 4/5] clk: renesas: r8a7792: Fix LB clock divider
Hi Fabrizio, On Tue, Apr 3, 2018 at 12:17 PM, Fabrizio Castro <fabrizio.cas...@bp.renesas.com> wrote: >> Subject: [PATCH 4/5] clk: renesas: r8a7792: Fix LB clock divider >> >> The CLK_TYPE_GEN2_LB clock type is meant for SoCs like R-Car H2, where >> the LB clock divider depends on the value of the MD18 pin. >> >> On R-Car V2H, the LB clock divider is fixed to 24. Hence model the >> clock as a fixed factor clock instead. >> >> Signed-off-by: Geert Uytterhoeven <geert+rene...@glider.be> >> --- >> drivers/clk/renesas/r8a7792-cpg-mssr.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/drivers/clk/renesas/r8a7792-cpg-mssr.c >> b/drivers/clk/renesas/r8a7792-cpg-mssr.c >> index 609a540804965c40..8b66e6f4b4584de1 100644 >> --- a/drivers/clk/renesas/r8a7792-cpg-mssr.c >> +++ b/drivers/clk/renesas/r8a7792-cpg-mssr.c >> @@ -53,7 +53,6 @@ static const struct cpg_core_clk r8a7792_core_clks[] >> __initconst = { >> DEF_FIXED(".pll1_div2", CLK_PLL1_DIV2, CLK_PLL1, 2, 1), >> >> /* Core Clock Outputs */ >> -DEF_BASE("lb", R8A7792_CLK_LB, CLK_TYPE_GEN2_LB, CLK_PLL1), >> DEF_BASE("qspi", R8A7792_CLK_QSPI, CLK_TYPE_GEN2_QSPI, CLK_PLL1_DIV2), >> >> DEF_FIXED("z", R8A7792_CLK_Z, CLK_PLL0, 1, 1), >> @@ -63,6 +62,7 @@ static const struct cpg_core_clk r8a7792_core_clks[] >> __initconst = { >> DEF_FIXED("hp", R8A7792_CLK_HP,CLK_PLL1, 12, 1), >> DEF_FIXED("i", R8A7792_CLK_I, CLK_PLL1, 3, 1), >> DEF_FIXED("b", R8A7792_CLK_B, CLK_PLL1, 12, 1), >> +DEF_FIXED("lb", R8A7792_CLK_B, CLK_PLL1, 24, 1), > > s/ R8A7792_CLK_B/ R8A7792_CLK_LB/ > > With that fixed: > Reviewed-by: Fabrizio Castro <fabrizio.cas...@bp.renesas.com> Thanks, nice catch. Fixing up (also for r8a7794). Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds
RE: [PATCH 4/5] clk: renesas: r8a7792: Fix LB clock divider
Hello Geert > Subject: [PATCH 4/5] clk: renesas: r8a7792: Fix LB clock divider > > The CLK_TYPE_GEN2_LB clock type is meant for SoCs like R-Car H2, where > the LB clock divider depends on the value of the MD18 pin. > > On R-Car V2H, the LB clock divider is fixed to 24. Hence model the > clock as a fixed factor clock instead. > > Signed-off-by: Geert Uytterhoeven <geert+rene...@glider.be> > --- > drivers/clk/renesas/r8a7792-cpg-mssr.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/clk/renesas/r8a7792-cpg-mssr.c > b/drivers/clk/renesas/r8a7792-cpg-mssr.c > index 609a540804965c40..8b66e6f4b4584de1 100644 > --- a/drivers/clk/renesas/r8a7792-cpg-mssr.c > +++ b/drivers/clk/renesas/r8a7792-cpg-mssr.c > @@ -53,7 +53,6 @@ static const struct cpg_core_clk r8a7792_core_clks[] > __initconst = { > DEF_FIXED(".pll1_div2", CLK_PLL1_DIV2, CLK_PLL1, 2, 1), > > /* Core Clock Outputs */ > -DEF_BASE("lb", R8A7792_CLK_LB, CLK_TYPE_GEN2_LB, CLK_PLL1), > DEF_BASE("qspi", R8A7792_CLK_QSPI, CLK_TYPE_GEN2_QSPI, CLK_PLL1_DIV2), > > DEF_FIXED("z", R8A7792_CLK_Z, CLK_PLL0, 1, 1), > @@ -63,6 +62,7 @@ static const struct cpg_core_clk r8a7792_core_clks[] > __initconst = { > DEF_FIXED("hp", R8A7792_CLK_HP,CLK_PLL1, 12, 1), > DEF_FIXED("i", R8A7792_CLK_I, CLK_PLL1, 3, 1), > DEF_FIXED("b", R8A7792_CLK_B, CLK_PLL1, 12, 1), > +DEF_FIXED("lb", R8A7792_CLK_B, CLK_PLL1, 24, 1), s/ R8A7792_CLK_B/ R8A7792_CLK_LB/ With that fixed: Reviewed-by: Fabrizio Castro <fabrizio.cas...@bp.renesas.com> > DEF_FIXED("p", R8A7792_CLK_P, CLK_PLL1, 24, 1), > DEF_FIXED("cl", R8A7792_CLK_CL,CLK_PLL1, 48, 1), > DEF_FIXED("m2", R8A7792_CLK_M2,CLK_PLL1, 8, 1), > -- > 2.7.4 Renesas Electronics Europe Ltd, Dukes Meadow, Millboard Road, Bourne End, Buckinghamshire, SL8 5FH, UK. Registered in England & Wales under Registered No. 04586709.
[PATCH 4/5] clk: renesas: r8a7792: Fix LB clock divider
The CLK_TYPE_GEN2_LB clock type is meant for SoCs like R-Car H2, where the LB clock divider depends on the value of the MD18 pin. On R-Car V2H, the LB clock divider is fixed to 24. Hence model the clock as a fixed factor clock instead. Signed-off-by: Geert Uytterhoeven--- drivers/clk/renesas/r8a7792-cpg-mssr.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/clk/renesas/r8a7792-cpg-mssr.c b/drivers/clk/renesas/r8a7792-cpg-mssr.c index 609a540804965c40..8b66e6f4b4584de1 100644 --- a/drivers/clk/renesas/r8a7792-cpg-mssr.c +++ b/drivers/clk/renesas/r8a7792-cpg-mssr.c @@ -53,7 +53,6 @@ static const struct cpg_core_clk r8a7792_core_clks[] __initconst = { DEF_FIXED(".pll1_div2", CLK_PLL1_DIV2, CLK_PLL1, 2, 1), /* Core Clock Outputs */ - DEF_BASE("lb", R8A7792_CLK_LB, CLK_TYPE_GEN2_LB, CLK_PLL1), DEF_BASE("qspi", R8A7792_CLK_QSPI, CLK_TYPE_GEN2_QSPI, CLK_PLL1_DIV2), DEF_FIXED("z", R8A7792_CLK_Z, CLK_PLL0, 1, 1), @@ -63,6 +62,7 @@ static const struct cpg_core_clk r8a7792_core_clks[] __initconst = { DEF_FIXED("hp", R8A7792_CLK_HP,CLK_PLL1, 12, 1), DEF_FIXED("i", R8A7792_CLK_I, CLK_PLL1, 3, 1), DEF_FIXED("b", R8A7792_CLK_B, CLK_PLL1, 12, 1), + DEF_FIXED("lb", R8A7792_CLK_B, CLK_PLL1, 24, 1), DEF_FIXED("p", R8A7792_CLK_P, CLK_PLL1, 24, 1), DEF_FIXED("cl", R8A7792_CLK_CL,CLK_PLL1, 48, 1), DEF_FIXED("m2", R8A7792_CLK_M2,CLK_PLL1, 8, 1), -- 2.7.4