diff mbox

[3/3] clk: renesas: r8a7796: Add DU and LVDS clocks

Message ID 1476738753-24304-4-git-send-email-laurent.pinchart+renesas@ideasonboard.com (mailing list archive)
State Superseded
Delegated to: Geert Uytterhoeven
Headers show

Commit Message

Laurent Pinchart Oct. 17, 2016, 9:12 p.m. UTC
Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
---
 drivers/clk/renesas/r8a7796-cpg-mssr.c | 4 ++++
 1 file changed, 4 insertions(+)

Comments

Geert Uytterhoeven Oct. 18, 2016, 8:31 a.m. UTC | #1
On Mon, Oct 17, 2016 at 11:12 PM, Laurent Pinchart
<laurent.pinchart+renesas@ideasonboard.com> wrote:
> Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>

Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

> ---
>  drivers/clk/renesas/r8a7796-cpg-mssr.c | 4 ++++
>  1 file changed, 4 insertions(+)
>
> diff --git a/drivers/clk/renesas/r8a7796-cpg-mssr.c b/drivers/clk/renesas/r8a7796-cpg-mssr.c
> index 679054658f99..20bd0643b238 100644
> --- a/drivers/clk/renesas/r8a7796-cpg-mssr.c
> +++ b/drivers/clk/renesas/r8a7796-cpg-mssr.c
> @@ -134,6 +134,10 @@ static const struct mssr_mod_clk r8a7796_mod_clks[] __initconst = {
>         DEF_MOD("vspd0",                 623,   R8A7796_CLK_S0D2),
>         DEF_MOD("vspb",                  626,   R8A7796_CLK_S0D1),
>         DEF_MOD("vspi0",                 631,   R8A7796_CLK_S0D1),
> +       DEF_MOD("du2",                   722,   R8A7796_CLK_S0D2),
> +       DEF_MOD("du1",                   723,   R8A7796_CLK_S0D2),
> +       DEF_MOD("du0",                   724,   R8A7796_CLK_S0D2),

I'll fix up the DU parent clock to S2D1 now we got confirmation.

> +       DEF_MOD("lvds",                  727,   R8A7796_CLK_S0D4),

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@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
Laurent Pinchart Oct. 18, 2016, 8:33 a.m. UTC | #2
Hi Geert,

On Tuesday 18 Oct 2016 10:31:10 Geert Uytterhoeven wrote:
> On Mon, Oct 17, 2016 at 11:12 PM, Laurent Pinchart wrote:
> > Signed-off-by: Laurent Pinchart
> > <laurent.pinchart+renesas@ideasonboard.com>
> 
> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
> 
> > ---
> > 
> >  drivers/clk/renesas/r8a7796-cpg-mssr.c | 4 ++++
> >  1 file changed, 4 insertions(+)
> > 
> > diff --git a/drivers/clk/renesas/r8a7796-cpg-mssr.c
> > b/drivers/clk/renesas/r8a7796-cpg-mssr.c index 679054658f99..20bd0643b238
> > 100644
> > --- a/drivers/clk/renesas/r8a7796-cpg-mssr.c
> > +++ b/drivers/clk/renesas/r8a7796-cpg-mssr.c
> > @@ -134,6 +134,10 @@ static const struct mssr_mod_clk r8a7796_mod_clks[]
> > __initconst = {
> >         DEF_MOD("vspd0",                 623,   R8A7796_CLK_S0D2),
> >         DEF_MOD("vspb",                  626,   R8A7796_CLK_S0D1),
> >         DEF_MOD("vspi0",                 631,   R8A7796_CLK_S0D1),
> > +       DEF_MOD("du2",                   722,   R8A7796_CLK_S0D2),
> > +       DEF_MOD("du1",                   723,   R8A7796_CLK_S0D2),
> > +       DEF_MOD("du0",                   724,   R8A7796_CLK_S0D2),
> 
> I'll fix up the DU parent clock to S2D1 now we got confirmation.

Fixed in v2, sent out to the list, and pushed to 
git://linuxtv.org/pinchartl/media.git drm/r8a7796/clk with your acks picked up 
:-)

Can I assume you'll take the patches in your tree for v4.10 ?

> > +       DEF_MOD("lvds",                  727,   R8A7796_CLK_S0D4),
diff mbox

Patch

diff --git a/drivers/clk/renesas/r8a7796-cpg-mssr.c b/drivers/clk/renesas/r8a7796-cpg-mssr.c
index 679054658f99..20bd0643b238 100644
--- a/drivers/clk/renesas/r8a7796-cpg-mssr.c
+++ b/drivers/clk/renesas/r8a7796-cpg-mssr.c
@@ -134,6 +134,10 @@  static const struct mssr_mod_clk r8a7796_mod_clks[] __initconst = {
 	DEF_MOD("vspd0",		 623,	R8A7796_CLK_S0D2),
 	DEF_MOD("vspb",			 626,	R8A7796_CLK_S0D1),
 	DEF_MOD("vspi0",		 631,	R8A7796_CLK_S0D1),
+	DEF_MOD("du2",			 722,	R8A7796_CLK_S0D2),
+	DEF_MOD("du1",			 723,	R8A7796_CLK_S0D2),
+	DEF_MOD("du0",			 724,	R8A7796_CLK_S0D2),
+	DEF_MOD("lvds",			 727,	R8A7796_CLK_S0D4),
 	DEF_MOD("etheravb",		 812,	R8A7796_CLK_S0D6),
 	DEF_MOD("gpio7",		 905,	R8A7796_CLK_S3D4),
 	DEF_MOD("gpio6",		 906,	R8A7796_CLK_S3D4),