ASoC: fsl_micfil: use GENMASK to define register bit fields
authorSascha Hauer <s.hauer@pengutronix.de>
Thu, 14 Apr 2022 16:22:33 +0000 (18:22 +0200)
committerMark Brown <broonie@kernel.org>
Tue, 19 Apr 2022 11:06:13 +0000 (12:06 +0100)
Use GENMASK along with FIELD_PREP and FIELD_GET to access bitfields in
registers to straighten register access and to drop a lot of defines.

Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
Acked-by: Shengjiu Wang <shengjiu.wang@gmail.com>
Link: https://lore.kernel.org/r/20220414162249.3934543-6-s.hauer@pengutronix.de
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/fsl/fsl_micfil.c
sound/soc/fsl/fsl_micfil.h

index 878d24f..cfa8af6 100644 (file)
@@ -1,6 +1,7 @@
 // SPDX-License-Identifier: GPL-2.0
 // Copyright 2018 NXP
 
+#include <linux/bitfield.h>
 #include <linux/clk.h>
 #include <linux/device.h>
 #include <linux/interrupt.h>
@@ -116,23 +117,22 @@ static inline int get_pdm_clk(struct fsl_micfil *micfil,
        int bclk;
 
        regmap_read(micfil->regmap, REG_MICFIL_CTRL2, &ctrl2_reg);
-       osr = 16 - ((ctrl2_reg & MICFIL_CTRL2_CICOSR_MASK)
-                   >> MICFIL_CTRL2_CICOSR_SHIFT);
-       qsel = ctrl2_reg & MICFIL_CTRL2_QSEL_MASK;
+       osr = 16 - FIELD_GET(MICFIL_CTRL2_CICOSR, ctrl2_reg);
+       qsel = FIELD_GET(MICFIL_CTRL2_QSEL, ctrl2_reg);
 
        switch (qsel) {
-       case MICFIL_HIGH_QUALITY:
+       case MICFIL_QSEL_HIGH_QUALITY:
                bclk = rate * 8 * osr / 2; /* kfactor = 0.5 */
                break;
-       case MICFIL_MEDIUM_QUALITY:
-       case MICFIL_VLOW0_QUALITY:
+       case MICFIL_QSEL_MEDIUM_QUALITY:
+       case MICFIL_QSEL_VLOW0_QUALITY:
                bclk = rate * 4 * osr * 1; /* kfactor = 1 */
                break;
-       case MICFIL_LOW_QUALITY:
-       case MICFIL_VLOW1_QUALITY:
+       case MICFIL_QSEL_LOW_QUALITY:
+       case MICFIL_QSEL_VLOW1_QUALITY:
                bclk = rate * 2 * osr * 2; /* kfactor = 2 */
                break;
-       case MICFIL_VLOW2_QUALITY:
+       case MICFIL_QSEL_VLOW2_QUALITY:
                bclk = rate * osr * 4; /* kfactor = 4 */
                break;
        default:
@@ -244,8 +244,8 @@ static int fsl_micfil_trigger(struct snd_pcm_substream *substream, int cmd,
                 * 11 - reserved
                 */
                ret = regmap_update_bits(micfil->regmap, REG_MICFIL_CTRL1,
-                                        MICFIL_CTRL1_DISEL_MASK,
-                                        (1 << MICFIL_CTRL1_DISEL_SHIFT));
+                               MICFIL_CTRL1_DISEL,
+                               FIELD_PREP(MICFIL_CTRL1_DISEL, MICFIL_CTRL1_DISEL_DMA));
                if (ret) {
                        dev_err(dev, "failed to update DISEL bits\n");
                        return ret;
@@ -274,8 +274,8 @@ static int fsl_micfil_trigger(struct snd_pcm_substream *substream, int cmd,
                }
 
                ret = regmap_update_bits(micfil->regmap, REG_MICFIL_CTRL1,
-                                        MICFIL_CTRL1_DISEL_MASK,
-                                        (0 << MICFIL_CTRL1_DISEL_SHIFT));
+                               MICFIL_CTRL1_DISEL,
+                               FIELD_PREP(MICFIL_CTRL1_DISEL, MICFIL_CTRL1_DISEL_DISABLE));
                if (ret) {
                        dev_err(dev, "failed to update DISEL bits\n");
                        return ret;
@@ -300,8 +300,8 @@ static int fsl_set_clock_params(struct device *dev, unsigned int rate)
 
        /* set CICOSR */
        ret |= regmap_update_bits(micfil->regmap, REG_MICFIL_CTRL2,
-                                MICFIL_CTRL2_CICOSR_MASK,
-                                MICFIL_CTRL2_OSR_DEFAULT);
+                                MICFIL_CTRL2_CICOSR,
+                                FIELD_PREP(MICFIL_CTRL2_CICOSR, MICFIL_CTRL2_CICOSR_DEFAULT));
        if (ret)
                dev_err(dev, "failed to set CICOSR in reg 0x%X\n",
                        REG_MICFIL_CTRL2);
@@ -312,7 +312,8 @@ static int fsl_set_clock_params(struct device *dev, unsigned int rate)
                ret = -EINVAL;
 
        ret |= regmap_update_bits(micfil->regmap, REG_MICFIL_CTRL2,
-                                MICFIL_CTRL2_CLKDIV_MASK, clk_div);
+                                MICFIL_CTRL2_CLKDIV,
+                                FIELD_PREP(MICFIL_CTRL2_CLKDIV, clk_div));
        if (ret)
                dev_err(dev, "failed to set CLKDIV in reg 0x%X\n",
                        REG_MICFIL_CTRL2);
@@ -368,13 +369,13 @@ static int fsl_micfil_dai_probe(struct snd_soc_dai *cpu_dai)
 {
        struct fsl_micfil *micfil = dev_get_drvdata(cpu_dai->dev);
        struct device *dev = cpu_dai->dev;
-       unsigned int val;
        int ret;
        int i;
 
        /* set qsel to medium */
        ret = regmap_update_bits(micfil->regmap, REG_MICFIL_CTRL2,
-                                MICFIL_CTRL2_QSEL_MASK, MICFIL_MEDIUM_QUALITY);
+                       MICFIL_CTRL2_QSEL,
+                       FIELD_PREP(MICFIL_CTRL2_QSEL, MICFIL_QSEL_MEDIUM_QUALITY));
        if (ret) {
                dev_err(dev, "failed to set quality mode bits, reg 0x%X\n",
                        REG_MICFIL_CTRL2);
@@ -390,10 +391,9 @@ static int fsl_micfil_dai_probe(struct snd_soc_dai *cpu_dai)
                                  &micfil->dma_params_rx);
 
        /* FIFO Watermark Control - FIFOWMK*/
-       val = MICFIL_FIFO_CTRL_FIFOWMK(micfil->soc->fifo_depth) - 1;
        ret = regmap_update_bits(micfil->regmap, REG_MICFIL_FIFO_CTRL,
-                                MICFIL_FIFO_CTRL_FIFOWMK_MASK,
-                                val);
+                       MICFIL_FIFO_CTRL_FIFOWMK,
+                       FIELD_PREP(MICFIL_FIFO_CTRL_FIFOWMK, micfil->soc->fifo_depth - 1));
        if (ret) {
                dev_err(dev, "failed to set FIFOWMK\n");
                return ret;
@@ -553,11 +553,11 @@ static irqreturn_t micfil_isr(int irq, void *devid)
        regmap_read(micfil->regmap, REG_MICFIL_CTRL1, &ctrl1_reg);
        regmap_read(micfil->regmap, REG_MICFIL_FIFO_STAT, &fifo_stat_reg);
 
-       dma_enabled = MICFIL_DMA_ENABLED(ctrl1_reg);
+       dma_enabled = FIELD_GET(MICFIL_CTRL1_DISEL, ctrl1_reg) == MICFIL_CTRL1_DISEL_DMA;
 
        /* Channel 0-7 Output Data Flags */
        for (i = 0; i < MICFIL_OUTPUT_CHANNELS; i++) {
-               if (stat_reg & MICFIL_STAT_CHXF_MASK(i))
+               if (stat_reg & MICFIL_STAT_CHXF(i))
                        dev_dbg(&pdev->dev,
                                "Data available in Data Channel %d\n", i);
                /* if DMA is not enabled, field must be written with 1
@@ -566,17 +566,17 @@ static irqreturn_t micfil_isr(int irq, void *devid)
                if (!dma_enabled)
                        regmap_write_bits(micfil->regmap,
                                          REG_MICFIL_STAT,
-                                         MICFIL_STAT_CHXF_MASK(i),
+                                         MICFIL_STAT_CHXF(i),
                                          1);
        }
 
        for (i = 0; i < MICFIL_FIFO_NUM; i++) {
-               if (fifo_stat_reg & MICFIL_FIFO_STAT_FIFOX_OVER_MASK(i))
+               if (fifo_stat_reg & MICFIL_FIFO_STAT_FIFOX_OVER(i))
                        dev_dbg(&pdev->dev,
                                "FIFO Overflow Exception flag for channel %d\n",
                                i);
 
-               if (fifo_stat_reg & MICFIL_FIFO_STAT_FIFOX_UNDER_MASK(i))
+               if (fifo_stat_reg & MICFIL_FIFO_STAT_FIFOX_UNDER(i))
                        dev_dbg(&pdev->dev,
                                "FIFO Underflow Exception flag for channel %d\n",
                                i);
index 11ccc08..5cecae2 100644 (file)
 #define MICFIL_CTRL1_DBG               BIT(28)
 #define MICFIL_CTRL1_SRES              BIT(27)
 #define MICFIL_CTRL1_DBGE              BIT(26)
-#define MICFIL_CTRL1_DISEL_SHIFT       24
-#define MICFIL_CTRL1_DISEL_WIDTH       2
-#define MICFIL_CTRL1_DISEL_MASK                ((BIT(MICFIL_CTRL1_DISEL_WIDTH) - 1) \
-                                        << MICFIL_CTRL1_DISEL_SHIFT)
+
+#define MICFIL_CTRL1_DISEL_DISABLE     0
+#define MICFIL_CTRL1_DISEL_DMA         1
+#define MICFIL_CTRL1_DISEL_IRQ         2
+#define MICFIL_CTRL1_DISEL             GENMASK(25, 24)
 #define MICFIL_CTRL1_ERREN             BIT(23)
-#define MICFIL_CTRL1_CHEN_SHIFT                0
-#define MICFIL_CTRL1_CHEN_WIDTH                8
-#define MICFIL_CTRL1_CHEN_MASK(x)      (BIT(x) << MICFIL_CTRL1_CHEN_SHIFT)
-#define MICFIL_CTRL1_CHEN(x)           (MICFIL_CTRL1_CHEN_MASK(x))
+#define MICFIL_CTRL1_CHEN(ch)          BIT(ch)
 
 /* MICFIL Control Register 2 -- REG_MICFILL_CTRL2 0x04 */
 #define MICFIL_CTRL2_QSEL_SHIFT                25
-#define MICFIL_CTRL2_QSEL_WIDTH                3
-#define MICFIL_CTRL2_QSEL_MASK         ((BIT(MICFIL_CTRL2_QSEL_WIDTH) - 1) \
-                                        << MICFIL_CTRL2_QSEL_SHIFT)
-#define MICFIL_HIGH_QUALITY            BIT(MICFIL_CTRL2_QSEL_SHIFT)
-#define MICFIL_MEDIUM_QUALITY          (0 << MICFIL_CTRL2_QSEL_SHIFT)
-#define MICFIL_LOW_QUALITY             (7 << MICFIL_CTRL2_QSEL_SHIFT)
-#define MICFIL_VLOW0_QUALITY           (6 << MICFIL_CTRL2_QSEL_SHIFT)
-#define MICFIL_VLOW1_QUALITY           (5 << MICFIL_CTRL2_QSEL_SHIFT)
-#define MICFIL_VLOW2_QUALITY           (4 << MICFIL_CTRL2_QSEL_SHIFT)
-
-#define MICFIL_CTRL2_CICOSR_SHIFT      16
-#define MICFIL_CTRL2_CICOSR_WIDTH      4
-#define MICFIL_CTRL2_CICOSR_MASK       ((BIT(MICFIL_CTRL2_CICOSR_WIDTH) - 1) \
-                                        << MICFIL_CTRL2_CICOSR_SHIFT)
-#define MICFIL_CTRL2_CICOSR(v)         (((v) << MICFIL_CTRL2_CICOSR_SHIFT) \
-                                        & MICFIL_CTRL2_CICOSR_MASK)
-#define MICFIL_CTRL2_CLKDIV_SHIFT      0
-#define MICFIL_CTRL2_CLKDIV_WIDTH      8
-#define MICFIL_CTRL2_CLKDIV_MASK       ((BIT(MICFIL_CTRL2_CLKDIV_WIDTH) - 1) \
-                                        << MICFIL_CTRL2_CLKDIV_SHIFT)
-#define MICFIL_CTRL2_CLKDIV(v)         (((v) << MICFIL_CTRL2_CLKDIV_SHIFT) \
-                                        & MICFIL_CTRL2_CLKDIV_MASK)
+#define MICFIL_CTRL2_QSEL              GENMASK(27, 25)
+#define MICFIL_QSEL_MEDIUM_QUALITY     0
+#define MICFIL_QSEL_HIGH_QUALITY       1
+#define MICFIL_QSEL_LOW_QUALITY                7
+#define MICFIL_QSEL_VLOW0_QUALITY      6
+#define MICFIL_QSEL_VLOW1_QUALITY      5
+#define MICFIL_QSEL_VLOW2_QUALITY      4
+
+#define MICFIL_CTRL2_CICOSR            GENMASK(19, 16)
+#define MICFIL_CTRL2_CICOSR_DEFAULT    0
+#define MICFIL_CTRL2_CLKDIV            GENMASK(7, 0)
 
 /* MICFIL Status Register -- REG_MICFIL_STAT 0x08 */
 #define MICFIL_STAT_BSY_FIL            BIT(31)
 #define MICFIL_STAT_FIR_RDY            BIT(30)
 #define MICFIL_STAT_LOWFREQF           BIT(29)
