linux-kernel - linux-next: build warning after merge of the sound-asoc tree

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Date:   Tue, 26 Mar 2019 10:01:45 +1100
From:   Stephen Rothwell <sfr@...b.auug.org.au>
To:     Mark Brown <broonie@...nel.org>,
        Liam Girdwood <lgirdwood@...il.com>
Cc:     Linux Next Mailing List <linux-next@...r.kernel.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        Annaliese McDermond <nh6z@...z.net>
Subject: linux-next: build warning after merge of the sound-asoc tree

Hi all,

After merging the sound-asoc tree, today's linux-next build (x86_64
allmodconfig) produced this warning:

sound/soc/codecs/tlv320aic32x4-clk.c: In function 'clk_aic32x4_pll_recalc_rate':
sound/soc/codecs/tlv320aic32x4-clk.c:149:38: warning: 'settings.d' may be used uninitialized in this function [-Wmaybe-uninitialized]
     ((settings->j * 10000) + settings->d);
                              ~~~~~~~~^~~
sound/soc/codecs/tlv320aic32x4-clk.c:197:32: note: 'settings.d' was declared here
  struct clk_aic32x4_pll_muldiv settings;
                                ^~~~~~~~
sound/soc/codecs/tlv320aic32x4-clk.c:149:15: warning: 'settings.j' may be used uninitialized in this function [-Wmaybe-uninitialized]
     ((settings->j * 10000) + settings->d);
       ~~~~~~~~^~~
sound/soc/codecs/tlv320aic32x4-clk.c:197:32: note: 'settings.j' was declared here
  struct clk_aic32x4_pll_muldiv settings;
                                ^~~~~~~~
sound/soc/codecs/tlv320aic32x4-clk.c:148:37: warning: 'settings.r' may be used uninitialized in this function [-Wmaybe-uninitialized]
  rate = (u64) parent_rate * settings->r *
                             ~~~~~~~~^~~
sound/soc/codecs/tlv320aic32x4-clk.c:197:32: note: 'settings.r' was declared here
  struct clk_aic32x4_pll_muldiv settings;
                                ^~~~~~~~
sound/soc/codecs/tlv320aic32x4-clk.c:151:56: warning: 'settings.p' may be used uninitialized in this function [-Wmaybe-uninitialized]
  return (unsigned long) DIV_ROUND_UP_ULL(rate, settings->p * 10000);
                                                        ^~
sound/soc/codecs/tlv320aic32x4-clk.c:197:32: note: 'settings.p' was declared here
  struct clk_aic32x4_pll_muldiv settings;
                                ^~~~~~~~

Introduced by commit

  514b044cba66 ("ASoC: tlv320aic32x4: Model PLL in CCF")

-- 
Cheers,
Stephen Rothwell

Content of type "application/pgp-signature" skipped