]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
ASoC: mt8365: Remove spurious unsigned long casts
authorMark Brown <broonie@kernel.org>
Sat, 7 Sep 2024 00:53:27 +0000 (01:53 +0100)
committerMark Brown <broonie@kernel.org>
Mon, 9 Sep 2024 15:58:13 +0000 (16:58 +0100)
The regmap APIs take unsigned ints not unsigned longs so casting their
arguments to unsigned longs is not a good choice, the constants being
cast here are all unsigned ints anyway.

Reviewed-by: Alexandre Mergnat <amergnat@baylibre.com>
Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
Tested-by: Nathan Chancellor <nathan@kernel.org> # build
Signed-off-by: Mark Brown <broonie@kernel.org>
Link: https://patch.msgid.link/20240907-asoc-fix-mt8365-build-v1-2-7ad0bac20161@kernel.org
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/mediatek/mt8365/mt8365-dai-i2s.c

index 5003fe5e5ccfe0439cae8ed3ab3dfede4703b40a..6b4d8b7e24caaea7a83d2f77ba4889777826fb3b 100644 (file)
@@ -385,7 +385,7 @@ static int mt8365_afe_set_2nd_i2s_asrc(struct mtk_base_afe *afe,
                /* disable IIR coeff SRAM access */
                regmap_update_bits(afe->regmap, AFE_ASRC_2CH_CON0,
                                   COEFF_SRAM_CTRL,
-                                  (unsigned long)~COEFF_SRAM_CTRL);
+                                  ~COEFF_SRAM_CTRL);
                regmap_update_bits(afe->regmap, AFE_ASRC_2CH_CON2,
                                   CLR_IIR_HISTORY | IIR_EN | IIR_STAGE_MASK,
                                   CLR_IIR_HISTORY | IIR_EN |
@@ -393,7 +393,7 @@ static int mt8365_afe_set_2nd_i2s_asrc(struct mtk_base_afe *afe,
        } else {
                /* disable IIR */
                regmap_update_bits(afe->regmap, AFE_ASRC_2CH_CON2,
-                                  IIR_EN, (unsigned long)~IIR_EN);
+                                  IIR_EN, ~IIR_EN);
        }
 
        /* CON3 setting (RX OFS) */
@@ -456,7 +456,7 @@ static int mt8365_afe_set_2nd_i2s_asrc_enable(struct mtk_base_afe *afe,
                                   ASM_ON, ASM_ON);
        else
                regmap_update_bits(afe->regmap, AFE_ASRC_2CH_CON0,
-                                  ASM_ON, (unsigned long)~ASM_ON);
+                                  ASM_ON, ~ASM_ON);
        return 0;
 }