All of lore.kernel.org
 help / color / mirror / Atom feed
From: Charles Keepax <ckeepax@opensource.cirrus.com>
To: <broonie@kernel.org>
Cc: <lgirdwood@gmail.com>, <codrin.ciubotariu@microchip.com>,
	<lars@metafoo.de>, <cychiang@chromium.org>, <tzungbi@google.com>,
	<bleung@chromium.org>, <matthias.bgg@gmail.com>,
	<kmarinushkin@birdec.com>, <oder_chiou@realtek.com>,
	<steven.eckhoff.opensource@gmail.com>,
	<srinivas.kandagatla@linaro.org>, <alexandre.belloni@bootlin.com>,
	<kuninori.morimoto.gx@renesas.com>, <jiaxin.yu@mediatek.com>,
	<alsa-devel@alsa-project.org>, <chrome-platform@lists.linux.dev>,
	<linux-mediatek@lists.infradead.org>,
	<patches@opensource.cirrus.com>
Subject: [PATCH 01/38] ASoC: soc-component: Add comment for the endianness flag
Date: Wed, 4 May 2022 18:08:28 +0100	[thread overview]
Message-ID: <20220504170905.332415-2-ckeepax@opensource.cirrus.com> (raw)
In-Reply-To: <20220504170905.332415-1-ckeepax@opensource.cirrus.com>

Add a comment to make the purpose of the endianness flag on the
snd_soc_component structure more clear.

Signed-off-by: Charles Keepax <ckeepax@opensource.cirrus.com>
---
 include/sound/soc-component.h | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/include/sound/soc-component.h b/include/sound/soc-component.h
