* [PATCH][RFC] arm64: renesas: Add EXTAL configuration to Salvator-X
@ 2015-09-08 4:49 Magnus Damm
2015-09-08 6:58 ` Geert Uytterhoeven
` (4 more replies)
0 siblings, 5 replies; 6+ messages in thread
From: Magnus Damm @ 2015-09-08 4:49 UTC (permalink / raw)
To: linux-sh
From: Magnus Damm <damm+renesas@opensource.se>
Configure EXTAL to 16.666 MHz on Salvator-X.
Signed-off-by: Magnus Damm <damm+renesas@opensource.se>
---
This sets the EXTAL frequency for the V9 integration series:
[PATCH v9 00/07][RFC] Renesas R-Car H3 (r8a7795) integration patches V9
To be used with the V7 CPG:
[PATCH v7 00/05] Renesas R-Car Gen3 CPG support V7
arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts | 4 ++++
1 file changed, 4 insertions(+)
--- 0007/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
+++ work/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts 2015-09-08 13:44:53.682366518 +0900
@@ -32,6 +32,10 @@
};
};
+&extal_clk {
+ clock-frequency = <16666666>;
+};
+
&pfc {
scif1_pins: scif1 {
renesas,groups = "scif1_data_a", "scif1_ctrl";
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH][RFC] arm64: renesas: Add EXTAL configuration to Salvator-X
2015-09-08 4:49 [PATCH][RFC] arm64: renesas: Add EXTAL configuration to Salvator-X Magnus Damm
@ 2015-09-08 6:58 ` Geert Uytterhoeven
2015-09-08 6:59 ` Laurent Pinchart
` (3 subsequent siblings)
4 siblings, 0 replies; 6+ messages in thread
From: Geert Uytterhoeven @ 2015-09-08 6:58 UTC (permalink / raw)
To: linux-sh
On Tue, Sep 8, 2015 at 6:49 AM, Magnus Damm <magnus.damm@gmail.com> wrote:
> From: Magnus Damm <damm+renesas@opensource.se>
>
> Configure EXTAL to 16.666 MHz on Salvator-X.
>
> Signed-off-by: Magnus Damm <damm+renesas@opensource.se>
Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>
Cfr. "[PATCH 3/3] arm64: renesas: salvator-x: Add missing extal clock override"
http://www.spinics.net/lists/linux-sh/msg44416.html
--
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
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH][RFC] arm64: renesas: Add EXTAL configuration to Salvator-X
2015-09-08 4:49 [PATCH][RFC] arm64: renesas: Add EXTAL configuration to Salvator-X Magnus Damm
2015-09-08 6:58 ` Geert Uytterhoeven
@ 2015-09-08 6:59 ` Laurent Pinchart
2015-09-08 8:23 ` Magnus Damm
` (2 subsequent siblings)
4 siblings, 0 replies; 6+ messages in thread
From: Laurent Pinchart @ 2015-09-08 6:59 UTC (permalink / raw)
To: linux-sh
Hi Magnus,
Thank you for the patch.
On Tuesday 08 September 2015 13:49:57 Magnus Damm wrote:
> From: Magnus Damm <damm+renesas@opensource.se>
>
> Configure EXTAL to 16.666 MHz on Salvator-X.
>
> Signed-off-by: Magnus Damm <damm+renesas@opensource.se>
This looks quite straightforward and good to me. I don't think it requires the
RFC status.
Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> ---
>
> This sets the EXTAL frequency for the V9 integration series:
> [PATCH v9 00/07][RFC] Renesas R-Car H3 (r8a7795) integration patches V9
>
> To be used with the V7 CPG:
> [PATCH v7 00/05] Renesas R-Car Gen3 CPG support V7
>
> arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts | 4 ++++
> 1 file changed, 4 insertions(+)
>
> --- 0007/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
> +++ work/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts 2015-09-08
> 13:44:53.682366518 +0900 @@ -32,6 +32,10 @@
> };
> };
>
> +&extal_clk {
> + clock-frequency = <16666666>;
> +};
> +
> &pfc {
> scif1_pins: scif1 {
> renesas,groups = "scif1_data_a", "scif1_ctrl";
--
Regards,
Laurent Pinchart
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH][RFC] arm64: renesas: Add EXTAL configuration to Salvator-X
2015-09-08 4:49 [PATCH][RFC] arm64: renesas: Add EXTAL configuration to Salvator-X Magnus Damm
2015-09-08 6:58 ` Geert Uytterhoeven
2015-09-08 6:59 ` Laurent Pinchart
@ 2015-09-08 8:23 ` Magnus Damm
2015-09-08 9:07 ` Khiem Nguyen
2015-09-09 0:31 ` Simon Horman
4 siblings, 0 replies; 6+ messages in thread
From: Magnus Damm @ 2015-09-08 8:23 UTC (permalink / raw)
To: linux-sh
Hi Geert,
On Tue, Sep 8, 2015 at 3:58 PM, Geert Uytterhoeven <geert@linux-m68k.org> wrote:
> On Tue, Sep 8, 2015 at 6:49 AM, Magnus Damm <magnus.damm@gmail.com> wrote:
>> From: Magnus Damm <damm+renesas@opensource.se>
>>
>> Configure EXTAL to 16.666 MHz on Salvator-X.
>>
>> Signed-off-by: Magnus Damm <damm+renesas@opensource.se>
>
> Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>
>
> Cfr. "[PATCH 3/3] arm64: renesas: salvator-x: Add missing extal clock override"
> http://www.spinics.net/lists/linux-sh/msg44416.html
Thanks, yeah I suspected this was done already. =)
To make things slightly more complicated it seems that some boards use
16.6666 MHz and others 16.0 Mhz...
Cheers,
/ magnus
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH][RFC] arm64: renesas: Add EXTAL configuration to Salvator-X
2015-09-08 4:49 [PATCH][RFC] arm64: renesas: Add EXTAL configuration to Salvator-X Magnus Damm
` (2 preceding siblings ...)
2015-09-08 8:23 ` Magnus Damm
@ 2015-09-08 9:07 ` Khiem Nguyen
2015-09-09 0:31 ` Simon Horman
4 siblings, 0 replies; 6+ messages in thread
From: Khiem Nguyen @ 2015-09-08 9:07 UTC (permalink / raw)
To: linux-sh
Hi Magnus,
> To make things slightly more complicated it seems that some boards use
> 16.6666 MHz and others 16.0 Mhz...
AFAIK, the 16.0 Mhz is only available in some initial Salvator-X boards.
So, 16.6666 MHz should be standard value, I guess.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH][RFC] arm64: renesas: Add EXTAL configuration to Salvator-X
2015-09-08 4:49 [PATCH][RFC] arm64: renesas: Add EXTAL configuration to Salvator-X Magnus Damm
` (3 preceding siblings ...)
2015-09-08 9:07 ` Khiem Nguyen
@ 2015-09-09 0:31 ` Simon Horman
4 siblings, 0 replies; 6+ messages in thread
From: Simon Horman @ 2015-09-09 0:31 UTC (permalink / raw)
To: linux-sh
On Tue, Sep 08, 2015 at 01:49:57PM +0900, Magnus Damm wrote:
> From: Magnus Damm <damm+renesas@opensource.se>
>
> Configure EXTAL to 16.666 MHz on Salvator-X.
>
> Signed-off-by: Magnus Damm <damm+renesas@opensource.se>
> ---
>
> This sets the EXTAL frequency for the V9 integration series:
> [PATCH v9 00/07][RFC] Renesas R-Car H3 (r8a7795) integration patches V9
>
> To be used with the V7 CPG:
> [PATCH v7 00/05] Renesas R-Car Gen3 CPG support V7
Thanks,
I have pushed this patch to the renesas tree in a new topic branch,
topic/arm64-rcar-gen3-v9.plus, which is based on topic/arm64-rcar-gen3-v9.
I am assuming you will include this in/squash this into the next posting of
"Renesas R-Car H3 (r8a7795) integration patches".
Disclaimer: To be quite clear none of these patches are queued up for
mainline yet and as such they are not included in the next or devel
branches of the renesas tree.
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2015-09-09 0:31 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-09-08 4:49 [PATCH][RFC] arm64: renesas: Add EXTAL configuration to Salvator-X Magnus Damm
2015-09-08 6:58 ` Geert Uytterhoeven
2015-09-08 6:59 ` Laurent Pinchart
2015-09-08 8:23 ` Magnus Damm
2015-09-08 9:07 ` Khiem Nguyen
2015-09-09 0:31 ` Simon Horman
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.