All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2 0/3] ASoC: sun50i-codec-analog: Internal bias support
@ 2022-06-21  3:54 ` Samuel Holland
  0 siblings, 0 replies; 33+ messages in thread
From: Samuel Holland @ 2022-06-21  3:54 UTC (permalink / raw)
  To: Chen-Yu Tsai, Jernej Skrabec, Liam Girdwood, Mark Brown
  Cc: Arnaud Ferraris, Samuel Holland, Jaroslav Kysela,
	Krzysztof Kozlowski, Maxime Ripard, Rob Herring, Takashi Iwai,
	alsa-devel, devicetree, linux-arm-kernel, linux-kernel,
	linux-sunxi

This series adds support for enabling the codec's internal microphone
bias, which is needed on at least some versions of the PinePhone.

Changes in v2:
 - Move register update from component probe to device probe

Arnaud Ferraris (2):
  ASoC: dt-bindings: sun50i-codec: Add binding for internal bias
  ASoC: sun50i-codec-analog: Add support for internal bias

Samuel Holland (1):
  arm64: dts: allwinner: pinephone: Enable internal HMIC bias

 .../bindings/sound/allwinner,sun50i-a64-codec-analog.yaml | 5 +++++
 .../arm64/boot/dts/allwinner/sun50i-a64-pinephone-1.0.dts | 4 ++++
 .../arm64/boot/dts/allwinner/sun50i-a64-pinephone-1.1.dts | 4 ++++
 sound/soc/sunxi/sun50i-codec-analog.c                     | 8 ++++++++
 4 files changed, 21 insertions(+)

-- 
2.35.1


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

* [PATCH v2 0/3] ASoC: sun50i-codec-analog: Internal bias support
@ 2022-06-21  3:54 ` Samuel Holland
  0 siblings, 0 replies; 33+ messages in thread
From: Samuel Holland @ 2022-06-21  3:54 UTC (permalink / raw)
  To: Chen-Yu Tsai, Jernej Skrabec, Liam Girdwood, Mark Brown
  Cc: devicetree, Arnaud Ferraris, alsa-devel, Samuel Holland,
	linux-kernel, Takashi Iwai, Maxime Ripard, Rob Herring,
	Krzysztof Kozlowski, linux-sunxi, linux-arm-kernel

This series adds support for enabling the codec's internal microphone
bias, which is needed on at least some versions of the PinePhone.

Changes in v2:
 - Move register update from component probe to device probe

Arnaud Ferraris (2):
  ASoC: dt-bindings: sun50i-codec: Add binding for internal bias
  ASoC: sun50i-codec-analog: Add support for internal bias

Samuel Holland (1):
  arm64: dts: allwinner: pinephone: Enable internal HMIC bias

 .../bindings/sound/allwinner,sun50i-a64-codec-analog.yaml | 5 +++++
 .../arm64/boot/dts/allwinner/sun50i-a64-pinephone-1.0.dts | 4 ++++
 .../arm64/boot/dts/allwinner/sun50i-a64-pinephone-1.1.dts | 4 ++++
 sound/soc/sunxi/sun50i-codec-analog.c                     | 8 ++++++++
 4 files changed, 21 insertions(+)

-- 
2.35.1


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

* [PATCH v2 0/3] ASoC: sun50i-codec-analog: Internal bias support
@ 2022-06-21  3:54 ` Samuel Holland
  0 siblings, 0 replies; 33+ messages in thread
From: Samuel Holland @ 2022-06-21  3:54 UTC (permalink / raw)
  To: Chen-Yu Tsai, Jernej Skrabec, Liam Girdwood, Mark Brown
  Cc: Arnaud Ferraris, Samuel Holland, Jaroslav Kysela,
	Krzysztof Kozlowski, Maxime Ripard, Rob Herring, Takashi Iwai,
	alsa-devel, devicetree, linux-arm-kernel, linux-kernel,
	linux-sunxi

This series adds support for enabling the codec's internal microphone
bias, which is needed on at least some versions of the PinePhone.

Changes in v2:
 - Move register update from component probe to device probe

Arnaud Ferraris (2):
  ASoC: dt-bindings: sun50i-codec: Add binding for internal bias
  ASoC: sun50i-codec-analog: Add support for internal bias

Samuel Holland (1):
  arm64: dts: allwinner: pinephone: Enable internal HMIC bias

 .../bindings/sound/allwinner,sun50i-a64-codec-analog.yaml | 5 +++++
 .../arm64/boot/dts/allwinner/sun50i-a64-pinephone-1.0.dts | 4 ++++
 .../arm64/boot/dts/allwinner/sun50i-a64-pinephone-1.1.dts | 4 ++++
 sound/soc/sunxi/sun50i-codec-analog.c                     | 8 ++++++++
 4 files changed, 21 insertions(+)

-- 
2.35.1


_______________________________________________
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] 33+ messages in thread

* [PATCH v2 1/3] ASoC: dt-bindings: sun50i-codec: Add binding for internal bias
  2022-06-21  3:54 ` Samuel Holland
  (?)
@ 2022-06-21  3:54   ` Samuel Holland
  -1 siblings, 0 replies; 33+ messages in thread
From: Samuel Holland @ 2022-06-21  3:54 UTC (permalink / raw)
  To: Chen-Yu Tsai, Jernej Skrabec, Liam Girdwood, Mark Brown
  Cc: Arnaud Ferraris, Samuel Holland, Jaroslav Kysela,
	Krzysztof Kozlowski, Maxime Ripard, Rob Herring, Takashi Iwai,
	alsa-devel, devicetree, linux-arm-kernel, linux-kernel,
	linux-sunxi

From: Arnaud Ferraris <arnaud.ferraris@collabora.com>

In order to properly bias headset microphones, there should be a pull-up
resistor between pins HBIAS and MIC2P. This can be an external resistor,
but the codec also provides an internal 2.2K resistor which is enabled
by a register.

This patch adds a device-tree property to the sun50i-codec-analog driver
to take advantage of this feature.

Signed-off-by: Arnaud Ferraris <arnaud.ferraris@collabora.com>
[Samuel: split binding and implementation patches]
Signed-off-by: Samuel Holland <samuel@sholland.org>
---

(no changes since v1)

 .../bindings/sound/allwinner,sun50i-a64-codec-analog.yaml    | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/Documentation/devicetree/bindings/sound/allwinner,sun50i-a64-codec-analog.yaml b/Documentation/devicetree/bindings/sound/allwinner,sun50i-a64-codec-analog.yaml
index 3b764415c9ab..66859eb8f79a 100644
--- a/Documentation/devicetree/bindings/sound/allwinner,sun50i-a64-codec-analog.yaml
+++ b/Documentation/devicetree/bindings/sound/allwinner,sun50i-a64-codec-analog.yaml
@@ -21,6 +21,11 @@ properties:
     description:
       Regulator for the headphone amplifier
 
+  allwinner,internal-bias-resistor:
+    description:
+      Enable the internal 2.2K bias resistor between HBIAS and MICDET pins
+    type: boolean
+
 required:
   - compatible
   - reg
-- 
2.35.1


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

* [PATCH v2 1/3] ASoC: dt-bindings: sun50i-codec: Add binding for internal bias
@ 2022-06-21  3:54   ` Samuel Holland
  0 siblings, 0 replies; 33+ messages in thread
From: Samuel Holland @ 2022-06-21  3:54 UTC (permalink / raw)
  To: Chen-Yu Tsai, Jernej Skrabec, Liam Girdwood, Mark Brown
  Cc: devicetree, Arnaud Ferraris, alsa-devel, Samuel Holland,
	linux-kernel, Takashi Iwai, Maxime Ripard, Rob Herring,
	Krzysztof Kozlowski, linux-sunxi, linux-arm-kernel

From: Arnaud Ferraris <arnaud.ferraris@collabora.com>

In order to properly bias headset microphones, there should be a pull-up
resistor between pins HBIAS and MIC2P. This can be an external resistor,
but the codec also provides an internal 2.2K resistor which is enabled
by a register.

This patch adds a device-tree property to the sun50i-codec-analog driver
to take advantage of this feature.

Signed-off-by: Arnaud Ferraris <arnaud.ferraris@collabora.com>
[Samuel: split binding and implementation patches]
Signed-off-by: Samuel Holland <samuel@sholland.org>
---

