From: Geert Uytterhoeven <geert@linux-m68k.org>
To: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Cc: Magnus <magnus.damm@gmail.com>,
Linux-Renesas <linux-renesas-soc@vger.kernel.org>,
Kazuya Mizuguchi <kazuya.mizuguchi.ks@renesas.com>,
Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>,
hoai.luu.ub@renesas.com,
Takeshi Kihara <takeshi.kihara.df@renesas.com>,
Harunobu Kurokawa <harunobu.kurokawa.dn@renesas.com>,
Khiem Nguyen <khiem.nguyen.xt@renesas.com>,
Hien Dang <hien.dang.eb@renesas.com>
Subject: Re: [PATCH 2/3] arm64: dts: renesas: r8a77995: add R-Car Sound support
Date: Mon, 17 May 2021 13:33:39 +0200 [thread overview]
Message-ID: <CAMuHMdXtnSenYk3sE6Pp89YeuUf=RsU5ueOLKX3ip9j1NMBKCQ@mail.gmail.com> (raw)
In-Reply-To: <87h7j2ci1k.wl-kuninori.morimoto.gx@renesas.com>
Hi Morimoto-san,
Thanks for your patch!
On Mon, May 17, 2021 at 2:36 AM Kuninori Morimoto
<kuninori.morimoto.gx@renesas.com> wrote:
> From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
>
> This patch adds R-Car Sound and Audio-DMAC support for D3.
>
> 1st note is that D3 doesn't have audio-clk-c, but is
> required from driver. This patch uses null-clk for it.
>
> 2nd note is that D3 has SSI3/4 and SRC5/6 only, but driver
> requres from SSI0/SRC0. This patch has disabled SSI/SRC for it.
I think it would be better to fix the driver instead.
>
> Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
> ---
> arch/arm64/boot/dts/renesas/r8a77995.dtsi | 176 ++++++++++++++++++++++
> 1 file changed, 176 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/renesas/r8a77995.dtsi b/arch/arm64/boot/dts/renesas/r8a77995.dtsi
> index 2319271c881b..0950f39711eb 100644
> --- a/arch/arm64/boot/dts/renesas/r8a77995.dtsi
> +++ b/arch/arm64/boot/dts/renesas/r8a77995.dtsi
> @@ -43,6 +43,33 @@ L2_CA53: cache-controller-1 {
> };
> };
>
> + /*
> + * The external audio clocks are configured as 0 Hz fixed frequency
> + * clocks by default.
> + * Boards that provide audio clocks should override them.
> + */
> + audio_clk_a: audio_clk_a {
Please use alphabetical sort order when adding nodes without unit
addresses.
> + compatible = "fixed-clock";
> + #clock-cells = <0>;
> + clock-frequency = <0>;
> + };
> +
> + audio_clk_b: audio_clk_b {
> + compatible = "fixed-clock";
> + #clock-cells = <0>;
> + clock-frequency = <0>;
> + };
> +
> + /*
> + * R8A77995 doesn't have audio_clk_c, * but is required from driver.
> + * Create NULL clock for it.
> + */
> + null_clk: null_clk {
> + compatible = "fixed-clock";
> + #clock-cells = <0>;
> + clock-frequency = <0>;
> + };
Please do not add nodes not matching the hardware description, but fix
the driver instead. ("DT describes hardware, not software policy").
> +
> extal_clk: extal {
> compatible = "fixed-clock";
> #clock-cells = <0>;
> @@ -642,6 +669,48 @@ dmac2: dma-controller@e7310000 {
> <&ipmmu_ds1 22>, <&ipmmu_ds1 23>;
> };
>
> + audma0: dma-controller@ec700000 {
Please use numerical sort order when adding nodes with unit addresses.
> @@ -1258,6 +1327,113 @@ prr: chipid@fff00044 {
> compatible = "renesas,prr";
> reg = <0 0xfff00044 0 4>;
> };
> +
> + rcar_sound: sound@ec500000 {
Please use numerical sort order when adding nodes with unit addresses.
> + /*
> + * #sound-dai-cells is required
> + *
> + * Single DAI : #sound-dai-cells = <0>; <&rcar_sound>;
> + * Multi DAI : #sound-dai-cells = <1>; <&rcar_sound N>;
> + */
> + /*
> + * #clock-cells is required for audio_clkout0/1/2/3
> + *
> + * clkout : #clock-cells = <0>; <&rcar_sound>;
> + * clkout0/1/2/3: #clock-cells = <1>; <&rcar_sound N>;
> + */
> + compatible = "renesas,rcar_sound-r8a77995", "renesas,rcar_sound-gen3";
> + reg = <0 0xec500000 0 0x1000>, /* SCU */
> + <0 0xec5a0000 0 0x100>, /* ADG */
> + <0 0xec540000 0 0x1000>, /* SSIU */
> + <0 0xec541000 0 0x280>, /* SSI */
> + <0 0xec740000 0 0x200>; /* Audio DMAC peri peri*/
> + reg-names = "scu", "adg", "ssiu", "ssi", "audmapp";
> +
> + clocks = <&cpg CPG_MOD 1005>,
> + <&cpg CPG_MOD 1011>, <&cpg CPG_MOD 1012>,
> + <&cpg CPG_MOD 1025>, <&cpg CPG_MOD 1026>,
> + <&cpg CPG_MOD 1020>, <&cpg CPG_MOD 1021>,
> + <&cpg CPG_MOD 1020>, <&cpg CPG_MOD 1021>,
> + <&cpg CPG_MOD 1019>, <&cpg CPG_MOD 1018>,
> + <&audio_clk_a>, <&audio_clk_b>, <&null_clk>, /* clk_c is NULL */
> + <&cpg CPG_CORE R8A77995_CLK_ZA2>;
> + clock-names = "ssi-all",
> + "ssi.4", "ssi.3",
> + "src.6", "src.5",
> + "mix.1", "mix.0",
> + "ctu.1", "ctu.0",
> + "dvc.0", "dvc.1",
> + "clk_a", "clk_b", "clk_c", "clk_i"; /* clk_c is NULL */
> + power-domains = <&sysc R8A77995_PD_ALWAYS_ON>;
> + resets = <&cpg 1005>,
> + <&cpg 1011>, <&cpg 1012>;
> + reset-names = "ssi-all",
> + "ssi.4", "ssi.3";
arch/arm64/boot/dts/renesas/r8a77995-draak.dt.yaml: sound@ec500000:
resets: [[4, 1005], [4, 1011], [4, 1012]] is too short
From schema: Documentation/devicetree/bindings/sound/renesas,rsnd.yaml
arch/arm64/boot/dts/renesas/r8a77995-draak.dt.yaml: sound@ec500000:
reset-names: ['ssi-all', 'ssi.4', 'ssi.3'] is too short
From schema: Documentation/devicetree/bindings/sound/renesas,rsnd.yaml
As the DTS is correct, the DT bindings should be updated.
> + rcar_sound,src {
> + src0: src-0 { status = "disabled"; };
> + src1: src-1 { status = "disabled"; };
> + src2: src-2 { status = "disabled"; };
> + src3: src-3 { status = "disabled"; };
> + src4: src-4 { status = "disabled"; };
Please drop nonexistent src channels.
> + src5: src-5 {
> + interrupts = <GIC_SPI 357 IRQ_TYPE_LEVEL_HIGH>;
> + dmas = <&audma0 0x8f>, <&audma0 0xb2>;
> + dma-names = "rx", "tx";
> + };
> + src6: src-6 {
> + interrupts = <GIC_SPI 358 IRQ_TYPE_LEVEL_HIGH>;
> + dmas = <&audma0 0x91>, <&audma0 0xb4>;
> + dma-names = "rx", "tx";
> + };
> + };
> +
> + rcar_sound,ssi {
> + ssi0: ssi-0 { status = "disabled"; };
> + ssi1: ssi-1 { status = "disabled"; };
> + ssi2: ssi-2 { status = "disabled"; };
Please drop nonexistent ssi channels.
> + ssi3: ssi-3 {
> + interrupts = <GIC_SPI 373 IRQ_TYPE_LEVEL_HIGH>;
> + dmas = <&audma0 0x07>, <&audma0 0x08>,
> + <&audma0 0x6f>, <&audma0 0x70>;
> + dma-names = "rx", "tx", "rxu", "txu";
> + };
> + ssi4: ssi-4 {
> + interrupts = <GIC_SPI 374 IRQ_TYPE_LEVEL_HIGH>;
> + dmas = <&audma0 0x09>, <&audma0 0x0a>,
> + <&audma0 0x71>, <&audma0 0x72>;
> + dma-names = "rx", "tx", "rxu", "txu";
> + };
> + };
> + };
> };
>
> thermal-zones {
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@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
next prev parent reply other threads:[~2021-05-17 11:33 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-05-17 0:36 [PATCH 0/3] arm64: dts: renesas: r8a77995: draak: add R-Car Sound support Kuninori Morimoto
2021-05-17 0:36 ` [PATCH 1/3] clk: renesas: r8a77995: Add ZA2 clock Kuninori Morimoto
2021-05-17 9:36 ` Geert Uytterhoeven
2021-05-17 22:20 ` Kuninori Morimoto
2021-05-24 0:57 ` Kuninori Morimoto
2021-05-25 7:42 ` Geert Uytterhoeven
2021-05-17 0:36 ` [PATCH 2/3] arm64: dts: renesas: r8a77995: add R-Car Sound support Kuninori Morimoto
2021-05-17 11:33 ` Geert Uytterhoeven [this message]
2021-05-17 22:27 ` Kuninori Morimoto
2021-05-24 5:43 ` Kuninori Morimoto
2021-05-25 7:40 ` Geert Uytterhoeven
2021-05-25 22:53 ` Kuninori Morimoto
2021-05-17 0:37 ` [PATCH 3/3] arm64: dts: renesas: r8a77995: draak: Add " Kuninori Morimoto
2021-05-17 11:46 ` Geert Uytterhoeven
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='CAMuHMdXtnSenYk3sE6Pp89YeuUf=RsU5ueOLKX3ip9j1NMBKCQ@mail.gmail.com' \
--to=geert@linux-m68k.org \
--cc=harunobu.kurokawa.dn@renesas.com \
--cc=hien.dang.eb@renesas.com \
--cc=hoai.luu.ub@renesas.com \
--cc=kazuya.mizuguchi.ks@renesas.com \
--cc=khiem.nguyen.xt@renesas.com \
--cc=kuninori.morimoto.gx@renesas.com \
--cc=linux-renesas-soc@vger.kernel.org \
--cc=magnus.damm@gmail.com \
--cc=takeshi.kihara.df@renesas.com \
--cc=yoshihiro.shimoda.uh@renesas.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).