Linux-Devicetree Archive on lore.kernel.org
 help / color / Atom feed
* [PATCH 0/3] Add NAU8812 support
@ 2019-11-15 16:08 Marco Felsch
  2019-11-15 16:08 ` [PATCH 1/3] dt-bindings: ASoC: add nau8812 documentation Marco Felsch
                   ` (2 more replies)
  0 siblings, 3 replies; 10+ messages in thread
From: Marco Felsch @ 2019-11-15 16:08 UTC (permalink / raw)
  To: robh+dt, KCHSU0, broonie, thomas.fehrenbacher
  Cc: kernel, alsa-devel, devicetree

Hi,

this small series adds the documentation and support for the nau8812
devices. The exisiting nau8810 driver can handle both devices so we only
need to add the dt-bindings and inform the user by adapting the Kconfig
entry.

Futhermore the series adds the support for the aux-port control.

@John Hsu
Please can you verify that register accesses to the aux-port control
bits don't affect the nau8810 devices? I don't have such a device but
the documentations says that those register fields are just '0'ed. So I
assume the nau8810 devices ignore such writes.

Marco Felsch (2):
  dt-bindings: ASoC: add nau8812 documentation
  ASoC: nau8810: add support for nau8812

Thomas Fehrenbacher (1):
  ASoC: nau8810: add aux input control, available on NAU8812

 Documentation/devicetree/bindings/sound/nau8810.txt | 2 +-
 sound/soc/codecs/Kconfig                            | 2 +-
 sound/soc/codecs/nau8810.c                          | 9 +++++++++
 sound/soc/codecs/nau8810.h                          | 2 ++
 4 files changed, 13 insertions(+), 2 deletions(-)

-- 
2.20.1


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

* [PATCH 1/3] dt-bindings: ASoC: add nau8812 documentation
  2019-11-15 16:08 [PATCH 0/3] Add NAU8812 support Marco Felsch
@ 2019-11-15 16:08 ` Marco Felsch
  2019-12-03 19:27   ` Rob Herring
  2019-11-15 16:08 ` [PATCH 2/3] ASoC: nau8810: add support for nau8812 Marco Felsch
  2019-11-15 16:08 ` [PATCH 3/3] ASoC: nau8810: add aux input control, available on NAU8812 Marco Felsch
  2 siblings, 1 reply; 10+ messages in thread
From: Marco Felsch @ 2019-11-15 16:08 UTC (permalink / raw)
  To: robh+dt, KCHSU0, broonie, thomas.fehrenbacher
  Cc: kernel, alsa-devel, devicetree

Add dt-bindings for the nau8812 device.

Signed-off-by: Marco Felsch <m.felsch@pengutronix.de>
---
 Documentation/devicetree/bindings/sound/nau8810.txt | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/sound/nau8810.txt b/Documentation/devicetree/bindings/sound/nau8810.txt
index 05830e477acd..ddb007a43e68 100644
--- a/Documentation/devicetree/bindings/sound/nau8810.txt
+++ b/Documentation/devicetree/bindings/sound/nau8810.txt
@@ -4,7 +4,7 @@ This device supports I2C only.
 
 Required properties:
 
-  - compatible : "nuvoton,nau8810"
+  - compatible : "nuvoton,nau8810" or "nuvoton,nau8812"
 
   - reg : the I2C address of the device.
 
-- 
2.20.1


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

* [PATCH 2/3] ASoC: nau8810: add support for nau8812
  2019-11-15 16:08 [PATCH 0/3] Add NAU8812 support Marco Felsch
  2019-11-15 16:08 ` [PATCH 1/3] dt-bindings: ASoC: add nau8812 documentation Marco Felsch
@ 2019-11-15 16:08 ` Marco Felsch
  2019-11-15 16:08 ` [PATCH 3/3] ASoC: nau8810: add aux input control, available on NAU8812 Marco Felsch
  2 siblings, 0 replies; 10+ messages in thread
From: Marco Felsch @ 2019-11-15 16:08 UTC (permalink / raw)
  To: robh+dt, KCHSU0, broonie, thomas.fehrenbacher
  Cc: kernel, alsa-devel, devicetree

The nau8812 device [1] uses exactly the same register layout as the
nau8810 so there is no need to differentiate those. The only difference
is the output power and an additional aux-port input on the nau8812.
The registers related to the aux-port don't affect the nau8810 in a
negative way. They are just ignored (set to '0').

[1] http://www.nuvoton.com/resource-files/NAU8812DatasheetRev2.7.pdf

Signed-off-by: Marco Felsch <m.felsch@pengutronix.de>
---
 sound/soc/codecs/Kconfig   | 2 +-
 sound/soc/codecs/nau8810.c | 2 ++
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/sound/soc/codecs/Kconfig b/sound/soc/codecs/Kconfig
index 229cc89f8c5a..d13a2e5d0afb 100644
--- a/sound/soc/codecs/Kconfig
+++ b/sound/soc/codecs/Kconfig
@@ -1470,7 +1470,7 @@ config SND_SOC_NAU8540
        depends on I2C
 
 config SND_SOC_NAU8810
-	tristate "Nuvoton Technology Corporation NAU88C10 CODEC"
+	tristate "Nuvoton Technology Corporation NAU88C10/12 CODEC"
 	depends on I2C
 
 config SND_SOC_NAU8822
diff --git a/sound/soc/codecs/nau8810.c b/sound/soc/codecs/nau8810.c
index de26758c30a8..a32a4a8d5f50 100644
--- a/sound/soc/codecs/nau8810.c
+++ b/sound/soc/codecs/nau8810.c
@@ -862,6 +862,7 @@ static int nau8810_i2c_probe(struct i2c_client *i2c,
 
 static const struct i2c_device_id nau8810_i2c_id[] = {
 	{ "nau8810", 0 },
+	{ "nau8812", 0 },
 	{ }
 };
 MODULE_DEVICE_TABLE(i2c, nau8810_i2c_id);
@@ -869,6 +870,7 @@ MODULE_DEVICE_TABLE(i2c, nau8810_i2c_id);
 #ifdef CONFIG_OF
 static const struct of_device_id nau8810_of_match[] = {
 	{ .compatible = "nuvoton,nau8810", },
+	{ .compatible = "nuvoton,nau8812", },
 	{ }
 };
 MODULE_DEVICE_TABLE(of, nau8810_of_match);
-- 
2.20.1


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

* [PATCH 3/3] ASoC: nau8810: add aux input control, available on NAU8812
  2019-11-15 16:08 [PATCH 0/3] Add NAU8812 support Marco Felsch
  2019-11-15 16:08 ` [PATCH 1/3] dt-bindings: ASoC: add nau8812 documentation Marco Felsch
  2019-11-15 16:08 ` [PATCH 2/3] ASoC: nau8810: add support for nau8812 Marco Felsch
@ 2019-11-15 16:08 ` Marco Felsch
  2019-11-15 16:12   ` Mark Brown
  2019-11-16  1:43   ` AS50 KCHsu0
  2 siblings, 2 replies; 10+ messages in thread
From: Marco Felsch @ 2019-11-15 16:08 UTC (permalink / raw)
  To: robh+dt, KCHSU0, broonie, thomas.fehrenbacher
  Cc: kernel, alsa-devel, devicetree

From: Thomas Fehrenbacher <thomas.fehrenbacher@siedle.de>

This commit adds the support to control the aux-port on the nau8812
devices. We don't need to differentiate the aux-port registers since
those bitfields are set to '0' on the nau8810 devices [1,2].

[1] http://www.nuvoton.com/resource-files/NAU8810_Datasheet_Rev_2.8.pdf
[2] http://www.nuvoton.com/resource-files/NAU8812DatasheetRev2.7.pdf

Signed-off-by: Thomas Fehrenbacher <thomas.fehrenbacher@siedle.de>
[m.felsch@pengutronix.de: add commit message]
Signed-off-by: Marco Felsch <m.felsch@pengutronix.de>
---
 sound/soc/codecs/nau8810.c | 7 +++++++
 sound/soc/codecs/nau8810.h | 2 ++
 2 files changed, 9 insertions(+)

diff --git a/sound/soc/codecs/nau8810.c b/sound/soc/codecs/nau8810.c
index a32a4a8d5f50..b1024d24d413 100644
--- a/sound/soc/codecs/nau8810.c
+++ b/sound/soc/codecs/nau8810.c
@@ -351,6 +351,9 @@ static const struct snd_kcontrol_new nau8810_snd_controls[] = {
 		NAU8810_DACOS_SFT, 1, 0),
 	SOC_SINGLE("ADC Oversampling Rate(128x) Switch", NAU8810_REG_ADC,
 		NAU8810_ADCOS_SFT, 1, 0),
+
+	SOC_SINGLE("AUX Input Enable Switch", NAU8810_REG_POWER1,
+		NAU8810_AUX_EN_SFT, 1, 0),
 };
 
 /* Speaker Output Mixer */
@@ -383,6 +386,8 @@ static const struct snd_kcontrol_new nau8810_inpga[] = {
 		NAU8810_NMICPGA_SFT, 1, 0),
 	SOC_DAPM_SINGLE("MicP Switch", NAU8810_REG_INPUT_SIGNAL,
 		NAU8810_PMICPGA_SFT, 1, 0),
+	SOC_DAPM_SINGLE("AUX Switch", NAU8810_REG_INPUT_SIGNAL,
+		NAU8810_AUXPGA_SFT, 1, 0),
 };
 
 /* Loopback Switch */