(no changes since v1)

 .../bindings/sound/allwinner,sun50i-a64-codec-analog.yaml    | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/Documentation/devicetree/bindings/sound/allwinner,sun50i-a64-codec-analog.yaml b/Documentation/devicetree/bindings/sound/allwinner,sun50i-a64-codec-analog.yaml
index 3b764415c9ab..66859eb8f79a 100644
--- a/Documentation/devicetree/bindings/sound/allwinner,sun50i-a64-codec-analog.yaml
+++ b/Documentation/devicetree/bindings/sound/allwinner,sun50i-a64-codec-analog.yaml
@@ -21,6 +21,11 @@ properties:
     description:
       Regulator for the headphone amplifier
 
+  allwinner,internal-bias-resistor:
+    description:
+      Enable the internal 2.2K bias resistor between HBIAS and MICDET pins
+    type: boolean
+
 required:
   - compatible
   - reg
-- 
2.35.1


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

* [PATCH v2 1/3] ASoC: dt-bindings: sun50i-codec: Add binding for internal bias
@ 2022-06-21  3:54   ` Samuel Holland
  0 siblings, 0 replies; 33+ messages in thread
From: Samuel Holland @ 2022-06-21  3:54 UTC (permalink / raw)
  To: Chen-Yu Tsai, Jernej Skrabec, Liam Girdwood, Mark Brown
  Cc: Arnaud Ferraris, Samuel Holland, Jaroslav Kysela,
	Krzysztof Kozlowski, Maxime Ripard, Rob Herring, Takashi Iwai,
	alsa-devel, devicetree, linux-arm-kernel, linux-kernel,
	linux-sunxi

From: Arnaud Ferraris <arnaud.ferraris@collabora.com>

In order to properly bias headset microphones, there should be a pull-up
resistor between pins HBIAS and MIC2P. This can be an external resistor,
but the codec also provides an internal 2.2K resistor which is enabled
by a register.

This patch adds a device-tree property to the sun50i-codec-analog driver
to take advantage of this feature.

Signed-off-by: Arnaud Ferraris <arnaud.ferraris@collabora.com>
[Samuel: split binding and implementation patches]
Signed-off-by: Samuel Holland <samuel@sholland.org>
---

(no changes since v1)

 .../bindings/sound/allwinner,sun50i-a64-codec-analog.yaml    | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/Documentation/devicetree/bindings/sound/allwinner,sun50i-a64-codec-analog.yaml b/Documentation/devicetree/bindings/sound/allwinner,sun50i-a64-codec-analog.yaml
index 3b764415c9ab..66859eb8f79a 100644
--- a/Documentation/devicetree/bindings/sound/allwinner,sun50i-a64-codec-analog.yaml
+++ b/Documentation/devicetree/bindings/sound/allwinner,sun50i-a64-codec-analog.yaml
@@ -21,6 +21,11 @@ properties:
     description:
       Regulator for the headphone amplifier
 
+  allwinner,internal-bias-resistor:
+    description:
+      Enable the internal 2.2K bias resistor between HBIAS and MICDET pins
+    type: boolean
+
 required:
   - compatible
   - reg
-- 
2.35.1


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

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

* [PATCH v2 2/3] ASoC: sun50i-codec-analog: Add support for internal bias
  2022-06-21  3:54 ` Samuel Holland
  (?)
@ 2022-06-21  3:54   ` Samuel Holland
  -1 siblings, 0 replies; 33+ messages in thread
From: Samuel Holland @ 2022-06-21  3:54 UTC (permalink / raw)
  To: Chen-Yu Tsai, Jernej Skrabec, Liam Girdwood, Mark Brown
  Cc: Arnaud Ferraris, Samuel Holland, Jaroslav Kysela,
	Krzysztof Kozlowski, Maxime Ripard, Rob Herring, Takashi Iwai,
	alsa-devel, devicetree, linux-arm-kernel, linux-kernel,
	linux-sunxi

From: Arnaud Ferraris <arnaud.ferraris@collabora.com>

In order to properly bias headset microphones, there should be a pull-up
resistor between pins HBIAS and MIC2P. This can be an external resistor,
but the codec also provides an internal 2.2K resistor which is enabled
by a register.

This patch enables or disables the internal bias resistor based on a
device tree property.

Signed-off-by: Arnaud Ferraris <arnaud.ferraris@collabora.com>
[Samuel: split binding and implementation; move to device probe]
Signed-off-by: Samuel Holland <samuel@sholland.org>
---

Changes in v2:
 - Move register update from component probe to device probe

 sound/soc/sunxi/sun50i-codec-analog.c | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/sound/soc/sunxi/sun50i-codec-analog.c b/sound/soc/sunxi/sun50i-codec-analog.c
index a41e25ad0aaf..e1e5e8de0130 100644
--- a/sound/soc/sunxi/sun50i-codec-analog.c
+++ b/sound/soc/sunxi/sun50i-codec-analog.c
@@ -117,6 +117,7 @@
 #define SUN50I_ADDA_HS_MBIAS_CTRL_MMICBIASEN	7
 
 #define SUN50I_ADDA_JACK_MIC_CTRL	0x1d
+#define SUN50I_ADDA_JACK_MIC_CTRL_INNERRESEN	6
 #define SUN50I_ADDA_JACK_MIC_CTRL_HMICBIASEN	5
 
 /* mixer controls */
@@ -507,6 +508,7 @@ static int sun50i_codec_analog_probe(struct platform_device *pdev)
 {
 	struct regmap *regmap;
 	void __iomem *base;
+	bool enable;
 
 	base = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(base)) {
@@ -520,6 +522,12 @@ static int sun50i_codec_analog_probe(struct platform_device *pdev)
 		return PTR_ERR(regmap);
 	}
 
+	enable = device_property_read_bool(&pdev->dev,
+					   "allwinner,internal-bias-resistor");
+	regmap_update_bits(regmap, SUN50I_ADDA_JACK_MIC_CTRL,
+			   BIT(SUN50I_ADDA_JACK_MIC_CTRL_INNERRESEN),
+			   enable << SUN50I_ADDA_JACK_MIC_CTRL_INNERRESEN);
+
 	return devm_snd_soc_register_component(&pdev->dev,
 					       &sun50i_codec_analog_cmpnt_drv,
 					       NULL, 0);
-- 
2.35.1


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

* [PATCH v2 2/3] ASoC: sun50i-codec-analog: Add support for internal bias
@ 2022-06-21  3:54   ` Samuel Holland
  0 siblings, 0 replies; 33+ messages in thread
From: Samuel Holland @ 2022-06-21  3:54 UTC (permalink / raw)
  To: Chen-Yu Tsai, Jernej Skrabec, Liam Girdwood, Mark Brown
  Cc: devicetree, Arnaud Ferraris, alsa-devel, Samuel Holland,
	linux-kernel, Takashi Iwai, Maxime Ripard, Rob Herring,
	Krzysztof Kozlowski, linux-sunxi, linux-arm-kernel

From: Arnaud Ferraris <arnaud.ferraris@collabora.com>

In order to properly bias headset microphones, there should be a pull-up
resistor between pins HBIAS and MIC2P. This can be an external resistor,
but the codec also provides an internal 2.2K resistor which is enabled
by a register.

This patch enables or disables the internal bias resistor based on a
device tree property.

Signed-off-by: Arnaud Ferraris <arnaud.ferraris@collabora.com>
[Samuel: split binding and implementation; move to device probe]
Signed-off-by: Samuel Holland <samuel@sholland.org>
---

Changes in v2:
 - Move register update from component probe to device probe

 sound/soc/sunxi/sun50i-codec-analog.c | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/sound/soc/sunxi/sun50i-codec-analog.c b/sound/soc/sunxi/sun50i-codec-analog.c
index a41e25ad0aaf..e1e5e8de0130 100644
--- a/sound/soc/sunxi/sun50i-codec-analog.c
+++ b/sound/soc/sunxi/sun50i-codec-analog.c
@@ -117,6 +117,7 @@
 #define SUN50I_ADDA_HS_MBIAS_CTRL_MMICBIASEN	7
 
 #define SUN50I_ADDA_JACK_MIC_CTRL	0x1d
