From: Charles Keepax <ckeepax@opensource.cirrus.com>
To: Mohammad Rafi Shaik <quic_mohs@quicinc.com>
Cc: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>,
Liam Girdwood <lgirdwood@gmail.com>,
Mark Brown <broonie@kernel.org>, Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>, Vinod Koul <vkoul@kernel.org>,
Bard Liao <yung-chuan.liao@linux.intel.com>,
Jaroslav Kysela <perex@perex.cz>, Takashi Iwai <tiwai@suse.com>,
Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>,
Sanyog Kale <sanyog.r.kale@intel.com>,
<linux-arm-msm@vger.kernel.org>, <alsa-devel@alsa-project.org>,
<linux-sound@vger.kernel.org>, <devicetree@vger.kernel.org>,
<linux-kernel@vger.kernel.org>, <quic_rohkumar@quicinc.com>,
<kernel@quicinc.com>, <quic_pkumpatl@quicinc.com>
Subject: Re: [PATCH v1 0/4] Add static channel mapping between soundwire master and slave
Date: Mon, 9 Sep 2024 15:49:47 +0100 [thread overview]
Message-ID: <Zt8LC4IY7DGq8Qom@opensource.cirrus.com> (raw)
In-Reply-To: <20240909105547.2691015-1-quic_mohs@quicinc.com>
On Mon, Sep 09, 2024 at 04:25:43PM +0530, Mohammad Rafi Shaik wrote:
> Add static channel map support between soundwire master and slave.
> This patch series will resolve channel mask mismatch between master and slave.
>
> Scenario: wcd937x AMIC2 usecase
>
> Master Slave (wcd937x)
> +--------------+ +--------------+
> | +--------+ | | +--------+ |
> AMIC1 ----->| | PORT1 | | | | TX1 | |<-----------AMIC1
> AMIC2 ----->| | | | | | | |
> | +--------+ | | +--------+ |
> | | | |
> AMIC3 ----->| +--------+ | | +--------+ |
> | | PORT2 | | | | TX2 | |<-----------AMIC2
> | | | | | | | |<-----------AMIC3
> | +--------+ | | +--------+ |
> | | | |
> | +--------+ | | +--------+ |
> DMIC0...DMIC3------>| | PORT3 | | | | TX3 | |<-----------DMIC0...DMIC3
> | | | | | | | |<-----------MBHC
> | +--------+ | | +--------+ |
> | | | |
> | +--------+ | | +--------+ |
> DMIC4...DMIC37----->| | PORT4 | | | | TX4 | |<-----------DMIC4...DMIC7
> | | | | | | | |
> | +--------+ | | +--------+ |
> | | | |
> +------------- + +--------------+
>
> For AMIC2 usecase, The Slave need to configure TX2 Port with channel mask 1 and
> for Master required PORT1 with channel mask 2,
>
> In existing design master and slave configured with same channel mask, it will fail
> AMIC2 usecase.
Apologies but I am not really following what exactly the issue is
here? How do these ports map to DAI links? It looks like you are
attempting to have AMIC2 produced by one DAI link, but consumed
by another?
Thanks,
Charles
next prev parent reply other threads:[~2024-09-09 14:50 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-09-09 10:55 [PATCH v1 0/4] Add static channel mapping between soundwire master and slave Mohammad Rafi Shaik
2024-09-09 10:55 ` [PATCH v1 1/4] ASoC: dt-bindings: wcd938x-sdw: Add static channel mapping support Mohammad Rafi Shaik
2024-09-10 7:45 ` Krzysztof Kozlowski
2024-09-11 11:33 ` Dmitry Baryshkov
2024-09-09 10:55 ` [PATCH v1 2/4] soundwire: stream: Add set_master_channel_map() to set static channel mapping Mohammad Rafi Shaik
2024-09-09 14:50 ` Charles Keepax
2024-09-09 10:55 ` [PATCH v1 3/4] soundwire: qcom: Add static channel mapping support in soundwire master Mohammad Rafi Shaik
2024-09-09 10:55 ` [PATCH v1 4/4] ASoC: codecs: wcd937x: Add static channel mapping support in wcd937x-sdw Mohammad Rafi Shaik
2024-09-09 14:49 ` Charles Keepax [this message]
2024-09-09 15:10 ` [PATCH v1 0/4] Add static channel mapping between soundwire master and slave Mark Brown
2024-09-09 16:03 ` Pierre-Louis Bossart
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=Zt8LC4IY7DGq8Qom@opensource.cirrus.com \
--to=ckeepax@opensource.cirrus.com \
--cc=alsa-devel@alsa-project.org \
--cc=broonie@kernel.org \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=kernel@quicinc.com \
--cc=krzk+dt@kernel.org \
--cc=lgirdwood@gmail.com \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-sound@vger.kernel.org \
--cc=perex@perex.cz \
--cc=pierre-louis.bossart@linux.intel.com \
--cc=quic_mohs@quicinc.com \
--cc=quic_pkumpatl@quicinc.com \
--cc=quic_rohkumar@quicinc.com \
--cc=robh@kernel.org \
--cc=sanyog.r.kale@intel.com \
--cc=srinivas.kandagatla@linaro.org \
--cc=tiwai@suse.com \
--cc=vkoul@kernel.org \
--cc=yung-chuan.liao@linux.intel.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