Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
kernel os linux

ASoC: Export OF module alias information in missing codec drivers

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>

authored by

Javier Martinez Canillas and committed by
Mark Brown
bf08f39e d770e558

+4
+1
sound/soc/codecs/da9055.c
··· 1533 1533 { .compatible = "dlg,da9055-codec", }, 1534 1534 { } 1535 1535 }; 1536 + MODULE_DEVICE_TABLE(of, da9055_of_match); 1536 1537 1537 1538 /* I2C codec control layer */ 1538 1539 static struct i2c_driver da9055_i2c_driver = {
+1
sound/soc/codecs/wm8510.c
··· 598 598 { .compatible = "wlf,wm8510" }, 599 599 { }, 600 600 }; 601 + MODULE_DEVICE_TABLE(of, wm8510_of_match); 601 602 602 603 static const struct regmap_config wm8510_regmap = { 603 604 .reg_bits = 7,
+1
sound/soc/codecs/wm8523.c
··· 430 430 { .compatible = "wlf,wm8523" }, 431 431 { }, 432 432 }; 433 + MODULE_DEVICE_TABLE(of, wm8523_of_match); 433 434 434 435 static const struct regmap_config wm8523_regmap = { 435 436 .reg_bits = 8,
+1
sound/soc/codecs/wm8580.c
··· 916 916 { .compatible = "wlf,wm8580" }, 917 917 { }, 918 918 }; 919 + MODULE_DEVICE_TABLE(of, wm8580_of_match); 919 920 920 921 static const struct regmap_config wm8580_regmap = { 921 922 .reg_bits = 7,