All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v1 0/2] Support built-in Mic on Tegra boards that use WM8903
@ 2020-03-20 20:55 ` Dmitry Osipenko
  0 siblings, 0 replies; 20+ messages in thread
From: Dmitry Osipenko @ 2020-03-20 20:55 UTC (permalink / raw)
  To: Rob Herring, Mark Brown, Jaroslav Kysela, Takashi Iwai,
	Liam Girdwood, Stephen Warren, Jonathan Hunter
  Cc: devicetree, linux-tegra, alsa-devel, linux-kernel

Hello,

This small series adds audio route for built-in microphone on NVIDIA Tegra
boards that use WM8903 CODEC. In particular this is needed in order to unmute
internal microphone on Acer A500 tablet device. I'm planning to send out the
device tree for the A500 for 5.8, so will be nice to get the microphone
sorted out. Please review and apply, thanks in advance.

Dmitry Osipenko (2):
  dt-bindings: sound: tegra-wm8903: Document built-in microphone audio
    source
  ASoC: tegra: tegra_wm8903: Support DAPM events for built-in microphone

 .../sound/nvidia,tegra-audio-wm8903.txt        |  1 +
 sound/soc/tegra/tegra_wm8903.c                 | 18 ++++++++++++++++++
 2 files changed, 19 insertions(+)

-- 
2.25.1

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

* [PATCH v1 0/2] Support built-in Mic on Tegra boards that use WM8903
@ 2020-03-20 20:55 ` Dmitry Osipenko
  0 siblings, 0 replies; 20+ messages in thread
From: Dmitry Osipenko @ 2020-03-20 20:55 UTC (permalink / raw)
  To: Rob Herring, Mark Brown, Jaroslav Kysela, Takashi Iwai,
	Liam Girdwood, Stephen Warren, Jonathan Hunter
  Cc: linux-tegra, devicetree, alsa-devel, linux-kernel

Hello,

This small series adds audio route for built-in microphone on NVIDIA Tegra
boards that use WM8903 CODEC. In particular this is needed in order to unmute
internal microphone on Acer A500 tablet device. I'm planning to send out the
device tree for the A500 for 5.8, so will be nice to get the microphone
sorted out. Please review and apply, thanks in advance.

Dmitry Osipenko (2):
  dt-bindings: sound: tegra-wm8903: Document built-in microphone audio
    source
  ASoC: tegra: tegra_wm8903: Support DAPM events for built-in microphone

 .../sound/nvidia,tegra-audio-wm8903.txt        |  1 +
 sound/soc/tegra/tegra_wm8903.c                 | 18 ++++++++++++++++++
 2 files changed, 19 insertions(+)

-- 
2.25.1


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

* [PATCH v1 1/2] dt-bindings: sound: tegra-wm8903: Document built-in microphone audio source
  2020-03-20 20:55 ` Dmitry Osipenko
  (?)
@ 2020-03-20 20:55     ` Dmitry Osipenko
  -1 siblings, 0 replies; 20+ messages in thread
From: Dmitry Osipenko @ 2020-03-20 20:55 UTC (permalink / raw)
  To: Rob Herring, Mark Brown, Jaroslav Kysela, Takashi Iwai,
	Liam Girdwood, Stephen Warren, Jonathan Hunter
  Cc: devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-tegra-u79uwXL29TY76Z2rM5mHXA,
	alsa-devel-K7yf7f+aM1XWsZ/bQMPhNw,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA

The internal microphone source is needed in order to be able to describe
the hardware audio routing for devices that have the built-in microphone
in addition to the external Mic Jack.

Signed-off-by: Dmitry Osipenko <digetx-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
---
 .../devicetree/bindings/sound/nvidia,tegra-audio-wm8903.txt      | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/sound/nvidia,tegra-audio-wm8903.txt b/Documentation/devicetree/bindings/sound/nvidia,tegra-audio-wm8903.txt
index b795d282818d..a8f2b0c56c79 100644
--- a/Documentation/devicetree/bindings/sound/nvidia,tegra-audio-wm8903.txt
+++ b/Documentation/devicetree/bindings/sound/nvidia,tegra-audio-wm8903.txt
@@ -18,6 +18,7 @@ Required properties:
   * Headphone Jack
   * Int Spk
   * Mic Jack
+  * Int Mic
 
 - nvidia,i2s-controller : The phandle of the Tegra I2S1 controller
 - nvidia,audio-codec : The phandle of the WM8903 audio codec
-- 
2.25.1

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

