From: niklas.soderlund@ragnatech.se (Niklas Söderlund)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] arm64: dts: renesas: r8a7795: Add missing SYS-DMAC2 dmas
Date: Fri, 9 Mar 2018 23:03:33 +0100 [thread overview]
Message-ID: <20180309220333.GM2205@bigcity.dyn.berto.se> (raw)
In-Reply-To: <1520518205-23351-1-git-send-email-geert+renesas@glider.be>
Hi Geert,
Thanks for your patch.
On 2018-03-08 15:10:05 +0100, Geert Uytterhoeven wrote:
> On R-Car H3, on-chip peripheral modules that can make use of DMA are
> wired to either SYS-DMAC0 only, or to both SYS-DMAC1 and SYS-DMAC2.
>
> Add the missing DMA properties pointing to SYS-DMAC2 for HSCIF[0-2],
> SCIF[015], and I2C[0-2]. These were initially left out because early
> firmware versions prohibited using SYS-DMAC2. This restriction has been
> lifted in IPL and Secure Monitor Rev1.0.6 (released on Feb 25, 2016).
>
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Reviewed-by: Niklas S?derlund <niklas.soderlund+renesas@ragnatech.se>
> ---
> arch/arm64/boot/dts/renesas/r8a7795.dtsi | 45 +++++++++++++++++++-------------
> 1 file changed, 27 insertions(+), 18 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/renesas/r8a7795.dtsi b/arch/arm64/boot/dts/renesas/r8a7795.dtsi
> index 0648fe451d097f4e..849c2c7ff101c202 100644
> --- a/arch/arm64/boot/dts/renesas/r8a7795.dtsi
> +++ b/arch/arm64/boot/dts/renesas/r8a7795.dtsi
> @@ -1076,8 +1076,9 @@
> <&cpg CPG_CORE R8A7795_CLK_S3D1>,
> <&scif_clk>;
> clock-names = "fck", "brg_int", "scif_clk";
> - dmas = <&dmac1 0x31>, <&dmac1 0x30>;
> - dma-names = "tx", "rx";
> + dmas = <&dmac1 0x31>, <&dmac1 0x30>,
> + <&dmac2 0x31>, <&dmac2 0x30>;
> + dma-names = "tx", "rx", "tx", "rx";
> power-domains = <&sysc R8A7795_PD_ALWAYS_ON>;
> resets = <&cpg 520>;
> status = "disabled";
> @@ -1093,8 +1094,9 @@
> <&cpg CPG_CORE R8A7795_CLK_S3D1>,
> <&scif_clk>;
> clock-names = "fck", "brg_int", "scif_clk";
> - dmas = <&dmac1 0x33>, <&dmac1 0x32>;
> - dma-names = "tx", "rx";
> + dmas = <&dmac1 0x33>, <&dmac1 0x32>,
> + <&dmac2 0x33>, <&dmac2 0x32>;
> + dma-names = "tx", "rx", "tx", "rx";
> power-domains = <&sysc R8A7795_PD_ALWAYS_ON>;
> resets = <&cpg 519>;
> status = "disabled";
> @@ -1110,8 +1112,9 @@
> <&cpg CPG_CORE R8A7795_CLK_S3D1>,
> <&scif_clk>;
> clock-names = "fck", "brg_int", "scif_clk";
> - dmas = <&dmac1 0x35>, <&dmac1 0x34>;
> - dma-names = "tx", "rx";
> + dmas = <&dmac1 0x35>, <&dmac1 0x34>,
> + <&dmac2 0x35>, <&dmac2 0x34>;
> + dma-names = "tx", "rx", "tx", "rx";
> power-domains = <&sysc R8A7795_PD_ALWAYS_ON>;
> resets = <&cpg 518>;
> status = "disabled";
> @@ -1222,8 +1225,9 @@
> <&cpg CPG_CORE R8A7795_CLK_S3D1>,
> <&scif_clk>;
> clock-names = "fck", "brg_int", "scif_clk";
> - dmas = <&dmac1 0x51>, <&dmac1 0x50>;
> - dma-names = "tx", "rx";
> + dmas = <&dmac1 0x51>, <&dmac1 0x50>,
> + <&dmac2 0x51>, <&dmac2 0x50>;
> + dma-names = "tx", "rx", "tx", "rx";
> power-domains = <&sysc R8A7795_PD_ALWAYS_ON>;
> resets = <&cpg 207>;
> status = "disabled";
> @@ -1238,8 +1242,9 @@
> <&cpg CPG_CORE R8A7795_CLK_S3D1>,
> <&scif_clk>;
> clock-names = "fck", "brg_int", "scif_clk";
> - dmas = <&dmac1 0x53>, <&dmac1 0x52>;
> - dma-names = "tx", "rx";
> + dmas = <&dmac1 0x53>, <&dmac1 0x52>,
> + <&dmac2 0x53>, <&dmac2 0x52>;
> + dma-names = "tx", "rx", "tx", "rx";
> power-domains = <&sysc R8A7795_PD_ALWAYS_ON>;
> resets = <&cpg 206>;
> status = "disabled";
> @@ -1302,8 +1307,9 @@
> <&cpg CPG_CORE R8A7795_CLK_S3D1>,
> <&scif_clk>;
> clock-names = "fck", "brg_int", "scif_clk";
> - dmas = <&dmac1 0x5b>, <&dmac1 0x5a>;
> - dma-names = "tx", "rx";
> + dmas = <&dmac1 0x5b>, <&dmac1 0x5a>,
> + <&dmac2 0x5b>, <&dmac2 0x5a>;
> + dma-names = "tx", "rx", "tx", "rx";
> power-domains = <&sysc R8A7795_PD_ALWAYS_ON>;
> resets = <&cpg 202>;
> status = "disabled";
> @@ -1335,8 +1341,9 @@
> clocks = <&cpg CPG_MOD 931>;
> power-domains = <&sysc R8A7795_PD_ALWAYS_ON>;
> resets = <&cpg 931>;
> - dmas = <&dmac1 0x91>, <&dmac1 0x90>;
> - dma-names = "tx", "rx";
> + dmas = <&dmac1 0x91>, <&dmac1 0x90>,
> + <&dmac2 0x91>, <&dmac2 0x90>;
> + dma-names = "tx", "rx", "tx", "rx";
> i2c-scl-internal-delay-ns = <110>;
> status = "disabled";
> };
> @@ -1351,8 +1358,9 @@
> clocks = <&cpg CPG_MOD 930>;
> power-domains = <&sysc R8A7795_PD_ALWAYS_ON>;
> resets = <&cpg 930>;
> - dmas = <&dmac1 0x93>, <&dmac1 0x92>;
> - dma-names = "tx", "rx";
> + dmas = <&dmac1 0x93>, <&dmac1 0x92>,
> + <&dmac2 0x93>, <&dmac2 0x92>;
> + dma-names = "tx", "rx", "tx", "rx";
> i2c-scl-internal-delay-ns = <6>;
> status = "disabled";
> };
> @@ -1367,8 +1375,9 @@
> clocks = <&cpg CPG_MOD 929>;
> power-domains = <&sysc R8A7795_PD_ALWAYS_ON>;
> resets = <&cpg 929>;
> - dmas = <&dmac1 0x95>, <&dmac1 0x94>;
> - dma-names = "tx", "rx";
> + dmas = <&dmac1 0x95>, <&dmac1 0x94>,
> + <&dmac2 0x95>, <&dmac2 0x94>;
> + dma-names = "tx", "rx", "tx", "rx";
> i2c-scl-internal-delay-ns = <6>;
> status = "disabled";
> };
> --
> 2.7.4
>
--
Regards,
Niklas S?derlund
next prev parent reply other threads:[~2018-03-09 22:03 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-03-08 14:10 [PATCH] arm64: dts: renesas: r8a7795: Add missing SYS-DMAC2 dmas Geert Uytterhoeven
2018-03-09 22:03 ` Niklas Söderlund [this message]
2018-03-13 18:43 ` Simon Horman
2018-03-13 19:23 ` Geert Uytterhoeven
2018-03-16 8:43 ` Simon Horman
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=20180309220333.GM2205@bigcity.dyn.berto.se \
--to=niklas.soderlund@ragnatech.se \
--cc=linux-arm-kernel@lists.infradead.org \
/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).