* [PATCH 1/3] ASoC: codecs: wm8904: add dt ids table
@ 2014-12-11 3:14 Bo Shen
2014-12-11 3:14 ` [PATCH 2/3] ARM: at91: sama5d3xek/dt: use dt ids for wm8904 Bo Shen
` (2 more replies)
0 siblings, 3 replies; 5+ messages in thread
From: Bo Shen @ 2014-12-11 3:14 UTC (permalink / raw)
To: nicolas.ferre, Mark Brown
Cc: linux-arm-kernel, linux-kernel, devicetree, alsa-devel,
linux-sound, Alexander Morozov, Bo Shen
From: Alexander Morozov <linux@meltdown.ru>
Signed-off-by: Alexander Morozov <linux@meltdown.ru>
Signed-off-by: Bo Shen <voice.shen@atmel.com>
---
sound/soc/codecs/wm8904.c | 9 +++++++++
1 file changed, 9 insertions(+)
diff --git a/sound/soc/codecs/wm8904.c b/sound/soc/codecs/wm8904.c
index 4d2d2b1..23831a5 100644
--- a/sound/soc/codecs/wm8904.c
+++ b/sound/soc/codecs/wm8904.c
@@ -2255,10 +2255,19 @@ static const struct i2c_device_id wm8904_i2c_id[] = {
};
MODULE_DEVICE_TABLE(i2c, wm8904_i2c_id);
+static const struct of_device_id wm8904_of_match[] = {
+ { .compatible = "wlf,wm8904", },
+ { .compatible = "wlf,wm8912", },
+ { .compatible = "wlf,wm8918", },
+ { }
+};
+MODULE_DEVICE_TABLE(of, wm8904_of_match);
+
static struct i2c_driver wm8904_i2c_driver = {
.driver = {
.name = "wm8904",
.owner = THIS_MODULE,
+ .of_match_table = wm8904_of_match,
},
.probe = wm8904_i2c_probe,
.remove = wm8904_i2c_remove,
--
2.1.0.24.g4109c28
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH 2/3] ARM: at91: sama5d3xek/dt: use dt ids for wm8904
2014-12-11 3:14 [PATCH 1/3] ASoC: codecs: wm8904: add dt ids table Bo Shen
@ 2014-12-11 3:14 ` Bo Shen
2014-12-11 3:14 ` [PATCH 3/3] ARM: at91: at91sam9n12ek/dt: " Bo Shen
2014-12-12 13:23 ` [PATCH 1/3] ASoC: codecs: wm8904: add dt ids table Mark Brown
2 siblings, 0 replies; 5+ messages in thread
From: Bo Shen @ 2014-12-11 3:14 UTC (permalink / raw)
To: nicolas.ferre, Mark Brown
Cc: devicetree, alsa-devel, linux-kernel, linux-sound,
Alexander Morozov, Bo Shen, linux-arm-kernel
From: Alexander Morozov <linux@meltdown.ru>
Using the wm8904 device tree compatible string.
Signed-off-by: Alexander Morozov <linux@meltdown.ru>
Signed-off-by: Bo Shen <voice.shen@atmel.com>
---
arch/arm/boot/dts/sama5d3xmb.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/sama5d3xmb.dtsi b/arch/arm/boot/dts/sama5d3xmb.dtsi
index 49c10d3..2a1aa58 100644
--- a/arch/arm/boot/dts/sama5d3xmb.dtsi
+++ b/arch/arm/boot/dts/sama5d3xmb.dtsi
@@ -45,7 +45,7 @@
*/
i2c0: i2c@f0014000 {
wm8904: wm8904@1a {
- compatible = "wm8904";
+ compatible = "wlf,wm8904";
reg = <0x1a>;
clocks = <&pck0>;
clock-names = "mclk";
--
2.1.0.24.g4109c28
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH 3/3] ARM: at91: at91sam9n12ek/dt: use dt ids for wm8904
2014-12-11 3:14 [PATCH 1/3] ASoC: codecs: wm8904: add dt ids table Bo Shen
2014-12-11 3:14 ` [PATCH 2/3] ARM: at91: sama5d3xek/dt: use dt ids for wm8904 Bo Shen
@ 2014-12-11 3:14 ` Bo Shen
2014-12-12 13:23 ` [PATCH 1/3] ASoC: codecs: wm8904: add dt ids table Mark Brown
2 siblings, 0 replies; 5+ messages in thread
From: Bo Shen @ 2014-12-11 3:14 UTC (permalink / raw)
To: nicolas.ferre, Mark Brown
Cc: linux-arm-kernel, linux-kernel, devicetree, alsa-devel,
linux-sound, Bo Shen
Using the device tree ids for wm8904 codec.
Signed-off-by: Bo Shen <voice.shen@atmel.com>
---
arch/arm/boot/dts/at91sam9n12ek.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/at91sam9n12ek.dts b/arch/arm/boot/dts/at91sam9n12ek.dts
index 13bb24e..9575c0d 100644
--- a/arch/arm/boot/dts/at91sam9n12ek.dts
+++ b/arch/arm/boot/dts/at91sam9n12ek.dts
@@ -54,7 +54,7 @@
status = "okay";
wm8904: codec@1a {
- compatible = "wm8904";
+ compatible = "wlf,wm8904";
reg = <0x1a>;
clocks = <&pck0>;
clock-names = "mclk";
--
2.1.0.24.g4109c28
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH 1/3] ASoC: codecs: wm8904: add dt ids table
2014-12-11 3:14 [PATCH 1/3] ASoC: codecs: wm8904: add dt ids table Bo Shen
2014-12-11 3:14 ` [PATCH 2/3] ARM: at91: sama5d3xek/dt: use dt ids for wm8904 Bo Shen
2014-12-11 3:14 ` [PATCH 3/3] ARM: at91: at91sam9n12ek/dt: " Bo Shen
@ 2014-12-12 13:23 ` Mark Brown
2014-12-15 3:28 ` Bo Shen
2 siblings, 1 reply; 5+ messages in thread
From: Mark Brown @ 2014-12-12 13:23 UTC (permalink / raw)
To: Bo Shen
Cc: nicolas.ferre, linux-arm-kernel, linux-kernel, devicetree,
alsa-devel, linux-sound, Alexander Morozov
[-- Attachment #1: Type: text/plain, Size: 590 bytes --]
On Thu, Dec 11, 2014 at 11:14:39AM +0800, Bo Shen wrote:
> +static const struct of_device_id wm8904_of_match[] = {
> + { .compatible = "wlf,wm8904", },
> + { .compatible = "wlf,wm8912", },
> + { .compatible = "wlf,wm8918", },
> + { }
> +};
> +MODULE_DEVICE_TABLE(of, wm8904_of_match);
> +
> static struct i2c_driver wm8904_i2c_driver = {
> .driver = {
> .name = "wm8904",
> .owner = THIS_MODULE,
> + .of_match_table = wm8904_of_match,
I'm not seeing anything here that ensures that devtype is set correctly
when probing using the OF IDs - currently we rely on i2c->driver_data.
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 473 bytes --]
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 1/3] ASoC: codecs: wm8904: add dt ids table
2014-12-12 13:23 ` [PATCH 1/3] ASoC: codecs: wm8904: add dt ids table Mark Brown
@ 2014-12-15 3:28 ` Bo Shen
0 siblings, 0 replies; 5+ messages in thread
From: Bo Shen @ 2014-12-15 3:28 UTC (permalink / raw)
To: Mark Brown
Cc: nicolas.ferre, linux-arm-kernel, linux-kernel, devicetree,
alsa-devel, linux-sound, Alexander Morozov
Hi Mark,
On 12/12/2014 09:23 PM, Mark Brown wrote:
> On Thu, Dec 11, 2014 at 11:14:39AM +0800, Bo Shen wrote:
>
>> +static const struct of_device_id wm8904_of_match[] = {
>> + { .compatible = "wlf,wm8904", },
>> + { .compatible = "wlf,wm8912", },
>> + { .compatible = "wlf,wm8918", },
>> + { }
>> +};
>> +MODULE_DEVICE_TABLE(of, wm8904_of_match);
>> +
>> static struct i2c_driver wm8904_i2c_driver = {
>> .driver = {
>> .name = "wm8904",
>> .owner = THIS_MODULE,
>> + .of_match_table = wm8904_of_match,
>
> I'm not seeing anything here that ensures that devtype is set correctly
> when probing using the OF IDs - currently we rely on i2c->driver_data.
>
I will add it in v2 patch.
Thanks.
Best Regards,
Bo Shen
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2014-12-15 3:28 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-12-11 3:14 [PATCH 1/3] ASoC: codecs: wm8904: add dt ids table Bo Shen
2014-12-11 3:14 ` [PATCH 2/3] ARM: at91: sama5d3xek/dt: use dt ids for wm8904 Bo Shen
2014-12-11 3:14 ` [PATCH 3/3] ARM: at91: at91sam9n12ek/dt: " Bo Shen
2014-12-12 13:23 ` [PATCH 1/3] ASoC: codecs: wm8904: add dt ids table Mark Brown
2014-12-15 3:28 ` Bo Shen
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).