]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
ASoC: codecs: nau8325: Silence uninitialized variables warnings
authorKrzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
Wed, 3 Dec 2025 14:06:12 +0000 (15:06 +0100)
committerMark Brown <broonie@kernel.org>
Wed, 3 Dec 2025 14:54:37 +0000 (14:54 +0000)
clang W=1 builds warn:

  nau8325.c:430:13: error: variable 'n2_max' is uninitialized when used here [-Werror,-Wuninitialized]

which are false positive, because the variables will be always
initialized when used (guarded by mclk_max!=0 check).  However
initializing them upfront makes the code more obvious and easier, plus
it silences the warning.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
Link: https://patch.msgid.link/20251203140611.87191-2-krzysztof.kozlowski@oss.qualcomm.com
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/codecs/nau8325.c

index 3bfdb448f8bd722bf219155e8e8d5bef8afc73d7..e651263a9812179b37138627e9603119162b22a2 100644 (file)
@@ -386,7 +386,8 @@ static int nau8325_clksrc_choose(struct nau8325 *nau8325,
                                 const struct nau8325_srate_attr **srate_table,
                                 int *n1_sel, int *mult_sel, int *n2_sel)
 {
-       int i, j, mclk, mclk_max, ratio, ratio_sel, n2_max;
+       int i, j, mclk, ratio;
+       int mclk_max = 0, ratio_sel = 0, n2_max = 0;
 
        if (!nau8325->mclk || !nau8325->fs)
                goto proc_err;
@@ -408,7 +409,6 @@ static int nau8325_clksrc_choose(struct nau8325 *nau8325,
        }
 
        /* Get MCLK_SRC through 1/N, Multiplier, and then 1/N2. */
-       mclk_max = 0;
        for (i = 0; i < ARRAY_SIZE(mclk_n1_div); i++) {
                for (j = 0; j < ARRAY_SIZE(mclk_n3_mult); j++) {
                        mclk = nau8325->mclk << mclk_n3_mult[j].param;