Hi Simon,
On Mon, Mar 14, 2016 at 3:29 AM, Simon Horman
<horms+renesas@xxxxxxxxxxxx> wrote:
> Add CAN nodes to r8a7794 device tree.
>
> Based on work by Sergei Shtylyov for the r8a7791 SoC.
>
> Cc: Sergei Shtylyov <sergei.shtylyov@xxxxxxxxxxxxxxxxxx>
> Signed-off-by: Simon Horman <horms+renesas@xxxxxxxxxxxx>
Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>
> ---
> 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@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@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@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
