| From: Shengjiu Wang <shengjiu.wang@nxp.com> |
| Date: Wed, 11 Dec 2019 19:57:22 +0800 |
| Subject: ASoC: wm8962: fix lambda value |
| |
| commit 556672d75ff486e0b6786056da624131679e0576 upstream. |
| |
| According to user manual, it is required that FLL_LAMBDA > 0 |
| in all cases (Integer and Franctional modes). |
| |
| Fixes: 9a76f1ff6e29 ("ASoC: Add initial WM8962 CODEC driver") |
| Signed-off-by: Shengjiu Wang <shengjiu.wang@nxp.com> |
| Acked-by: Charles Keepax <ckeepax@opensource.cirrus.com> |
| Link: https://lore.kernel.org/r/1576065442-19763-1-git-send-email-shengjiu.wang@nxp.com |
| Signed-off-by: Mark Brown <broonie@kernel.org> |
| Signed-off-by: Ben Hutchings <ben@decadent.org.uk> |
| --- |
| sound/soc/codecs/wm8962.c | 4 ++-- |
| 1 file changed, 2 insertions(+), 2 deletions(-) |
| |
| --- a/sound/soc/codecs/wm8962.c |
| +++ b/sound/soc/codecs/wm8962.c |
| @@ -2795,7 +2795,7 @@ static int fll_factors(struct _fll_div * |
| |
| if (target % Fref == 0) { |
| fll_div->theta = 0; |
| - fll_div->lambda = 0; |
| + fll_div->lambda = 1; |
| } else { |
| gcd_fll = gcd(target, fratio * Fref); |
| |
| @@ -2865,7 +2865,7 @@ static int wm8962_set_fll(struct snd_soc |
| return -EINVAL; |
| } |
| |
| - if (fll_div.theta || fll_div.lambda) |
| + if (fll_div.theta) |
| fll1 |= WM8962_FLL_FRAC; |
| |
| /* Stop the FLL while we reconfigure */ |