-#define MICFIL_STAT_CHXF_SHIFT(v)      (v)
-#define MICFIL_STAT_CHXF_MASK(v)       BIT(MICFIL_STAT_CHXF_SHIFT(v))
-#define MICFIL_STAT_CHXF(v)            BIT(MICFIL_STAT_CHXF_SHIFT(v))
+#define MICFIL_STAT_CHXF(ch)           BIT(ch)
 
 /* MICFIL FIFO Control Register -- REG_MICFIL_FIFO_CTRL 0x10 */
-#define MICFIL_FIFO_CTRL_FIFOWMK_SHIFT 0
-#define MICFIL_FIFO_CTRL_FIFOWMK_WIDTH 3
-#define MICFIL_FIFO_CTRL_FIFOWMK_MASK  ((BIT(MICFIL_FIFO_CTRL_FIFOWMK_WIDTH) - 1) \
-                                        << MICFIL_FIFO_CTRL_FIFOWMK_SHIFT)
-#define MICFIL_FIFO_CTRL_FIFOWMK(v)    (((v) << MICFIL_FIFO_CTRL_FIFOWMK_SHIFT) \
-                                        & MICFIL_FIFO_CTRL_FIFOWMK_MASK)
+#define MICFIL_FIFO_CTRL_FIFOWMK       GENMASK(2, 0)
 
 /* MICFIL FIFO Status Register -- REG_MICFIL_FIFO_STAT 0x14 */
-#define MICFIL_FIFO_STAT_FIFOX_OVER_SHIFT(v)   (v)
-#define MICFIL_FIFO_STAT_FIFOX_OVER_MASK(v)    BIT(MICFIL_FIFO_STAT_FIFOX_OVER_SHIFT(v))
-#define MICFIL_FIFO_STAT_FIFOX_UNDER_SHIFT(v)  ((v) + 8)
-#define MICFIL_FIFO_STAT_FIFOX_UNDER_MASK(v)   BIT(MICFIL_FIFO_STAT_FIFOX_UNDER_SHIFT(v))
+#define MICFIL_FIFO_STAT_FIFOX_OVER(ch)        BIT(ch)
+#define MICFIL_FIFO_STAT_FIFOX_UNDER(ch)       BIT((ch) + 8)
 
 /* MICFIL HWVAD0 Control 1 Register -- REG_MICFIL_VAD0_CTRL1*/
-#define MICFIL_VAD0_CTRL1_CHSEL_SHIFT  24
-#define MICFIL_VAD0_CTRL1_CHSEL_WIDTH  3
-#define MICFIL_VAD0_CTRL1_CHSEL_MASK   ((BIT(MICFIL_VAD0_CTRL1_CHSEL_WIDTH) - 1) \
-                                        << MICFIL_VAD0_CTRL1_CHSEL_SHIFT)
-#define MICFIL_VAD0_CTRL1_CHSEL(v)     (((v) << MICFIL_VAD0_CTRL1_CHSEL_SHIFT) \
-                                        & MICFIL_VAD0_CTRL1_CHSEL_MASK)
-#define MICFIL_VAD0_CTRL1_CICOSR_SHIFT 16
-#define MICFIL_VAD0_CTRL1_CICOSR_WIDTH 4
-#define MICFIL_VAD0_CTRL1_CICOSR_MASK  ((BIT(MICFIL_VAD0_CTRL1_CICOSR_WIDTH) - 1) \
-                                        << MICFIL_VAD0_CTRL1_CICOSR_SHIFT)
-#define MICFIL_VAD0_CTRL1_CICOSR(v)    (((v) << MICFIL_VAD0_CTRL1_CICOSR_SHIFT) \
-                                        & MICFIL_VAD0_CTRL1_CICOSR_MASK)
-#define MICFIL_VAD0_CTRL1_INITT_SHIFT  8
-#define MICFIL_VAD0_CTRL1_INITT_WIDTH  5
-#define MICFIL_VAD0_CTRL1_INITT_MASK   ((BIT(MICFIL_VAD0_CTRL1_INITT_WIDTH) - 1) \
-                                        << MICFIL_VAD0_CTRL1_INITT_SHIFT)
-#define MICFIL_VAD0_CTRL1_INITT(v)     (((v) << MICFIL_VAD0_CTRL1_INITT_SHIFT) \
-                                        & MICFIL_VAD0_CTRL1_INITT_MASK)
+#define MICFIL_VAD0_CTRL1_CHSEL_SHIFT  GENMASK(26, 24)
+#define MICFIL_VAD0_CTRL1_CICOSR_SHIFT GENMASK(19, 16)
+#define MICFIL_VAD0_CTRL1_INITT_SHIFT  GENMASK(12, 8)
 #define MICFIL_VAD0_CTRL1_ST10         BIT(4)
 #define MICFIL_VAD0_CTRL1_ERIE         BIT(3)
 #define MICFIL_VAD0_CTRL1_IE           BIT(2)
 #define MICFIL_VAD0_CTRL2_FRENDIS      BIT(31)
 #define MICFIL_VAD0_CTRL2_PREFEN       BIT(30)
 #define MICFIL_VAD0_CTRL2_FOUTDIS      BIT(28)
-#define MICFIL_VAD0_CTRL2_FRAMET_SHIFT 16
-#define MICFIL_VAD0_CTRL2_FRAMET_WIDTH 6
-#define MICFIL_VAD0_CTRL2_FRAMET_MASK  ((BIT(MICFIL_VAD0_CTRL2_FRAMET_WIDTH) - 1) \
-                                        << MICFIL_VAD0_CTRL2_FRAMET_SHIFT)
-#define MICFIL_VAD0_CTRL2_FRAMET(v)    (((v) << MICFIL_VAD0_CTRL2_FRAMET_SHIFT) \
-                                        & MICFIL_VAD0_CTRL2_FRAMET_MASK)
-#define MICFIL_VAD0_CTRL2_INPGAIN_SHIFT        8
-#define MICFIL_VAD0_CTRL2_INPGAIN_WIDTH        4
-#define MICFIL_VAD0_CTRL2_INPGAIN_MASK ((BIT(MICFIL_VAD0_CTRL2_INPGAIN_WIDTH) - 1) \
-                                        << MICFIL_VAD0_CTRL2_INPGAIN_SHIFT)
-#define MICFIL_VAD0_CTRL2_INPGAIN(v)   (((v) << MICFIL_VAD0_CTRL2_INPGAIN_SHIFT) \
-                                       & MICFIL_VAD0_CTRL2_INPGAIN_MASK)
-#define MICFIL_VAD0_CTRL2_HPF_SHIFT    0
-#define MICFIL_VAD0_CTRL2_HPF_WIDTH    2
-#define MICFIL_VAD0_CTRL2_HPF_MASK     ((BIT(MICFIL_VAD0_CTRL2_HPF_WIDTH) - 1) \
-                                        << MICFIL_VAD0_CTRL2_HPF_SHIFT)
-#define MICFIL_VAD0_CTRL2_HPF(v)       (((v) << MICFIL_VAD0_CTRL2_HPF_SHIFT) \
-                                        & MICFIL_VAD0_CTRL2_HPF_MASK)
+#define MICFIL_VAD0_CTRL2_FRAMET       GENMASK(21, 16)
+#define MICFIL_VAD0_CTRL2_INPGAIN      GENMASK(11, 8)
+#define MICFIL_VAD0_CTRL2_HPF          GENMASK(1, 0)
 
 /* MICFIL HWVAD0 Signal CONFIG Register -- REG_MICFIL_VAD0_SCONFIG */
 #define MICFIL_VAD0_SCONFIG_SFILEN             BIT(31)
 #define MICFIL_VAD0_SCONFIG_SMAXEN             BIT(30)
