All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/2] ASoC: bindings: fsl-asoc-card: Add compatible for tlv320aic31xx codec
@ 2022-02-07 16:49 ` Apertis package maintainers
  0 siblings, 0 replies; 13+ messages in thread
From: Apertis package maintainers @ 2022-02-07 16:49 UTC (permalink / raw)
  To: alsa-devel, devicetree, kernel, linux-arm-kernel, linux-imx,
	linux-kernel
  Cc: ariel.dalessandro, broonie, festevam, krzysztof.kozlowski,
	lgirdwood, michael, robh+dt, s.hauer, shawnguo

From: Ariel D'Alessandro <ariel.dalessandro@collabora.com>

Commit 8c9b9cfb7724 ("ASoC: fsl-asoc-card: Support
fsl,imx-audio-tlv320aic31xx codec")' added support for tlv320aic31xx
codec to fsl-asoc-card, but missed the related device-tree compatible
string documentation. Fix this.

Signed-off-by: Ariel D'Alessandro <ariel.dalessandro@collabora.com>
---
 Documentation/devicetree/bindings/sound/fsl-asoc-card.txt | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/Documentation/devicetree/bindings/sound/fsl-asoc-card.txt b/Documentation/devicetree/bindings/sound/fsl-asoc-card.txt
index b219626a5403..8b4f4015cfe4 100644
--- a/Documentation/devicetree/bindings/sound/fsl-asoc-card.txt
+++ b/Documentation/devicetree/bindings/sound/fsl-asoc-card.txt
@@ -40,6 +40,8 @@ The compatible list for this generic sound card currently:
 
  "fsl,imx-audio-tlv320aic32x4"
 
+ "fsl,imx-audio-tlv320aic31xx"
+
  "fsl,imx-audio-si476x"
 
  "fsl,imx-audio-wm8958"
-- 
2.34.1


^ permalink raw reply related	[flat|nested] 13+ messages in thread

* [PATCH 1/2] ASoC: bindings: fsl-asoc-card: Add compatible for tlv320aic31xx codec
@ 2022-02-07 16:49 ` Apertis package maintainers
  0 siblings, 0 replies; 13+ messages in thread
From: Apertis package maintainers @ 2022-02-07 16:49 UTC (permalink / raw)
  To: alsa-devel, devicetree, kernel, linux-arm-kernel, linux-imx,
	linux-kernel
  Cc: ariel.dalessandro, krzysztof.kozlowski, shawnguo, s.hauer,
	lgirdwood, robh+dt, broonie, michael, festevam

From: Ariel D'Alessandro <ariel.dalessandro@collabora.com>

