From: Geert Uytterhoeven <geert@linux-m68k.org>
To: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Cc: Linux-Renesas <linux-renesas-soc@vger.kernel.org>,
Rob Herring <robh+dt@kernel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
Mark Brown <broonie@kernel.org>,
Liam Girdwood <lgirdwood@gmail.com>,
"open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS"
<devicetree@vger.kernel.org>,
ALSA Development Mailing List <alsa-devel@alsa-project.org>,
Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Subject: Re: [PATCH 0/8] arm64: dts: renesas: Simple-Card / Audio-Graph-Card/Card2 dts for ULCB/KF
Date: Wed, 11 Jan 2023 15:45:48 +0100 [thread overview]
Message-ID: <CAMuHMdUH-CSmaKuM2HA-5j5cP_c2pygp9DqyQiOC9XkCUuzCKw@mail.gmail.com> (raw)
In-Reply-To: <87edt2pxhd.wl-kuninori.morimoto.gx@renesas.com>
Hi Morimoto-san,
CC DT/ASoC
On Wed, Dec 14, 2022 at 2:47 AM Kuninori Morimoto
<kuninori.morimoto.gx@renesas.com> wrote:
> ALSA SoC is now supporting generic Sound Card driver such as
> Simple-Card / Audio-Graph-Card/Card2, and it supports
> "normal connection" and "special connections".
> I'm its maintainer, thus I need/want to test these.
>
> I have been tested these by using my local patches to switching card,
> but maintenance it locally is not useful for me, and users who want to
> know the sample DT settings.
>
> And I have noticed that some local patch had bug, but no one noticed
> about it (of course :)
>
> ULCB/KF is very good board to test these.
> So, I want to upstream these.
>
> One note here is that this patch *should not* have effect to
> existing normal users, but useful for test user.
> So, this patch-set adds "3 card" x "2 use case" dtsi file,
> and ulcb/ulcb-kf "include" one of them which doesn't change anything.
> test user can switch to other card or other use case by changing
> "include" dtsi.
>
> [1/8][2/8] are small tidyup patches for all SoC sound.
Thanks for your series!
I really wanted to review this earlier, but I am held back by my lack
of understanding of the audio subsystem and the audio bindings.
It is also not clear to me what really is hardware description
(i.e. to be described in DT), and what is software policy (i.e. not
to be described in DT)...
Hence I have lots of questions ("A fool can ask more questions in a
minute than a wise man can answer in an hour." ;-)
What is the difference between "Simple Audio Card", "Audio Graph Card",
and "Audio Graph Card2"? Why do we need to support all three of them?
How come we can have no #sound-dai-cells, #sound-dai-cells = <0>,
and #sound-dai-cells = <1>, for the same hardware?
What is the real difference between Single DAI and Multi DAI?
Why do we need to support both?
> Kuninori Morimoto (8):
> ARM: dts: renesas: #sound-dai-cells is used when simple-card
> arm64: dts: renesas: #sound-dai-cells is used when simple-card
> arm64: dts: renesas: add ulcb{-kf} Audio Graph Card dtsi
> arm64: dts: renesas: add ulcb{-kf} Audio Graph Card2 dtsi
> arm64: dts: renesas: add ulcb{-kf} Simple Audio Card dtsi
The above three all describe the same hardware configuration?
> arm64: dts: renesas: add ulcb{-kf} Audio Graph Card2 dtsi MIX + TDM Split dtsi
> arm64: dts: renesas: add ulcb{-kf} Audio Graph Card dtsi MIX + TDM Split dtsi
> arm64: dts: renesas: add ulcb{-kf} Simple Audio Card MIX + TDM Split dtsi
Is MIX and TDM Split a different hardware configuration, or software
policy?
Thanks!
> arch/arm/boot/dts/r8a7742.dtsi | 2 +-
> arch/arm/boot/dts/r8a7743.dtsi | 2 +-
> arch/arm/boot/dts/r8a7744.dtsi | 2 +-
> arch/arm/boot/dts/r8a7745.dtsi | 2 +-
> arch/arm/boot/dts/r8a7778.dtsi | 2 +-
> arch/arm/boot/dts/r8a7790.dtsi | 2 +-
> arch/arm/boot/dts/r8a7791.dtsi | 2 +-
> arch/arm/boot/dts/r8a7793.dtsi | 2 +-
> arch/arm/boot/dts/r8a7794.dtsi | 2 +-
> arch/arm64/boot/dts/renesas/r8a774a1.dtsi | 2 +-
> arch/arm64/boot/dts/renesas/r8a774b1.dtsi | 2 +-
> arch/arm64/boot/dts/renesas/r8a774c0.dtsi | 2 +-
> arch/arm64/boot/dts/renesas/r8a774e1.dtsi | 2 +-
> arch/arm64/boot/dts/renesas/r8a77951.dtsi | 2 +-
> arch/arm64/boot/dts/renesas/r8a77960.dtsi | 2 +-
> arch/arm64/boot/dts/renesas/r8a77961.dtsi | 2 +-
> arch/arm64/boot/dts/renesas/r8a77965.dtsi | 2 +-
> arch/arm64/boot/dts/renesas/r8a77990.dtsi | 2 +-
> arch/arm64/boot/dts/renesas/r8a77995.dtsi | 2 +-
> .../boot/dts/renesas/salvator-common.dtsi | 3 -
> .../ulcb-audio-graph-card-mix+split.dtsi | 89 +++++++++
> .../dts/renesas/ulcb-audio-graph-card.dtsi | 85 +++++++++
> .../ulcb-audio-graph-card2-mix+split.dtsi | 111 +++++++++++
> .../dts/renesas/ulcb-audio-graph-card2.dtsi | 26 +++
> .../ulcb-kf-audio-graph-card-mix+split.dtsi | 162 ++++++++++++++++
> .../dts/renesas/ulcb-kf-audio-graph-card.dtsi | 88 +++++++++
> .../ulcb-kf-audio-graph-card2-mix+split.dtsi | 177 ++++++++++++++++++
> .../renesas/ulcb-kf-audio-graph-card2.dtsi | 30 +++
> .../ulcb-kf-simple-audio-card-mix+split.dtsi | 137 ++++++++++++++
> .../renesas/ulcb-kf-simple-audio-card.dtsi | 85 +++++++++
> arch/arm64/boot/dts/renesas/ulcb-kf.dtsi | 66 ++-----
> .../ulcb-simple-audio-card-mix+split.dtsi | 86 +++++++++
> .../dts/renesas/ulcb-simple-audio-card.dtsi | 89 +++++++++
> arch/arm64/boot/dts/renesas/ulcb.dtsi | 60 ++----
> 34 files changed, 1212 insertions(+), 120 deletions(-)
> create mode 100644 arch/arm64/boot/dts/renesas/ulcb-audio-graph-card-mix+split.dtsi
> create mode 100644 arch/arm64/boot/dts/renesas/ulcb-audio-graph-card.dtsi
> create mode 100644 arch/arm64/boot/dts/renesas/ulcb-audio-graph-card2-mix+split.dtsi
> create mode 100644 arch/arm64/boot/dts/renesas/ulcb-audio-graph-card2.dtsi
> create mode 100644 arch/arm64/boot/dts/renesas/ulcb-kf-audio-graph-card-mix+split.dtsi
> create mode 100644 arch/arm64/boot/dts/renesas/ulcb-kf-audio-graph-card.dtsi
> create mode 100644 arch/arm64/boot/dts/renesas/ulcb-kf-audio-graph-card2-mix+split.dtsi
> create mode 100644 arch/arm64/boot/dts/renesas/ulcb-kf-audio-graph-card2.dtsi
> create mode 100644 arch/arm64/boot/dts/renesas/ulcb-kf-simple-audio-card-mix+split.dtsi
> create mode 100644 arch/arm64/boot/dts/renesas/ulcb-kf-simple-audio-card.dtsi
> create mode 100644 arch/arm64/boot/dts/renesas/ulcb-simple-audio-card-mix+split.dtsi
> create mode 100644 arch/arm64/boot/dts/renesas/ulcb-simple-audio-card.dtsi
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 parent reply other threads:[~2023-01-11 14:46 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <87edt2pxhd.wl-kuninori.morimoto.gx@renesas.com>
2023-01-11 14:45 ` Geert Uytterhoeven [this message]
2023-01-12 0:17 ` [PATCH 0/8] arm64: dts: renesas: Simple-Card / Audio-Graph-Card/Card2 dts for ULCB/KF Kuninori Morimoto
2023-01-12 10:59 ` Geert Uytterhoeven
2023-01-12 23:07 ` Kuninori Morimoto
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=CAMuHMdUH-CSmaKuM2HA-5j5cP_c2pygp9DqyQiOC9XkCUuzCKw@mail.gmail.com \
--to=geert@linux-m68k.org \
--cc=alsa-devel@alsa-project.org \
--cc=broonie@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=kuninori.morimoto.gx@renesas.com \
--cc=laurent.pinchart+renesas@ideasonboard.com \
--cc=lgirdwood@gmail.com \
--cc=linux-renesas-soc@vger.kernel.org \
--cc=robh+dt@kernel.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).