* [PATCH v1 1/2] dt-bindings: sound: tegra-wm8903: Document built-in microphone audio source
@ 2020-03-20 20:55     ` Dmitry Osipenko
  0 siblings, 0 replies; 20+ messages in thread
From: Dmitry Osipenko @ 2020-03-20 20:55 UTC (permalink / raw)
  To: Rob Herring, Mark Brown, Jaroslav Kysela, Takashi Iwai,
	Liam Girdwood, Stephen Warren, Jonathan Hunter
  Cc: devicetree, linux-tegra, alsa-devel, linux-kernel

The internal microphone source is needed in order to be able to describe
the hardware audio routing for devices that have the built-in microphone
in addition to the external Mic Jack.

Signed-off-by: Dmitry Osipenko <digetx@gmail.com>
---
 .../devicetree/bindings/sound/nvidia,tegra-audio-wm8903.txt      | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/sound/nvidia,tegra-audio-wm8903.txt b/Documentation/devicetree/bindings/sound/nvidia,tegra-audio-wm8903.txt
index b795d282818d..a8f2b0c56c79 100644
--- a/Documentation/devicetree/bindings/sound/nvidia,tegra-audio-wm8903.txt
+++ b/Documentation/devicetree/bindings/sound/nvidia,tegra-audio-wm8903.txt
@@ -18,6 +18,7 @@ Required properties:
   * Headphone Jack
   * Int Spk
   * Mic Jack
+  * Int Mic
 
 - nvidia,i2s-controller : The phandle of the Tegra I2S1 controller
 - nvidia,audio-codec : The phandle of the WM8903 audio codec
-- 
2.25.1


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

* [PATCH v1 1/2] dt-bindings: sound: tegra-wm8903: Document built-in microphone audio source
@ 2020-03-20 20:55     ` Dmitry Osipenko
  0 siblings, 0 replies; 20+ messages in thread
From: Dmitry Osipenko @ 2020-03-20 20:55 UTC (permalink / raw)
  To: Rob Herring, Mark Brown, Jaroslav Kysela, Takashi Iwai,
	Liam Girdwood, Stephen Warren, Jonathan Hunter
  Cc: linux-tegra, devicetree, alsa-devel, linux-kernel

The internal microphone source is needed in order to be able to describe
the hardware audio routing for devices that have the built-in microphone
in addition to the external Mic Jack.

Signed-off-by: Dmitry Osipenko <digetx@gmail.com>
---
 .../devicetree/bindings/sound/nvidia,tegra-audio-wm8903.txt      | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/sound/nvidia,tegra-audio-wm8903.txt b/Documentation/devicetree/bindings/sound/nvidia,tegra-audio-wm8903.txt
index b795d282818d..a8f2b0c56c79 100644
--- a/Documentation/devicetree/bindings/sound/nvidia,tegra-audio-wm8903.txt
+++ b/Documentation/devicetree/bindings/sound/nvidia,tegra-audio-wm8903.txt
@@ -18,6 +18,7 @@ Required properties:
   * Headphone Jack
   * Int Spk
   * Mic Jack
+  * Int Mic
 
 - nvidia,i2s-controller : The phandle of the Tegra I2S1 controller
 - nvidia,audio-codec : The phandle of the WM8903 audio codec
-- 
2.25.1


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

* [PATCH v1 2/2] ASoC: tegra: tegra_wm8903: Support DAPM events for built-in microphone
  2020-03-20 20:55 ` Dmitry Osipenko
  (?)
@ 2020-03-20 20:55     ` Dmitry Osipenko
  -1 siblings, 0 replies; 20+ messages in thread
From: Dmitry Osipenko @ 2020-03-20 20:55 UTC (permalink / raw)
  To: Rob Herring, Mark Brown, Jaroslav Kysela, Takashi Iwai,
	Liam Girdwood, Stephen Warren, Jonathan Hunter
  Cc: devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-tegra-u79uwXL29TY76Z2rM5mHXA,
	alsa-devel-K7yf7f+aM1XWsZ/bQMPhNw,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA

The enable-GPIO needs to be toggled on a DAPM event in order to turn
microphone ON/OFF, otherwise microphone won't work.

Signed-off-by: Dmitry Osipenko <digetx-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
---
 sound/soc/tegra/tegra_wm8903.c | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)

diff --git a/sound/soc/tegra/tegra_wm8903.c b/sound/soc/tegra/tegra_wm8903.c
index f08d3489c3cf..071c7d2de77c 100644
--- a/sound/soc/tegra/tegra_wm8903.c
+++ b/sound/soc/tegra/tegra_wm8903.c
@@ -143,14 +143,32 @@ static int tegra_wm8903_event_hp(struct snd_soc_dapm_widget *w,
 	return 0;
 }
 
