2022-05-09 12:37:19

by Lukasz Majewski

[permalink] [raw]
Subject: [PATCH 1/2] ASoC: wm8940: add devicetree support

This adds devicetree support to the wm8940 codec driver.
With a DT-based kernel, there is no board-specific setting
to select the driver so allow it to be manually chosen.

Signed-off-by: Lukasz Majewski <[email protected]>
---
sound/soc/codecs/Kconfig | 2 +-
sound/soc/codecs/wm8940.c | 7 +++++++
2 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/sound/soc/codecs/Kconfig b/sound/soc/codecs/Kconfig
index f46a22660103..5120b15139bc 100644
--- a/sound/soc/codecs/Kconfig
+++ b/sound/soc/codecs/Kconfig
@@ -1811,7 +1811,7 @@ config SND_SOC_WM8904
depends on I2C

config SND_SOC_WM8940
- tristate
+ tristate "Wolfson Microelectronics WM8940 codec"
depends on I2C

config SND_SOC_WM8955
diff --git a/sound/soc/codecs/wm8940.c b/sound/soc/codecs/wm8940.c
index 440d048ef0c0..8615a55f84dd 100644
--- a/sound/soc/codecs/wm8940.c
+++ b/sound/soc/codecs/wm8940.c
@@ -779,9 +779,16 @@ static const struct i2c_device_id wm8940_i2c_id[] = {
};
MODULE_DEVICE_TABLE(i2c, wm8940_i2c_id);

+static const struct of_device_id wm8940_of_match[] = {
+ { .compatible = "wlf,wm8940", },
+ { }
+};
+MODULE_DEVICE_TABLE(of, wm8940_of_match);
+
static struct i2c_driver wm8940_i2c_driver = {
.driver = {
.name = "wm8940",
+ .of_match_table = wm8940_of_match,
},
.probe = wm8940_i2c_probe,
.id_table = wm8940_i2c_id,
--
2.35.1



2022-05-09 12:48:20

by Charles Keepax

[permalink] [raw]
Subject: Re: [PATCH 1/2] ASoC: wm8940: add devicetree support

On Mon, May 09, 2022 at 02:10:55PM +0200, Lukasz Majewski wrote:
> This adds devicetree support to the wm8940 codec driver.
> With a DT-based kernel, there is no board-specific setting
> to select the driver so allow it to be manually chosen.
>
> Signed-off-by: Lukasz Majewski <[email protected]>
> ---

Acked-by: Charles Keepax <[email protected]>

Thanks,
Charles

2022-05-09 15:25:07

by Mark Brown

[permalink] [raw]
Subject: Re: [PATCH 1/2] ASoC: wm8940: add devicetree support

On Mon, May 09, 2022 at 02:10:55PM +0200, Lukasz Majewski wrote:

> This adds devicetree support to the wm8940 codec driver.
> With a DT-based kernel, there is no board-specific setting
> to select the driver so allow it to be manually chosen.
>
> Signed-off-by: Lukasz Majewski <[email protected]>
> ---
> sound/soc/codecs/Kconfig | 2 +-
> sound/soc/codecs/wm8940.c | 7 +++++++
> 2 files changed, 8 insertions(+), 1 deletion(-)

You need to provide a binding document for any new bindings you add in
code.


Attachments:
(No filename) (522.00 B)
signature.asc (499.00 B)
Download all attachments

2022-05-09 18:13:54

by Mark Brown

[permalink] [raw]
Subject: Re: [PATCH 1/2] ASoC: wm8940: add devicetree support

On Mon, 9 May 2022 14:10:55 +0200, Lukasz Majewski wrote:
> This adds devicetree support to the wm8940 codec driver.
> With a DT-based kernel, there is no board-specific setting
> to select the driver so allow it to be manually chosen.
>
>

Applied to

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

Thanks!

[1/2] ASoC: wm8940: add devicetree support
commit: 3a3610aaa9dce8ef1257bb42ac7f0fe2b5809a54

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

2022-05-10 12:26:47

by Mark Brown

[permalink] [raw]
Subject: Re: [PATCH 1/2] ASoC: wm8940: add devicetree support

On Tue, May 10, 2022 at 09:24:38AM +0200, Lukasz Majewski wrote:
> > On Mon, May 09, 2022 at 02:10:55PM +0200, Lukasz Majewski wrote:

> > > This adds devicetree support to the wm8940 codec driver.
> > > With a DT-based kernel, there is no board-specific setting
> > > to select the driver so allow it to be manually chosen.

> > You need to provide a binding document for any new bindings you add in
> > code.

> The second patch in this series adds proper *.yaml file to Linux source
> tree.

You didn't send the patches as a series, you sent two separate threads
so they got totally separated in my inbox. I did eventually connect
them.


Attachments:
(No filename) (657.00 B)
signature.asc (499.00 B)
Download all attachments

2022-05-10 13:15:07

by Lukasz Majewski

[permalink] [raw]
Subject: Re: [PATCH 1/2] ASoC: wm8940: add devicetree support

Hi Mark,

> On Mon, May 09, 2022 at 02:10:55PM +0200, Lukasz Majewski wrote:
>
> > This adds devicetree support to the wm8940 codec driver.
> > With a DT-based kernel, there is no board-specific setting
> > to select the driver so allow it to be manually chosen.
> >
> > Signed-off-by: Lukasz Majewski <[email protected]>
> > ---
> > sound/soc/codecs/Kconfig | 2 +-
> > sound/soc/codecs/wm8940.c | 7 +++++++
> > 2 files changed, 8 insertions(+), 1 deletion(-)
>
> You need to provide a binding document for any new bindings you add in
> code.

The second patch in this series adds proper *.yaml file to Linux source
tree.


Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: [email protected]


Attachments:
(No filename) (499.00 B)
OpenPGP digital signature

2022-05-10 13:48:57

by Lukasz Majewski

[permalink] [raw]
Subject: Re: [PATCH 1/2] ASoC: wm8940: add devicetree support

Hi Mark,

> On Tue, May 10, 2022 at 09:24:38AM +0200, Lukasz Majewski wrote:
> > > On Mon, May 09, 2022 at 02:10:55PM +0200, Lukasz Majewski wrote:
>
> > > > This adds devicetree support to the wm8940 codec driver.
> > > > With a DT-based kernel, there is no board-specific setting
> > > > to select the driver so allow it to be manually chosen.
>
> > > You need to provide a binding document for any new bindings you
> > > add in code.
>
> > The second patch in this series adds proper *.yaml file to Linux
> > source tree.
>
> You didn't send the patches as a series, you sent two separate threads
> so they got totally separated in my inbox. I did eventually connect
> them.

Thanks :-)


Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: [email protected]


Attachments:
(No filename) (499.00 B)
OpenPGP digital signature