Re: [PATCH 07/16] dt-bindings: clock: Add r8a779f0 CPG Core Clock Definitions

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Shimoda-san,

On Tue, Nov 16, 2021 at 8:42 AM Yoshihiro Shimoda
<yoshihiro.shimoda.uh@xxxxxxxxxxx> wrote:
> Add all Clock Pulse Generator Core Clock Outputs for the Renesas
> R-Car S4-8 (R8A779F0) SoC.
>
> Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@xxxxxxxxxxx>

Thanks for your patch!

> --- /dev/null
> +++ b/include/dt-bindings/clock/r8a779f0-cpg-mssr.h
> @@ -0,0 +1,65 @@
> +/* SPDX-License-Identifier: (GPL-2.0 or MIT) */
> +/*
> + * Copyright (C) 2021 Renesas Electronics Corp.
> + */
> +#ifndef __DT_BINDINGS_CLOCK_R8A779F0_CPG_MSSR_H__
> +#define __DT_BINDINGS_CLOCK_R8A779F0_CPG_MSSR_H__
> +
> +#include <dt-bindings/clock/renesas-cpg-mssr.h>
> +
> +/* r8a779f0 CPG Core Clocks */
> +
> +#define R8A779F0_CLK_ZX                        0
> +#define R8A779F0_CLK_ZS                        1
> +#define R8A779F0_CLK_ZT                        2
> +#define R8A779F0_CLK_ZTR               3
> +#define R8A779F0_CLK_S0D2              4
> +#define R8A779F0_CLK_S0D3              5
> +#define R8A779F0_CLK_S0D4              6
> +#define R8A779F0_CLK_S0D2_MM           7
> +#define R8A779F0_CLK_S0D3_MM           8
> +#define R8A779F0_CLK_S0D4_MM           9
> +#define R8A779F0_CLK_S0D2_RT           10
> +#define R8A779F0_CLK_S0D3_RT           11
> +#define R8A779F0_CLK_S0D4_RT           12
> +#define R8A779F0_CLK_S0D6_RT           13
> +#define R8A779F0_CLK_S0D3_PER          14
> +#define R8A779F0_CLK_S0D6_PER          15
> +#define R8A779F0_CLK_S0D12_PER         16
> +#define R8A779F0_CLK_S0D24_PER         17
> +#define R8A779F0_CLK_S0D2_HSC          18
> +#define R8A779F0_CLK_S0D3_HSC          19
> +#define R8A779F0_CLK_S0D4_HSC          20
> +#define R8A779F0_CLK_S0D6_HSC          21
> +#define R8A779F0_CLK_S0D12_HSC         22
> +#define R8A779F0_CLK_S0D2_CC           23
> +#define R8A779F0_CLK_CL                        24
> +#define R8A779F0_CLK_CL16M             25
> +#define R8A779F0_CLK_CL16M_MM          26
> +#define R8A779F0_CLK_CL16M_RT          27
> +#define R8A779F0_CLK_CL16M_PER         28
> +#define R8A779F0_CLK_CL16M_HSC         29
> +#define R8A779F0_CLK_Z0                        30
> +#define R8A779F0_CLK_Z1                        31
> +#define R8A779F0_CLK_ZB3               32
> +#define R8A779F0_CLK_ZB3D2             33
> +#define R8A779F0_CLK_ZB3D4             34
> +#define R8A779F0_CLK_SDSRC             35

I think we can leave out SDSRC, like on the other SoCs, as it's an
internal clock.

> +#define R8A779F0_CLK_SD0H              36
> +#define R8A779F0_CLK_SD0               37
> +#define R8A779F0_CLK_RPC               38
> +#define R8A779F0_CLK_RPCD2             39
> +#define R8A779F0_CLK_MSO               40
> +#define R8A779F0_CLK_SASYNCRT          41
> +#define R8A779F0_CLK_SASYNCPERD1       42
> +#define R8A779F0_CLK_SASYNCPERD2       43
> +#define R8A779F0_CLK_SASYNCPERD4       44
> +#define R8A779F0_CLK_DBGSOC_HSC                45
> +#define R8A779F0_CLK_RSW2              46
> +#define R8A779F0_CLK_OSC               47
> +#define R8A779F0_CLK_ZR                        48
> +#define R8A779F0_CLK_CPEX              49
> +#define R8A779F0_CLK_CBFUSA            50
> +#define R8A779F0_CLK_R                 51

The rest looks good to me.

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx

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



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux