ASoC: adau17x1: Add support for specifying the MCLK using the CCF
authorLars-Peter Clausen <lars@metafoo.de>
Wed, 15 Jun 2016 13:07:27 +0000 (15:07 +0200)
committerMark Brown <broonie@kernel.org>
Wed, 15 Jun 2016 13:42:40 +0000 (14:42 +0100)
The devices from the ADAU17X1 family all have a MCLK clock input which
supplies the master clock for the device. The master clock is used as the
input clock for the PLL. Currently the MCLK rate as well as the desired PLL
output frequency need to be supplied by calling snd_soc_dai_set_pll() form
a machine driver.

Add support for specifying the MCLK using the common clock framework. In
addition to that also automatically configure the PLL to a suitable rate
if the master clock was provided using the CCW. This allows to use the
CODEC driver without any special configuration requirements from the
machine driver.

While the PLL output frequency can be configured over a (more or less)
continuous range the narrowness of the range and the other constraints of
the clocking tree usually only result in two output frequencies that will
actually be chosen. One for 44.1kHz based rates and one for 48kHz based
rates, these are the rates that the automatic PLL configuration will use.
For the rare case where a non-standard setup is required a machine driver
can disable the auto-configuration and configure a custom frequency using
the existing mechanisms.

If the common clock framework is not enabled clk_get() will return NULL and
the driver will function as before and the clock rate needs to be
configured manually.

Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
Signed-off-by: Mark Brown <broonie@kernel.org>
Documentation/devicetree/bindings/sound/adi,adau17x1.txt
sound/soc/codecs/adau1761-i2c.c
sound/soc/codecs/adau1761-spi.c
sound/soc/codecs/adau1781-i2c.c
sound/soc/codecs/adau1781-spi.c
sound/soc/codecs/adau17x1.c
sound/soc/codecs/adau17x1.h

index 8dbce0e18dda266b3c824e1c34c5ad9b744ca2d2..1447dec281252cb1f71374742e54331ebebd7c5b 100644 (file)
@@ -13,6 +13,11 @@ Required properties:
  - reg:                        The i2c address. Value depends on the state of ADDR0
                        and ADDR1, as wired in hardware.
 
+Optional properties:
+ - clock-names:                If provided must be "mclk".
+ - clocks:             phandle + clock-specifiers for the clock that provides
+                       the audio master clock for the device.
+
 Examples:
 #include <dt-bindings/sound/adau17x1.h>
 
@@ -20,5 +25,8 @@ Examples:
                adau1361@38 {
                        compatible = "adi,adau1761";
                        reg = <0x38>;
+
+                       clock-names = "mclk";
+                       clocks = <&audio_clock>;
                };
        };
index 8de010f758cd8d5007cf661a590f6e7c473da385..9e7f257f17f87591f380152553fc4c41725b1e2e 100644 (file)
@@ -31,7 +31,7 @@ static int adau1761_i2c_probe(struct i2c_client *client,
 
 static int adau1761_i2c_remove(struct i2c_client *client)
 {
-       snd_soc_unregister_codec(&client->dev);
+       adau17x1_remove(&client->dev);
        return 0;
 }
 
index d9171245bd9f11a177ce2429c0cbf50ef159af9d..a0b214be759a8700d3ad81eb098ed111b4b4d6c8 100644 (file)
@@ -48,7 +48,7 @@ static int adau1761_spi_probe(struct spi_device *spi)
 
 static int adau1761_spi_remove(struct spi_device *spi)
 {
-       snd_soc_unregister_codec(&spi->dev);
+       adau17x1_remove(&spi->dev);
        return 0;
 }
 
index 06cbca84cf02150ab0a83bbf114955ffe6ea62d1..7b9d1802d1598e9c170d28df0d28014c06642742 100644 (file)
@@ -31,7 +31,7 @@ static int adau1781_i2c_probe(struct i2c_client *client,
 
 static int adau1781_i2c_remove(struct i2c_client *client)
 {
-       snd_soc_unregister_codec(&client->dev);
+       adau17x1_remove(&client->dev);
        return 0;
 }
 
index 3d965a01b99cee318da30a080baebc8a060852cc..9b233544d2e8f34a49791237162cb8f965b453d5 100644 (file)
@@ -48,7 +48,7 @@ static int adau1781_spi_probe(struct spi_device *spi)
 
 static int adau1781_spi_remove(struct spi_device *spi)
 {
-       snd_soc_unregister_codec(&spi->dev);
+       adau17x1_remove(&spi->dev);
        return 0;
 }
 
index 66a6e061923d7b19b6896e8f4ee35442502281e1..439aa3ff1f99cd517f28988f80fe2b34c6489ab2 100644 (file)
@@ -9,6 +9,7 @@
 
 #include <linux/module.h>
 #include <linux/init.h>
+#include <linux/clk.h>
 #include <linux/delay.h>
 #include <linux/slab.h>
 #include <sound/core.h>
@@ -303,6 +304,116 @@ bool adau17x1_has_dsp(struct adau *adau)
 }
 EXPORT_SYMBOL_GPL(adau17x1_has_dsp);
 
+static int adau17x1_set_dai_pll(struct snd_soc_dai *dai, int pll_id,
+       int source, unsigned int freq_in, unsigned int freq_out)
+{
+       struct snd_soc_codec *codec = dai->codec;
+       struct adau *adau = snd_soc_codec_get_drvdata(codec);
+       int ret;
+
+       if (freq_in < 8000000 || freq_in > 27000000)
+               return -EINVAL;
+
+       ret = adau_calc_pll_cfg(freq_in, freq_out, adau->pll_regs);
+       if (ret < 0)
+               return ret;
+
+       /* The PLL register is 6 bytes long and can only be written at once. */
+       ret = regmap_raw_write(adau->regmap, ADAU17X1_PLL_CONTROL,
+                       adau->pll_regs, ARRAY_SIZE(adau->pll_regs));
+       if (ret)
+               return ret;
+
+       adau->pll_freq = freq_out;
+
+       return 0;
+}
+
+static int adau17x1_set_dai_sysclk(struct snd_soc_dai *dai,
+               int clk_id, unsigned int freq, int dir)
+{
+       struct snd_soc_dapm_context *dapm = snd_soc_codec_get_dapm(dai->codec);
+       struct adau *adau = snd_soc_codec_get_drvdata(dai->codec);
+       bool is_pll;
+       bool was_pll;
+
+       switch (clk_id) {
+       case ADAU17X1_CLK_SRC_MCLK:
+               is_pll = false;
+               break;
+       case ADAU17X1_CLK_SRC_PLL_AUTO:
+               if (!adau->mclk)
+                       return -EINVAL;
+               /* Fall-through */
+       case ADAU17X1_CLK_SRC_PLL:
+               is_pll = true;
+               break;
+       default:
+               return -EINVAL;
+       }
+
+       switch (adau->clk_src) {
+       case ADAU17X1_CLK_SRC_MCLK:
+               was_pll = false;
+               break;
+       case ADAU17X1_CLK_SRC_PLL:
+       case ADAU17X1_CLK_SRC_PLL_AUTO:
+               was_pll = true;
+               break;
+       default:
+               return -EINVAL;
+       }
+
+       adau->sysclk = freq;
+
+       if (is_pll != was_pll) {
+               if (is_pll) {
+                       snd_soc_dapm_add_routes(dapm,
+                               &adau17x1_dapm_pll_route, 1);
+               } else {
+                       snd_soc_dapm_del_routes(dapm,
+                               &adau17x1_dapm_pll_route, 1);
+               }
+       }
+
+       adau->clk_src = clk_id;
+
+       return 0;
+}
+
+static int adau17x1_auto_pll(struct snd_soc_dai *dai,
+       struct snd_pcm_hw_params *params)
+{
+       struct adau *adau = snd_soc_dai_get_drvdata(dai);
+       unsigned int pll_rate;
+
+       switch (params_rate(params)) {
+       case 48000:
+       case 8000:
+       case 12000:
+       case 16000:
+       case 24000:
+       case 32000:
+       case 96000:
+               pll_rate = 48000 * 1024;
+               break;
+       case 44100:
+       case 7350:
+       case 11025:
+       case 14700:
+       case 22050:
+       case 29400:
+       case 88200:
+               pll_rate = 44100 * 1024;
+               break;
+       default:
+               return -EINVAL;
+       }
+
+       return adau17x1_set_dai_pll(dai, ADAU17X1_PLL, ADAU17X1_PLL_SRC_MCLK,
+               clk_get_rate(adau->mclk), pll_rate);
+}
+
 static int adau17x1_hw_params(struct snd_pcm_substream *substream,
        struct snd_pcm_hw_params *params, struct snd_soc_dai *dai)
 {
@@ -312,10 +423,19 @@ static int adau17x1_hw_params(struct snd_pcm_substream *substream,
        unsigned int freq;
        int ret;
 
-       if (adau->clk_src == ADAU17X1_CLK_SRC_PLL)
+       switch (adau->clk_src) {
+       case ADAU17X1_CLK_SRC_PLL_AUTO:
+               ret = adau17x1_auto_pll(dai, params);
+               if (ret)
+                       return ret;
+               /* Fall-through */
+       case ADAU17X1_CLK_SRC_PLL:
                freq = adau->pll_freq;
-       else
+               break;
+       default:
                freq = adau->sysclk;
+               break;
+       }
 
        if (freq % params_rate(params) != 0)
                return -EINVAL;
@@ -387,62 +507,6 @@ static int adau17x1_hw_params(struct snd_pcm_substream *substream,
                        ADAU17X1_SERIAL_PORT1_DELAY_MASK, val);
 }
 
-static int adau17x1_set_dai_pll(struct snd_soc_dai *dai, int pll_id,
-       int source, unsigned int freq_in, unsigned int freq_out)
-{
-       struct snd_soc_codec *codec = dai->codec;
-       struct adau *adau = snd_soc_codec_get_drvdata(codec);
-       int ret;
-
-       if (freq_in < 8000000 || freq_in > 27000000)
-               return -EINVAL;
-
-       ret = adau_calc_pll_cfg(freq_in, freq_out, adau->pll_regs);
-       if (ret < 0)
-               return ret;
-
-       /* The PLL register is 6 bytes long and can only be written at once. */
-       ret = regmap_raw_write(adau->regmap, ADAU17X1_PLL_CONTROL,
-                       adau->pll_regs, ARRAY_SIZE(adau->pll_regs));
-       if (ret)
-               return ret;
-
-       adau->pll_freq = freq_out;
-
-       return 0;
-}
-
-static int adau17x1_set_dai_sysclk(struct snd_soc_dai *dai,
-               int clk_id, unsigned int freq, int dir)
-{
-       struct snd_soc_dapm_context *dapm = snd_soc_codec_get_dapm(dai->codec);
-       struct adau *adau = snd_soc_codec_get_drvdata(dai->codec);
-
-       switch (clk_id) {
-       case ADAU17X1_CLK_SRC_MCLK:
-       case ADAU17X1_CLK_SRC_PLL:
-               break;
-       default:
-               return -EINVAL;
-       }
-
-       adau->sysclk = freq;
-
-       if (adau->clk_src != clk_id) {
-               if (clk_id == ADAU17X1_CLK_SRC_PLL) {
-                       snd_soc_dapm_add_routes(dapm,
-                               &adau17x1_dapm_pll_route, 1);
-               } else {
-                       snd_soc_dapm_del_routes(dapm,
-                               &adau17x1_dapm_pll_route, 1);
-               }
-       }
-
-       adau->clk_src = clk_id;
-
-       return 0;
-}
-
 static int adau17x1_set_dai_fmt(struct snd_soc_dai *dai,
                unsigned int fmt)
 {
@@ -827,6 +891,10 @@ int adau17x1_add_routes(struct snd_soc_codec *codec)
                ret = snd_soc_dapm_add_routes(dapm, adau17x1_no_dsp_dapm_routes,
                        ARRAY_SIZE(adau17x1_no_dsp_dapm_routes));
        }
+
+       if (adau->clk_src != ADAU17X1_CLK_SRC_MCLK)
+               snd_soc_dapm_add_routes(dapm, &adau17x1_dapm_pll_route, 1);
+
        return ret;
 }
 EXPORT_SYMBOL_GPL(adau17x1_add_routes);
@@ -849,6 +917,7 @@ int adau17x1_probe(struct device *dev, struct regmap *regmap,
        const char *firmware_name)
 {
        struct adau *adau;
+       int ret;
 
        if (IS_ERR(regmap))
                return PTR_ERR(regmap);
@@ -857,6 +926,30 @@ int adau17x1_probe(struct device *dev, struct regmap *regmap,
        if (!adau)
                return -ENOMEM;
 
+       adau->mclk = devm_clk_get(dev, "mclk");
+       if (IS_ERR(adau->mclk)) {
+               if (PTR_ERR(adau->mclk) != -ENOENT)
+                       return PTR_ERR(adau->mclk);
+               /* Clock is optional (for the driver) */
+               adau->mclk = NULL;
+       } else if (adau->mclk) {
+               adau->clk_src = ADAU17X1_CLK_SRC_PLL_AUTO;
+
+               /*
+                * Any valid PLL output rate will work at this point, use one
+                * that is likely to be chosen later as well. The register will
+                * be written when the PLL is powered up for the first time.
+                */
+               ret = adau_calc_pll_cfg(clk_get_rate(adau->mclk), 48000 * 1024,
+                               adau->pll_regs);
+               if (ret < 0)
+                       return ret;
+
+               ret = clk_prepare_enable(adau->mclk);
+               if (ret)
+                       return ret;
+       }
+
        adau->regmap = regmap;
        adau->switch_mode = switch_mode;
        adau->type = type;
@@ -880,6 +973,16 @@ int adau17x1_probe(struct device *dev, struct regmap *regmap,
 }
 EXPORT_SYMBOL_GPL(adau17x1_probe);
 
+void adau17x1_remove(struct device *dev)
+{
+       struct adau *adau = dev_get_drvdata(dev);
+
+       snd_soc_unregister_codec(dev);
+       if (adau->mclk)
+               clk_disable_unprepare(adau->mclk);
+}
+EXPORT_SYMBOL_GPL(adau17x1_remove);
+
 MODULE_DESCRIPTION("ASoC ADAU1X61/ADAU1X81 common code");
 MODULE_AUTHOR("Lars-Peter Clausen <lars@metafoo.de>");
 MODULE_LICENSE("GPL");
index 5ae87a084d97e24a6aa243ba6cb1afc45290ac43..bf04b7efee4084cf77fa934eadf6ab7a3506b1ad 100644 (file)
@@ -22,13 +22,18 @@ enum adau17x1_pll_src {
 };
 
 enum adau17x1_clk_src {
+       /* Automatically configure PLL based on the sample rate */
+       ADAU17X1_CLK_SRC_PLL_AUTO,
        ADAU17X1_CLK_SRC_MCLK,
        ADAU17X1_CLK_SRC_PLL,
 };
 
+struct clk;
+
 struct adau {
        unsigned int sysclk;
        unsigned int pll_freq;
+       struct clk *mclk;
 
        enum adau17x1_clk_src clk_src;
        enum adau17x1_type type;
@@ -52,6 +57,7 @@ int adau17x1_add_routes(struct snd_soc_codec *codec);
 int adau17x1_probe(struct device *dev, struct regmap *regmap,
        enum adau17x1_type type, void (*switch_mode)(struct device *dev),
        const char *firmware_name);
+void adau17x1_remove(struct device *dev);
 int adau17x1_set_micbias_voltage(struct snd_soc_codec *codec,
        enum adau17x1_micbias_voltage micbias);
 bool adau17x1_readable_register(struct device *dev, unsigned int reg);