ASoC: pcm512x: Split out bus drivers
authorMark Brown <broonie@linaro.org>
Fri, 7 Mar 2014 03:44:08 +0000 (11:44 +0800)
committerMark Brown <broonie@linaro.org>
Sun, 9 Mar 2014 07:38:22 +0000 (07:38 +0000)
Move to the new style of defining the bus interfaces in separate modules
in order to simplify dependencies.

Signed-off-by: Mark Brown <broonie@linaro.org>
sound/soc/codecs/Kconfig
sound/soc/codecs/Makefile
sound/soc/codecs/pcm512x-i2c.c [new file with mode: 0644]
sound/soc/codecs/pcm512x-spi.c [new file with mode: 0644]
sound/soc/codecs/pcm512x.c
sound/soc/codecs/pcm512x.h

index fa47a8336be34c4329586e236b25b435ef3b1bd3..cebe3ceef4bd0b7559ceb53d7c4820492a1e4ee3 100644 (file)
@@ -59,7 +59,8 @@ config SND_SOC_ALL_CODECS
        select SND_SOC_PCM1681 if I2C
        select SND_SOC_PCM1792A if SPI_MASTER
        select SND_SOC_PCM3008
-       select SND_SOC_PCM512x if SND_SOC_I2C_AND_SPI
+       select SND_SOC_PCM512x_I2C if I2C
+       select SND_SOC_PCM512x_SPI if SPI_MASTER
        select SND_SOC_RT5631 if I2C
        select SND_SOC_RT5640 if I2C
        select SND_SOC_SGTL5000 if I2C
@@ -315,8 +316,19 @@ config SND_SOC_PCM3008
        tristate
 
 config SND_SOC_PCM512x
-       tristate "Texas Instruments PCM512x CODECs"
-       select REGMAP
+       tristate
+
+config SND_SOC_PCM512x_I2C
+       tristate "Texas Instruments PCM512x CODECs - I2C"
+       depends on I2C
+       select SND_SOC_PCM512x
+       select REGMAP_I2C
+
+config SND_SOC_PCM512x_SPI
+       tristate "Texas Instruments PCM512x CODECs - SPI"
+       depends on SPI_MASTER
+       select SND_SOC_PCM512x
+       select REGMAP_SPI
 
 config SND_SOC_RT5631
        tristate
index d3b536fc075d07bc74c4a7fa8d9745bba8e978aa..c1191c05c88eb1641ff97ef007078f31b957e929 100644 (file)
@@ -47,6 +47,8 @@ snd-soc-pcm1681-objs := pcm1681.o
 snd-soc-pcm1792a-codec-objs := pcm1792a.o
 snd-soc-pcm3008-objs := pcm3008.o
 snd-soc-pcm512x-objs := pcm512x.o
+snd-soc-pcm512x-i2c-objs := pcm512x-i2c.o
+snd-soc-pcm512x-spi-objs := pcm512x-spi.o
 snd-soc-rt5631-objs := rt5631.o
 snd-soc-rt5640-objs := rt5640.o
 snd-soc-sgtl5000-objs := sgtl5000.o
@@ -181,6 +183,8 @@ obj-$(CONFIG_SND_SOC_PCM1681)       += snd-soc-pcm1681.o
 obj-$(CONFIG_SND_SOC_PCM1792A) += snd-soc-pcm1792a-codec.o
 obj-$(CONFIG_SND_SOC_PCM3008)  += snd-soc-pcm3008.o
 obj-$(CONFIG_SND_SOC_PCM512x)  += snd-soc-pcm512x.o
+obj-$(CONFIG_SND_SOC_PCM512x_I2C)      += snd-soc-pcm512x-i2c.o
+obj-$(CONFIG_SND_SOC_PCM512x_SPI)      += snd-soc-pcm512x-spi.o
 obj-$(CONFIG_SND_SOC_RT5631)   += snd-soc-rt5631.o
 obj-$(CONFIG_SND_SOC_RT5640)   += snd-soc-rt5640.o
 obj-$(CONFIG_SND_SOC_SGTL5000)  += snd-soc-sgtl5000.o
diff --git a/sound/soc/codecs/pcm512x-i2c.c b/sound/soc/codecs/pcm512x-i2c.c
new file mode 100644 (file)
index 0000000..4d62230
--- /dev/null
@@ -0,0 +1,71 @@
+/*
+ * Driver for the PCM512x CODECs
+ *
+ * Author:     Mark Brown <broonie@linaro.org>
+ *             Copyright 2014 Linaro Ltd
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License
+ * version 2 as published by the Free Software Foundation.
+ *
+ * This program is distributed in the hope that it will be useful, but
+ * WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * General Public License for more details.
+ */
+
+#include <linux/init.h>
+#include <linux/module.h>
+#include <linux/i2c.h>
+
+#include "pcm512x.h"
+
+static int pcm512x_i2c_probe(struct i2c_client *i2c,
+                            const struct i2c_device_id *id)
+{
+       struct regmap *regmap;
+
+       regmap = devm_regmap_init_i2c(i2c, &pcm512x_regmap);
+       if (IS_ERR(regmap))
+               return PTR_ERR(regmap);
+
+       return pcm512x_probe(&i2c->dev, regmap);
+}
+
+static int pcm512x_i2c_remove(struct i2c_client *i2c)
+{
+       pcm512x_remove(&i2c->dev);
+       return 0;
+}
+
+static const struct i2c_device_id pcm512x_i2c_id[] = {
+       { "pcm5121", },
+       { "pcm5122", },
+       { }
+};
+MODULE_DEVICE_TABLE(i2c, pcm512x_i2c_id);
+
+static const struct of_device_id pcm512x_of_match[] = {
+       { .compatible = "ti,pcm5121", },
+       { .compatible = "ti,pcm5122", },
+       { }
+};
+MODULE_DEVICE_TABLE(of, pcm512x_of_match);
+
+static struct i2c_driver pcm512x_i2c_driver = {
+       .probe          = pcm512x_i2c_probe,
+       .remove         = pcm512x_i2c_remove,
+       .id_table       = pcm512x_i2c_id,
+       .driver         = {
+               .name   = "pcm512x",
+               .owner  = THIS_MODULE,
+               .of_match_table = pcm512x_of_match,
+               .pm     = &pcm512x_pm_ops,
+       },
+};
+
+module_i2c_driver(pcm512x_i2c_driver);
+
+MODULE_DESCRIPTION("ASoC PCM512x codec driver - I2C");
+MODULE_AUTHOR("Mark Brown <broonie@linaro.org>");
+MODULE_LICENSE("GPL v2");
diff --git a/sound/soc/codecs/pcm512x-spi.c b/sound/soc/codecs/pcm512x-spi.c
new file mode 100644 (file)
index 0000000..f297058
--- /dev/null
@@ -0,0 +1,69 @@
+/*
+ * Driver for the PCM512x CODECs
+ *
+ * Author:     Mark Brown <broonie@linaro.org>
+ *             Copyright 2014 Linaro Ltd
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License
+ * version 2 as published by the Free Software Foundation.
+ *
+ * This program is distributed in the hope that it will be useful, but
+ * WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * General Public License for more details.
+ */
+
+#include <linux/init.h>
+#include <linux/module.h>
+#include <linux/spi/spi.h>
+
+#include "pcm512x.h"
+
+static int pcm512x_spi_probe(struct spi_device *spi)
+{
+       struct regmap *regmap;
+       int ret;
+
+       regmap = devm_regmap_init_spi(spi, &pcm512x_regmap);
+       if (IS_ERR(regmap)) {
+               ret = PTR_ERR(regmap);
+               return ret;
+       }
+
+       return pcm512x_probe(&spi->dev, regmap);
+}
+
+static int pcm512x_spi_remove(struct spi_device *spi)
+{
+       pcm512x_remove(&spi->dev);
+       return 0;
+}
+
+static const struct spi_device_id pcm512x_spi_id[] = {
+       { "pcm5121", },
+       { "pcm5122", },
+       { },
+};
+MODULE_DEVICE_TABLE(spi, pcm512x_spi_id);
+
+static const struct of_device_id pcm512x_of_match[] = {
+       { .compatible = "ti,pcm5121", },
+       { .compatible = "ti,pcm5122", },
+       { }
+};
+MODULE_DEVICE_TABLE(of, pcm512x_of_match);
+
+static struct spi_driver pcm512x_spi_driver = {
+       .probe          = pcm512x_spi_probe,
+       .remove         = pcm512x_spi_remove,
+       .id_table       = pcm512x_spi_id,
+       .driver = {
+               .name   = "pcm512x",
+               .owner  = THIS_MODULE,
+               .of_match_table = pcm512x_of_match,
+               .pm     = &pcm512x_pm_ops,
+       },
+};
+
+module_spi_driver(pcm512x_spi_driver);
index 3a0bbb6ab242ff4c1cd325948f06f6788d0a2a59..0c907051cc7b5266a04cf120da308b3c9862a9d4 100644 (file)
 #include <linux/init.h>
 #include <linux/module.h>
 #include <linux/clk.h>
