Commit 06109f47 authored by Daniel Matuschek's avatar Daniel Matuschek Committed by Mark Brown

ASoC: wm8804: Allow control of master clock divider in PLL generation

WM8804 can run with PLL frequencies of 256xfs and 128xfs for
most sample rates. At 192kHz only 128xfs is supported. The
existing driver selects 128xfs automatically for some lower
samples rates. By using an additional mclk_div divider, it
is now possible to control the behaviour. This allows using
256xfs PLL frequency on all sample rates up to 96kHz. It
should allow lower jitter and better signal quality. The
behavior has to be controlled by the sound card driver,
because some sample frequency share the same setting. e.g.
192kHz and 96kHz use 24.576MHz master clock. The only
difference is the MCLK divider.
Signed-off-by: default avatarDaniel Matuschek <daniel@matuschek.net>
Tested-by: default avatarFlorian Meier <florian.meier@koalo.de>
Signed-off-by: default avatarCharles Keepax <ckeepax@opensource.wolfsonmicro.com>
Signed-off-by: default avatarMark Brown <broonie@linaro.org>
parent a3086791
...@@ -63,6 +63,7 @@ struct wm8804_priv { ...@@ -63,6 +63,7 @@ struct wm8804_priv {
struct regmap *regmap; struct regmap *regmap;
struct regulator_bulk_data supplies[WM8804_NUM_SUPPLIES]; struct regulator_bulk_data supplies[WM8804_NUM_SUPPLIES];
struct notifier_block disable_nb[WM8804_NUM_SUPPLIES]; struct notifier_block disable_nb[WM8804_NUM_SUPPLIES];
int mclk_div;
}; };
static int txsrc_get(struct snd_kcontrol *kcontrol, static int txsrc_get(struct snd_kcontrol *kcontrol,
...@@ -318,7 +319,7 @@ static struct { ...@@ -318,7 +319,7 @@ static struct {
#define FIXED_PLL_SIZE ((1ULL << 22) * 10) #define FIXED_PLL_SIZE ((1ULL << 22) * 10)
static int pll_factors(struct pll_div *pll_div, unsigned int target, static int pll_factors(struct pll_div *pll_div, unsigned int target,
unsigned int source) unsigned int source, unsigned int mclk_div)
{ {
u64 Kpart; u64 Kpart;
unsigned long int K, Ndiv, Nmod, tmp; unsigned long int K, Ndiv, Nmod, tmp;
...@@ -330,7 +331,8 @@ static int pll_factors(struct pll_div *pll_div, unsigned int target, ...@@ -330,7 +331,8 @@ static int pll_factors(struct pll_div *pll_div, unsigned int target,
*/ */
for (i = 0; i < ARRAY_SIZE(post_table); i++) { for (i = 0; i < ARRAY_SIZE(post_table); i++) {
tmp = target * post_table[i].div; tmp = target * post_table[i].div;
if (tmp >= 90000000 && tmp <= 100000000) { if ((tmp >= 90000000 && tmp <= 100000000) &&
(mclk_div == post_table[i].mclkdiv)) {
pll_div->freqmode = post_table[i].freqmode; pll_div->freqmode = post_table[i].freqmode;
pll_div->mclkdiv = post_table[i].mclkdiv; pll_div->mclkdiv = post_table[i].mclkdiv;
target *= post_table[i].div; target *= post_table[i].div;
...@@ -387,8 +389,12 @@ static int wm8804_set_pll(struct snd_soc_dai *dai, int pll_id, ...@@ -387,8 +389,12 @@ static int wm8804_set_pll(struct snd_soc_dai *dai, int pll_id,
} else { } else {
int ret; int ret;
struct pll_div pll_div; struct pll_div pll_div;
struct wm8804_priv *wm8804;
ret = pll_factors(&pll_div, freq_out, freq_in); wm8804 = snd_soc_codec_get_drvdata(codec);
ret = pll_factors(&pll_div, freq_out, freq_in,
wm8804->mclk_div);
if (ret) if (ret)
return ret; return ret;
...@@ -452,6 +458,7 @@ static int wm8804_set_clkdiv(struct snd_soc_dai *dai, ...@@ -452,6 +458,7 @@ static int wm8804_set_clkdiv(struct snd_soc_dai *dai,
int div_id, int div) int div_id, int div)
{ {
struct snd_soc_codec *codec; struct snd_soc_codec *codec;
struct wm8804_priv *wm8804;
codec = dai->codec; codec = dai->codec;
switch (div_id) { switch (div_id) {
...@@ -459,6 +466,10 @@ static int wm8804_set_clkdiv(struct snd_soc_dai *dai, ...@@ -459,6 +466,10 @@ static int wm8804_set_clkdiv(struct snd_soc_dai *dai,
snd_soc_update_bits(codec, WM8804_PLL5, 0x30, snd_soc_update_bits(codec, WM8804_PLL5, 0x30,
(div & 0x3) << 4); (div & 0x3) << 4);
break; break;
case WM8804_MCLK_DIV:
wm8804 = snd_soc_codec_get_drvdata(codec);
wm8804->mclk_div = div;
break;
default: default:
dev_err(dai->dev, "Unknown clock divider: %d\n", div_id); dev_err(dai->dev, "Unknown clock divider: %d\n", div_id);
return -EINVAL; return -EINVAL;
......
...@@ -57,5 +57,9 @@ ...@@ -57,5 +57,9 @@
#define WM8804_CLKOUT_SRC_OSCCLK 4 #define WM8804_CLKOUT_SRC_OSCCLK 4
#define WM8804_CLKOUT_DIV 1 #define WM8804_CLKOUT_DIV 1
#define WM8804_MCLK_DIV 2
#define WM8804_MCLKDIV_256FS 0
#define WM8804_MCLKDIV_128FS 1
#endif /* _WM8804_H */ #endif /* _WM8804_H */
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment