From: Rob Herring <robh@kernel.org>
To: Chris Brandt <chris.brandt@renesas.com>
Cc: Simon Horman <horms@verge.net.au>,
Mark Rutland <mark.rutland@arm.com>,
devicetree@vger.kernel.org, linux-renesas-soc@vger.kernel.org
Subject: Re: [PATCH v3 1/2] ARM: dts: r7s9210: Initial SoC device tree
Date: Tue, 18 Dec 2018 09:05:51 -0600 [thread overview]
Message-ID: <20181218150551.GA15519@bogus> (raw)
In-Reply-To: <20181217144324.99422-2-chris.brandt@renesas.com>
On Mon, Dec 17, 2018 at 09:43:23AM -0500, Chris Brandt wrote:
> Basic support for the RZ/A2 (R7S9210) SoC.
>
> Signed-off-by: Chris Brandt <chris.brandt@renesas.com>
> ---
> v2:
> * Fixed cpg node name to match reg address
> * Removed the clocks subnode
> * SCIF register range 18 to 0x18
> * Removed 'reset-cells' from cpg because resets not supported (yet?)
> * Sorted nodes by address (per group of device
> ---
> arch/arm/boot/dts/r7s9210.dtsi | 204 +++++++++++++++++++++++++++++++++++++++++
> 1 file changed, 204 insertions(+)
> create mode 100644 arch/arm/boot/dts/r7s9210.dtsi
>
> diff --git a/arch/arm/boot/dts/r7s9210.dtsi b/arch/arm/boot/dts/r7s9210.dtsi
> new file mode 100644
> index 000000000000..2b589226895e
> --- /dev/null
> +++ b/arch/arm/boot/dts/r7s9210.dtsi
> @@ -0,0 +1,204 @@
> +// SPDX-License-Identifier: GPL-2.0
> +/*
> + * Device Tree Source for the R7S9210 SoC
> + *
> + * Copyright (C) 2018 Renesas Electronics Corporation
> + *
> + */
> +
> +#include <dt-bindings/interrupt-controller/arm-gic.h>
> +#include <dt-bindings/clock/r7s9210-cpg-mssr.h>
> +
> +/ {
> + compatible = "renesas,r7s9210";
> + interrupt-parent = <&gic>;
> + #address-cells = <1>;
> + #size-cells = <1>;
> +
> + bsid: chipid@fcfe8004 {
Put all the mmio peripherals under a simple-bus node or nodes.
> + compatible = "renesas,bsid";
> + reg = <0xfcfe8004 4>;
> + };
> +
> + /* External clocks */
> + extal_clk: extal {
> + #clock-cells = <0>;
> + compatible = "fixed-clock";
> + /* Value must be set by board */
> + clock-frequency = <0>;
> + };
> +
> + rtc_x1_clk: rtc_x1 {
> + #clock-cells = <0>;
> + compatible = "fixed-clock";
> + /* If clk present, value (32678) must be set by board */
> + clock-frequency = <0>;
> + };
> +
> + cpus {
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> + cpu@0 {
> + device_type = "cpu";
> + compatible = "arm,cortex-a9";
> + reg = <0>;
> + clock-frequency = <528000000>;
> + next-level-cache = <&L2>;
> + };
> + };
> +
> + L2: cache-controller@1f003000 {
> + compatible = "arm,pl310-cache";
> + reg = <0x1f003000 0x1000>;
> + interrupts = <GIC_SPI 12 IRQ_TYPE_LEVEL_HIGH>;
> + arm,early-bresp-disable;
> + arm,full-line-zero-disable;
> + cache-unified;
> + cache-level = <2>;
> + };
> +
> + gic: interrupt-controller@e8221000 {
> + compatible = "arm,gic-400";
Kind of strange that a single core A9 uses an external GIC rather than
the built-in one.
> + #interrupt-cells = <3>;
> + #address-cells = <0>;
> + interrupt-controller;
> + reg = <0xe8221000 0x1000>,
> + <0xe8222000 0x1000>;
> + };
> +
> + cpg: clock-controller@fcfe0010 {
> + compatible = "renesas,r7s9210-cpg-mssr";
> + reg = <0xfcfe0010 0x455>;
> + clocks = <&extal_clk>;
> + clock-names = "extal";
> + #clock-cells = <2>;
> + #power-domain-cells = <0>;
> + };
> +
> + wdt: watchdog@fcfe7000 {
> + compatible = "renesas,r7s9210-wdt", "renesas,rza-wdt";
> + reg = <0xfcfe7000 0x26>;
> + interrupts = <GIC_SPI 61 IRQ_TYPE_LEVEL_HIGH>;
> + clocks = <&cpg CPG_CORE R7S9210_CLK_P0>;
> + };
> +
> + pinctrl: pin-controller@fcffe000 {
> + compatible = "renesas,r7s9210-pinctrl";
> + reg = <0xfcffe000 0x1000>;
> +
> + gpio-controller;
> + #gpio-cells = <2>;
> + gpio-ranges = <&pinctrl 0 0 176>;
> + };
> +
> + scif0: serial@e8007000 {
> + compatible = "renesas,scif-r7s9210";
> + reg = <0xe8007000 0x18>;
> + interrupts = <GIC_SPI 265 IRQ_TYPE_LEVEL_HIGH>,
> + <GIC_SPI 266 IRQ_TYPE_LEVEL_HIGH>,
> + <GIC_SPI 267 IRQ_TYPE_LEVEL_HIGH>,
> + <GIC_SPI 265 IRQ_TYPE_LEVEL_HIGH>,
> + <GIC_SPI 268 IRQ_TYPE_LEVEL_HIGH>,
> + <GIC_SPI 268 IRQ_TYPE_LEVEL_HIGH>;
> + interrupt-names = "eri", "rxi", "txi", "bri", "dri", "tei";
> + clocks = <&cpg CPG_MOD 47>;
> + clock-names = "fck";
> + power-domains = <&cpg>;
> + status = "disabled";
> + };
> +
> + scif1: serial@e8007800 {
> + compatible = "renesas,scif-r7s9210";
> + reg = <0xe8007800 0x18>;
> + interrupts = <GIC_SPI 271 IRQ_TYPE_LEVEL_HIGH>,
> + <GIC_SPI 272 IRQ_TYPE_LEVEL_HIGH>,
> + <GIC_SPI 273 IRQ_TYPE_LEVEL_HIGH>,
> + <GIC_SPI 271 IRQ_TYPE_LEVEL_HIGH>,
> + <GIC_SPI 274 IRQ_TYPE_LEVEL_HIGH>,
> + <GIC_SPI 274 IRQ_TYPE_LEVEL_HIGH>;
> + interrupt-names = "eri", "rxi", "txi", "bri", "dri", "tei";
> + clocks = <&cpg CPG_MOD 46>;
> + clock-names = "fck";
> + power-domains = <&cpg>;
> + status = "disabled";
> + };
> +
> + scif2: serial@e8008000 {
> + compatible = "renesas,scif-r7s9210";
> + reg = <0xe8008000 0x18>;
> + interrupts = <GIC_SPI 277 IRQ_TYPE_LEVEL_HIGH>,
> + <GIC_SPI 278 IRQ_TYPE_LEVEL_HIGH>,
> + <GIC_SPI 279 IRQ_TYPE_LEVEL_HIGH>,
> + <GIC_SPI 277 IRQ_TYPE_LEVEL_HIGH>,
> + <GIC_SPI 280 IRQ_TYPE_LEVEL_HIGH>,
> + <GIC_SPI 280 IRQ_TYPE_LEVEL_HIGH>;
> + interrupt-names = "eri", "rxi", "txi", "bri", "dri", "tei";
> + clocks = <&cpg CPG_MOD 45>;
> + clock-names = "fck";
> + power-domains = <&cpg>;
> + status = "disabled";
> + };
> +
> + scif3: serial@e8008800 {
> + compatible = "renesas,scif-r7s9210";
> + reg = <0xe8008800 0x18>;
> + interrupts = <GIC_SPI 283 IRQ_TYPE_LEVEL_HIGH>,
> + <GIC_SPI 284 IRQ_TYPE_LEVEL_HIGH>,
> + <GIC_SPI 285 IRQ_TYPE_LEVEL_HIGH>,
> + <GIC_SPI 283 IRQ_TYPE_LEVEL_HIGH>,
> + <GIC_SPI 286 IRQ_TYPE_LEVEL_HIGH>,
> + <GIC_SPI 286 IRQ_TYPE_LEVEL_HIGH>;
> + interrupt-names = "eri", "rxi", "txi", "bri", "dri", "tei";
> + clocks = <&cpg CPG_MOD 44>;
> + clock-names = "fck";
> + power-domains = <&cpg>;
> + status = "disabled";
> + };
> +
> + scif4: serial@e8009000 {
> + compatible = "renesas,scif-r7s9210";
> + reg = <0xe8009000 0x18>;
> + interrupts = <GIC_SPI 289 IRQ_TYPE_LEVEL_HIGH>,
> + <GIC_SPI 290 IRQ_TYPE_LEVEL_HIGH>,
> + <GIC_SPI 291 IRQ_TYPE_LEVEL_HIGH>,
> + <GIC_SPI 289 IRQ_TYPE_LEVEL_HIGH>,
> + <GIC_SPI 292 IRQ_TYPE_LEVEL_HIGH>,
> + <GIC_SPI 292 IRQ_TYPE_LEVEL_HIGH>;
> + interrupt-names = "eri", "rxi", "txi", "bri", "dri", "tei";
> + clocks = <&cpg CPG_MOD 43>;
> + clock-names = "fck";
> + power-domains = <&cpg>;
> + status = "disabled";
> + };
> +
> + ostm0: timer@e803b000 {
> + compatible = "renesas,r7s9210-ostm", "renesas,ostm";
> + reg = <0xe803b000 0x30>;
> + interrupts = <GIC_SPI 56 IRQ_TYPE_EDGE_RISING>;
> + clocks = <&cpg CPG_MOD 36>;
> + clock-names = "ostm0";
> + power-domains = <&cpg>;
> + status = "disabled";
> + };
> +
> + ostm1: timer@e803c000 {
> + compatible = "renesas,r7s9210-ostm", "renesas,ostm";
> + reg = <0xe803c000 0x30>;
> + interrupts = <GIC_SPI 57 IRQ_TYPE_EDGE_RISING>;
> + clocks = <&cpg CPG_MOD 35>;
> + clock-names = "ostm1";
> + power-domains = <&cpg>;
> + status = "disabled";
> + };
> +
> + ostm2: timer@e803d000 {
> + compatible = "renesas,r7s9210-ostm", "renesas,ostm";
> + reg = <0xe803d000 0x30>;
> + interrupts = <GIC_SPI 58 IRQ_TYPE_EDGE_RISING>;
> + clocks = <&cpg CPG_MOD 34>;
> + clock-names = "ostm2";
> + power-domains = <&cpg>;
> + status = "disabled";
> + };
> +};
> --
> 2.16.1
>
next prev parent reply other threads:[~2018-12-18 15:05 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-12-17 14:43 [PATCH v3 0/2] Add Initial Device Tree for RZ/A2 Chris Brandt
2018-12-17 14:43 ` [PATCH v3 1/2] ARM: dts: r7s9210: Initial SoC device tree Chris Brandt
2018-12-18 15:05 ` Rob Herring [this message]
2018-12-18 15:40 ` Chris Brandt
2018-12-18 15:44 ` Rob Herring
2018-12-19 14:05 ` Rob Herring
2018-12-19 14:22 ` Geert Uytterhoeven
2018-12-17 14:43 ` [PATCH v3 2/2] ARM: dts: r7s9210-rza2mevb: Add support for RZ/A2M EVB Chris Brandt
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20181218150551.GA15519@bogus \
--to=robh@kernel.org \
--cc=chris.brandt@renesas.com \
--cc=devicetree@vger.kernel.org \
--cc=horms@verge.net.au \
--cc=linux-renesas-soc@vger.kernel.org \
--cc=mark.rutland@arm.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).