Commit 8c9b9cfb7724 ("ASoC: fsl-asoc-card: Support
fsl,imx-audio-tlv320aic31xx codec")' added support for tlv320aic31xx
codec to fsl-asoc-card, but missed the related device-tree compatible
string documentation. Fix this.

Signed-off-by: Ariel D'Alessandro <ariel.dalessandro@collabora.com>
---
 Documentation/devicetree/bindings/sound/fsl-asoc-card.txt | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/Documentation/devicetree/bindings/sound/fsl-asoc-card.txt b/Documentation/devicetree/bindings/sound/fsl-asoc-card.txt
index b219626a5403..8b4f4015cfe4 100644
--- a/Documentation/devicetree/bindings/sound/fsl-asoc-card.txt
+++ b/Documentation/devicetree/bindings/sound/fsl-asoc-card.txt
@@ -40,6 +40,8 @@ The compatible list for this generic sound card currently:
 
  "fsl,imx-audio-tlv320aic32x4"
 
+ "fsl,imx-audio-tlv320aic31xx"
+
  "fsl,imx-audio-si476x"
 
  "fsl,imx-audio-wm8958"
-- 
2.34.1


^ permalink raw reply related	[flat|nested] 13+ messages in thread

* [PATCH 2/2] arm64: dts: imx8mn-bsh-smm-s2pro: Add tlv320aic31xx audio card node
  2022-02-07 16:49 ` Apertis package maintainers
@ 2022-02-07 16:49   ` Apertis package maintainers
  -1 siblings, 0 replies; 13+ messages in thread
From: Apertis package maintainers @ 2022-02-07 16:49 UTC (permalink / raw)
  To: alsa-devel, devicetree, kernel, linux-arm-kernel, linux-imx,
	linux-kernel
  Cc: ariel.dalessandro, broonie, festevam, krzysztof.kozlowski,
	lgirdwood, michael, robh+dt, s.hauer, shawnguo

From: Ariel D'Alessandro <ariel.dalessandro@collabora.com>

BSH SystemMaster (SMM) S2 PRO board comes with an audio card based on
tlv320aic31xx family codec.

The audio card exposes two playback devices, one of them using the EASRC
(Enhanced Asynchronous Sample Rate Converter) module. Note that this
would require SDMA and EASRC firmware in order to work.

Signed-off-by: Ariel D'Alessandro <ariel.dalessandro@collabora.com>
Signed-off-by: Michael Trimarchi <michael@amarulasolutions.com>
---
 .../dts/freescale/imx8mn-bsh-smm-s2pro.dts    | 94 +++++++++++++++++++
 1 file changed, 94 insertions(+)

diff --git a/arch/arm64/boot/dts/freescale/imx8mn-bsh-smm-s2pro.dts b/arch/arm64/boot/dts/freescale/imx8mn-bsh-smm-s2pro.dts
index c6a8ed6745c1..3621354b4a92 100644
--- a/arch/arm64/boot/dts/freescale/imx8mn-bsh-smm-s2pro.dts
+++ b/arch/arm64/boot/dts/freescale/imx8mn-bsh-smm-s2pro.dts
@@ -7,6 +7,7 @@
 /dts-v1/;
 
 #include "imx8mn-bsh-smm-s2-common.dtsi"
+#include <dt-bindings/sound/tlv320aic31xx.h>
 
 / {
 	model = "BSH SMM S2 PRO";
@@ -16,6 +17,69 @@ memory@40000000 {
 		device_type = "memory";
 		reg = <0x0 0x40000000 0x0 0x20000000>;
 	};
+
+	sound-tlv320aic31xx {
+		compatible = "fsl,imx-audio-tlv320aic31xx";
+		model = "tlv320aic31xx-hifi";
+		audio-cpu = <&sai3>;
+		audio-codec = <&codec>;
+		audio-asrc = <&easrc>;
+		audio-routing =
+			"Ext Spk", "SPL",
+			"Ext Spk", "SPR";
+		mclk-id = <PLL_CLKIN_BCLK>;
+	};
+
+	vdd_input: vdd_input {
+		compatible = "regulator-fixed";
+		regulator-name = "vdd_input";
+		regulator-min-microvolt = <5000000>;
+		regulator-max-microvolt = <5000000>;
+	};
+};
+
+&easrc {
+	fsl,asrc-rate  = <48000>;
+	fsl,asrc-format = <10>;
+	status = "okay";
+};
+
+&i2c2 {
+	clock-frequency = <400000>;
+	pinctrl-names = "default";
+	pinctrl-0 = <&pinctrl_i2c2>;
+	status = "okay";
+
+	codec: tlv320dac3101@18 {
+		#sound-dai-cells = <0>;
+		compatible = "ti,tlv320dac3101";
+		pinctrl-names = "default";
+		pinctrl-0 = <&pinctrl_dac_rst>;
+		reg = <0x18>;
+
+		ai31xx-micbias-vg = <MICBIAS_AVDDV>;
+
+		HPVDD-supply = <&buck4_reg>;
+		SPRVDD-supply = <&vdd_input>;
+		SPLVDD-supply = <&vdd_input>;
+		AVDD-supply = <&buck4_reg>;
+		IOVDD-supply = <&buck4_reg>;
+		DVDD-supply = <&buck5_reg>;
+		reset-gpios = <&gpio1 6 GPIO_ACTIVE_LOW>;
+
+		clocks = <&clk IMX8MN_CLK_SAI3_ROOT>;
+		clock-names = "mclk";
+	};
+};
+
+&sai3 {
+	pinctrl-names = "default";
+	pinctrl-0 = <&pinctrl_sai3>;
+	assigned-clocks = <&clk IMX8MN_CLK_SAI3>;
+	assigned-clock-parents = <&clk IMX8MN_AUDIO_PLL1_OUT>;
+	assigned-clock-rates = <24576000>;
+	fsl,sai-mclk-direction-output;
+	status = "okay";
 };
 
 /* eMMC */
@@ -30,6 +94,36 @@ &usdhc1 {
 };
 
 &iomuxc {
+	pinctrl_dac_rst: dac_rst {
+		fsl,pins = <
+			MX8MN_IOMUXC_GPIO1_IO06_GPIO1_IO6		0x19 /* DAC_RST */
+		>;
+	};
+
+	pinctrl_espi2: espi2grp {
+		fsl,pins = <
+			MX8MN_IOMUXC_ECSPI2_SCLK_ECSPI2_SCLK            0x082
+			MX8MN_IOMUXC_ECSPI2_MOSI_ECSPI2_MOSI            0x082
+			MX8MN_IOMUXC_ECSPI2_MISO_ECSPI2_MISO            0x082
+			MX8MN_IOMUXC_ECSPI2_SS0_ECSPI2_SS0		0x040
+		>;
+	};
+
+	pinctrl_i2c2: i2c2grp {
+		fsl,pins = <
+			MX8MN_IOMUXC_I2C2_SCL_I2C2_SCL			0x400000c3
+			MX8MN_IOMUXC_I2C2_SDA_I2C2_SDA			0x400000c3
+		>;
+	};
+
+	pinctrl_sai3: sai3grp {
+		fsl,pins = <
+			MX8MN_IOMUXC_SAI3_TXFS_SAI3_TX_SYNC		0xd6
+			MX8MN_IOMUXC_SAI3_TXC_SAI3_TX_BCLK		0xd6
+			MX8MN_IOMUXC_SAI3_TXD_SAI3_TX_DATA0		0xd6
+		>;
+	};
+
 	pinctrl_usdhc1: usdhc1grp {
 		fsl,pins = <
 			MX8MN_IOMUXC_SD1_CLK_USDHC1_CLK			0x40000090
-- 
2.34.1


^ permalink raw reply related	[flat|nested] 13+ messages in thread

* [PATCH 2/2] arm64: dts: imx8mn-bsh-smm-s2pro: Add tlv320aic31xx audio card node
@ 2022-02-07 16:49   ` Apertis package maintainers
  0 siblings, 0 replies; 13+ messages in thread
From: Apertis package maintainers @ 2022-02-07 16:49 UTC (permalink / raw)
  To: alsa-devel, devicetree, kernel, linux-arm-kernel, linux-imx,
	linux-kernel
  Cc: ariel.dalessandro, krzysztof.kozlowski, shawnguo, s.hauer,
	lgirdwood, robh+dt, broonie, michael, festevam

From: Ariel D'Alessandro <ariel.dalessandro@collabora.com>

BSH SystemMaster (SMM) S2 PRO board comes with an audio card based on
tlv320aic31xx family codec.

The audio card exposes two playback devices, one of them using the EASRC
(Enhanced Asynchronous Sample Rate Converter) module. Note that this
would require SDMA and EASRC firmware in order to work.

Signed-off-by: Ariel D'Alessandro <ariel.dalessandro@collabora.com>
Signed-off-by: Michael Trimarchi <michael@amarulasolutions.com>
---
 .../dts/freescale/imx8mn-bsh-smm-s2pro.dts    | 94 +++++++++++++++++++
 1 file changed, 94 insertions(+)

diff --git a/arch/arm64/boot/dts/freescale/imx8mn-bsh-smm-s2pro.dts b/arch/arm64/boot/dts/freescale/imx8mn-bsh-smm-s2pro.dts
index c6a8ed6745c1..3621354b4a92 100644
--- a/arch/arm64/boot/dts/freescale/imx8mn-bsh-smm-s2pro.dts
+++ b/arch/arm64/boot/dts/freescale/imx8mn-bsh-smm-s2pro.dts
@@ -7,6 +7,7 @@
 /dts-v1/;
 
 #include "imx8mn-bsh-smm-s2-common.dtsi"
+#include <dt-bindings/sound/tlv320aic31xx.h>
 
 / {
 	model = "BSH SMM S2 PRO";
@@ -16,6 +17,69 @@ memory@40000000 {
 		device_type = "memory";
 		reg = <0x0 0x40000000 0x0 0x20000000>;
 	};
+
+	sound-tlv320aic31xx {
+		compatible = "fsl,imx-audio-tlv320aic31xx";
+		model = "tlv320aic31xx-hifi";
+		audio-cpu = <&sai3>;
+		audio-codec = <&codec>;
+		audio-asrc = <&easrc>;
+		audio-routing =
+			"Ext Spk", "SPL",
+			"Ext Spk", "SPR";
+		mclk-id = <PLL_CLKIN_BCLK>;
+	};
+
+	vdd_input: vdd_input {
+		compatible = "regulator-fixed";
+		regulator-name = "vdd_input";
+		regulator-min-microvolt = <5000000>;
+		regulator-max-microvolt = <5000000>;
+	};
+};
+
+&easrc {
+	fsl,asrc-rate  = <48000>;
+	fsl,asrc-format = <10>;
+	status = "okay";
+};
+
+&i2c2 {
+	clock-frequency = <400000>;
+	pinctrl-names = "default";
+	pinctrl-0 = <&pinctrl_i2c2>;
+	status = "okay";
+
+	codec: tlv320dac3101@18 {
+		#sound-dai-cells = <0>;
+		compatible = "ti,tlv320dac3101";
+		pinctrl-names = "default";
+		pinctrl-0 = <&pinctrl_dac_rst>;
+		reg = <0x18>;
+
+		ai31xx-micbias-vg = <MICBIAS_AVDDV>;
+
+		HPVDD-supply = <&buck4_reg>;
+		SPRVDD-supply = <&vdd_input>;
+		SPLVDD-supply = <&vdd_input>;
+		AVDD-supply = <&buck4_reg>;
+		IOVDD-supply = <&buck4_reg>;
+		DVDD-supply = <&buck5_reg>;
+		reset-gpios = <&gpio1 6 GPIO_ACTIVE_LOW>;
+
+		clocks = <&clk IMX8MN_CLK_SAI3_ROOT>;
+		clock-names = "mclk";
+	};
+};
+
+&sai3 {
+	pinctrl-names = "default";
+	pinctrl-0 = <&pinctrl_sai3>;
+	assigned-clocks = <&clk IMX8MN_CLK_SAI3>;
+	assigned-clock-parents = <&clk IMX8MN_AUDIO_PLL1_OUT>;
+	assigned-clock-rates = <24576000>;
+	fsl,sai-mclk-direction-output;
+	status = "okay";
 };
 
 /* eMMC */
@@ -30,6 +94,36 @@ &usdhc1 {
 };
 
 &iomuxc {
+	pinctrl_dac_rst: dac_rst {
+		fsl,pins = <
+			MX8MN_IOMUXC_GPIO1_IO06_GPIO1_IO6		0x19 /* DAC_RST */
+		>;
+	};
+
+	pinctrl_espi2: espi2grp {
+		fsl,pins = <
+			MX8MN_IOMUXC_ECSPI2_SCLK_ECSPI2_SCLK            0x082
+			MX8MN_IOMUXC_ECSPI2_MOSI_ECSPI2_MOSI            0x082
+			MX8MN_IOMUXC_ECSPI2_MISO_ECSPI2_MISO            0x082
+			MX8MN_IOMUXC_ECSPI2_SS0_ECSPI2_SS0		0x040
+		>;
+	};
+
+	pinctrl_i2c2: i2c2grp {
+		fsl,pins = <
+			MX8MN_IOMUXC_I2C2_SCL_I2C2_SCL			0x400000c3
+			MX8MN_IOMUXC_I2C2_SDA_I2C2_SDA			0x400000c3
+		>;
+	};
+
+	pinctrl_sai3: sai3grp {
+		fsl,pins = <
+			MX8MN_IOMUXC_SAI3_TXFS_SAI3_TX_SYNC		0xd6
+			MX8MN_IOMUXC_SAI3_TXC_SAI3_TX_BCLK		0xd6
+			MX8MN_IOMUXC_SAI3_TXD_SAI3_TX_DATA0		0xd6
+		>;
+	};
+
 	pinctrl_usdhc1: usdhc1grp {
 		fsl,pins = <
 			MX8MN_IOMUXC_SD1_CLK_USDHC1_CLK			0x40000090
-- 
2.34.1


^ permalink raw reply related	[flat|nested] 13+ messages in thread

* Re: [PATCH 1/2] ASoC: bindings: fsl-asoc-card: Add compatible for tlv320aic31xx codec
  2022-02-07 16:49 ` Apertis package maintainers
  (?)
@ 2022-02-10 13:24   ` Mark Brown
  -1 siblings, 0 replies; 13+ messages in thread
From: Mark Brown @ 2022-02-10 13:24 UTC (permalink / raw)
  To: Apertis package maintainers
  Cc: alsa-devel, devicetree, kernel, linux-arm-kernel, linux-imx,
	linux-kernel, ariel.dalessandro, festevam, krzysztof.kozlowski,
	lgirdwood, michael, robh+dt, s.hauer, shawnguo

[-- Attachment #1: Type: text/plain, Size: 824 bytes --]

On Mon, Feb 07, 2022 at 01:49:45PM -0300, Apertis package maintainers wrote:
> From: Ariel D'Alessandro <ariel.dalessandro@collabora.com>
> 
> Commit 8c9b9cfb7724 ("ASoC: fsl-asoc-card: Support
> fsl,imx-audio-tlv320aic31xx codec")' added support for tlv320aic31xx
> codec to fsl-asoc-card, but missed the related device-tree compatible
> string documentation. Fix this.
> 
> Signed-off-by: Ariel D'Alessandro <ariel.dalessandro@collabora.com>
> ---

This has a signoff from Ariel but the mail comes from something called
"Apertis package maintainers" and I really can't tell if there's a good
signoff chain here, please see Documentation/process/submitting-patches.rst
for details on what this is and why it's important.  The submission
really needs to come from an actual person who's providing a signoff.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 1/2] ASoC: bindings: fsl-asoc-card: Add compatible for tlv320aic31xx codec
@ 2022-02-10 13:24   ` Mark Brown
  0 siblings, 0 replies; 13+ messages in thread
From: Mark Brown @ 2022-02-10 13:24 UTC (permalink / raw)
  To: Apertis package maintainers
  Cc: devicetree, alsa-devel, ariel.dalessandro, krzysztof.kozlowski,
	shawnguo, s.hauer, linux-kernel, lgirdwood, robh+dt, linux-imx,
	kernel, michael, festevam, linux-arm-kernel

[-- Attachment #1: Type: text/plain, Size: 824 bytes --]

On Mon, Feb 07, 2022 at 01:49:45PM -0300, Apertis package maintainers wrote:
> From: Ariel D'Alessandro <ariel.dalessandro@collabora.com>
> 
> Commit 8c9b9cfb7724 ("ASoC: fsl-asoc-card: Support
> fsl,imx-audio-tlv320aic31xx codec")' added support for tlv320aic31xx
> codec to fsl-asoc-card, but missed the related device-tree compatible
> string documentation. Fix this.
> 
> Signed-off-by: Ariel D'Alessandro <ariel.dalessandro@collabora.com>
> ---

This has a signoff from Ariel but the mail comes from something called
"Apertis package maintainers" and I really can't tell if there's a good
signoff chain here, please see Documentation/process/submitting-patches.rst
for details on what this is and why it's important.  The submission
really needs to come from an actual person who's providing a signoff.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 1/2] ASoC: bindings: fsl-asoc-card: Add compatible for tlv320aic31xx codec
@ 2022-02-10 13:24   ` Mark Brown
  0 siblings, 0 replies; 13+ messages in thread
From: Mark Brown @ 2022-02-10 13:24 UTC (permalink / raw)
  To: Apertis package maintainers
  Cc: alsa-devel, devicetree, kernel, linux-arm-kernel, linux-imx,
	linux-kernel, ariel.dalessandro, festevam, krzysztof.kozlowski,
	lgirdwood, michael, robh+dt, s.hauer, shawnguo


[-- Attachment #1.1: Type: text/plain, Size: 824 bytes --]

On Mon, Feb 07, 2022 at 01:49:45PM -0300, Apertis package maintainers wrote:
> From: Ariel D'Alessandro <ariel.dalessandro@collabora.com>
> 
> Commit 8c9b9cfb7724 ("ASoC: fsl-asoc-card: Support
> fsl,imx-audio-tlv320aic31xx codec")' added support for tlv320aic31xx
> codec to fsl-asoc-card, but missed the related device-tree compatible
> string documentation. Fix this.
> 
> Signed-off-by: Ariel D'Alessandro <ariel.dalessandro@collabora.com>
> ---

This has a signoff from Ariel but the mail comes from something called
"Apertis package maintainers" and I really can't tell if there's a good
signoff chain here, please see Documentation/process/submitting-patches.rst
for details on what this is and why it's important.  The submission
really needs to come from an actual person who's providing a signoff.

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

[-- Attachment #2: Type: text/plain, Size: 176 bytes --]

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 1/2] ASoC: bindings: fsl-asoc-card: Add compatible for tlv320aic31xx codec
  2022-02-10 13:24   ` Mark Brown
  (?)
@ 2022-02-10 13:29     ` Mark Brown
  -1 siblings, 0 replies; 13+ messages in thread
From: Mark Brown @ 2022-02-10 13:29 UTC (permalink / raw)
  To: Apertis package maintainers
  Cc: alsa-devel, devicetree, kernel, linux-arm-kernel, linux-imx,
	linux-kernel, ariel.dalessandro, festevam, krzysztof.kozlowski,
	lgirdwood, michael, robh+dt, s.hauer, shawnguo

[-- Attachment #1: Type: text/plain, Size: 991 bytes --]

On Thu, Feb 10, 2022 at 01:24:34PM +0000, Mark Brown wrote:
> On Mon, Feb 07, 2022 at 01:49:45PM -0300, Apertis package maintainers wrote:
> > From: Ariel D'Alessandro <ariel.dalessandro@collabora.com>
> > 
> > Commit 8c9b9cfb7724 ("ASoC: fsl-asoc-card: Support
> > fsl,imx-audio-tlv320aic31xx codec")' added support for tlv320aic31xx
> > codec to fsl-asoc-card, but missed the related device-tree compatible
> > string documentation. Fix this.
> > 
> > Signed-off-by: Ariel D'Alessandro <ariel.dalessandro@collabora.com>
> > ---
> 
> This has a signoff from Ariel but the mail comes from something called
> "Apertis package maintainers" and I really can't tell if there's a good
> signoff chain here, please see Documentation/process/submitting-patches.rst
> for details on what this is and why it's important.  The submission
> really needs to come from an actual person who's providing a signoff.

...and in fact the address for this group(?) isn't even deliverable :(

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 1/2] ASoC: bindings: fsl-asoc-card: Add compatible for tlv320aic31xx codec
@ 2022-02-10 13:29     ` Mark Brown
  0 siblings, 0 replies; 13+ messages in thread
From: Mark Brown @ 2022-02-10 13:29 UTC (permalink / raw)
  To: Apertis package maintainers
  Cc: devicetree, alsa-devel, ariel.dalessandro, krzysztof.kozlowski,
	shawnguo, s.hauer, linux-kernel, lgirdwood, robh+dt, linux-imx,
	kernel, michael, festevam, linux-arm-kernel

[-- Attachment #1: Type: text/plain, Size: 991 bytes --]

On Thu, Feb 10, 2022 at 01:24:34PM +0000, Mark Brown wrote:
> On Mon, Feb 07, 2022 at 01:49:45PM -0300, Apertis package maintainers wrote:
> > From: Ariel D'Alessandro <ariel.dalessandro@collabora.com>
> > 
> > Commit 8c9b9cfb7724 ("ASoC: fsl-asoc-card: Support
> > fsl,imx-audio-tlv320aic31xx codec")' added support for tlv320aic31xx
> > codec to fsl-asoc-card, but missed the related device-tree compatible
> > string documentation. Fix this.
> > 
> > Signed-off-by: Ariel D'Alessandro <ariel.dalessandro@collabora.com>
> > ---
> 
> This has a signoff from Ariel but the mail comes from something called
> "Apertis package maintainers" and I really can't tell if there's a good
> signoff chain here, please see Documentation/process/submitting-patches.rst
> for details on what this is and why it's important.  The submission
> really needs to come from an actual person who's providing a signoff.

...and in fact the address for this group(?) isn't even deliverable :(

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 1/2] ASoC: bindings: fsl-asoc-card: Add compatible for tlv320aic31xx codec
@ 2022-02-10 13:29     ` Mark Brown
  0 siblings, 0 replies; 13+ messages in thread
From: Mark Brown @ 2022-02-10 13:29 UTC (permalink / raw)
  To: Apertis package maintainers
  Cc: alsa-devel, devicetree, kernel, linux-arm-kernel, linux-imx,
	linux-kernel, ariel.dalessandro, festevam, krzysztof.kozlowski,
	lgirdwood, michael, robh+dt, s.hauer, shawnguo


[-- Attachment #1.1: Type: text/plain, Size: 991 bytes --]

On Thu, Feb 10, 2022 at 01:24:34PM +0000, Mark Brown wrote:
> On Mon, Feb 07, 2022 at 01:49:45PM -0300, Apertis package maintainers wrote:
> > From: Ariel D'Alessandro <ariel.dalessandro@collabora.com>
> > 
> > Commit 8c9b9cfb7724 ("ASoC: fsl-asoc-card: Support
> > fsl,imx-audio-tlv320aic31xx codec")' added support for tlv320aic31xx
> > codec to fsl-asoc-card, but missed the related device-tree compatible
> > string documentation. Fix this.
> > 
> > Signed-off-by: Ariel D'Alessandro <ariel.dalessandro@collabora.com>
> > ---
> 
> This has a signoff from Ariel but the mail comes from something called
> "Apertis package maintainers" and I really can't tell if there's a good
> signoff chain here, please see Documentation/process/submitting-patches.rst
> for details on what this is and why it's important.  The submission
> really needs to come from an actual person who's providing a signoff.

...and in fact the address for this group(?) isn't even deliverable :(

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

[-- Attachment #2: Type: text/plain, Size: 176 bytes --]

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 1/2] ASoC: bindings: fsl-asoc-card: Add compatible for tlv320aic31xx codec
  2022-02-10 13:24   ` Mark Brown
  (?)
@ 2022-02-10 13:30     ` Ariel D'Alessandro
  -1 siblings, 0 replies; 13+ messages in thread
From: Ariel D'Alessandro @ 2022-02-10 13:30 UTC (permalink / raw)
  To: Mark Brown
  Cc: alsa-devel, devicetree, kernel, linux-arm-kernel, linux-imx,
	linux-kernel, festevam, krzysztof.kozlowski, lgirdwood, michael,
	robh+dt, s.hauer, shawnguo

Hi Mark,

On 2/10/22 10:24, Mark Brown wrote:
> On Mon, Feb 07, 2022 at 01:49:45PM -0300, Apertis package maintainers wrote:
>> From: Ariel D'Alessandro <ariel.dalessandro@collabora.com>
>>
>> Commit 8c9b9cfb7724 ("ASoC: fsl-asoc-card: Support
>> fsl,imx-audio-tlv320aic31xx codec")' added support for tlv320aic31xx
>> codec to fsl-asoc-card, but missed the related device-tree compatible
>> string documentation. Fix this.
>>
>> Signed-off-by: Ariel D'Alessandro <ariel.dalessandro@collabora.com>
>> ---
> 
> This has a signoff from Ariel but the mail comes from something called
> "Apertis package maintainers" and I really can't tell if there's a good
> signoff chain here, please see Documentation/process/submitting-patches.rst
> for details on what this is and why it's important.  The submission
> really needs to come from an actual person who's providing a signoff.

Ugh (:hard-facepalm:) totally a misconfiguration on my mail client. Will 
resubmit right away. Sorry for not noticing it when I sent the patchset.

Thanks,
Ariel

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 1/2] ASoC: bindings: fsl-asoc-card: Add compatible for tlv320aic31xx codec
@ 2022-02-10 13:30     ` Ariel D'Alessandro
  0 siblings, 0 replies; 13+ messages in thread
From: Ariel D'Alessandro @ 2022-02-10 13:30 UTC (permalink / raw)
  To: Mark Brown
  Cc: devicetree, alsa-devel, krzysztof.kozlowski, shawnguo, s.hauer,
	linux-kernel, lgirdwood, robh+dt, linux-imx, kernel, michael,
	festevam, linux-arm-kernel

Hi Mark,

On 2/10/22 10:24, Mark Brown wrote:
> On Mon, Feb 07, 2022 at 01:49:45PM -0300, Apertis package maintainers wrote:
>> From: Ariel D'Alessandro <ariel.dalessandro@collabora.com>
>>
>> Commit 8c9b9cfb7724 ("ASoC: fsl-asoc-card: Support
>> fsl,imx-audio-tlv320aic31xx codec")' added support for tlv320aic31xx
>> codec to fsl-asoc-card, but missed the related device-tree compatible
>> string documentation. Fix this.
>>
>> Signed-off-by: Ariel D'Alessandro <ariel.dalessandro@collabora.com>
>> ---
> 
> This has a signoff from Ariel but the mail comes from something called
> "Apertis package maintainers" and I really can't tell if there's a good
> signoff chain here, please see Documentation/process/submitting-patches.rst
> for details on what this is and why it's important.  The submission
> really needs to come from an actual person who's providing a signoff.

Ugh (:hard-facepalm:) totally a misconfiguration on my mail client. Will 
resubmit right away. Sorry for not noticing it when I sent the patchset.

Thanks,
Ariel

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 1/2] ASoC: bindings: fsl-asoc-card: Add compatible for tlv320aic31xx codec
@ 2022-02-10 13:30     ` Ariel D'Alessandro
  0 siblings, 0 replies; 13+ messages in thread
From: Ariel D'Alessandro @ 2022-02-10 13:30 UTC (permalink / raw)
  To: Mark Brown
  Cc: alsa-devel, devicetree, kernel, linux-arm-kernel, linux-imx,
	linux-kernel, festevam, krzysztof.kozlowski, lgirdwood, michael,
	robh+dt, s.hauer, shawnguo

Hi Mark,

On 2/10/22 10:24, Mark Brown wrote:
> On Mon, Feb 07, 2022 at 01:49:45PM -0300, Apertis package maintainers wrote:
>> From: Ariel D'Alessandro <ariel.dalessandro@collabora.com>
>>
>> Commit 8c9b9cfb7724 ("ASoC: fsl-asoc-card: Support
>> fsl,imx-audio-tlv320aic31xx codec")' added support for tlv320aic31xx
>> codec to fsl-asoc-card, but missed the related device-tree compatible
>> string documentation. Fix this.
>>
>> Signed-off-by: Ariel D'Alessandro <ariel.dalessandro@collabora.com>
>> ---
> 
> This has a signoff from Ariel but the mail comes from something called
> "Apertis package maintainers" and I really can't tell if there's a good
> signoff chain here, please see Documentation/process/submitting-patches.rst
> for details on what this is and why it's important.  The submission
> really needs to come from an actual person who's providing a signoff.

Ugh (:hard-facepalm:) totally a misconfiguration on my mail client. Will 
resubmit right away. Sorry for not noticing it when I sent the patchset.

Thanks,
Ariel

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 13+ messages in thread

end of thread, other threads:[~2022-02-10 13:32 UTC | newest]

Thread overview: 13+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-02-07 16:49 [PATCH 1/2] ASoC: bindings: fsl-asoc-card: Add compatible for tlv320aic31xx codec Apertis package maintainers
2022-02-07 16:49 ` Apertis package maintainers
2022-02-07 16:49 ` [PATCH 2/2] arm64: dts: imx8mn-bsh-smm-s2pro: Add tlv320aic31xx audio card node Apertis package maintainers
2022-02-07 16:49   ` Apertis package maintainers
2022-02-10 13:24 ` [PATCH 1/2] ASoC: bindings: fsl-asoc-card: Add compatible for tlv320aic31xx codec Mark Brown
2022-02-10 13:24   ` Mark Brown
2022-02-10 13:24   ` Mark Brown
2022-02-10 13:29   ` Mark Brown
2022-02-10 13:29     ` Mark Brown
2022-02-10 13:29     ` Mark Brown
2022-02-10 13:30   ` Ariel D'Alessandro
2022-02-10 13:30     ` Ariel D'Alessandro
2022-02-10 13:30     ` Ariel D'Alessandro

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.