When shared_lrclk is set, LRCM will be enabled. But the following software
reset in wm8960_probe will reset it to the default state. So LRCM operation
should after software reset.

Signed-off-by: Zidan Wang <b50...@freescale.com>
---
 sound/soc/codecs/wm8960.c | 17 ++++++++---------
 1 file changed, 8 insertions(+), 9 deletions(-)

diff --git a/sound/soc/codecs/wm8960.c b/sound/soc/codecs/wm8960.c
index bc8793cd..21db90c 100644
--- a/sound/soc/codecs/wm8960.c
+++ b/sound/soc/codecs/wm8960.c
@@ -960,6 +960,14 @@ static int wm8960_probe(struct snd_soc_codec *codec)
                return ret;
        }
 
+       if (pdata->shared_lrclk) {
+               ret = snd_soc_update_bits(codec, WM8960_ADDCTL2, 0x4, 0x4);
+               if (ret < 0) {
+                       dev_err(codec->dev, "Failed to enable LRCM: %d\n", ret);
+                       return ret;
+               }
+       }
+
        /* Latch the update bits */
        snd_soc_update_bits(codec, WM8960_LINVOL, 0x100, 0x100);
        snd_soc_update_bits(codec, WM8960_RINVOL, 0x100, 0x100);
@@ -1030,15 +1038,6 @@ static int wm8960_i2c_probe(struct i2c_client *i2c,
        else if (i2c->dev.of_node)
                wm8960_set_pdata_from_of(i2c, &wm8960->pdata);
 
-       if (pdata && pdata->shared_lrclk) {
-               ret = regmap_update_bits(wm8960->regmap, WM8960_ADDCTL2,
-                                        0x4, 0x4);
-               if (ret != 0) {
-                       dev_err(&i2c->dev, "Failed to enable LRCM: %d\n",
-                               ret);
-                       return ret;
-               }
-       }
 
        i2c_set_clientdata(i2c, wm8960);
 
-- 
1.9.1

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to