On Thu, May 05, 2022 at 03:34:07PM +0800, Shengjiu Wang wrote: > Remove the _SHIFT for the mask definition. > > Fixes: 17f2142bae4b ("ASoC: fsl_micfil: use GENMASK to define register bit fields") > Signed-off-by: Shengjiu Wang <shengjiu.wang@xxxxxxx> > --- Acked-by: Sascha Hauer <s.hauer@xxxxxxxxxxxxxx> Sascha > sound/soc/fsl/fsl_micfil.h | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/sound/soc/fsl/fsl_micfil.h b/sound/soc/fsl/fsl_micfil.h > index 08901827047d..053caba3caf3 100644 > --- a/sound/soc/fsl/fsl_micfil.h > +++ b/sound/soc/fsl/fsl_micfil.h > @@ -74,9 +74,9 @@ > #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 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_CHSEL GENMASK(26, 24) > +#define MICFIL_VAD0_CTRL1_CICOSR GENMASK(19, 16) > +#define MICFIL_VAD0_CTRL1_INITT GENMASK(12, 8) > #define MICFIL_VAD0_CTRL1_ST10 BIT(4) > #define MICFIL_VAD0_CTRL1_ERIE BIT(3) > #define MICFIL_VAD0_CTRL1_IE BIT(2) > @@ -106,7 +106,7 @@ > > /* MICFIL HWVAD0 Zero-Crossing Detector - REG_MICFIL_VAD0_ZCD */ > #define MICFIL_VAD0_ZCD_ZCDTH GENMASK(25, 16) > -#define MICFIL_VAD0_ZCD_ZCDADJ_SHIFT GENMASK(11, 8) > +#define MICFIL_VAD0_ZCD_ZCDADJ GENMASK(11, 8) > #define MICFIL_VAD0_ZCD_ZCDAND BIT(4) > #define MICFIL_VAD0_ZCD_ZCDAUT BIT(2) > #define MICFIL_VAD0_ZCD_ZCDEN BIT(0) > -- > 2.17.1 > > -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |