* [PATCH] clk: renesas: Add r8a774b1 CPG Core Clock Definitions
@ 2019-09-05 6:52 Biju Das
2019-09-09 17:25 ` Geert Uytterhoeven
0 siblings, 1 reply; 2+ messages in thread
From: Biju Das @ 2019-09-05 6:52 UTC (permalink / raw)
To: Rob Herring, Mark Rutland
Cc: Biju Das, devicetree, Simon Horman, Geert Uytterhoeven,
Chris Paterson, Fabrizio Castro, linux-renesas-soc
Add all RZ/G2N Clock Pulse Generator Core Clock Outputs, as listed in
Table 8.2d ("List of Clocks [RZ/G2N]") of the RZ/G2N Hardware User's
Manual.
Signed-off-by: Biju Das <biju.das@bp.renesas.com>
---
include/dt-bindings/clock/r8a774b1-cpg-mssr.h | 57 +++++++++++++++++++++++++++
1 file changed, 57 insertions(+)
create mode 100644 include/dt-bindings/clock/r8a774b1-cpg-mssr.h
diff --git a/include/dt-bindings/clock/r8a774b1-cpg-mssr.h b/include/dt-bindings/clock/r8a774b1-cpg-mssr.h
new file mode 100644
index 0000000..1355451
--- /dev/null
+++ b/include/dt-bindings/clock/r8a774b1-cpg-mssr.h
@@ -0,0 +1,57 @@
+/* SPDX-License-Identifier: GPL-2.0
+ *
+ * Copyright (C) 2019 Renesas Electronics Corp.
+ */
+#ifndef __DT_BINDINGS_CLOCK_R8A774B1_CPG_MSSR_H__
+#define __DT_BINDINGS_CLOCK_R8A774B1_CPG_MSSR_H__
+
+#include <dt-bindings/clock/renesas-cpg-mssr.h>
+
+/* r8a774b1 CPG Core Clocks */
+#define R8A774B1_CLK_Z 0
+#define R8A774B1_CLK_ZG 1
+#define R8A774B1_CLK_ZTR 2
+#define R8A774B1_CLK_ZTRD2 3
+#define R8A774B1_CLK_ZT 4
+#define R8A774B1_CLK_ZX 5
+#define R8A774B1_CLK_S0D1 6
+#define R8A774B1_CLK_S0D2 7
+#define R8A774B1_CLK_S0D3 8
+#define R8A774B1_CLK_S0D4 9
+#define R8A774B1_CLK_S0D6 10
+#define R8A774B1_CLK_S0D8 11
+#define R8A774B1_CLK_S0D12 12
+#define R8A774B1_CLK_S1D2 13
+#define R8A774B1_CLK_S1D4 14
+#define R8A774B1_CLK_S2D1 15
+#define R8A774B1_CLK_S2D2 16
+#define R8A774B1_CLK_S2D4 17
+#define R8A774B1_CLK_S3D1 18
+#define R8A774B1_CLK_S3D2 19
+#define R8A774B1_CLK_S3D4 20
+#define R8A774B1_CLK_LB 21
+#define R8A774B1_CLK_CL 22
+#define R8A774B1_CLK_ZB3 23
+#define R8A774B1_CLK_ZB3D2 24
+#define R8A774B1_CLK_CR 25
+#define R8A774B1_CLK_DDR 26
+#define R8A774B1_CLK_SD0H 27
+#define R8A774B1_CLK_SD0 28
+#define R8A774B1_CLK_SD1H 29
+#define R8A774B1_CLK_SD1 30
+#define R8A774B1_CLK_SD2H 31
+#define R8A774B1_CLK_SD2 32
+#define R8A774B1_CLK_SD3H 33
+#define R8A774B1_CLK_SD3 34
+#define R8A774B1_CLK_RPC 35
+#define R8A774B1_CLK_RPCD2 36
+#define R8A774B1_CLK_MSO 37
+#define R8A774B1_CLK_HDMI 38
+#define R8A774B1_CLK_CSI0 39
+#define R8A774B1_CLK_CP 40
+#define R8A774B1_CLK_CPEX 41
+#define R8A774B1_CLK_R 42
+#define R8A774B1_CLK_OSC 43
+#define R8A774B1_CLK_CANFD 44
+
+#endif /* __DT_BINDINGS_CLOCK_R8A774B1_CPG_MSSR_H__ */
--
2.7.4
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] clk: renesas: Add r8a774b1 CPG Core Clock Definitions
2019-09-05 6:52 [PATCH] clk: renesas: Add r8a774b1 CPG Core Clock Definitions Biju Das
@ 2019-09-09 17:25 ` Geert Uytterhoeven
0 siblings, 0 replies; 2+ messages in thread
From: Geert Uytterhoeven @ 2019-09-09 17:25 UTC (permalink / raw)
To: Biju Das
Cc: Rob Herring, Mark Rutland,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
Simon Horman, Chris Paterson, Fabrizio Castro, Linux-Renesas
Hi Biju,
On Thu, Sep 5, 2019 at 9:00 AM Biju Das <biju.das@bp.renesas.com> wrote:
> Add all RZ/G2N Clock Pulse Generator Core Clock Outputs, as listed in
> Table 8.2d ("List of Clocks [RZ/G2N]") of the RZ/G2N Hardware User's
> Manual.
>
> Signed-off-by: Biju Das <biju.das@bp.renesas.com>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
One comment below...
> --- /dev/null
> +++ b/include/dt-bindings/clock/r8a774b1-cpg-mssr.h
> @@ -0,0 +1,57 @@
> +/* SPDX-License-Identifier: GPL-2.0
> + *
> + * Copyright (C) 2019 Renesas Electronics Corp.
> + */
> +#ifndef __DT_BINDINGS_CLOCK_R8A774B1_CPG_MSSR_H__
> +#define __DT_BINDINGS_CLOCK_R8A774B1_CPG_MSSR_H__
> +
> +#include <dt-bindings/clock/renesas-cpg-mssr.h>
> +
> +/* r8a774b1 CPG Core Clocks */
> +#define R8A774B1_CLK_Z 0
> +#define R8A774B1_CLK_ZG 1
> +#define R8A774B1_CLK_ZTR 2
> +#define R8A774B1_CLK_ZTRD2 3
> +#define R8A774B1_CLK_ZT 4
> +#define R8A774B1_CLK_ZX 5
> +#define R8A774B1_CLK_S0D1 6
> +#define R8A774B1_CLK_S0D2 7
> +#define R8A774B1_CLK_S0D3 8
> +#define R8A774B1_CLK_S0D4 9
> +#define R8A774B1_CLK_S0D6 10
> +#define R8A774B1_CLK_S0D8 11
> +#define R8A774B1_CLK_S0D12 12
> +#define R8A774B1_CLK_S1D2 13
> +#define R8A774B1_CLK_S1D4 14
> +#define R8A774B1_CLK_S2D1 15
> +#define R8A774B1_CLK_S2D2 16
> +#define R8A774B1_CLK_S2D4 17
> +#define R8A774B1_CLK_S3D1 18
> +#define R8A774B1_CLK_S3D2 19
> +#define R8A774B1_CLK_S3D4 20
> +#define R8A774B1_CLK_LB 21
> +#define R8A774B1_CLK_CL 22
> +#define R8A774B1_CLK_ZB3 23
> +#define R8A774B1_CLK_ZB3D2 24
> +#define R8A774B1_CLK_CR 25
> +#define R8A774B1_CLK_DDR 26
> +#define R8A774B1_CLK_SD0H 27
> +#define R8A774B1_CLK_SD0 28
> +#define R8A774B1_CLK_SD1H 29
> +#define R8A774B1_CLK_SD1 30
> +#define R8A774B1_CLK_SD2H 31
> +#define R8A774B1_CLK_SD2 32
> +#define R8A774B1_CLK_SD3H 33
> +#define R8A774B1_CLK_SD3 34
> +#define R8A774B1_CLK_RPC 35
> +#define R8A774B1_CLK_RPCD2 36
> +#define R8A774B1_CLK_MSO 37
> +#define R8A774B1_CLK_HDMI 38
> +#define R8A774B1_CLK_CSI0 39
> +#define R8A774B1_CLK_CP 40
> +#define R8A774B1_CLK_CPEX 41
> +#define R8A774B1_CLK_R 42
> +#define R8A774B1_CLK_OSC 43
> +#define R8A774B1_CLK_CANFD 44
The only thing I can comment on is the location of the CANFD clock.
For RZ/G2M and RZ/G2E, the clock was added to the end of the list,
as it wasn't listed in the Hardware User's Manual.
In the mean time, Rev. 0.80 of the Hardware User's Manual was released,
which does include the CANFD clock.
If no one objects against this location, for consistency with other
RZ/G2 SoCs, i.e. will queue it in renesas-devel for v5.5, on a branch to
be shared by the clock driver and the DTS user.
> +
> +#endif /* __DT_BINDINGS_CLOCK_R8A774B1_CPG_MSSR_H__ */
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] 2+ messages in thread
end of thread, other threads:[~2019-09-09 17:25 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-09-05 6:52 [PATCH] clk: renesas: Add r8a774b1 CPG Core Clock Definitions Biju Das
2019-09-09 17:25 ` Geert Uytterhoeven
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).