-#define MICFIL_VAD0_SCONFIG_SGAIN_SHIFT                0
-#define MICFIL_VAD0_SCONFIG_SGAIN_WIDTH                4
-#define MICFIL_VAD0_SCONFIG_SGAIN_MASK         ((BIT(MICFIL_VAD0_SCONFIG_SGAIN_WIDTH) - 1) \
-                                               << MICFIL_VAD0_SCONFIG_SGAIN_SHIFT)
-#define MICFIL_VAD0_SCONFIG_SGAIN(v)           (((v) << MICFIL_VAD0_SCONFIG_SGAIN_SHIFT) \
-                                                & MICFIL_VAD0_SCONFIG_SGAIN_MASK)
+#define MICFIL_VAD0_SCONFIG_SGAIN              GENMASK(3, 0)
 
 /* MICFIL HWVAD0 Noise CONFIG Register -- REG_MICFIL_VAD0_NCONFIG */
 #define MICFIL_VAD0_NCONFIG_NFILAUT            BIT(31)
 #define MICFIL_VAD0_NCONFIG_NMINEN             BIT(30)
 #define MICFIL_VAD0_NCONFIG_NDECEN             BIT(29)
 #define MICFIL_VAD0_NCONFIG_NOREN              BIT(28)
-#define MICFIL_VAD0_NCONFIG_NFILADJ_SHIFT      8
-#define MICFIL_VAD0_NCONFIG_NFILADJ_WIDTH      5
-#define MICFIL_VAD0_NCONFIG_NFILADJ_MASK       ((BIT(MICFIL_VAD0_NCONFIG_NFILADJ_WIDTH) - 1) \
-                                                << MICFIL_VAD0_NCONFIG_NFILADJ_SHIFT)
-#define MICFIL_VAD0_NCONFIG_NFILADJ(v)         (((v) << MICFIL_VAD0_NCONFIG_NFILADJ_SHIFT) \
-                                                & MICFIL_VAD0_NCONFIG_NFILADJ_MASK)
-#define MICFIL_VAD0_NCONFIG_NGAIN_SHIFT                0
-#define MICFIL_VAD0_NCONFIG_NGAIN_WIDTH                4
-#define MICFIL_VAD0_NCONFIG_NGAIN_MASK         ((BIT(MICFIL_VAD0_NCONFIG_NGAIN_WIDTH) - 1) \
-                                                << MICFIL_VAD0_NCONFIG_NGAIN_SHIFT)
-#define MICFIL_VAD0_NCONFIG_NGAIN(v)           (((v) << MICFIL_VAD0_NCONFIG_NGAIN_SHIFT) \
-                                                & MICFIL_VAD0_NCONFIG_NGAIN_MASK)
+#define MICFIL_VAD0_NCONFIG_NFILADJ            GENMASK(12, 8)
+#define MICFIL_VAD0_NCONFIG_NGAIN              GENMASK(3, 0)
 
 /* MICFIL HWVAD0 Zero-Crossing Detector - REG_MICFIL_VAD0_ZCD */