@@ -436,6 +441,7 @@ static const struct snd_soc_dapm_widget nau8810_dapm_widgets[] = {
 
 	SND_SOC_DAPM_INPUT("MICN"),
 	SND_SOC_DAPM_INPUT("MICP"),
+	SND_SOC_DAPM_INPUT("AUX"),
 	SND_SOC_DAPM_OUTPUT("MONOOUT"),
 	SND_SOC_DAPM_OUTPUT("SPKOUTP"),
 	SND_SOC_DAPM_OUTPUT("SPKOUTN"),
@@ -470,6 +476,7 @@ static const struct snd_soc_dapm_route nau8810_dapm_routes[] = {
 	{"Input PGA", NULL, "Mic Bias"},
 	{"Input PGA", "MicN Switch", "MICN"},
 	{"Input PGA", "MicP Switch", "MICP"},
+	{"Input PGA", "AUX Switch", "AUX"},
 
 	/* Digital Looptack */
 	{"Digital Loopback", "Switch", "ADC"},
diff --git a/sound/soc/codecs/nau8810.h b/sound/soc/codecs/nau8810.h
index 1ada31883dc6..7b5ecad173d3 100644
--- a/sound/soc/codecs/nau8810.h
+++ b/sound/soc/codecs/nau8810.h
@@ -69,6 +69,7 @@
 
 /* NAU8810_REG_POWER1 (0x1) */
 #define NAU8810_DCBUF_EN		(0x1 << 8)
+#define NAU8810_AUX_EN_SFT		6
 #define NAU8810_PLL_EN_SFT		5
 #define NAU8810_MICBIAS_EN_SFT	4
 #define NAU8810_ABIAS_EN		(0x1 << 3)
@@ -229,6 +230,7 @@
 /* NAU8810_REG_INPUT_SIGNAL (0x2C) */
 #define NAU8810_PMICPGA_SFT		0
 #define NAU8810_NMICPGA_SFT		1
+#define NAU8810_AUXPGA_SFT		2
 
 /* NAU8810_REG_PGAGAIN (0x2D) */
 #define NAU8810_PGAGAIN_SFT		0
-- 
2.20.1


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

* Re: [PATCH 3/3] ASoC: nau8810: add aux input control, available on NAU8812
  2019-11-15 16:08 ` [PATCH 3/3] ASoC: nau8810: add aux input control, available on NAU8812 Marco Felsch
@ 2019-11-15 16:12   ` Mark Brown
  2019-11-15 16:20     ` Marco Felsch
  2019-11-16  1:43   ` AS50 KCHsu0
  1 sibling, 1 reply; 10+ messages in thread
From: Mark Brown @ 2019-11-15 16:12 UTC (permalink / raw)
  To: Marco Felsch
  Cc: robh+dt, KCHSU0, thomas.fehrenbacher, kernel, alsa-devel, devicetree

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

On Fri, Nov 15, 2019 at 05:08:19PM +0100, Marco Felsch wrote:

> This commit adds the support to control the aux-port on the nau8812
> devices. We don't need to differentiate the aux-port registers since
> those bitfields are set to '0' on the nau8810 devices [1,2].

It'd still be better to only register these controls if the device isn't
a nau8812 so that userspace doesn't see them.

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

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

* Re: [PATCH 3/3] ASoC: nau8810: add aux input control, available on NAU8812
  2019-11-15 16:12   ` Mark Brown
@ 2019-11-15 16:20     ` Marco Felsch
  2019-11-15 17:14       ` Mark Brown
  0 siblings, 1 reply; 10+ messages in thread
From: Marco Felsch @ 2019-11-15 16:20 UTC (permalink / raw)
  To: Mark Brown
  Cc: robh+dt, KCHSU0, thomas.fehrenbacher, kernel, alsa-devel, devicetree

Hi Mark,

On 19-11-15 16:12, Mark Brown wrote:
> On Fri, Nov 15, 2019 at 05:08:19PM +0100, Marco Felsch wrote:
> 
> > This commit adds the support to control the aux-port on the nau8812
> > devices. We don't need to differentiate the aux-port registers since
> > those bitfields are set to '0' on the nau8810 devices [1,2].
> 
> It'd still be better to only register these controls if the device isn't
> a nau8812 so that userspace doesn't see them.

That's the thing I tought about 10min.. Imagine that: You have an
embedded device with a very good codec driver (all codec controlls are
available) but the hw-guys didn't connect all the in-/outputs. So
switching those controls are useless too. IMHO this is exactly the same
thing here.

Regards,
  Marco

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

* Re: [PATCH 3/3] ASoC: nau8810: add aux input control, available on NAU8812
  2019-11-15 16:20     ` Marco Felsch
@ 2019-11-15 17:14       ` Mark Brown
  0 siblings, 0 replies; 10+ messages in thread
From: Mark Brown @ 2019-11-15 17:14 UTC (permalink / raw)
  To: Marco Felsch
  Cc: robh+dt, KCHSU0, thomas.fehrenbacher, kernel, alsa-devel, devicetree

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

On Fri, Nov 15, 2019 at 05:20:53PM +0100, Marco Felsch wrote:
> On 19-11-15 16:12, Mark Brown wrote:

> > It'd still be better to only register these controls if the device isn't
> > a nau8812 so that userspace doesn't see them.

> That's the thing I tought about 10min.. Imagine that: You have an
> embedded device with a very good codec driver (all codec controlls are
> available) but the hw-guys didn't connect all the in-/outputs. So
> switching those controls are useless too. IMHO this is exactly the same
> thing here.

We do have the start of facilities to mask out unreachable controls
already (a machine driver could use them manually as things stand).  If
we ever get the graph API done properly that work is more likely to get
finished.

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

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

* Re: [PATCH 3/3] ASoC: nau8810: add aux input control, available on NAU8812
  2019-11-15 16:08 ` [PATCH 3/3] ASoC: nau8810: add aux input control, available on NAU8812 Marco Felsch
  2019-11-15 16:12   ` Mark Brown
@ 2019-11-16  1:43   ` AS50 KCHsu0
  2019-11-18  6:46     ` Marco Felsch
  1 sibling, 1 reply; 10+ messages in thread
From: AS50 KCHsu0 @ 2019-11-16  1:43 UTC (permalink / raw)
  To: Marco Felsch, robh+dt, broonie, thomas.fehrenbacher
  Cc: kernel, alsa-devel, devicetree, AS50 CTLin0, AS50 WTLi


Marco Felsch 於 2019/11/16 上午 12:08 寫道:
> From: Thomas Fehrenbacher <thomas.fehrenbacher@siedle.de>
>
> This commit adds the support to control the aux-port on the nau8812
> devices. We don't need to differentiate the aux-port registers since
> those bitfields are set to '0' on the nau8810 devices [1,2].
>
> [1] http://www.nuvoton.com/resource-files/NAU8810_Datasheet_Rev_2.8.pdf
> [2] http://www.nuvoton.com/resource-files/NAU8812DatasheetRev2.7.pdf
>
> Signed-off-by: Thomas Fehrenbacher <thomas.fehrenbacher@siedle.de>
> [m.felsch@pengutronix.de: add commit message]
> Signed-off-by: Marco Felsch <m.felsch@pengutronix.de>
> ---
>   sound/soc/codecs/nau8810.c | 7 +++++++
>   sound/soc/codecs/nau8810.h | 2 ++
>   2 files changed, 9 insertions(+)
>
> diff --git a/sound/soc/codecs/nau8810.c b/sound/soc/codecs/nau8810.c
> index a32a4a8d5f50..b1024d24d413 100644
> --- a/sound/soc/codecs/nau8810.c
> +++ b/sound/soc/codecs/nau8810.c
> @@ -351,6 +351,9 @@ static const struct snd_kcontrol_new nau8810_snd_controls[] = {
>   NAU8810_DACOS_SFT, 1, 0),
>   SOC_SINGLE("ADC Oversampling Rate(128x) Switch", NAU8810_REG_ADC,
>   NAU8810_ADCOS_SFT, 1, 0),
> +
> +SOC_SINGLE("AUX Input Enable Switch", NAU8810_REG_POWER1,
> +NAU8810_AUX_EN_SFT, 1, 0),
>   };
>


The power control has to move to the nau8810_dapm_widgets.

The DAPM can just control the power switch of AUXI dynamically.


>   /* Speaker Output Mixer */
> @@ -383,6 +386,8 @@ static const struct snd_kcontrol_new nau8810_inpga[] = {
>   NAU8810_NMICPGA_SFT, 1, 0),
>   SOC_DAPM_SINGLE("MicP Switch", NAU8810_REG_INPUT_SIGNAL,
>   NAU8810_PMICPGA_SFT, 1, 0),
> +SOC_DAPM_SINGLE("AUX Switch", NAU8810_REG_INPUT_SIGNAL,
> +NAU8810_AUXPGA_SFT, 1, 0),
>   };
>
>   /* Loopback Switch */
> @@ -436,6 +441,7 @@ static const struct snd_soc_dapm_widget nau8810_dapm_widgets[] = {
>
>   SND_SOC_DAPM_INPUT("MICN"),
>   SND_SOC_DAPM_INPUT("MICP"),
> +SND_SOC_DAPM_INPUT("AUX"),
>   SND_SOC_DAPM_OUTPUT("MONOOUT"),
>   SND_SOC_DAPM_OUTPUT("SPKOUTP"),
>   SND_SOC_DAPM_OUTPUT("SPKOUTN"),
> @@ -470,6 +476,7 @@ static const struct snd_soc_dapm_route nau8810_dapm_routes[] = {
>   {"Input PGA", NULL, "Mic Bias"},
>   {"Input PGA", "MicN Switch", "MICN"},
>   {"Input PGA", "MicP Switch", "MICP"},
> +{"Input PGA", "AUX Switch", "AUX"},
>
>   /* Digital Looptack */
>   {"Digital Loopback", "Switch", "ADC"},
> diff --git a/sound/soc/codecs/nau8810.h b/sound/soc/codecs/nau8810.h
> index 1ada31883dc6..7b5ecad173d3 100644
> --- a/sound/soc/codecs/nau8810.h
> +++ b/sound/soc/codecs/nau8810.h
> @@ -69,6 +69,7 @@
>
>   /* NAU8810_REG_POWER1 (0x1) */
>   #define NAU8810_DCBUF_EN(0x1 << 8)
> +#define NAU8810_AUX_EN_SFT6
>   #define NAU8810_PLL_EN_SFT5
>   #define NAU8810_MICBIAS_EN_SFT4
>   #define NAU8810_ABIAS_EN(0x1 << 3)
> @@ -229,6 +230,7 @@
>   /* NAU8810_REG_INPUT_SIGNAL (0x2C) */
>   #define NAU8810_PMICPGA_SFT0
>   #define NAU8810_NMICPGA_SFT1
> +#define NAU8810_AUXPGA_SFT2
>
>   /* NAU8810_REG_PGAGAIN (0x2D) */
>   #define NAU8810_PGAGAIN_SFT0
________________________________
________________________________
 The privileged confidential information contained in this email is intended for use only by the addressees as indicated by the original sender of this email. If you are not the addressee indicated in this email or are not responsible for delivery of the email to such a person, please kindly reply to the sender indicating this fact and delete all copies of it from your computer and network server immediately. Your cooperation is highly appreciated. It is advised that any unauthorized use of confidential information of Nuvoton is strictly prohibited; and any information in this email irrelevant to the official business of Nuvoton shall be deemed as neither given nor endorsed by Nuvoton.

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

* Re: [PATCH 3/3] ASoC: nau8810: add aux input control, available on NAU8812
  2019-11-16  1:43   ` AS50 KCHsu0
@ 2019-11-18  6:46     ` Marco Felsch
  0 siblings, 0 replies; 10+ messages in thread
From: Marco Felsch @ 2019-11-18  6:46 UTC (permalink / raw)
  To: AS50 KCHsu0
  Cc: robh+dt, broonie, thomas.fehrenbacher, kernel, alsa-devel,
	devicetree, AS50 CTLin0, AS50 WTLi

Hi,

On 19-11-16 01:43, AS50 KCHsu0 wrote:
> 
> Marco Felsch 於 2019/11/16 上午 12:08 寫道:
> > From: Thomas Fehrenbacher <thomas.fehrenbacher@siedle.de>
> >
> > This commit adds the support to control the aux-port on the nau8812
> > devices. We don't need to differentiate the aux-port registers since
> > those bitfields are set to '0' on the nau8810 devices [1,2].
> >
> > [1] http://www.nuvoton.com/resource-files/NAU8810_Datasheet_Rev_2.8.pdf
> > [2] http://www.nuvoton.com/resource-files/NAU8812DatasheetRev2.7.pdf
> >
> > Signed-off-by: Thomas Fehrenbacher <thomas.fehrenbacher@siedle.de>
> > [m.felsch@pengutronix.de: add commit message]
> > Signed-off-by: Marco Felsch <m.felsch@pengutronix.de>
> > ---
> >   sound/soc/codecs/nau8810.c | 7 +++++++
> >   sound/soc/codecs/nau8810.h | 2 ++
> >   2 files changed, 9 insertions(+)
> >
> > diff --git a/sound/soc/codecs/nau8810.c b/sound/soc/codecs/nau8810.c
> > index a32a4a8d5f50..b1024d24d413 100644
> > --- a/sound/soc/codecs/nau8810.c
> > +++ b/sound/soc/codecs/nau8810.c
> > @@ -351,6 +351,9 @@ static const struct snd_kcontrol_new nau8810_snd_controls[] = {
> >   NAU8810_DACOS_SFT, 1, 0),
> >   SOC_SINGLE("ADC Oversampling Rate(128x) Switch", NAU8810_REG_ADC,
> >   NAU8810_ADCOS_SFT, 1, 0),
> > +
> > +SOC_SINGLE("AUX Input Enable Switch", NAU8810_REG_POWER1,
> > +NAU8810_AUX_EN_SFT, 1, 0),
> >   };
> >
> 
> 
> The power control has to move to the nau8810_dapm_widgets.
> 
> The DAPM can just control the power switch of AUXI dynamically.

Okay, thanks for covering that. I will prepare a v2 with your and Mark's
feedback.

Regards,
  Marco

> >   /* Speaker Output Mixer */
> > @@ -383,6 +386,8 @@ static const struct snd_kcontrol_new nau8810_inpga[] = {
> >   NAU8810_NMICPGA_SFT, 1, 0),
> >   SOC_DAPM_SINGLE("MicP Switch", NAU8810_REG_INPUT_SIGNAL,
> >   NAU8810_PMICPGA_SFT, 1, 0),
> > +SOC_DAPM_SINGLE("AUX Switch", NAU8810_REG_INPUT_SIGNAL,
> > +NAU8810_AUXPGA_SFT, 1, 0),
> >   };
> >
> >   /* Loopback Switch */
> > @@ -436,6 +441,7 @@ static const struct snd_soc_dapm_widget nau8810_dapm_widgets[] = {
> >
> >   SND_SOC_DAPM_INPUT("MICN"),
> >   SND_SOC_DAPM_INPUT("MICP"),
> > +SND_SOC_DAPM_INPUT("AUX"),
> >   SND_SOC_DAPM_OUTPUT("MONOOUT"),
> >   SND_SOC_DAPM_OUTPUT("SPKOUTP"),
> >   SND_SOC_DAPM_OUTPUT("SPKOUTN"),
> > @@ -470,6 +476,7 @@ static const struct snd_soc_dapm_route nau8810_dapm_routes[] = {
> >   {"Input PGA", NULL, "Mic Bias"},
> >   {"Input PGA", "MicN Switch", "MICN"},
> >   {"Input PGA", "MicP Switch", "MICP"},
> > +{"Input PGA", "AUX Switch", "AUX"},
> >
> >   /* Digital Looptack */
> >   {"Digital Loopback", "Switch", "ADC"},
> > diff --git a/sound/soc/codecs/nau8810.h b/sound/soc/codecs/nau8810.h
> > index 1ada31883dc6..7b5ecad173d3 100644
> > --- a/sound/soc/codecs/nau8810.h
> > +++ b/sound/soc/codecs/nau8810.h
> > @@ -69,6 +69,7 @@
> >
> >   /* NAU8810_REG_POWER1 (0x1) */
> >   #define NAU8810_DCBUF_EN(0x1 << 8)
> > +#define NAU8810_AUX_EN_SFT6
> >   #define NAU8810_PLL_EN_SFT5
> >   #define NAU8810_MICBIAS_EN_SFT4
> >   #define NAU8810_ABIAS_EN(0x1 << 3)
> > @@ -229,6 +230,7 @@
> >   /* NAU8810_REG_INPUT_SIGNAL (0x2C) */
> >   #define NAU8810_PMICPGA_SFT0
> >   #define NAU8810_NMICPGA_SFT1
> > +#define NAU8810_AUXPGA_SFT2
> >
> >   /* NAU8810_REG_PGAGAIN (0x2D) */
> >   #define NAU8810_PGAGAIN_SFT0
> ________________________________
> ________________________________
>  The privileged confidential information contained in this email is intended for use only by the addressees as indicated by the original sender of this email. If you are not the addressee indicated in this email or are not responsible for delivery of the email to such a person, please kindly reply to the sender indicating this fact and delete all copies of it from your computer and network server immediately. Your cooperation is highly appreciated. It is advised that any unauthorized use of confidential information of Nuvoton is strictly prohibited; and any information in this email irrelevant to the official business of Nuvoton shall be deemed as neither given nor endorsed by Nuvoton.

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

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

* Re: [PATCH 1/3] dt-bindings: ASoC: add nau8812 documentation
  2019-11-15 16:08 ` [PATCH 1/3] dt-bindings: ASoC: add nau8812 documentation Marco Felsch
@ 2019-12-03 19:27   ` Rob Herring
  0 siblings, 0 replies; 10+ messages in thread
From: Rob Herring @ 2019-12-03 19:27 UTC (permalink / raw)
  To: Marco Felsch
  Cc: robh+dt, KCHSU0, broonie, thomas.fehrenbacher, kernel,
	alsa-devel, devicetree

On Fri, 15 Nov 2019 17:08:17 +0100, Marco Felsch wrote:
> Add dt-bindings for the nau8812 device.
> 
> Signed-off-by: Marco Felsch <m.felsch@pengutronix.de>
> ---
>  Documentation/devicetree/bindings/sound/nau8810.txt | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 

Acked-by: Rob Herring <robh@kernel.org>

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

end of thread, back to index

Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-11-15 16:08 [PATCH 0/3] Add NAU8812 support Marco Felsch
2019-11-15 16:08 ` [PATCH 1/3] dt-bindings: ASoC: add nau8812 documentation Marco Felsch
2019-12-03 19:27   ` Rob Herring
2019-11-15 16:08 ` [PATCH 2/3] ASoC: nau8810: add support for nau8812 Marco Felsch
2019-11-15 16:08 ` [PATCH 3/3] ASoC: nau8810: add aux input control, available on NAU8812 Marco Felsch
2019-11-15 16:12   ` Mark Brown
2019-11-15 16:20     ` Marco Felsch
2019-11-15 17:14       ` Mark Brown
2019-11-16  1:43   ` AS50 KCHsu0
2019-11-18  6:46     ` Marco Felsch

Linux-Devicetree Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-devicetree/0 linux-devicetree/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-devicetree linux-devicetree/ https://lore.kernel.org/linux-devicetree \
		devicetree@vger.kernel.org
	public-inbox-index linux-devicetree

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.linux-devicetree


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git