[PATCH 2/2] ARM: dts: r8a7794: add CAN nodes to device tree

[Geert Uytterhoeven]

Hi Simon,

On Mon, Mar 14, 2016 at 3:29 AM, Simon Horman
<horms+renesas at verge.net.au> wrote:
> Add CAN nodes to r8a7794 device tree.
>
> Based on work by Sergei Shtylyov for the r8a7791 SoC.
>
> Cc: Sergei Shtylyov <sergei.shtylyov at cogentembedded.com>
> Signed-off-by: Simon Horman <horms+renesas at verge.net.au>

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

> ---
>  arch/arm/boot/dts/r8a7794.dtsi            | 22 ++++++++++++++++++++++
>  include/dt-bindings/clock/r8a7794-clock.h |  2 ++
>  2 files changed, 24 insertions(+)
>
> diff --git a/arch/arm/boot/dts/r8a7794.dtsi b/arch/arm/boot/dts/r8a7794.dtsi
> index c8742a599330..ab755b754a9e 100644
> --- a/arch/arm/boot/dts/r8a7794.dtsi
> +++ b/arch/arm/boot/dts/r8a7794.dtsi
> @@ -830,6 +830,28 @@
>                 };
>         };
>
> +       can0: can at e6e80000 {
> +               compatible = "renesas,can-r8a7794", "renesas,rcar-gen2-can";
> +               reg = <0 0xe6e80000 0 0x1000>;
> +               interrupts = <GIC_SPI 186 IRQ_TYPE_LEVEL_HIGH>;
> +               clocks = <&mstp9_clks R8A7794_CLK_RCAN0>,

I don't think mstp9_clks already defines the can module clocks?

> +                        <&cpg_clocks R8A7794_CLK_RCAN>, <&can_clk>;
> +               clock-names = "clkp1", "clkp2", "can_clk";
> +               power-domains = <&cpg_clocks>;
> +               status = "disabled";
> +       };
> +
> +       can1: can at e6e88000 {
> +               compatible = "renesas,can-r8a7794", "renesas,rcar-gen2-can";
> +               reg = <0 0xe6e88000 0 0x1000>;
> +               interrupts = <GIC_SPI 187 IRQ_TYPE_LEVEL_HIGH>;
> +               clocks = <&mstp9_clks R8A7794_CLK_RCAN1>,

Likewise.

> +                        <&cpg_clocks R8A7794_CLK_RCAN>, <&can_clk>;
> +               clock-names = "clkp1", "clkp2", "can_clk";
> +               power-domains = <&cpg_clocks>;
> +               status = "disabled";
> +       };
> +
>         clocks {
>                 #address-cells = <2>;
>                 #size-cells = <2>;

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at 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