ASoC: Export OF module alias information in missing codec drivers
authorJavier Martinez Canillas <javier@osg.samsung.com>
Thu, 30 Jul 2015 16:18:45 +0000 (18:18 +0200)
committerMark Brown <broonie@kernel.org>
Fri, 31 Jul 2015 18:11:26 +0000 (19:11 +0100)
The I2C core always reports the MODALIAS uevent as "i2c:<client name"
regardless if the driver was matched using the I2C id_table or the
of_match_table. So technically there's no need for a driver to export
the OF table since currently it's not used.

In fact, the I2C device ID table is mandatory for I2C drivers since
a i2c_device_id is passed to the driver's probe function even if the
I2C core used the OF table to match the driver.

And since the I2C core uses different tables, OF-only drivers needs to
have duplicated data that has to be kept in sync and also the dev node
compatible manufacturer prefix is stripped when reporting the MODALIAS.

To avoid the above, the I2C core behavior may be changed in the future
to not require an I2C device table for OF-only drivers and report the
OF module alias. So, it's better to also export the OF table to prevent
breaking module autoloading if that happens.

Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/codecs/da9055.c
sound/soc/codecs/wm8510.c
sound/soc/codecs/wm8523.c
sound/soc/codecs/wm8580.c

index 66bb446473b868532ef7bee0c117c9584f5e995f..ab39de6cd0eb9fdcd07f0ac64130ead564665a6b 100644 (file)
@@ -1533,6 +1533,7 @@ static const struct of_device_id da9055_of_match[] = {
        { .compatible = "dlg,da9055-codec", },
        { }
 };
+MODULE_DEVICE_TABLE(of, da9055_of_match);
 
 /* I2C codec control layer */
 static struct i2c_driver da9055_i2c_driver = {
index dac5beb4d02356f7e8eef2fbf8e464ab77aa4f6b..0617415514cd69a1bf42ddac6e5e840a03d6cf5f 100644 (file)
@@ -598,6 +598,7 @@ static const struct of_device_id wm8510_of_match[] = {
        { .compatible = "wlf,wm8510" },
        { },
 };
+MODULE_DEVICE_TABLE(of, wm8510_of_match);
 
 static const struct regmap_config wm8510_regmap = {
        .reg_bits = 7,
index 43ea8ae5f871d2ce4185c41b9a2c3384707c245e..483ec72c34ab203a6ae59411056751f20d9181ee 100644 (file)
@@ -430,6 +430,7 @@ static const struct of_device_id wm8523_of_match[] = {
        { .compatible = "wlf,wm8523" },
        { },
 };
+MODULE_DEVICE_TABLE(of, wm8523_of_match);
 
 static const struct regmap_config wm8523_regmap = {
        .reg_bits = 8,
index 759a7928ac3ee48a43ecf7f626683042d5126534..a9f3900e0dce1dd1470fde86bf679992424e35ab 100644 (file)
@@ -916,6 +916,7 @@ static const struct of_device_id wm8580_of_match[] = {
        { .compatible = "wlf,wm8580" },
        { },
 };
+MODULE_DEVICE_TABLE(of, wm8580_of_match);
 
 static const struct regmap_config wm8580_regmap = {
        .reg_bits = 7,