[U-Boot] [PATCH] mcf: Fix cppcheck errors about uninitialized variables
Tom Rini
trini at ti.com
Tue Nov 18 15:17:17 CET 2014
On Fri, Nov 14, 2014 at 02:23:46PM +0100, Wolfgang Denk wrote:
> Dear Albert,
>
> In message <20141114134733.01c82023 at lilith> you wrote:
> >
> > > - u8 bootmode;
> > > + u8 bootmode = 0;
> >
> > If you set bootmode to the default 0 value at declaration, then the code
> > that follows in the source file can be simplified:
> >
> > u8 bootmode = 0;
> > ---8<---
> > pcrvalue = in_be32(&pll->pcr) & 0xFF0F0FFF;
> > pfdr = pcrvalue >> 24;
> >
> > if (pfdr == 0x1E)
> > bootmode = 0; /* Normal Mode */
> > ---8<---
> >
> > The part between scissors is useless now, and in turn, pfdr becomes unneeded.
>
> Exactly. This was my concern, too - otherwise I had provided auch a
> trivial fix. But I think here some deeper understanding of the
> intended logic is needed.
For clarity, yes, please do a v2 where you've re-organized the code
based on what the cppcheck errors point out rather than just a simple
silence of the warning, thanks!
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141118/0eaea90f/attachment.pgp>
More information about the U-Boot
mailing list