+#define SUN50I_ADDA_JACK_MIC_CTRL_INNERRESEN	6
 #define SUN50I_ADDA_JACK_MIC_CTRL_HMICBIASEN	5
 
 /* mixer controls */
@@ -507,6 +508,7 @@ static int sun50i_codec_analog_probe(struct platform_device *pdev)
 {
 	struct regmap *regmap;
 	void __iomem *base;
+	bool enable;
 
 	base = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(base)) {
@@ -520,6 +522,12 @@ static int sun50i_codec_analog_probe(struct platform_device *pdev)
 		return PTR_ERR(regmap);
 	}
 
+	enable = device_property_read_bool(&pdev->dev,
+					   "allwinner,internal-bias-resistor");
+	regmap_update_bits(regmap, SUN50I_ADDA_JACK_MIC_CTRL,
+			   BIT(SUN50I_ADDA_JACK_MIC_CTRL_INNERRESEN),
+			   enable << SUN50I_ADDA_JACK_MIC_CTRL_INNERRESEN);
+
 	return devm_snd_soc_register_component(&pdev->dev,
 					       &sun50i_codec_analog_cmpnt_drv,
 					       NULL, 0);
-- 
2.35.1


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

* [PATCH v2 2/3] ASoC: sun50i-codec-analog: Add support for internal bias
@ 2022-06-21  3:54   ` Samuel Holland
  0 siblings, 0 replies; 33+ messages in thread
From: Samuel Holland @ 2022-06-21  3:54 UTC (permalink / raw)
  To: Chen-Yu Tsai, Jernej Skrabec, Liam Girdwood, Mark Brown
  Cc: Arnaud Ferraris, Samuel Holland, Jaroslav Kysela,
	Krzysztof Kozlowski, Maxime Ripard, Rob Herring, Takashi Iwai,
	alsa-devel, devicetree, linux-arm-kernel, linux-kernel,
	linux-sunxi

From: Arnaud Ferraris <arnaud.ferraris@collabora.com>

In order to properly bias headset microphones, there should be a pull-up
resistor between pins HBIAS and MIC2P. This can be an external resistor,
but the codec also provides an internal 2.2K resistor which is enabled
by a register.

This patch enables or disables the internal bias resistor based on a
device tree property.

Signed-off-by: Arnaud Ferraris <arnaud.ferraris@collabora.com>
[Samuel: split binding and implementation; move to device probe]
Signed-off-by: Samuel Holland <samuel@sholland.org>
---

Changes in v2:
 - Move register update from component probe to device probe

 sound/soc/sunxi/sun50i-codec-analog.c | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/sound/soc/sunxi/sun50i-codec-analog.c b/sound/soc/sunxi/sun50i-codec-analog.c
index a41e25ad0aaf..e1e5e8de0130 100644
--- a/sound/soc/sunxi/sun50i-codec-analog.c
+++ b/sound/soc/sunxi/sun50i-codec-analog.c
@@ -117,6 +117,7 @@
 #define SUN50I_ADDA_HS_MBIAS_CTRL_MMICBIASEN	7
 
 #define SUN50I_ADDA_JACK_MIC_CTRL	0x1d
+#define SUN50I_ADDA_JACK_MIC_CTRL_INNERRESEN	6
 #define SUN50I_ADDA_JACK_MIC_CTRL_HMICBIASEN	5
 
 /* mixer controls */
@@ -507,6 +508,7 @@ static int sun50i_codec_analog_probe(struct platform_device *pdev)
 {
 	struct regmap *regmap;
 	void __iomem *base;
+	bool enable;
 
 	base = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(base)) {
@@ -520,6 +522,12 @@ static int sun50i_codec_analog_probe(struct platform_device *pdev)
 		return PTR_ERR(regmap);
 	}
 
+	enable = device_property_read_bool(&pdev->dev,
+					   "allwinner,internal-bias-resistor");
+	regmap_update_bits(regmap, SUN50I_ADDA_JACK_MIC_CTRL,
+			   BIT(SUN50I_ADDA_JACK_MIC_CTRL_INNERRESEN),
+			   enable << SUN50I_ADDA_JACK_MIC_CTRL_INNERRESEN);
+
 	return devm_snd_soc_register_component(&pdev->dev,
 					       &sun50i_codec_analog_cmpnt_drv,
 					       NULL, 0);
-- 
2.35.1


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

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

* [PATCH v2 3/3] arm64: dts: allwinner: pinephone: Enable internal HMIC bias
  2022-06-21  3:54 ` Samuel Holland
  (?)
@ 2022-06-21  3:54   ` Samuel Holland
  -1 siblings, 0 replies; 33+ messages in thread
From: Samuel Holland @ 2022-06-21  3:54 UTC (permalink / raw)
  To: Chen-Yu Tsai, Jernej Skrabec, Liam Girdwood, Mark Brown
  Cc: Arnaud Ferraris, Samuel Holland, Jaroslav Kysela,
	Krzysztof Kozlowski, Maxime Ripard, Rob Herring, Takashi Iwai,
	alsa-devel, devicetree, linux-arm-kernel, linux-kernel,
	linux-sunxi

Revisions 1.0 and 1.1 of the PinePhone mainboard do not have an external
resistor connecting HBIAS to MIC2P. Enable the internal resistor to
provide the necessary headeset microphone bias.

Signed-off-by: Samuel Holland <samuel@sholland.org>
---

(no changes since v1)

 arch/arm64/boot/dts/allwinner/sun50i-a64-pinephone-1.0.dts | 4 ++++
 arch/arm64/boot/dts/allwinner/sun50i-a64-pinephone-1.1.dts | 4 ++++
 2 files changed, 8 insertions(+)

diff --git a/arch/arm64/boot/dts/allwinner/sun50i-a64-pinephone-1.0.dts b/arch/arm64/boot/dts/allwinner/sun50i-a64-pinephone-1.0.dts
index fb65319a3bd3..219f720b8b7d 100644
--- a/arch/arm64/boot/dts/allwinner/sun50i-a64-pinephone-1.0.dts
+++ b/arch/arm64/boot/dts/allwinner/sun50i-a64-pinephone-1.0.dts
@@ -10,6 +10,10 @@ / {
 	compatible = "pine64,pinephone-1.0", "pine64,pinephone", "allwinner,sun50i-a64";
 };
 
+&codec_analog {
+	allwinner,internal-bias-resistor;
+};
+
 &sgm3140 {
 	enable-gpios = <&pio 2 3 GPIO_ACTIVE_HIGH>; /* PC3 */
 	flash-gpios = <&pio 3 24 GPIO_ACTIVE_HIGH>; /* PD24 */
diff --git a/arch/arm64/boot/dts/allwinner/sun50i-a64-pinephone-1.1.dts b/arch/arm64/boot/dts/allwinner/sun50i-a64-pinephone-1.1.dts
index 5e59d3752178..723af64a9cee 100644
--- a/arch/arm64/boot/dts/allwinner/sun50i-a64-pinephone-1.1.dts
+++ b/arch/arm64/boot/dts/allwinner/sun50i-a64-pinephone-1.1.dts
@@ -29,6 +29,10 @@ &backlight {
 	default-brightness-level = <400>;
 };
 
+&codec_analog {
+	allwinner,internal-bias-resistor;
+};
+
 &sgm3140 {
 	enable-gpios = <&pio 3 24 GPIO_ACTIVE_HIGH>; /* PD24 */
 	flash-gpios = <&pio 2 3 GPIO_ACTIVE_HIGH>; /* PC3 */
-- 
2.35.1


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

* [PATCH v2 3/3] arm64: dts: allwinner: pinephone: Enable internal HMIC bias
@ 2022-06-21  3:54   ` Samuel Holland
  0 siblings, 0 replies; 33+ messages in thread
From: Samuel Holland @ 2022-06-21  3:54 UTC (permalink / raw)
  To: Chen-Yu Tsai, Jernej Skrabec, Liam Girdwood, Mark Brown
  Cc: devicetree, Arnaud Ferraris, alsa-devel, Samuel Holland,
	linux-kernel, Takashi Iwai, Maxime Ripard, Rob Herring,
	Krzysztof Kozlowski, linux-sunxi, linux-arm-kernel

Revisions 1.0 and 1.1 of the PinePhone mainboard do not have an external
resistor connecting HBIAS to MIC2P. Enable the internal resistor to
provide the necessary headeset microphone bias.