-#include <linux/i2c.h>
 #include <linux/pm_runtime.h>
 #include <linux/regmap.h>
 #include <linux/regulator/consumer.h>
-#include <linux/spi/spi.h>
 #include <sound/soc.h>
 #include <sound/soc-dapm.h>
 #include <sound/tlv.h>
@@ -375,7 +373,7 @@ static const struct regmap_range_cfg pcm512x_range = {
        .window_start = 0, .window_len = 0x100,
 };
 
-static const struct regmap_config pcm512x_regmap = {
+const struct regmap_config pcm512x_regmap = {
        .reg_bits = 8,
        .val_bits = 8,
 
@@ -390,15 +388,9 @@ static const struct regmap_config pcm512x_regmap = {
        .num_reg_defaults = ARRAY_SIZE(pcm512x_reg_defaults),
        .cache_type = REGCACHE_RBTREE,
 };
+EXPORT_SYMBOL_GPL(pcm512x_regmap);
 
-static const struct of_device_id pcm512x_of_match[] = {
-       { .compatible = "ti,pcm5121", },
-       { .compatible = "ti,pcm5122", },
-       { }
-};
-MODULE_DEVICE_TABLE(of, pcm512x_of_match);
-
-static int pcm512x_probe(struct device *dev, struct regmap *regmap)
+int pcm512x_probe(struct device *dev, struct regmap *regmap)
 {
        struct pcm512x_priv *pcm512x;
        int i, ret;
@@ -510,8 +502,9 @@ err:
                                     pcm512x->supplies);
        return ret;
 }
+EXPORT_SYMBOL_GPL(pcm512x_probe);
 
-static void pcm512x_remove(struct device *dev)
+void pcm512x_remove(struct device *dev)
 {
        struct pcm512x_priv *pcm512x = dev_get_drvdata(dev);
 
@@ -522,6 +515,7 @@ static void pcm512x_remove(struct device *dev)
        regulator_bulk_disable(ARRAY_SIZE(pcm512x->supplies),
                               pcm512x->supplies);
 }
+EXPORT_SYMBOL_GPL(pcm512x_remove);
 
 static int pcm512x_suspend(struct device *dev)
 {
@@ -585,122 +579,10 @@ static int pcm512x_resume(struct device *dev)
        return 0;
 }
 
-static const struct dev_pm_ops pcm512x_pm_ops = {
+const struct dev_pm_ops pcm512x_pm_ops = {
        SET_RUNTIME_PM_OPS(pcm512x_suspend, pcm512x_resume, NULL)
 };
-
-#if IS_ENABLED(CONFIG_I2C)
-static int pcm512x_i2c_probe(struct i2c_client *i2c,
-                            const struct i2c_device_id *id)
-{
-       struct regmap *regmap;
-
-       regmap = devm_regmap_init_i2c(i2c, &pcm512x_regmap);
-       if (IS_ERR(regmap))
-               return PTR_ERR(regmap);
-
-       return pcm512x_probe(&i2c->dev, regmap);
-}
-
-static int pcm512x_i2c_remove(struct i2c_client *i2c)
-{
-       pcm512x_remove(&i2c->dev);
-       return 0;
-}
-
-static const struct i2c_device_id pcm512x_i2c_id[] = {
-       { "pcm5121", },
-       { "pcm5122", },
-       { }
-};
-MODULE_DEVICE_TABLE(i2c, pcm512x_i2c_id);
-
-static struct i2c_driver pcm512x_i2c_driver = {
-       .probe          = pcm512x_i2c_probe,
-       .remove         = pcm512x_i2c_remove,
-       .id_table       = pcm512x_i2c_id,
-       .driver         = {
-               .name   = "pcm512x",
-               .owner  = THIS_MODULE,
-               .of_match_table = pcm512x_of_match,
-               .pm     = &pcm512x_pm_ops,
-       },
-};
-#endif
-
-#if defined(CONFIG_SPI_MASTER)
-static int pcm512x_spi_probe(struct spi_device *spi)
-{
-       struct regmap *regmap;
-       int ret;
-
-       regmap = devm_regmap_init_spi(spi, &pcm512x_regmap);
-       if (IS_ERR(regmap)) {
-               ret = PTR_ERR(regmap);
-               return ret;
-       }
-
-       return pcm512x_probe(&spi->dev, regmap);
-}
-
-static int pcm512x_spi_remove(struct spi_device *spi)
-{
-       pcm512x_remove(&spi->dev);
-       return 0;
-}
-
-static const struct spi_device_id pcm512x_spi_id[] = {
-       { "pcm5121", },
-       { "pcm5122", },
-       { },
-};
-MODULE_DEVICE_TABLE(spi, pcm512x_spi_id);
-
-static struct spi_driver pcm512x_spi_driver = {
-       .probe          = pcm512x_spi_probe,
-       .remove         = pcm512x_spi_remove,
-       .id_table       = pcm512x_spi_id,
-       .driver = {
-               .name   = "pcm512x",
-               .owner  = THIS_MODULE,
-               .of_match_table = pcm512x_of_match,
-               .pm     = &pcm512x_pm_ops,
-       },
-};
-#endif
-
-static int __init pcm512x_modinit(void)
-{
-       int ret = 0;
-
-#if IS_ENABLED(CONFIG_I2C)
-       ret = i2c_add_driver(&pcm512x_i2c_driver);
-       if (ret) {
-               printk(KERN_ERR "Failed to register pcm512x I2C driver: %d\n",
-                      ret);
-       }
-#endif
-#if defined(CONFIG_SPI_MASTER)
-       ret = spi_register_driver(&pcm512x_spi_driver);
-       if (ret != 0) {
-               printk(KERN_ERR "Failed to register pcm512x SPI driver: %d\n",
-                      ret);
-       }
-#endif
-       return ret;
-}
-module_init(pcm512x_modinit);
-
-static void __exit pcm512x_exit(void)
-{
-#if IS_ENABLED(CONFIG_I2C)
-       i2c_del_driver(&pcm512x_i2c_driver);
-#endif
-#if defined(CONFIG_SPI_MASTER)
-       spi_unregister_driver(&pcm512x_spi_driver);
-#endif
-}
-module_exit(pcm512x_exit);
+EXPORT_SYMBOL_GPL(pcm512x_pm_ops);
 
 MODULE_DESCRIPTION("ASoC PCM512x codec driver");
 MODULE_AUTHOR("Mark Brown <broonie@linaro.org>");
index ac4a52c9ccf4fcc6b2f9791d1f127bd42c289f54..6ee76aaca09a0a5ebca6fde6b885e7d10cb558b0 100644 (file)
@@ -17,6 +17,9 @@
 #ifndef _SND_SOC_PCM512X
 #define _SND_SOC_PCM512X
 
+#include <linux/pm.h>
+#include <linux/regmap.h>
+
 #define PCM512x_VIRT_BASE 0x100
 #define PCM512x_PAGE_LEN  0x100
 #define PCM512x_PAGE_BASE(n)  (PCM512x_VIRT_BASE + (PCM512x_PAGE_LEN * n))
 #define PCM512x_AGBR_SHIFT 0
 #define PCM512x_AGBL_SHIFT 4
 
+extern const struct dev_pm_ops pcm512x_pm_ops;
+extern const struct regmap_config pcm512x_regmap;
+
+int pcm512x_probe(struct device *dev, struct regmap *regmap);
+void pcm512x_remove(struct device *dev);
+
 #endif