+static int tegra_wm8903_event_int_mic(struct snd_soc_dapm_widget *w,
+				      struct snd_kcontrol *k, int event)
+{
+	struct snd_soc_dapm_context *dapm = w->dapm;
+	struct snd_soc_card *card = dapm->card;
+	struct tegra_wm8903 *machine = snd_soc_card_get_drvdata(card);
+
+	if (!gpio_is_valid(machine->gpio_int_mic_en))
+		return 0;
+
+	gpio_set_value_cansleep(machine->gpio_int_mic_en,
+				SND_SOC_DAPM_EVENT_ON(event));
+
+	return 0;
+}
+
 static const struct snd_soc_dapm_widget tegra_wm8903_dapm_widgets[] = {
 	SND_SOC_DAPM_SPK("Int Spk", tegra_wm8903_event_int_spk),
 	SND_SOC_DAPM_HP("Headphone Jack", tegra_wm8903_event_hp),
 	SND_SOC_DAPM_MIC("Mic Jack", NULL),
+	SND_SOC_DAPM_MIC("Int Mic", tegra_wm8903_event_int_mic),
 };
 
 static const struct snd_kcontrol_new tegra_wm8903_controls[] = {
 	SOC_DAPM_PIN_SWITCH("Int Spk"),
+	SOC_DAPM_PIN_SWITCH("Int Mic"),
 };
 
 static int tegra_wm8903_init(struct snd_soc_pcm_runtime *rtd)
-- 
2.25.1

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

* [PATCH v1 2/2] ASoC: tegra: tegra_wm8903: Support DAPM events for built-in microphone
@ 2020-03-20 20:55     ` Dmitry Osipenko
  0 siblings, 0 replies; 20+ messages in thread
From: Dmitry Osipenko @ 2020-03-20 20:55 UTC (permalink / raw)
  To: Rob Herring, Mark Brown, Jaroslav Kysela, Takashi Iwai,
	Liam Girdwood, Stephen Warren, Jonathan Hunter
  Cc: devicetree, linux-tegra, alsa-devel, linux-kernel

The enable-GPIO needs to be toggled on a DAPM event in order to turn
microphone ON/OFF, otherwise microphone won't work.

Signed-off-by: Dmitry Osipenko <digetx@gmail.com>
---
 sound/soc/tegra/tegra_wm8903.c | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)

diff --git a/sound/soc/tegra/tegra_wm8903.c b/sound/soc/tegra/tegra_wm8903.c
index f08d3489c3cf..071c7d2de77c 100644
--- a/sound/soc/tegra/tegra_wm8903.c
+++ b/sound/soc/tegra/tegra_wm8903.c
@@ -143,14 +143,32 @@ static int tegra_wm8903_event_hp(struct snd_soc_dapm_widget *w,
 	return 0;
 }
 
+static int tegra_wm8903_event_int_mic(struct snd_soc_dapm_widget *w,
+				      struct snd_kcontrol *k, int event)
+{
+	struct snd_soc_dapm_context *dapm = w->dapm;
+	struct snd_soc_card *card = dapm->card;
+	struct tegra_wm8903 *machine = snd_soc_card_get_drvdata(card);
+
+	if (!gpio_is_valid(machine->gpio_int_mic_en))
+		return 0;
+
+	gpio_set_value_cansleep(machine->gpio_int_mic_en,
+				SND_SOC_DAPM_EVENT_ON(event));
+
+	return 0;
+}
+
 static const struct snd_soc_dapm_widget tegra_wm8903_dapm_widgets[] = {
 	SND_SOC_DAPM_SPK("Int Spk", tegra_wm8903_event_int_spk),
 	SND_SOC_DAPM_HP("Headphone Jack", tegra_wm8903_event_hp),
 	SND_SOC_DAPM_MIC("Mic Jack", NULL),
+	SND_SOC_DAPM_MIC("Int Mic", tegra_wm8903_event_int_mic),
 };
 
 static const struct snd_kcontrol_new tegra_wm8903_controls[] = {
 	SOC_DAPM_PIN_SWITCH("Int Spk"),
+	SOC_DAPM_PIN_SWITCH("Int Mic"),
 };
 
 static int tegra_wm8903_init(struct snd_soc_pcm_runtime *rtd)
-- 
2.25.1


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

* [PATCH v1 2/2] ASoC: tegra: tegra_wm8903: Support DAPM events for built-in microphone
@ 2020-03-20 20:55     ` Dmitry Osipenko
  0 siblings, 0 replies; 20+ messages in thread
From: Dmitry Osipenko @ 2020-03-20 20:55 UTC (permalink / raw)
  To: Rob Herring, Mark Brown, Jaroslav Kysela, Takashi Iwai,
	Liam Girdwood, Stephen Warren, Jonathan Hunter
  Cc: linux-tegra, devicetree, alsa-devel, linux-kernel

The enable-GPIO needs to be toggled on a DAPM event in order to turn
microphone ON/OFF, otherwise microphone won't work.

Signed-off-by: Dmitry Osipenko <digetx@gmail.com>
---
 sound/soc/tegra/tegra_wm8903.c | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)

diff --git a/sound/soc/tegra/tegra_wm8903.c b/sound/soc/tegra/tegra_wm8903.c
index f08d3489c3cf..071c7d2de77c 100644
--- a/sound/soc/tegra/tegra_wm8903.c
+++ b/sound/soc/tegra/tegra_wm8903.c
@@ -143,14 +143,32 @@ static int tegra_wm8903_event_hp(struct snd_soc_dapm_widget *w,
 	return 0;
 }
 
+static int tegra_wm8903_event_int_mic(struct snd_soc_dapm_widget *w,
+				      struct snd_kcontrol *k, int event)
+{
+	struct snd_soc_dapm_context *dapm = w->dapm;
+	struct snd_soc_card *card = dapm->card;
+	struct tegra_wm8903 *machine = snd_soc_card_get_drvdata(card);
+
+	if (!gpio_is_valid(machine->gpio_int_mic_en))
+		return 0;
+
+	gpio_set_value_cansleep(machine->gpio_int_mic_en,
+				SND_SOC_DAPM_EVENT_ON(event));
+
+	return 0;
+}
+
 static const struct snd_soc_dapm_widget tegra_wm8903_dapm_widgets[] = {
 	SND_SOC_DAPM_SPK("Int Spk", tegra_wm8903_event_int_spk),
 	SND_SOC_DAPM_HP("Headphone Jack", tegra_wm8903_event_hp),
 	SND_SOC_DAPM_MIC("Mic Jack", NULL),
+	SND_SOC_DAPM_MIC("Int Mic", tegra_wm8903_event_int_mic),
 };
 
 static const struct snd_kcontrol_new tegra_wm8903_controls[] = {
 	SOC_DAPM_PIN_SWITCH("Int Spk"),
+	SOC_DAPM_PIN_SWITCH("Int Mic"),
 };
 
 static int tegra_wm8903_init(struct snd_soc_pcm_runtime *rtd)
-- 
2.25.1


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

* Re: [PATCH v1 0/2] Support built-in Mic on Tegra boards that use WM8903
  2020-03-20 20:55 ` Dmitry Osipenko
  (?)
@ 2020-03-20 22:30     ` Stephen Warren
  -1 siblings, 0 replies; 20+ messages in thread
From: Stephen Warren @ 2020-03-20 22:30 UTC (permalink / raw)
  To: Dmitry Osipenko
  Cc: Rob Herring, Mark Brown, Jaroslav Kysela, Takashi Iwai,
	Liam Girdwood, Jonathan Hunter,
	devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-tegra-u79uwXL29TY76Z2rM5mHXA,
	alsa-devel-K7yf7f+aM1XWsZ/bQMPhNw,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA

On 3/20/20 2:55 PM, Dmitry Osipenko wrote:
> Hello,
> 
> This small series adds audio route for built-in microphone on NVIDIA Tegra
> boards that use WM8903 CODEC. In particular this is needed in order to unmute
> internal microphone on Acer A500 tablet device. I'm planning to send out the
> device tree for the A500 for 5.8, so will be nice to get the microphone
> sorted out. Please review and apply, thanks in advance.

It's been a long time since I looked at this code, but the series looks
plausible,
Acked-by: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>

(I wonder why machine->gpio_int_mic_en was already parse but never used!)

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