Signed-off-by: Samuel Holland <samuel@sholland.org>
---

(no changes since v1)

 arch/arm64/boot/dts/allwinner/sun50i-a64-pinephone-1.0.dts | 4 ++++
 arch/arm64/boot/dts/allwinner/sun50i-a64-pinephone-1.1.dts | 4 ++++
 2 files changed, 8 insertions(+)

diff --git a/arch/arm64/boot/dts/allwinner/sun50i-a64-pinephone-1.0.dts b/arch/arm64/boot/dts/allwinner/sun50i-a64-pinephone-1.0.dts
index fb65319a3bd3..219f720b8b7d 100644
--- a/arch/arm64/boot/dts/allwinner/sun50i-a64-pinephone-1.0.dts
+++ b/arch/arm64/boot/dts/allwinner/sun50i-a64-pinephone-1.0.dts
@@ -10,6 +10,10 @@ / {
 	compatible = "pine64,pinephone-1.0", "pine64,pinephone", "allwinner,sun50i-a64";
 };
 
+&codec_analog {
+	allwinner,internal-bias-resistor;
+};
+
 &sgm3140 {
 	enable-gpios = <&pio 2 3 GPIO_ACTIVE_HIGH>; /* PC3 */
 	flash-gpios = <&pio 3 24 GPIO_ACTIVE_HIGH>; /* PD24 */
diff --git a/arch/arm64/boot/dts/allwinner/sun50i-a64-pinephone-1.1.dts b/arch/arm64/boot/dts/allwinner/sun50i-a64-pinephone-1.1.dts
index 5e59d3752178..723af64a9cee 100644
--- a/arch/arm64/boot/dts/allwinner/sun50i-a64-pinephone-1.1.dts
+++ b/arch/arm64/boot/dts/allwinner/sun50i-a64-pinephone-1.1.dts
@@ -29,6 +29,10 @@ &backlight {
 	default-brightness-level = <400>;
 };
 
+&codec_analog {
+	allwinner,internal-bias-resistor;
+};
+
 &sgm3140 {
 	enable-gpios = <&pio 3 24 GPIO_ACTIVE_HIGH>; /* PD24 */
 	flash-gpios = <&pio 2 3 GPIO_ACTIVE_HIGH>; /* PC3 */
-- 
2.35.1


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

* [PATCH v2 3/3] arm64: dts: allwinner: pinephone: Enable internal HMIC bias
@ 2022-06-21  3:54   ` Samuel Holland
  0 siblings, 0 replies; 33+ messages in thread
From: Samuel Holland @ 2022-06-21  3:54 UTC (permalink / raw)
  To: Chen-Yu Tsai, Jernej Skrabec, Liam Girdwood, Mark Brown
  Cc: Arnaud Ferraris, Samuel Holland, Jaroslav Kysela,
	Krzysztof Kozlowski, Maxime Ripard, Rob Herring, Takashi Iwai,
	alsa-devel, devicetree, linux-arm-kernel, linux-kernel,
	linux-sunxi

Revisions 1.0 and 1.1 of the PinePhone mainboard do not have an external
resistor connecting HBIAS to MIC2P. Enable the internal resistor to
provide the necessary headeset microphone bias.

Signed-off-by: Samuel Holland <samuel@sholland.org>
---

(no changes since v1)

 arch/arm64/boot/dts/allwinner/sun50i-a64-pinephone-1.0.dts | 4 ++++
 arch/arm64/boot/dts/allwinner/sun50i-a64-pinephone-1.1.dts | 4 ++++
 2 files changed, 8 insertions(+)

diff --git a/arch/arm64/boot/dts/allwinner/sun50i-a64-pinephone-1.0.dts b/arch/arm64/boot/dts/allwinner/sun50i-a64-pinephone-1.0.dts
index fb65319a3bd3..219f720b8b7d 100644
--- a/arch/arm64/boot/dts/allwinner/sun50i-a64-pinephone-1.0.dts
+++ b/arch/arm64/boot/dts/allwinner/sun50i-a64-pinephone-1.0.dts
@@ -10,6 +10,10 @@ / {
 	compatible = "pine64,pinephone-1.0", "pine64,pinephone", "allwinner,sun50i-a64";
 };
 
+&codec_analog {
+	allwinner,internal-bias-resistor;
+};
+
 &sgm3140 {
 	enable-gpios = <&pio 2 3 GPIO_ACTIVE_HIGH>; /* PC3 */
 	flash-gpios = <&pio 3 24 GPIO_ACTIVE_HIGH>; /* PD24 */
diff --git a/arch/arm64/boot/dts/allwinner/sun50i-a64-pinephone-1.1.dts b/arch/arm64/boot/dts/allwinner/sun50i-a64-pinephone-1.1.dts
index 5e59d3752178..723af64a9cee 100644
--- a/arch/arm64/boot/dts/allwinner/sun50i-a64-pinephone-1.1.dts
+++ b/arch/arm64/boot/dts/allwinner/sun50i-a64-pinephone-1.1.dts
@@ -29,6 +29,10 @@ &backlight {
 	default-brightness-level = <400>;
 };
 
+&codec_analog {
+	allwinner,internal-bias-resistor;
+};
+
 &sgm3140 {
 	enable-gpios = <&pio 3 24 GPIO_ACTIVE_HIGH>; /* PD24 */
 	flash-gpios = <&pio 2 3 GPIO_ACTIVE_HIGH>; /* PC3 */
-- 
2.35.1


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

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

* Re: (subset) [PATCH v2 0/3] ASoC: sun50i-codec-analog: Internal bias support
  2022-06-21  3:54 ` Samuel Holland
  (?)
@ 2022-06-27 20:46   ` Mark Brown
  -1 siblings, 0 replies; 33+ messages in thread
From: Mark Brown @ 2022-06-27 20:46 UTC (permalink / raw)
  To: jernej.skrabec, lgirdwood, samuel, wens
  Cc: robh+dt, linux-arm-kernel, perex, arnaud.ferraris, devicetree,
	tiwai, linux-sunxi, mripard, alsa-devel, linux-kernel,
	krzysztof.kozlowski+dt

On Mon, 20 Jun 2022 22:54:49 -0500, Samuel Holland wrote:
> This series adds support for enabling the codec's internal microphone
> bias, which is needed on at least some versions of the PinePhone.
> 
> Changes in v2:
>  - Move register update from component probe to device probe
> 
> Arnaud Ferraris (2):
>   ASoC: dt-bindings: sun50i-codec: Add binding for internal bias
>   ASoC: sun50i-codec-analog: Add support for internal bias
> 
> [...]

Applied to

   https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next

Thanks!

[1/3] ASoC: dt-bindings: sun50i-codec: Add binding for internal bias
      commit: 24e0b04dd42be34ec4b18dc1a1e139d66eb572a3
[2/3] ASoC: sun50i-codec-analog: Add support for internal bias
      commit: 25ae1a04da0d32c22db0b018e5668129b91fa104

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

Thanks,
Mark

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

* Re: (subset) [PATCH v2 0/3] ASoC: sun50i-codec-analog: Internal bias support
@ 2022-06-27 20:46   ` Mark Brown
  0 siblings, 0 replies; 33+ messages in thread
From: Mark Brown @ 2022-06-27 20:46 UTC (permalink / raw)
  To: jernej.skrabec, lgirdwood, samuel, wens
  Cc: devicetree, arnaud.ferraris, alsa-devel, linux-kernel, tiwai,
	mripard, robh+dt, krzysztof.kozlowski+dt, linux-sunxi,
	linux-arm-kernel

On Mon, 20 Jun 2022 22:54:49 -0500, Samuel Holland wrote:
> This series adds support for enabling the codec's internal microphone
> bias, which is needed on at least some versions of the PinePhone.
> 
> Changes in v2:
>  - Move register update from component probe to device probe
> 
> Arnaud Ferraris (2):
>   ASoC: dt-bindings: sun50i-codec: Add binding for internal bias
>   ASoC: sun50i-codec-analog: Add support for internal bias
> 
> [...]

Applied to

   https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next

Thanks!

[1/3] ASoC: dt-bindings: sun50i-codec: Add binding for internal bias
      commit: 24e0b04dd42be34ec4b18dc1a1e139d66eb572a3
[2/3] ASoC: sun50i-codec-analog: Add support for internal bias
      commit: 25ae1a04da0d32c22db0b018e5668129b91fa104

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

Thanks,
Mark

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

* Re: (subset) [PATCH v2 0/3] ASoC: sun50i-codec-analog: Internal bias support
@ 2022-06-27 20:46   ` Mark Brown
  0 siblings, 0 replies; 33+ messages in thread
From: Mark Brown @ 2022-06-27 20:46 UTC (permalink / raw)
  To: jernej.skrabec, lgirdwood, samuel, wens
  Cc: robh+dt, linux-arm-kernel, perex, arnaud.ferraris, devicetree,
	tiwai, linux-sunxi, mripard, alsa-devel, linux-kernel,
	krzysztof.kozlowski+dt

On Mon, 20 Jun 2022 22:54:49 -0500, Samuel Holland wrote:
> This series adds support for enabling the codec's internal microphone
> bias, which is needed on at least some versions of the PinePhone.
> 
> Changes in v2:
>  - Move register update from component probe to device probe
> 
> Arnaud Ferraris (2):
>   ASoC: dt-bindings: sun50i-codec: Add binding for internal bias
>   ASoC: sun50i-codec-analog: Add support for internal bias
> 
> [...]

Applied to

   https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next

Thanks!

[1/3] ASoC: dt-bindings: sun50i-codec: Add binding for internal bias
      commit: 24e0b04dd42be34ec4b18dc1a1e139d66eb572a3
[2/3] ASoC: sun50i-codec-analog: Add support for internal bias
      commit: 25ae1a04da0d32c22db0b018e5668129b91fa104

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

Thanks,
Mark

_______________________________________________
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] 33+ messages in thread

* Re: [PATCH v2 3/3] arm64: dts: allwinner: pinephone: Enable internal HMIC bias
  2022-06-21  3:54   ` Samuel Holland
  (?)
@ 2022-06-27 20:53     ` Jernej Škrabec
  -1 siblings, 0 replies; 33+ messages in thread
From: Jernej Škrabec @ 2022-06-27 20:53 UTC (permalink / raw)
  To: Chen-Yu Tsai, Liam Girdwood, Mark Brown, Samuel Holland
  Cc: Arnaud Ferraris, Samuel Holland, Jaroslav Kysela,
	Krzysztof Kozlowski, Maxime Ripard, Rob Herring, Takashi Iwai,
	alsa-devel, devicetree, linux-arm-kernel, linux-kernel,
	linux-sunxi

Dne torek, 21. junij 2022 ob 05:54:52 CEST je Samuel Holland napisal(a):
> Revisions 1.0 and 1.1 of the PinePhone mainboard do not have an external
> resistor connecting HBIAS to MIC2P. Enable the internal resistor to
> provide the necessary headeset microphone bias.
> 
> Signed-off-by: Samuel Holland <samuel@sholland.org>

Reviewed-by: Jernej Skrabec <jernej.skrabec@gmail.com>

Best regards,
Jernej



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

* Re: [PATCH v2 3/3] arm64: dts: allwinner: pinephone: Enable internal HMIC bias
@ 2022-06-27 20:53     ` Jernej Škrabec
  0 siblings, 0 replies; 33+ messages in thread
From: Jernej Škrabec @ 2022-06-27 20:53 UTC (permalink / raw)
  To: Chen-Yu Tsai, Liam Girdwood, Mark Brown, Samuel Holland
  Cc: devicetree, Arnaud Ferraris, alsa-devel, Samuel Holland,
	linux-kernel, Takashi Iwai, Maxime Ripard, Rob Herring,
	Krzysztof Kozlowski, linux-sunxi, linux-arm-kernel

Dne torek, 21. junij 2022 ob 05:54:52 CEST je Samuel Holland napisal(a):
> Revisions 1.0 and 1.1 of the PinePhone mainboard do not have an external
> resistor connecting HBIAS to MIC2P. Enable the internal resistor to
> provide the necessary headeset microphone bias.
> 
> Signed-off-by: Samuel Holland <samuel@sholland.org>

Reviewed-by: Jernej Skrabec <jernej.skrabec@gmail.com>

Best regards,
Jernej



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

* Re: [PATCH v2 3/3] arm64: dts: allwinner: pinephone: Enable internal HMIC bias
@ 2022-06-27 20:53     ` Jernej Škrabec
  0 siblings, 0 replies; 33+ messages in thread
From: Jernej Škrabec @ 2022-06-27 20:53 UTC (permalink / raw)
  To: Chen-Yu Tsai, Liam Girdwood, Mark Brown, Samuel Holland
  Cc: Arnaud Ferraris, Samuel Holland, Jaroslav Kysela,
	Krzysztof Kozlowski, Maxime Ripard, Rob Herring, Takashi Iwai,
	alsa-devel, devicetree, linux-arm-kernel, linux-kernel,
	linux-sunxi

Dne torek, 21. junij 2022 ob 05:54:52 CEST je Samuel Holland napisal(a):
> Revisions 1.0 and 1.1 of the PinePhone mainboard do not have an external
> resistor connecting HBIAS to MIC2P. Enable the internal resistor to
> provide the necessary headeset microphone bias.
> 
> Signed-off-by: Samuel Holland <samuel@sholland.org>

Reviewed-by: Jernej Skrabec <jernej.skrabec@gmail.com>

Best regards,
Jernej



_______________________________________________
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] 33+ messages in thread

* Re: Re: (subset) [PATCH v2 0/3] ASoC: sun50i-codec-analog: Internal bias support
  2022-06-27 20:46   ` Mark Brown
  (?)
@ 2022-07-05 20:12     ` Jernej Škrabec
  -1 siblings, 0 replies; 33+ messages in thread
From: Jernej Škrabec @ 2022-07-05 20:12 UTC (permalink / raw)
  To: lgirdwood, samuel, wens, Mark Brown
  Cc: robh+dt, linux-arm-kernel, perex, arnaud.ferraris, devicetree,
	tiwai, linux-sunxi, mripard, alsa-devel, linux-kernel,
	krzysztof.kozlowski+dt

Hi Mark!

Dne ponedeljek, 27. junij 2022 ob 22:46:30 CEST je Mark Brown napisal(a):
> On Mon, 20 Jun 2022 22:54:49 -0500, Samuel Holland wrote:
> > This series adds support for enabling the codec's internal microphone
> > bias, which is needed on at least some versions of the PinePhone.
> > 
> > Changes in v2:
> >  - Move register update from component probe to device probe
> > 
> > Arnaud Ferraris (2):
> >   ASoC: dt-bindings: sun50i-codec: Add binding for internal bias
> >   ASoC: sun50i-codec-analog: Add support for internal bias
> > 
> > [...]
> 
> Applied to
> 
>    https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git
> for-next
> 
> Thanks!
> 
> [1/3] ASoC: dt-bindings: sun50i-codec: Add binding for internal bias
>       commit: 24e0b04dd42be34ec4b18dc1a1e139d66eb572a3
> [2/3] ASoC: sun50i-codec-analog: Add support for internal bias
>       commit: 25ae1a04da0d32c22db0b018e5668129b91fa104

Can you also take patch 3? You picked bindings change, so it's easiest if DT 
change goes through your tree too.

Best regards,
Jernej

> 
> All being well this means that it will be integrated into the linux-next
> tree (usually sometime in the next 24 hours) and sent to Linus during
> the next merge window (or sooner if it is a bug fix), however if
> problems are discovered then the patch may be dropped or reverted.
> 
> You may get further e-mails resulting from automated or manual testing
> and review of the tree, please engage with people reporting problems and
> send followup patches addressing any issues that are reported if needed.
> 
> If any updates are required or you are submitting further changes they
> should be sent as incremental updates against current git, existing
> patches will not be replaced.
> 
> Please add any relevant lists and maintainers to the CCs when replying
> to this mail.
> 
> Thanks,
> Mark



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

* Re: Re: (subset) [PATCH v2 0/3] ASoC: sun50i-codec-analog: Internal bias support
@ 2022-07-05 20:12     ` Jernej Škrabec
  0 siblings, 0 replies; 33+ messages in thread
From: Jernej Škrabec @ 2022-07-05 20:12 UTC (permalink / raw)
  To: lgirdwood, samuel, wens, Mark Brown
  Cc: devicetree, arnaud.ferraris, alsa-devel, linux-kernel, tiwai,
	mripard, robh+dt, krzysztof.kozlowski+dt, linux-sunxi,
	linux-arm-kernel

Hi Mark!

Dne ponedeljek, 27. junij 2022 ob 22:46:30 CEST je Mark Brown napisal(a):
> On Mon, 20 Jun 2022 22:54:49 -0500, Samuel Holland wrote:
> > This series adds support for enabling the codec's internal microphone
> > bias, which is needed on at least some versions of the PinePhone.
> > 
> > Changes in v2:
> >  - Move register update from component probe to device probe
> > 
> > Arnaud Ferraris (2):
> >   ASoC: dt-bindings: sun50i-codec: Add binding for internal bias
> >   ASoC: sun50i-codec-analog: Add support for internal bias
> > 
> > [...]
> 
> Applied to
> 
>    https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git
> for-next
> 
> Thanks!
> 
> [1/3] ASoC: dt-bindings: sun50i-codec: Add binding for internal bias
>       commit: 24e0b04dd42be34ec4b18dc1a1e139d66eb572a3
> [2/3] ASoC: sun50i-codec-analog: Add support for internal bias
>       commit: 25ae1a04da0d32c22db0b018e5668129b91fa104

Can you also take patch 3? You picked bindings change, so it's easiest if DT 
change goes through your tree too.

Best regards,
Jernej

> 
> All being well this means that it will be integrated into the linux-next
> tree (usually sometime in the next 24 hours) and sent to Linus during
> the next merge window (or sooner if it is a bug fix), however if
> problems are discovered then the patch may be dropped or reverted.
> 
> You may get further e-mails resulting from automated or manual testing
> and review of the tree, please engage with people reporting problems and
> send followup patches addressing any issues that are reported if needed.
> 
> If any updates are required or you are submitting further changes they
> should be sent as incremental updates against current git, existing
> patches will not be replaced.
> 
> Please add any relevant lists and maintainers to the CCs when replying
> to this mail.
> 
> Thanks,
> Mark



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

* Re: Re: (subset) [PATCH v2 0/3] ASoC: sun50i-codec-analog: Internal bias support
@ 2022-07-05 20:12     ` Jernej Škrabec
  0 siblings, 0 replies; 33+ messages in thread
From: Jernej Škrabec @ 2022-07-05 20:12 UTC (permalink / raw)
  To: lgirdwood, samuel, wens, Mark Brown
  Cc: robh+dt, linux-arm-kernel, perex, arnaud.ferraris, devicetree,
	tiwai, linux-sunxi, mripard, alsa-devel, linux-kernel,
	krzysztof.kozlowski+dt

Hi Mark!

Dne ponedeljek, 27. junij 2022 ob 22:46:30 CEST je Mark Brown napisal(a):
> On Mon, 20 Jun 2022 22:54:49 -0500, Samuel Holland wrote:
> > This series adds support for enabling the codec's internal microphone
> > bias, which is needed on at least some versions of the PinePhone.
> > 
> > Changes in v2:
> >  - Move register update from component probe to device probe
> > 
> > Arnaud Ferraris (2):
> >   ASoC: dt-bindings: sun50i-codec: Add binding for internal bias
> >   ASoC: sun50i-codec-analog: Add support for internal bias
> > 
> > [...]
> 
> Applied to
> 
>    https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git
> for-next
> 
> Thanks!
> 
> [1/3] ASoC: dt-bindings: sun50i-codec: Add binding for internal bias
>       commit: 24e0b04dd42be34ec4b18dc1a1e139d66eb572a3
> [2/3] ASoC: sun50i-codec-analog: Add support for internal bias
>       commit: 25ae1a04da0d32c22db0b018e5668129b91fa104

Can you also take patch 3? You picked bindings change, so it's easiest if DT 
change goes through your tree too.

Best regards,
Jernej

> 
> All being well this means that it will be integrated into the linux-next
> tree (usually sometime in the next 24 hours) and sent to Linus during
> the next merge window (or sooner if it is a bug fix), however if
> problems are discovered then the patch may be dropped or reverted.
> 
> You may get further e-mails resulting from automated or manual testing
> and review of the tree, please engage with people reporting problems and
> send followup patches addressing any issues that are reported if needed.
> 
> If any updates are required or you are submitting further changes they
> should be sent as incremental updates against current git, existing
> patches will not be replaced.
> 
> Please add any relevant lists and maintainers to the CCs when replying
> to this mail.
> 
> Thanks,
> Mark



_______________________________________________
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] 33+ messages in thread

* Re: Re: (subset) [PATCH v2 0/3] ASoC: sun50i-codec-analog: Internal bias support
  2022-07-05 20:12     ` Jernej Škrabec
  (?)
@ 2022-07-05 20:58       ` Mark Brown
  -1 siblings, 0 replies; 33+ messages in thread
From: Mark Brown @ 2022-07-05 20:58 UTC (permalink / raw)
  To: Jernej Škrabec
  Cc: lgirdwood, samuel, wens, robh+dt, linux-arm-kernel, perex,
	arnaud.ferraris, devicetree, tiwai, linux-sunxi, mripard,
	alsa-devel, linux-kernel, krzysztof.kozlowski+dt

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

On Tue, Jul 05, 2022 at 10:12:56PM +0200, Jernej Škrabec wrote:
> Dne ponedeljek, 27. junij 2022 ob 22:46:30 CEST je Mark Brown napisal(a):
> > On Mon, 20 Jun 2022 22:54:49 -0500, Samuel Holland wrote:

> > [1/3] ASoC: dt-bindings: sun50i-codec: Add binding for internal bias
> >       commit: 24e0b04dd42be34ec4b18dc1a1e139d66eb572a3
> > [2/3] ASoC: sun50i-codec-analog: Add support for internal bias
> >       commit: 25ae1a04da0d32c22db0b018e5668129b91fa104

> Can you also take patch 3? You picked bindings change, so it's easiest if DT 
> change goes through your tree too.

I can if the sunxi maintainers like, but normally changes to the DTs go
via the platform tree.

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

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

* Re: Re: (subset) [PATCH v2 0/3] ASoC: sun50i-codec-analog: Internal bias support
@ 2022-07-05 20:58       ` Mark Brown
  0 siblings, 0 replies; 33+ messages in thread
From: Mark Brown @ 2022-07-05 20:58 UTC (permalink / raw)
  To: Jernej Škrabec
  Cc: devicetree, arnaud.ferraris, alsa-devel, samuel, linux-kernel,
	tiwai, lgirdwood, mripard, wens, robh+dt, krzysztof.kozlowski+dt,
	linux-sunxi, linux-arm-kernel

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

On Tue, Jul 05, 2022 at 10:12:56PM +0200, Jernej Škrabec wrote:
> Dne ponedeljek, 27. junij 2022 ob 22:46:30 CEST je Mark Brown napisal(a):
> > On Mon, 20 Jun 2022 22:54:49 -0500, Samuel Holland wrote:

> > [1/3] ASoC: dt-bindings: sun50i-codec: Add binding for internal bias
> >       commit: 24e0b04dd42be34ec4b18dc1a1e139d66eb572a3
> > [2/3] ASoC: sun50i-codec-analog: Add support for internal bias
> >       commit: 25ae1a04da0d32c22db0b018e5668129b91fa104

> Can you also take patch 3? You picked bindings change, so it's easiest if DT 
> change goes through your tree too.

I can if the sunxi maintainers like, but normally changes to the DTs go
via the platform tree.

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

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

* Re: Re: (subset) [PATCH v2 0/3] ASoC: sun50i-codec-analog: Internal bias support
@ 2022-07-05 20:58       ` Mark Brown
  0 siblings, 0 replies; 33+ messages in thread
From: Mark Brown @ 2022-07-05 20:58 UTC (permalink / raw)
  To: Jernej Škrabec
  Cc: lgirdwood, samuel, wens, robh+dt, linux-arm-kernel, perex,
	arnaud.ferraris, devicetree, tiwai, linux-sunxi, mripard,
	alsa-devel, linux-kernel, krzysztof.kozlowski+dt


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

On Tue, Jul 05, 2022 at 10:12:56PM +0200, Jernej Škrabec wrote:
> Dne ponedeljek, 27. junij 2022 ob 22:46:30 CEST je Mark Brown napisal(a):
> > On Mon, 20 Jun 2022 22:54:49 -0500, Samuel Holland wrote:

> > [1/3] ASoC: dt-bindings: sun50i-codec: Add binding for internal bias
> >       commit: 24e0b04dd42be34ec4b18dc1a1e139d66eb572a3
> > [2/3] ASoC: sun50i-codec-analog: Add support for internal bias
> >       commit: 25ae1a04da0d32c22db0b018e5668129b91fa104

> Can you also take patch 3? You picked bindings change, so it's easiest if DT 
> change goes through your tree too.

I can if the sunxi maintainers like, but normally changes to the DTs go
via the platform tree.

[-- 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] 33+ messages in thread

* Re: (subset) [PATCH v2 0/3] ASoC: sun50i-codec-analog: Internal bias support
  2022-07-05 20:58       ` Mark Brown
  (?)
@ 2022-07-05 21:04         ` Jernej Škrabec
  -1 siblings, 0 replies; 33+ messages in thread
From: Jernej Škrabec @ 2022-07-05 21:04 UTC (permalink / raw)
  To: Mark Brown
  Cc: lgirdwood, samuel, wens, robh+dt, linux-arm-kernel, perex,
	arnaud.ferraris, devicetree, tiwai, linux-sunxi, mripard,
	alsa-devel, linux-kernel, krzysztof.kozlowski+dt

Dne torek, 05. julij 2022 ob 22:58:32 CEST je Mark Brown napisal(a):
> On Tue, Jul 05, 2022 at 10:12:56PM +0200, Jernej Škrabec wrote:
> > Dne ponedeljek, 27. junij 2022 ob 22:46:30 CEST je Mark Brown napisal(a):
> > > On Mon, 20 Jun 2022 22:54:49 -0500, Samuel Holland wrote:
> > > 
> > > [1/3] ASoC: dt-bindings: sun50i-codec: Add binding for internal bias
> > > 
> > >       commit: 24e0b04dd42be34ec4b18dc1a1e139d66eb572a3
> > > 
> > > [2/3] ASoC: sun50i-codec-analog: Add support for internal bias
> > > 
> > >       commit: 25ae1a04da0d32c22db0b018e5668129b91fa104
> > 
> > Can you also take patch 3? You picked bindings change, so it's easiest if
> > DT change goes through your tree too.
> 
> I can if the sunxi maintainers like, but normally changes to the DTs go
> via the platform tree.

I am sunxi maintainer. Wouldn't my branch generate warnings, if I have only DT 
changes but not bindings?

Best regards,
Jernej



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

* Re: (subset) [PATCH v2 0/3] ASoC: sun50i-codec-analog: Internal bias support
@ 2022-07-05 21:04         ` Jernej Škrabec
  0 siblings, 0 replies; 33+ messages in thread
From: Jernej Škrabec @ 2022-07-05 21:04 UTC (permalink / raw)
  To: Mark Brown
  Cc: devicetree, arnaud.ferraris, alsa-devel, samuel, linux-kernel,
	tiwai, lgirdwood, mripard, wens, robh+dt, krzysztof.kozlowski+dt,
	linux-sunxi, linux-arm-kernel

Dne torek, 05. julij 2022 ob 22:58:32 CEST je Mark Brown napisal(a):
> On Tue, Jul 05, 2022 at 10:12:56PM +0200, Jernej Škrabec wrote:
> > Dne ponedeljek, 27. junij 2022 ob 22:46:30 CEST je Mark Brown napisal(a):
> > > On Mon, 20 Jun 2022 22:54:49 -0500, Samuel Holland wrote:
> > > 
> > > [1/3] ASoC: dt-bindings: sun50i-codec: Add binding for internal bias
> > > 
> > >       commit: 24e0b04dd42be34ec4b18dc1a1e139d66eb572a3
> > > 
> > > [2/3] ASoC: sun50i-codec-analog: Add support for internal bias
> > > 
> > >       commit: 25ae1a04da0d32c22db0b018e5668129b91fa104
> > 
> > Can you also take patch 3? You picked bindings change, so it's easiest if
> > DT change goes through your tree too.
> 
> I can if the sunxi maintainers like, but normally changes to the DTs go
> via the platform tree.

I am sunxi maintainer. Wouldn't my branch generate warnings, if I have only DT 
changes but not bindings?

Best regards,
Jernej



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

* Re: (subset) [PATCH v2 0/3] ASoC: sun50i-codec-analog: Internal bias support
@ 2022-07-05 21:04         ` Jernej Škrabec
  0 siblings, 0 replies; 33+ messages in thread
From: Jernej Škrabec @ 2022-07-05 21:04 UTC (permalink / raw)
  To: Mark Brown
  Cc: lgirdwood, samuel, wens, robh+dt, linux-arm-kernel, perex,
	arnaud.ferraris, devicetree, tiwai, linux-sunxi, mripard,
	alsa-devel, linux-kernel, krzysztof.kozlowski+dt

Dne torek, 05. julij 2022 ob 22:58:32 CEST je Mark Brown napisal(a):
> On Tue, Jul 05, 2022 at 10:12:56PM +0200, Jernej Škrabec wrote:
> > Dne ponedeljek, 27. junij 2022 ob 22:46:30 CEST je Mark Brown napisal(a):
> > > On Mon, 20 Jun 2022 22:54:49 -0500, Samuel Holland wrote:
> > > 
> > > [1/3] ASoC: dt-bindings: sun50i-codec: Add binding for internal bias
> > > 
> > >       commit: 24e0b04dd42be34ec4b18dc1a1e139d66eb572a3
> > > 
> > > [2/3] ASoC: sun50i-codec-analog: Add support for internal bias
> > > 
> > >       commit: 25ae1a04da0d32c22db0b018e5668129b91fa104
> > 
> > Can you also take patch 3? You picked bindings change, so it's easiest if
> > DT change goes through your tree too.
> 
> I can if the sunxi maintainers like, but normally changes to the DTs go
> via the platform tree.

I am sunxi maintainer. Wouldn't my branch generate warnings, if I have only DT 
changes but not bindings?

Best regards,
Jernej



_______________________________________________
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] 33+ messages in thread

* Re: (subset) [PATCH v2 0/3] ASoC: sun50i-codec-analog: Internal bias support
  2022-07-05 21:04         ` Jernej Škrabec
  (?)
@ 2022-07-05 21:14           ` Mark Brown
  -1 siblings, 0 replies; 33+ messages in thread
From: Mark Brown @ 2022-07-05 21:14 UTC (permalink / raw)
  To: Jernej Škrabec
  Cc: lgirdwood, samuel, wens, robh+dt, linux-arm-kernel, perex,
	arnaud.ferraris, devicetree, tiwai, linux-sunxi, mripard,
	alsa-devel, linux-kernel, krzysztof.kozlowski+dt

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

On Tue, Jul 05, 2022 at 11:04:54PM +0200, Jernej Škrabec wrote:
> Dne torek, 05. julij 2022 ob 22:58:32 CEST je Mark Brown napisal(a):

> > I can if the sunxi maintainers like, but normally changes to the DTs go
> > via the platform tree.

> I am sunxi maintainer. Wouldn't my branch generate warnings, if I have only DT 
> changes but not bindings?

I guess, I think people tend not to worry about it so long as -next is
OK.  The issue with merging things through subsystem trees is that it's
very easy to generate lots of conflicts which tends to cause more issues
overall.

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

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

* Re: (subset) [PATCH v2 0/3] ASoC: sun50i-codec-analog: Internal bias support
@ 2022-07-05 21:14           ` Mark Brown
  0 siblings, 0 replies; 33+ messages in thread
From: Mark Brown @ 2022-07-05 21:14 UTC (permalink / raw)
  To: Jernej Škrabec
  Cc: devicetree, arnaud.ferraris, alsa-devel, samuel, linux-kernel,
	tiwai, lgirdwood, mripard, wens, robh+dt, krzysztof.kozlowski+dt,
	linux-sunxi, linux-arm-kernel

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

On Tue, Jul 05, 2022 at 11:04:54PM +0200, Jernej Škrabec wrote:
> Dne torek, 05. julij 2022 ob 22:58:32 CEST je Mark Brown napisal(a):

> > I can if the sunxi maintainers like, but normally changes to the DTs go
> > via the platform tree.

> I am sunxi maintainer. Wouldn't my branch generate warnings, if I have only DT 
> changes but not bindings?

I guess, I think people tend not to worry about it so long as -next is
OK.  The issue with merging things through subsystem trees is that it's
very easy to generate lots of conflicts which tends to cause more issues
overall.

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

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

* Re: (subset) [PATCH v2 0/3] ASoC: sun50i-codec-analog: Internal bias support
@ 2022-07-05 21:14           ` Mark Brown
  0 siblings, 0 replies; 33+ messages in thread
From: Mark Brown @ 2022-07-05 21:14 UTC (permalink / raw)
  To: Jernej Škrabec
  Cc: lgirdwood, samuel, wens, robh+dt, linux-arm-kernel, perex,
	arnaud.ferraris, devicetree, tiwai, linux-sunxi, mripard,
	alsa-devel, linux-kernel, krzysztof.kozlowski+dt


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

On Tue, Jul 05, 2022 at 11:04:54PM +0200, Jernej Škrabec wrote:
> Dne torek, 05. julij 2022 ob 22:58:32 CEST je Mark Brown napisal(a):

> > I can if the sunxi maintainers like, but normally changes to the DTs go
> > via the platform tree.

> I am sunxi maintainer. Wouldn't my branch generate warnings, if I have only DT 
> changes but not bindings?

I guess, I think people tend not to worry about it so long as -next is
OK.  The issue with merging things through subsystem trees is that it's
very easy to generate lots of conflicts which tends to cause more issues
overall.

[-- 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] 33+ messages in thread

* Re: Re: [PATCH v2 3/3] arm64: dts: allwinner: pinephone: Enable internal HMIC bias
  2022-06-27 20:53     ` Jernej Škrabec
  (?)
@ 2022-07-08 16:52       ` Jernej Škrabec
  -1 siblings, 0 replies; 33+ messages in thread
From: Jernej Škrabec @ 2022-07-08 16:52 UTC (permalink / raw)
  To: Chen-Yu Tsai, Liam Girdwood, Mark Brown, Samuel Holland
  Cc: Arnaud Ferraris, Samuel Holland, Jaroslav Kysela,
	Krzysztof Kozlowski, Maxime Ripard, Rob Herring, Takashi Iwai,
	alsa-devel, devicetree, linux-arm-kernel, linux-kernel,
	linux-sunxi

Dne ponedeljek, 27. junij 2022 ob 22:53:05 CEST je Jernej Škrabec napisal(a):
> Dne torek, 21. junij 2022 ob 05:54:52 CEST je Samuel Holland napisal(a):
> > Revisions 1.0 and 1.1 of the PinePhone mainboard do not have an external
> > resistor connecting HBIAS to MIC2P. Enable the internal resistor to
> > provide the necessary headeset microphone bias.
> > 
> > Signed-off-by: Samuel Holland <samuel@sholland.org>
> 
> Reviewed-by: Jernej Skrabec <jernej.skrabec@gmail.com>

Applied, thanks!
 
Best regards,
 Jernej



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

* Re: Re: [PATCH v2 3/3] arm64: dts: allwinner: pinephone: Enable internal HMIC bias
@ 2022-07-08 16:52       ` Jernej Škrabec
  0 siblings, 0 replies; 33+ messages in thread
From: Jernej Škrabec @ 2022-07-08 16:52 UTC (permalink / raw)
  To: Chen-Yu Tsai, Liam Girdwood, Mark Brown, Samuel Holland
  Cc: devicetree, Arnaud Ferraris, alsa-devel, Samuel Holland,
	linux-kernel, Takashi Iwai, Maxime Ripard, Rob Herring,
	Krzysztof Kozlowski, linux-sunxi, linux-arm-kernel

Dne ponedeljek, 27. junij 2022 ob 22:53:05 CEST je Jernej Škrabec napisal(a):
> Dne torek, 21. junij 2022 ob 05:54:52 CEST je Samuel Holland napisal(a):
> > Revisions 1.0 and 1.1 of the PinePhone mainboard do not have an external
> > resistor connecting HBIAS to MIC2P. Enable the internal resistor to
> > provide the necessary headeset microphone bias.
> > 
> > Signed-off-by: Samuel Holland <samuel@sholland.org>
> 
> Reviewed-by: Jernej Skrabec <jernej.skrabec@gmail.com>

Applied, thanks!
 
Best regards,
 Jernej



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

* Re: Re: [PATCH v2 3/3] arm64: dts: allwinner: pinephone: Enable internal HMIC bias
@ 2022-07-08 16:52       ` Jernej Škrabec
  0 siblings, 0 replies; 33+ messages in thread
From: Jernej Škrabec @ 2022-07-08 16:52 UTC (permalink / raw)
  To: Chen-Yu Tsai, Liam Girdwood, Mark Brown, Samuel Holland
  Cc: Arnaud Ferraris, Samuel Holland, Jaroslav Kysela,
	Krzysztof Kozlowski, Maxime Ripard, Rob Herring, Takashi Iwai,
	alsa-devel, devicetree, linux-arm-kernel, linux-kernel,
	linux-sunxi

Dne ponedeljek, 27. junij 2022 ob 22:53:05 CEST je Jernej Škrabec napisal(a):
> Dne torek, 21. junij 2022 ob 05:54:52 CEST je Samuel Holland napisal(a):
> > Revisions 1.0 and 1.1 of the PinePhone mainboard do not have an external
> > resistor connecting HBIAS to MIC2P. Enable the internal resistor to
> > provide the necessary headeset microphone bias.
> > 
> > Signed-off-by: Samuel Holland <samuel@sholland.org>
> 
> Reviewed-by: Jernej Skrabec <jernej.skrabec@gmail.com>

Applied, thanks!
 
Best regards,
 Jernej



_______________________________________________
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] 33+ messages in thread

end of thread, other threads:[~2022-07-08 16:55 UTC | newest]

Thread overview: 33+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-06-21  3:54 [PATCH v2 0/3] ASoC: sun50i-codec-analog: Internal bias support Samuel Holland
2022-06-21  3:54 ` Samuel Holland
2022-06-21  3:54 ` Samuel Holland
2022-06-21  3:54 ` [PATCH v2 1/3] ASoC: dt-bindings: sun50i-codec: Add binding for internal bias Samuel Holland
2022-06-21  3:54   ` Samuel Holland
2022-06-21  3:54   ` Samuel Holland
2022-06-21  3:54 ` [PATCH v2 2/3] ASoC: sun50i-codec-analog: Add support " Samuel Holland
2022-06-21  3:54   ` Samuel Holland
2022-06-21  3:54   ` Samuel Holland
2022-06-21  3:54 ` [PATCH v2 3/3] arm64: dts: allwinner: pinephone: Enable internal HMIC bias Samuel Holland
2022-06-21  3:54   ` Samuel Holland
2022-06-21  3:54   ` Samuel Holland
2022-06-27 20:53   ` Jernej Škrabec
2022-06-27 20:53     ` Jernej Škrabec
2022-06-27 20:53     ` Jernej Škrabec
2022-07-08 16:52     ` Jernej Škrabec
2022-07-08 16:52       ` Jernej Škrabec
2022-07-08 16:52       ` Jernej Škrabec
2022-06-27 20:46 ` (subset) [PATCH v2 0/3] ASoC: sun50i-codec-analog: Internal bias support Mark Brown
2022-06-27 20:46   ` Mark Brown
2022-06-27 20:46   ` Mark Brown
2022-07-05 20:12   ` Jernej Škrabec
2022-07-05 20:12     ` Jernej Škrabec
2022-07-05 20:12     ` Jernej Škrabec
2022-07-05 20:58     ` Mark Brown
2022-07-05 20:58       ` Mark Brown
2022-07-05 20:58       ` Mark Brown
2022-07-05 21:04       ` Jernej Škrabec
2022-07-05 21:04         ` Jernej Škrabec
2022-07-05 21:04         ` Jernej Škrabec
2022-07-05 21:14         ` Mark Brown
2022-07-05 21:14           ` Mark Brown
2022-07-05 21:14           ` Mark Brown

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.