[alsa-devel] [PATCH 1/3] ASoC: codec: Add WM8987 device id to WM8750 driver
The WM8987 codec is register compatible with the WM8750, so just add it to the SPI and I²C device table.
Signed-off-by: Maurus Cuelenaere mcuelenaere@gmail.com --- sound/soc/codecs/wm8750.c | 11 ++++++++++- 1 files changed, 10 insertions(+), 1 deletions(-)
diff --git a/sound/soc/codecs/wm8750.c b/sound/soc/codecs/wm8750.c index 9407e19..e2c05e3 100644 --- a/sound/soc/codecs/wm8750.c +++ b/sound/soc/codecs/wm8750.c @@ -884,6 +884,7 @@ static int wm8750_i2c_remove(struct i2c_client *client)
static const struct i2c_device_id wm8750_i2c_id[] = { { "wm8750", 0 }, + { "wm8987", 0 }, /* WM8987 is register compatible with WM8750 */ { } }; MODULE_DEVICE_TABLE(i2c, wm8750_i2c_id); @@ -925,14 +926,22 @@ static int __devexit wm8750_spi_remove(struct spi_device *spi) return 0; }
+static const struct spi_device_id wm8750_spi_id[] = { + { "wm8750", 0 }, + { "wm8987", 0 }, + { } +}; +MODULE_DEVICE_TABLE(spi, wm8750_spi_id); + static struct spi_driver wm8750_spi_driver = { .driver = { - .name = "wm8750", + .name = "WM8750 SPI Codec", .bus = &spi_bus_type, .owner = THIS_MODULE, }, .probe = wm8750_spi_probe, .remove = __devexit_p(wm8750_spi_remove), + .id_table = wm8750_spi_id, }; #endif
On Sat, Jul 03, 2010 at 02:46:10AM +0200, Maurus Cuelenaere wrote:
The WM8987 codec is register compatible with the WM8750, so just add it to the SPI and I²C device table.
Signed-off-by: Maurus Cuelenaere mcuelenaere@gmail.com
Applied, thanks. Though...
.driver = {
.name = "wm8750",
.name = "WM8750 SPI Codec",
...that naming isn't going to look pretty in kernel logs with dev_() print statements.
Op 04-07-10 10:59, Mark Brown schreef:
On Sat, Jul 03, 2010 at 02:46:10AM +0200, Maurus Cuelenaere wrote:
The WM8987 codec is register compatible with the WM8750, so just add it to the SPI and I²C device table. Signed-off-by: Maurus Cuelenaere mcuelenaere@gmail.com
Applied, thanks. Though...
.driver = {
.name = "wm8750",
.name = "WM8750 SPI Codec",
...that naming isn't going to look pretty in kernel logs with dev_() print statements.
It's the same as how the I²C driver is named, so I guess that has the same problem then.
participants (2)
-
Mark Brown
-
Maurus Cuelenaere