* Re: [PATCH v1 0/2] Support built-in Mic on Tegra boards that use WM8903
@ 2020-03-20 22:30     ` Stephen Warren
  0 siblings, 0 replies; 20+ messages in thread
From: Stephen Warren @ 2020-03-20 22:30 UTC (permalink / raw)
  To: Dmitry Osipenko
  Cc: Rob Herring, Mark Brown, Jaroslav Kysela, Takashi Iwai,
	Liam Girdwood, Jonathan Hunter, devicetree, linux-tegra,
	alsa-devel, linux-kernel

On 3/20/20 2:55 PM, Dmitry Osipenko wrote:
> Hello,
> 
> This small series adds audio route for built-in microphone on NVIDIA Tegra
> boards that use WM8903 CODEC. In particular this is needed in order to unmute
> internal microphone on Acer A500 tablet device. I'm planning to send out the
> device tree for the A500 for 5.8, so will be nice to get the microphone
> sorted out. Please review and apply, thanks in advance.

It's been a long time since I looked at this code, but the series looks
plausible,
Acked-by: Stephen Warren <swarren@nvidia.com>

(I wonder why machine->gpio_int_mic_en was already parse but never used!)

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

* Re: [PATCH v1 0/2] Support built-in Mic on Tegra boards that use WM8903
@ 2020-03-20 22:30     ` Stephen Warren
  0 siblings, 0 replies; 20+ messages in thread
From: Stephen Warren @ 2020-03-20 22:30 UTC (permalink / raw)
  To: Dmitry Osipenko
  Cc: devicetree, alsa-devel, linux-kernel, Takashi Iwai,
	Liam Girdwood, Rob Herring, Jonathan Hunter, Mark Brown,
	linux-tegra

On 3/20/20 2:55 PM, Dmitry Osipenko wrote:
> Hello,
> 
> This small series adds audio route for built-in microphone on NVIDIA Tegra
> boards that use WM8903 CODEC. In particular this is needed in order to unmute
> internal microphone on Acer A500 tablet device. I'm planning to send out the
> device tree for the A500 for 5.8, so will be nice to get the microphone
> sorted out. Please review and apply, thanks in advance.

It's been a long time since I looked at this code, but the series looks
plausible,
Acked-by: Stephen Warren <swarren@nvidia.com>

(I wonder why machine->gpio_int_mic_en was already parse but never used!)

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

* Re: [PATCH v1 0/2] Support built-in Mic on Tegra boards that use WM8903
  2020-03-20 22:30     ` Stephen Warren
  (?)
@ 2020-03-21  0:49         ` Dmitry Osipenko
  -1 siblings, 0 replies; 20+ messages in thread
From: Dmitry Osipenko @ 2020-03-21  0:49 UTC (permalink / raw)
  To: Stephen Warren
  Cc: Rob Herring, Mark Brown, Jaroslav Kysela, Takashi Iwai,
	Liam Girdwood, Jonathan Hunter,
	devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-tegra-u79uwXL29TY76Z2rM5mHXA,
	alsa-devel-K7yf7f+aM1XWsZ/bQMPhNw,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA

21.03.2020 01:30, Stephen Warren пишет:
> On 3/20/20 2:55 PM, Dmitry Osipenko wrote:
>> Hello,
>>
>> This small series adds audio route for built-in microphone on NVIDIA Tegra
>> boards that use WM8903 CODEC. In particular this is needed in order to unmute
>> internal microphone on Acer A500 tablet device. I'm planning to send out the
>> device tree for the A500 for 5.8, so will be nice to get the microphone
>> sorted out. Please review and apply, thanks in advance.
> 
> It's been a long time since I looked at this code, but the series looks
> plausible,
> Acked-by: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>

Thank you!

> (I wonder why machine->gpio_int_mic_en was already parse but never used!)

Perhaps there were plans to support it later on, but that never
materialized.

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

* Re: [PATCH v1 0/2] Support built-in Mic on Tegra boards that use WM8903
@ 2020-03-21  0:49         ` Dmitry Osipenko
  0 siblings, 0 replies; 20+ messages in thread
From: Dmitry Osipenko @ 2020-03-21  0:49 UTC (permalink / raw)
  To: Stephen Warren
  Cc: Rob Herring, Mark Brown, Jaroslav Kysela, Takashi Iwai,
	Liam Girdwood, Jonathan Hunter, devicetree, linux-tegra,
	alsa-devel, linux-kernel

21.03.2020 01:30, Stephen Warren пишет:
> On 3/20/20 2:55 PM, Dmitry Osipenko wrote:
>> Hello,
>>
>> This small series adds audio route for built-in microphone on NVIDIA Tegra
>> boards that use WM8903 CODEC. In particular this is needed in order to unmute
>> internal microphone on Acer A500 tablet device. I'm planning to send out the
>> device tree for the A500 for 5.8, so will be nice to get the microphone
>> sorted out. Please review and apply, thanks in advance.
> 
> It's been a long time since I looked at this code, but the series looks
> plausible,
> Acked-by: Stephen Warren <swarren@nvidia.com>

Thank you!

> (I wonder why machine->gpio_int_mic_en was already parse but never used!)

Perhaps there were plans to support it later on, but that never
materialized.

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

* Re: [PATCH v1 0/2] Support built-in Mic on Tegra boards that use WM8903
@ 2020-03-21  0:49         ` Dmitry Osipenko
  0 siblings, 0 replies; 20+ messages in thread