-#define MICFIL_VAD0_ZCD_ZCDTH_SHIFT    16
-#define MICFIL_VAD0_ZCD_ZCDTH_WIDTH    10
-#define MICFIL_VAD0_ZCD_ZCDTH_MASK     ((BIT(MICFIL_VAD0_ZCD_ZCDTH_WIDTH) - 1) \
-                                        << MICFIL_VAD0_ZCD_ZCDTH_SHIFT)
-#define MICFIL_VAD0_ZCD_ZCDTH(v)       (((v) << MICFIL_VAD0_ZCD_ZCDTH_SHIFT)\
-                                        & MICFIL_VAD0_ZCD_ZCDTH_MASK)
-#define MICFIL_VAD0_ZCD_ZCDADJ_SHIFT   8
-#define MICFIL_VAD0_ZCD_ZCDADJ_WIDTH   4
-#define MICFIL_VAD0_ZCD_ZCDADJ_MASK    ((BIT(MICFIL_VAD0_ZCD_ZCDADJ_WIDTH) - 1)\
-                                        << MICFIL_VAD0_ZCD_ZCDADJ_SHIFT)
-#define MICFIL_VAD0_ZCD_ZCDADJ(v)      (((v) << MICFIL_VAD0_ZCD_ZCDADJ_SHIFT)\
-                                        & MICFIL_VAD0_ZCD_ZCDADJ_MASK)
+#define MICFIL_VAD0_ZCD_ZCDTH          GENMASK(25, 16)
+#define MICFIL_VAD0_ZCD_ZCDADJ_SHIFT   GENMASK(11, 8)
 #define MICFIL_VAD0_ZCD_ZCDAND         BIT(4)
 #define MICFIL_VAD0_ZCD_ZCDAUT         BIT(2)
 #define MICFIL_VAD0_ZCD_ZCDEN          BIT(0)
 #define MICFIL_OUTGAIN_CHX_SHIFT(v)    (4 * (v))
 
 /* Constants */
-#define MICFIL_DMA_IRQ_DISABLED(v)     ((v) & MICFIL_CTRL1_DISEL_MASK)
-#define MICFIL_DMA_ENABLED(v)          ((0x1 << MICFIL_CTRL1_DISEL_SHIFT) \
-                                        == ((v) & MICFIL_CTRL1_DISEL_MASK))
-#define MICFIL_IRQ_ENABLED(v)          ((0x2 << MICFIL_CTRL1_DISEL_SHIFT) \
-                                        == ((v) & MICFIL_CTRL1_DISEL_MASK))
 #define MICFIL_OUTPUT_CHANNELS         8
 #define MICFIL_FIFO_NUM                        8
 
 #define MICFIL_SLEEP_MIN               90000 /* in us */
 #define MICFIL_SLEEP_MAX               100000 /* in us */
 #define MICFIL_DMA_MAXBURST_RX         6
-#define MICFIL_CTRL2_OSR_DEFAULT       (0 << MICFIL_CTRL2_CICOSR_SHIFT)
 
 #endif /* _FSL_MICFIL_H */