linux-kernel - Re: 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-prev] [thread-next>] [day] [month] [year] [list]

Date:   Tue, 9 Apr 2019 09:12:31 +1000
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: Re: linux-next: build warning after merge of the sound-asoc tree

Hi all,

On Tue, 26 Mar 2019 10:01:45 +1100 Stephen Rothwell <sfr@...b.auug.org.au> wrote:
>
> 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")

I am still getting these warnings.

-- 
Cheers,
Stephen Rothwell

Content of type "application/pgp-signature" skipped