From: Dmitry Osipenko @ 2020-03-21  0:49 UTC (permalink / raw)
  To: Stephen Warren
  Cc: devicetree, alsa-devel, linux-kernel, Takashi Iwai,
	Liam Girdwood, Rob Herring, Jonathan Hunter, Mark Brown,
	linux-tegra

21.03.2020 01:30, Stephen Warren пишет:
> On 3/20/20 2:55 PM, Dmitry Osipenko wrote:
>> Hello,
>>
>> This small series adds audio route for built-in microphone on NVIDIA Tegra
>> boards that use WM8903 CODEC. In particular this is needed in order to unmute
>> internal microphone on Acer A500 tablet device. I'm planning to send out the
>> device tree for the A500 for 5.8, so will be nice to get the microphone
>> sorted out. Please review and apply, thanks in advance.
> 
> It's been a long time since I looked at this code, but the series looks
> plausible,
> Acked-by: Stephen Warren <swarren@nvidia.com>

Thank you!

> (I wonder why machine->gpio_int_mic_en was already parse but never used!)

Perhaps there were plans to support it later on, but that never
materialized.

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

* Re: [PATCH v1 0/2] Support built-in Mic on Tegra boards that use WM8903
  2020-03-20 22:30     ` Stephen Warren
  (?)
@ 2020-03-23 15:59         ` Jon Hunter
  -1 siblings, 0 replies; 20+ messages in thread
From: Jon Hunter @ 2020-03-23 15:59 UTC (permalink / raw)
  To: Stephen Warren, Dmitry Osipenko
  Cc: devicetree-u79uwXL29TY76Z2rM5mHXA,
	alsa-devel-K7yf7f+aM1XWsZ/bQMPhNw,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA, Takashi Iwai, Liam Girdwood,
	Rob Herring, Mark Brown, linux-tegra-u79uwXL29TY76Z2rM5mHXA



On 20/03/2020 22:30, Stephen Warren wrote:
> On 3/20/20 2:55 PM, Dmitry Osipenko wrote:
>> Hello,
>>
>> This small series adds audio route for built-in microphone on NVIDIA Tegra
>> boards that use WM8903 CODEC. In particular this is needed in order to unmute
>> internal microphone on Acer A500 tablet device. I'm planning to send out the
>> device tree for the A500 for 5.8, so will be nice to get the microphone
>> sorted out. Please review and apply, thanks in advance.
> 
> It's been a long time since I looked at this code, but the series looks
> plausible,
> Acked-by: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
> 
> (I wonder why machine->gpio_int_mic_en was already parse but never used!)

Looking at the ventana schematics, it appears that the internal-mic
signals was connected to a header and not an actual mic, so maybe we
never had a proper internal-mic on any board but allowed one to be
connected.

Anyway, looks good to me as well.

Acked-by: Jon Hunter <jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>

Cheers
Jon

-- 
nvpublic

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

* Re: [PATCH v1 0/2] Support built-in Mic on Tegra boards that use WM8903
@ 2020-03-23 15:59         ` Jon Hunter
  0 siblings, 0 replies; 20+ messages in thread
From: Jon Hunter @ 2020-03-23 15:59 UTC (permalink / raw)
  To: Stephen Warren, Dmitry Osipenko
  Cc: devicetree, alsa-devel, linux-kernel, Takashi Iwai,
	Liam Girdwood, Rob Herring, Mark Brown, linux-tegra



On 20/03/2020 22:30, Stephen Warren wrote:
> On 3/20/20 2:55 PM, Dmitry Osipenko wrote:
>> Hello,
>>
>> This small series adds audio route for built-in microphone on NVIDIA Tegra
>> boards that use WM8903 CODEC. In particular this is needed in order to unmute
>> internal microphone on Acer A500 tablet device. I'm planning to send out the
>> device tree for the A500 for 5.8, so will be nice to get the microphone
>> sorted out. Please review and apply, thanks in advance.
> 
> It's been a long time since I looked at this code, but the series looks
> plausible,
> Acked-by: Stephen Warren <swarren@nvidia.com>
> 
> (I wonder why machine->gpio_int_mic_en was already parse but never used!)

Looking at the ventana schematics, it appears that the internal-mic
signals was connected to a header and not an actual mic, so maybe we
never had a proper internal-mic on any board but allowed one to be
connected.

Anyway, looks good to me as well.

Acked-by: Jon Hunter <jonathanh@nvidia.com>

Cheers
Jon

-- 
nvpublic

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

* Re: [PATCH v1 0/2] Support built-in Mic on Tegra boards that use WM8903
@ 2020-03-23 15:59         ` Jon Hunter
  0 siblings, 0 replies; 20+ messages in thread
From: Jon Hunter @ 2020-03-23 15:59 UTC (permalink / raw)
  To: Stephen Warren, Dmitry Osipenko
  Cc: devicetree, alsa-devel, Mark Brown, Takashi Iwai, Liam Girdwood,
	linux-kernel, Rob Herring, linux-tegra



On 20/03/2020 22:30, Stephen Warren wrote:
> On 3/20/20 2:55 PM, Dmitry Osipenko wrote:
>> Hello,
>>
>> This small series adds audio route for built-in microphone on NVIDIA Tegra
>> boards that use WM8903 CODEC. In particular this is needed in order to unmute
>> internal microphone on Acer A500 tablet device. I'm planning to send out the
>> device tree for the A500 for 5.8, so will be nice to get the microphone
>> sorted out. Please review and apply, thanks in advance.
> 
> It's been a long time since I looked at this code, but the series looks
> plausible,
> Acked-by: Stephen Warren <swarren@nvidia.com>
> 
> (I wonder why machine->gpio_int_mic_en was already parse but never used!)

Looking at the ventana schematics, it appears that the internal-mic
signals was connected to a header and not an actual mic, so maybe we
never had a proper internal-mic on any board but allowed one to be
connected.

Anyway, looks good to me as well.

Acked-by: Jon Hunter <jonathanh@nvidia.com>

Cheers
Jon

-- 
nvpublic

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

* Re: [PATCH v1 0/2] Support built-in Mic on Tegra boards that use WM8903
  2020-03-23 15:59         ` Jon Hunter
  (?)
@ 2020-03-23 16:05             ` Dmitry Osipenko
  -1 siblings, 0 replies; 20+ messages in thread
From: Dmitry Osipenko @ 2020-03-23 16:05 UTC (permalink / raw)
  To: Jon Hunter, Stephen Warren
  Cc: devicetree-u79uwXL29TY76Z2rM5mHXA,
	alsa-devel-K7yf7f+aM1XWsZ/bQMPhNw,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA, Takashi Iwai, Liam Girdwood,
	Rob Herring, Mark Brown, linux-tegra-u79uwXL29TY76Z2rM5mHXA

23.03.2020 18:59, Jon Hunter пишет:
> 
> 
> On 20/03/2020 22:30, Stephen Warren wrote:
>> On 3/20/20 2:55 PM, Dmitry Osipenko wrote:
>>> Hello,
>>>
>>> This small series adds audio route for built-in microphone on NVIDIA Tegra
>>> boards that use WM8903 CODEC. In particular this is needed in order to unmute
>>> internal microphone on Acer A500 tablet device. I'm planning to send out the
>>> device tree for the A500 for 5.8, so will be nice to get the microphone
>>> sorted out. Please review and apply, thanks in advance.
>>
>> It's been a long time since I looked at this code, but the series looks
>> plausible,
>> Acked-by: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
>>
>> (I wonder why machine->gpio_int_mic_en was already parse but never used!)
> 
> Looking at the ventana schematics, it appears that the internal-mic
> signals was connected to a header and not an actual mic, so maybe we
> never had a proper internal-mic on any board but allowed one to be
> connected.

I guess cardhu may have a mic, judging by downstream kernel sources (I
don't have cardhu schematics).

> Anyway, looks good to me as well.
> 
> Acked-by: Jon Hunter <jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>

Thanks

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

* Re: [PATCH v1 0/2] Support built-in Mic on Tegra boards that use WM8903
@ 2020-03-23 16:05             ` Dmitry Osipenko
  0 siblings, 0 replies; 20+ messages in thread
From: Dmitry Osipenko @ 2020-03-23 16:05 UTC (permalink / raw)
  To: Jon Hunter, Stephen Warren
  Cc: devicetree, alsa-devel, linux-kernel, Takashi Iwai,
	Liam Girdwood, Rob Herring, Mark Brown, linux-tegra

23.03.2020 18:59, Jon Hunter пишет:
> 
> 
> On 20/03/2020 22:30, Stephen Warren wrote:
>> On 3/20/20 2:55 PM, Dmitry Osipenko wrote:
>>> Hello,
>>>
>>> This small series adds audio route for built-in microphone on NVIDIA Tegra
>>> boards that use WM8903 CODEC. In particular this is needed in order to unmute
>>> internal microphone on Acer A500 tablet device. I'm planning to send out the
>>> device tree for the A500 for 5.8, so will be nice to get the microphone
>>> sorted out. Please review and apply, thanks in advance.
>>
>> It's been a long time since I looked at this code, but the series looks
>> plausible,
>> Acked-by: Stephen Warren <swarren@nvidia.com>
>>
>> (I wonder why machine->gpio_int_mic_en was already parse but never used!)
> 
> Looking at the ventana schematics, it appears that the internal-mic
> signals was connected to a header and not an actual mic, so maybe we
> never had a proper internal-mic on any board but allowed one to be
> connected.

I guess cardhu may have a mic, judging by downstream kernel sources (I
don't have cardhu schematics).

> Anyway, looks good to me as well.
> 
> Acked-by: Jon Hunter <jonathanh@nvidia.com>

Thanks

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

* Re: [PATCH v1 0/2] Support built-in Mic on Tegra boards that use WM8903
@ 2020-03-23 16:05             ` Dmitry Osipenko
  0 siblings, 0 replies; 20+ messages in thread
From: Dmitry Osipenko @ 2020-03-23 16:05 UTC (permalink / raw)
  To: Jon Hunter, Stephen Warren
  Cc: devicetree, alsa-devel, Mark Brown, Takashi Iwai, Liam Girdwood,
	linux-kernel, Rob Herring, linux-tegra

23.03.2020 18:59, Jon Hunter пишет:
> 
> 
> On 20/03/2020 22:30, Stephen Warren wrote:
>> On 3/20/20 2:55 PM, Dmitry Osipenko wrote:
>>> Hello,
>>>
>>> This small series adds audio route for built-in microphone on NVIDIA Tegra
>>> boards that use WM8903 CODEC. In particular this is needed in order to unmute
>>> internal microphone on Acer A500 tablet device. I'm planning to send out the
>>> device tree for the A500 for 5.8, so will be nice to get the microphone
>>> sorted out. Please review and apply, thanks in advance.
>>
>> It's been a long time since I looked at this code, but the series looks
>> plausible,
>> Acked-by: Stephen Warren <swarren@nvidia.com>
>>
>> (I wonder why machine->gpio_int_mic_en was already parse but never used!)
> 
> Looking at the ventana schematics, it appears that the internal-mic
> signals was connected to a header and not an actual mic, so maybe we
> never had a proper internal-mic on any board but allowed one to be
> connected.

I guess cardhu may have a mic, judging by downstream kernel sources (I
don't have cardhu schematics).

> Anyway, looks good to me as well.
> 
> Acked-by: Jon Hunter <jonathanh@nvidia.com>

Thanks

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

end of thread, other threads:[~2020-03-23 16:06 UTC | newest]

Thread overview: 20+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-03-20 20:55 [PATCH v1 0/2] Support built-in Mic on Tegra boards that use WM8903 Dmitry Osipenko
2020-03-20 20:55 ` Dmitry Osipenko
     [not found] ` <20200320205504.30466-1-digetx-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2020-03-20 20:55   ` [PATCH v1 1/2] dt-bindings: sound: tegra-wm8903: Document built-in microphone audio source Dmitry Osipenko
2020-03-20 20:55     ` Dmitry Osipenko
2020-03-20 20:55     ` Dmitry Osipenko
2020-03-20 20:55   ` [PATCH v1 2/2] ASoC: tegra: tegra_wm8903: Support DAPM events for built-in microphone Dmitry Osipenko
2020-03-20 20:55     ` Dmitry Osipenko
2020-03-20 20:55     ` Dmitry Osipenko
2020-03-20 22:30   ` [PATCH v1 0/2] Support built-in Mic on Tegra boards that use WM8903 Stephen Warren
2020-03-20 22:30     ` Stephen Warren
2020-03-20 22:30     ` Stephen Warren
     [not found]     ` <c27c2087-14cf-614d-a8c0-05072a54f24b-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2020-03-21  0:49       ` Dmitry Osipenko
2020-03-21  0:49         ` Dmitry Osipenko
2020-03-21  0:49         ` Dmitry Osipenko
2020-03-23 15:59       ` Jon Hunter
2020-03-23 15:59         ` Jon Hunter
2020-03-23 15:59         ` Jon Hunter
     [not found]         ` <da88ddff-e665-8cee-6f03-1a396602b076-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2020-03-23 16:05           ` Dmitry Osipenko
2020-03-23 16:05             ` Dmitry Osipenko
2020-03-23 16:05             ` Dmitry Osipenko

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.