ASoC: tas2552: Correct CFG1 register bit definitions
authorPeter Ujfalusi <peter.ujfalusi@ti.com>
Thu, 4 Jun 2015 13:04:17 +0000 (16:04 +0300)
committerMark Brown <broonie@kernel.org>
Thu, 4 Jun 2015 16:50:02 +0000 (17:50 +0100)
Remove the _MASK postfix of the bit definitions, collect the CFG1 bit
definition in one place and correct the bit shifts at the same time.

Signed-off-by: Peter Ujfalusi <peter.ujfalusi@ti.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/codecs/tas2552.c
sound/soc/codecs/tas2552.h

index 07a0ec0..681b868 100644 (file)
@@ -126,10 +126,10 @@ static void tas2552_sw_shutdown(struct tas2552_data *tas_data, int sw_shutdown)
        if (sw_shutdown)
                cfg1_reg = 0;
        else
-               cfg1_reg = TAS2552_SWS_MASK;
+               cfg1_reg = TAS2552_SWS;
 
-       snd_soc_update_bits(tas_data->codec, TAS2552_CFG_1,
-                                                TAS2552_SWS_MASK, cfg1_reg);
+       snd_soc_update_bits(tas_data->codec, TAS2552_CFG_1, TAS2552_SWS,
+                           cfg1_reg);
 }
 #endif
 
@@ -258,11 +258,11 @@ static int tas2552_mute(struct snd_soc_dai *dai, int mute)
        struct snd_soc_codec *codec = dai->codec;
 
        if (mute)
-               cfg1_reg = TAS2552_MUTE_MASK;
+               cfg1_reg = TAS2552_MUTE;
        else
-               cfg1_reg = ~TAS2552_MUTE_MASK;
+               cfg1_reg = ~TAS2552_MUTE;
 
-       snd_soc_update_bits(codec, TAS2552_CFG_1, TAS2552_MUTE_MASK, cfg1_reg);
+       snd_soc_update_bits(codec, TAS2552_CFG_1, TAS2552_MUTE, cfg1_reg);
 
        return 0;
 }
@@ -370,7 +370,7 @@ static int tas2552_codec_probe(struct snd_soc_codec *codec)
                goto probe_fail;
        }
 
-       snd_soc_write(codec, TAS2552_CFG_1, TAS2552_MUTE_MASK |
+       snd_soc_write(codec, TAS2552_CFG_1, TAS2552_MUTE |
                                TAS2552_PLL_SRC_BCLK);
        snd_soc_write(codec, TAS2552_CFG_3, TAS2552_I2S_OUT_SEL |
                                TAS2552_DIN_SRC_SEL_AVG_L_R | TAS2552_88_96KHZ);
index 938d90f..0725bef 100644 (file)
 #define TAS2552_MAX_REG                        0x20
 
 /* CFG1 Register Masks */
-#define TAS2552_MUTE_MASK              (1 << 2)
-#define TAS2552_SWS_MASK               (1 << 1)
-#define TAS2552_WCLK_MASK              0x07
-#define TAS2552_CLASSD_EN_MASK (1 << 7)
+#define TAS2552_DEV_RESET              (1 << 0)
+#define TAS2552_SWS                    (1 << 1)
+#define TAS2552_MUTE                   (1 << 2)
+#define TAS2552_PLL_SRC_MCLK           (0x0 << 4)
+#define TAS2552_PLL_SRC_BCLK           (0x1 << 4)
+#define TAS2552_PLL_SRC_IVCLKIN                (0x2 << 4)
+#define TAS2552_PLL_SRC_1_8_FIXED      (0x3 << 4)
+#define TAS2552_PLL_SRC_MASK           TAS2552_PLL_SRC_1_8_FIXED
 
 /* CFG2 Register Masks */
 #define TAS2552_CLASSD_EN              (1 << 7)
 #define TAS2552_DAIFMT_RIGHT_J         (1 << 4)
 #define TAS2552_DAIFMT_LEFT_J          (0x11 << 3)
 
-#define TAS2552_PLL_SRC_MCLK   0x00
-#define TAS2552_PLL_SRC_BCLK   (1 << 3)
-#define TAS2552_PLL_SRC_IVCLKIN        (1 << 4)
-#define TAS2552_PLL_SRC_1_8_FIXED (0x11 << 3)
-
 #define TAS2552_DIN_SRC_SEL_MUTED      0x00
 #define TAS2552_DIN_SRC_SEL_LEFT       (1 << 4)
 #define TAS2552_DIN_SRC_SEL_RIGHT      (1 << 5)