index 766dc6f009c0b..5a764c3099d3e 100644
--- a/include/sound/soc-component.h
+++ b/include/sound/soc-component.h
@@ -169,6 +169,15 @@ struct snd_soc_component_driver {
 	unsigned int idle_bias_on:1;
 	unsigned int suspend_bias_off:1;
 	unsigned int use_pmdown_time:1; /* care pmdown_time at stop */
+	/*
+	 * Indicates that the component does not care about the endianness of
+	 * PCM audio data and the core will ensure that both LE and BE variants
+	 * of each used format are present. Typically this is because the
+	 * component sits behind a bus that abstracts away the endian of the
+	 * original data, ie. one for which the transmission endian is defined
+	 * (I2S/SLIMbus/SoundWire), or the concept of endian doesn't exist (PDM,
+	 * analogue).
+	 */
 	unsigned int endianness:1;
 	unsigned int non_legacy_dai_naming:1;
 
-- 
2.30.2


_______________________________________________
Linux-mediatek mailing list
Linux-mediatek@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-mediatek

WARNING: multiple messages have this Message-ID (diff)
From: Charles Keepax <ckeepax@opensource.cirrus.com>
To: <broonie@kernel.org>
Cc: oder_chiou@realtek.com, steven.eckhoff.opensource@gmail.com,
	alexandre.belloni@bootlin.com, lars@metafoo.de,
	kuninori.morimoto.gx@renesas.com, kmarinushkin@birdec.com,
	patches@opensource.cirrus.com,
	linux-mediatek@lists.infradead.org, lgirdwood@gmail.com,
	jiaxin.yu@mediatek.com, tzungbi@google.com,
	srinivas.kandagatla@linaro.org, matthias.bgg@gmail.com,
	chrome-platform@lists.linux.dev, codrin.ciubotariu@microchip.com,
	alsa-devel@alsa-project.org, bleung@chromium.org,
	cychiang@chromium.org
Subject: [PATCH 01/38] ASoC: soc-component: Add comment for the endianness flag
Date: Wed, 4 May 2022 18:08:28 +0100	[thread overview]
Message-ID: <20220504170905.332415-2-ckeepax@opensource.cirrus.com> (raw)
In-Reply-To: <20220504170905.332415-1-ckeepax@opensource.cirrus.com>

Add a comment to make the purpose of the endianness flag on the
snd_soc_component structure more clear.

Signed-off-by: Charles Keepax <ckeepax@opensource.cirrus.com>
---
 include/sound/soc-component.h | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/include/sound/soc-component.h b/include/sound/soc-component.h
index 766dc6f009c0b..5a764c3099d3e 100644
--- a/include/sound/soc-component.h
+++ b/include/sound/soc-component.h
@@ -169,6 +169,15 @@ struct snd_soc_component_driver {
 	unsigned int idle_bias_on:1;
 	unsigned int suspend_bias_off:1;
 	unsigned int use_pmdown_time:1; /* care pmdown_time at stop */
+	/*
+	 * Indicates that the component does not care about the endianness of
+	 * PCM audio data and the core will ensure that both LE and BE variants
+	 * of each used format are present. Typically this is because the
+	 * component sits behind a bus that abstracts away the endian of the
+	 * original data, ie. one for which the transmission endian is defined
+	 * (I2S/SLIMbus/SoundWire), or the concept of endian doesn't exist (PDM,
+	 * analogue).
+	 */
 	unsigned int endianness:1;
 	unsigned int non_legacy_dai_naming:1;
 
-- 
2.30.2


WARNING: multiple messages have this Message-ID (diff)
From: Charles Keepax <ckeepax@opensource.cirrus.com>
To: <broonie@kernel.org>
Cc: <lgirdwood@gmail.com>, <codrin.ciubotariu@microchip.com>,
	<lars@metafoo.de>, <cychiang@chromium.org>, <tzungbi@google.com>,
	<bleung@chromium.org>, <matthias.bgg@gmail.com>,
	<kmarinushkin@birdec.com>, <oder_chiou@realtek.com>,
	<steven.eckhoff.opensource@gmail.com>,
	<srinivas.kandagatla@linaro.org>, <alexandre.belloni@bootlin.com>,
	<kuninori.morimoto.gx@renesas.com>, <jiaxin.yu@mediatek.com>,
	<alsa-devel@alsa-project.org>, <chrome-platform@lists.linux.dev>,
	<linux-mediatek@lists.infradead.org>,
	<patches@opensource.cirrus.com>
Subject: [PATCH 01/38] ASoC: soc-component: Add comment for the endianness flag
Date: Wed, 4 May 2022 18:08:28 +0100	[thread overview]
Message-ID: <20220504170905.332415-2-ckeepax@opensource.cirrus.com> (raw)
In-Reply-To: <20220504170905.332415-1-ckeepax@opensource.cirrus.com>

Add a comment to make the purpose of the endianness flag on the
snd_soc_component structure more clear.

Signed-off-by: Charles Keepax <ckeepax@opensource.cirrus.com>
---
 include/sound/soc-component.h | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/include/sound/soc-component.h b/include/sound/soc-component.h
index 766dc6f009c0b..5a764c3099d3e 100644
--- a/include/sound/soc-component.h
+++ b/include/sound/soc-component.h
@@ -169,6 +169,15 @@ struct snd_soc_component_driver {
 	unsigned int idle_bias_on:1;
 	unsigned int suspend_bias_off:1;
 	unsigned int use_pmdown_time:1; /* care pmdown_time at stop */
+	/*
+	 * Indicates that the component does not care about the endianness of
+	 * PCM audio data and the core will ensure that both LE and BE variants
+	 * of each used format are present. Typically this is because the
+	 * component sits behind a bus that abstracts away the endian of the
+	 * original data, ie. one for which the transmission endian is defined
+	 * (I2S/SLIMbus/SoundWire), or the concept of endian doesn't exist (PDM,
+	 * analogue).
+	 */
 	unsigned int endianness:1;
 	unsigned int non_legacy_dai_naming:1;
 
-- 
2.30.2


  reply	other threads:[~2022-05-04 17:10 UTC|newest]

Thread overview: 141+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-05-04 17:08 [PATCH 00/38] Clean up usage of the endianness flag Charles Keepax
2022-05-04 17:08 ` Charles Keepax
2022-05-04 17:08 ` Charles Keepax
2022-05-04 17:08 ` Charles Keepax [this message]
2022-05-04 17:08   ` [PATCH 01/38] ASoC: soc-component: Add comment for " Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-08 20:34   ` Kirill Marinushkin
2022-05-08 20:34     ` Kirill Marinushkin
2022-05-08 20:34     ` Kirill Marinushkin
2022-05-09  8:37     ` Charles Keepax
2022-05-09  8:37       ` Charles Keepax
2022-05-09  8:37       ` Charles Keepax
2022-05-09 19:22       ` Kirill Marinushkin
2022-05-09 19:22         ` Kirill Marinushkin
2022-05-09 19:22         ` Kirill Marinushkin
2022-05-09 19:30         ` Mark Brown
2022-05-09 19:30           ` Mark Brown
2022-05-09 19:30           ` Mark Brown
2022-05-09 20:11           ` Kirill Marinushkin
2022-05-09 20:11             ` Kirill Marinushkin
2022-05-09 20:11             ` Kirill Marinushkin
2022-05-09 20:18             ` Mark Brown
2022-05-09 20:18               ` Mark Brown
2022-05-09 20:18               ` Mark Brown
2022-05-10  8:53         ` Charles Keepax
2022-05-10  8:53           ` Charles Keepax
2022-05-10  8:53           ` Charles Keepax
2022-05-04 17:08 ` [PATCH 02/38] ASoC: atmel-pdmic: Remove endianness flag on pdmic component Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08 ` [PATCH 03/38] ASoC: atmel-classd: Remove endianness flag on class d component Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08 ` [PATCH 04/38] ASoC: cs4270: Remove redundant big endian formats Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08 ` [PATCH 05/38] ASoC: cs42l51: " Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08 ` [PATCH 06/38] ASoC: cs4349: " Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08 ` [PATCH 07/38] ASoC: hdmi-codec: " Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08 ` [PATCH 08/38] ASoC: sta32x: " Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08 ` [PATCH 09/38] ASoC: sta350: " Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08 ` [PATCH 10/38] ASoC: hdac_hda: Add endianness flag in snd_soc_component_driver Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08 ` [PATCH 11/38] ASoC: max98504: " Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08 ` [PATCH 12/38] ASoC: adau1372: " Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08 ` [PATCH 13/38] ASoC: cs4234: " Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08 ` [PATCH 14/38] ASoC: cs35l41: " Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08 ` [PATCH 15/38] ASoC: cx2072x: " Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08 ` [PATCH 16/38] ASoC: lochnagar: " Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08 ` [PATCH 17/38] ASoC: mt6351: " Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08 ` [PATCH 18/38] ASoC: mt6358: " Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08 ` [PATCH 19/38] ASoC: mt6359: " Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08 ` [PATCH 20/38] ASoC: mt6660: " Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08 ` [PATCH 21/38] ASoC: pcm3060: " Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08 ` [PATCH 22/38] ASoC: rt1019: " Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08 ` [PATCH 23/38] ASoC: rt9120: " Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08 ` [PATCH 24/38] ASoC: tlv320adc3xxx: " Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08 ` [PATCH 25/38] ASoC: tscs454: " Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08 ` [PATCH 26/38] ASoC: cros_ec_codec: Add endianness flag in i2s_rx_component_driver Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08 ` [PATCH 27/38] ASoC: wcd934x: Add endianness flag in snd_soc_component_driver Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08 ` [PATCH 28/38] ASoC: wcd9335: " Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08 ` [PATCH 29/38] ASoC: rt700: " Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08 ` [PATCH 30/38] ASoC: rt711: " Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08 ` [PATCH 31/38] ASoC: rt711-sdca: " Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08 ` [PATCH 32/38] ASoC: rt715: " Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:08   ` Charles Keepax
2022-05-04 17:09 ` [PATCH 33/38] ASoC: rt715-sdca: " Charles Keepax
2022-05-04 17:09   ` Charles Keepax
2022-05-04 17:09   ` Charles Keepax
2022-05-04 17:09 ` [PATCH 34/38] ASoC: rt1308-sdw: " Charles Keepax
2022-05-04 17:09   ` Charles Keepax
2022-05-04 17:09   ` Charles Keepax
2022-05-04 17:09 ` [PATCH 35/38] ASoC: rt1316-sdw: " Charles Keepax
2022-05-04 17:09   ` Charles Keepax
2022-05-04 17:09   ` Charles Keepax
2022-05-04 17:09 ` [PATCH 36/38] ASoC: wcd938x: " Charles Keepax
2022-05-04 17:09   ` Charles Keepax
2022-05-04 17:09   ` Charles Keepax
2022-05-04 17:09 ` [PATCH 37/38] ASoC: wsa881x: " Charles Keepax
2022-05-04 17:09   ` Charles Keepax
2022-05-04 17:09   ` Charles Keepax
2022-05-04 17:09 ` [PATCH 38/38] ASoC: sdw-mockup: " Charles Keepax
2022-05-04 17:09   ` Charles Keepax
2022-05-04 17:09   ` Charles Keepax
2022-05-10 11:13 ` [PATCH 00/38] Clean up usage of the endianness flag Mark Brown
2022-05-10 11:13   ` Mark Brown
2022-05-10 11:13   ` Mark Brown

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=20220504170905.332415-2-ckeepax@opensource.cirrus.com \
    --to=ckeepax@opensource.cirrus.com \
    --cc=alexandre.belloni@bootlin.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=bleung@chromium.org \
    --cc=broonie@kernel.org \
    --cc=chrome-platform@lists.linux.dev \
    --cc=codrin.ciubotariu@microchip.com \
    --cc=cychiang@chromium.org \
    --cc=jiaxin.yu@mediatek.com \
    --cc=kmarinushkin@birdec.com \
    --cc=kuninori.morimoto.gx@renesas.com \
    --cc=lars@metafoo.de \
    --cc=lgirdwood@gmail.com \
    --cc=linux-mediatek@lists.infradead.org \
    --cc=matthias.bgg@gmail.com \
    --cc=oder_chiou@realtek.com \
    --cc=patches@opensource.cirrus.com \
    --cc=srinivas.kandagatla@linaro.org \
    --cc=steven.eckhoff.opensource@gmail.com \
    --cc=tzungbi@google.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.