All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] clk: renesas: r8a7795: Correct lvds clock parent
@ 2016-06-10  7:44 Geert Uytterhoeven
  2016-06-10 13:22 ` Laurent Pinchart
  0 siblings, 1 reply; 2+ messages in thread
From: Geert Uytterhoeven @ 2016-06-10  7:44 UTC (permalink / raw)
  To: Michael Turquette, Stephen Boyd, Laurent Pinchart
  Cc: Kuninori Morimoto, linux-clk, linux-renesas-soc, Geert Uytterhoeven

According to the latest information, the parent clock of the LVDS module
clock is the S0D4 clock, not the S2D1 clock.

Note that this change has no influence on actual operation, as the
rcar-du LVDS encoder driver doesn't use the parent clock's rate.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
Will queue in clk-renesas-for-v4.8.

 drivers/clk/renesas/r8a7795-cpg-mssr.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/clk/renesas/r8a7795-cpg-mssr.c b/drivers/clk/renesas/r8a7795-cpg-mssr.c
index 8f76af6f454d0823..ad01b0b1bc5fa172 100644
--- a/drivers/clk/renesas/r8a7795-cpg-mssr.c
+++ b/drivers/clk/renesas/r8a7795-cpg-mssr.c
@@ -180,7 +180,7 @@ static const struct mssr_mod_clk r8a7795_mod_clks[] __initconst = {
 	DEF_MOD("du2",			 722,	R8A7795_CLK_S2D1),
 	DEF_MOD("du1",			 723,	R8A7795_CLK_S2D1),
 	DEF_MOD("du0",			 724,	R8A7795_CLK_S2D1),
-	DEF_MOD("lvds",			 727,	R8A7795_CLK_S2D1),
+	DEF_MOD("lvds",			 727,	R8A7795_CLK_S0D4),
 	DEF_MOD("hdmi1",		 728,	R8A7795_CLK_HDMI),
 	DEF_MOD("hdmi0",		 729,	R8A7795_CLK_HDMI),
 	DEF_MOD("vin7",			 804,	R8A7795_CLK_S2D1),
-- 
1.9.1

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] clk: renesas: r8a7795: Correct lvds clock parent
  2016-06-10  7:44 [PATCH] clk: renesas: r8a7795: Correct lvds clock parent Geert Uytterhoeven
@ 2016-06-10 13:22 ` Laurent Pinchart
  0 siblings, 0 replies; 2+ messages in thread
From: Laurent Pinchart @ 2016-06-10 13:22 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: Michael Turquette, Stephen Boyd, Laurent Pinchart,
	Kuninori Morimoto, linux-clk, linux-renesas-soc

Hi Geert,

Thank you for the patch.

On Friday 10 Jun 2016 09:44:33 Geert Uytterhoeven wrote:
> According to the latest information, the parent clock of the LVDS module
> clock is the S0D4 clock, not the S2D1 clock.
> 
> Note that this change has no influence on actual operation, as the
> rcar-du LVDS encoder driver doesn't use the parent clock's rate.
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>

Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

> ---
> Will queue in clk-renesas-for-v4.8.
> 
>  drivers/clk/renesas/r8a7795-cpg-mssr.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/clk/renesas/r8a7795-cpg-mssr.c
> b/drivers/clk/renesas/r8a7795-cpg-mssr.c index
> 8f76af6f454d0823..ad01b0b1bc5fa172 100644
> --- a/drivers/clk/renesas/r8a7795-cpg-mssr.c
> +++ b/drivers/clk/renesas/r8a7795-cpg-mssr.c
> @@ -180,7 +180,7 @@ static const struct mssr_mod_clk r8a7795_mod_clks[]
> __initconst = { DEF_MOD("du2",			 722,	R8A7795_CLK_S2D1),
>  	DEF_MOD("du1",			 723,	R8A7795_CLK_S2D1),
>  	DEF_MOD("du0",			 724,	R8A7795_CLK_S2D1),
> -	DEF_MOD("lvds",			 727,	R8A7795_CLK_S2D1),
> +	DEF_MOD("lvds",			 727,	R8A7795_CLK_S0D4),
>  	DEF_MOD("hdmi1",		 728,	R8A7795_CLK_HDMI),
>  	DEF_MOD("hdmi0",		 729,	R8A7795_CLK_HDMI),
>  	DEF_MOD("vin7",			 804,	R8A7795_CLK_S2D1),

-- 
Regards,

Laurent Pinchart

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2016-06-10 13:22 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-06-10  7:44 [PATCH] clk: renesas: r8a7795: Correct lvds clock parent Geert Uytterhoeven
2016-06-10 13:22 ` Laurent Pinchart

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.