ASoC: wm8523: Remove #if IS_ENABLED(CONFIG_I2C)
authorKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Thu, 17 Nov 2016 01:13:35 +0000 (01:13 +0000)
committerMark Brown <broonie@kernel.org>
Mon, 21 Nov 2016 19:19:13 +0000 (19:19 +0000)
wm8523 driver doesn't work without CONFIG_I2C anyway.
Let's remove #if IS_ENABLED(CONFIG_I2C), and use module_i2c_driver().

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Acked-by: Charles Keepax <ckeepax@opensource.wolfsonmicro.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/codecs/wm8523.c

index deb2e075428ec9e29acdbcb981dc34630b0df0ff..6d0a2723bfde7e06956934d1792f719263353699 100644 (file)
@@ -446,7 +446,6 @@ static const struct regmap_config wm8523_regmap = {
        .volatile_reg = wm8523_volatile_register,
 };
 
-#if IS_ENABLED(CONFIG_I2C)
 static int wm8523_i2c_probe(struct i2c_client *i2c,
                            const struct i2c_device_id *id)
 {
@@ -543,29 +542,8 @@ static struct i2c_driver wm8523_i2c_driver = {
        .remove =   wm8523_i2c_remove,
        .id_table = wm8523_i2c_id,
 };
-#endif
 
-static int __init wm8523_modinit(void)
-{
-       int ret;
-#if IS_ENABLED(CONFIG_I2C)
-       ret = i2c_add_driver(&wm8523_i2c_driver);
-       if (ret != 0) {
-               printk(KERN_ERR "Failed to register WM8523 I2C driver: %d\n",
-                      ret);
-       }
-#endif
-       return 0;
-}
-module_init(wm8523_modinit);
-
-static void __exit wm8523_exit(void)
-{
-#if IS_ENABLED(CONFIG_I2C)
-       i2c_del_driver(&wm8523_i2c_driver);
-#endif
-}
-module_exit(wm8523_exit);
+module_i2c_driver(wm8523_i2c_driver);
 
 MODULE_DESCRIPTION("ASoC WM8523 driver");
 MODULE_AUTHOR("Mark Brown <broonie@opensource.wolfsonmicro.com>");