[U-Boot] [PATCH] mcf: Fix cppcheck errors about uninitialized variables

Albert ARIBAUD albert.u.boot at aribaud.net
Fri Nov 14 13:47:33 CET 2014


Hello Alison,

On Fri, 14 Nov 2014 14:56:12 +0800, Alison Wang <b18965 at freescale.com>
wrote:
> This patch will initialize the variables to fix the cppcheck errors as
> follows.
> 
> [arch/m68k/cpu/mcf5227x/speed.c:89]: (error) Uninitialized variable: bootmode
> [arch/m68k/cpu/mcf532x/speed.c:248]: (error) Uninitialized variable: fout
> [arch/m68k/cpu/mcf5445x/speed.c:194]: (error) Uninitialized variable: bootmode
> 
> Signed-off-by: Alison Wang <alison.wang at freescale.com>
> ---
>  arch/m68k/cpu/mcf5227x/speed.c | 2 +-
>  arch/m68k/cpu/mcf532x/speed.c  | 3 ++-
>  arch/m68k/cpu/mcf5445x/speed.c | 2 +-
>  3 files changed, 4 insertions(+), 3 deletions(-)
> 
> diff --git a/arch/m68k/cpu/mcf5227x/speed.c b/arch/m68k/cpu/mcf5227x/speed.c
> index 44de4a6..e991cd1 100644
> --- a/arch/m68k/cpu/mcf5227x/speed.c
> +++ b/arch/m68k/cpu/mcf5227x/speed.c
> @@ -74,7 +74,7 @@ int get_clocks(void)
>  	ccm_t *ccm = (ccm_t *)MMAP_CCM;
>  	pll_t *pll = (pll_t *)MMAP_PLL;
>  	int vco, temp, pcrvalue, pfdr;
> -	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.

>  	pcrvalue = in_be32(&pll->pcr) & 0xFF0F0FFF;
>  	pfdr = pcrvalue >> 24;
> diff --git a/arch/m68k/cpu/mcf532x/speed.c b/arch/m68k/cpu/mcf532x/speed.c
> index a440bbb..0495b16 100644
> --- a/arch/m68k/cpu/mcf532x/speed.c
> +++ b/arch/m68k/cpu/mcf532x/speed.c
> @@ -143,7 +143,8 @@ int clock_pll(int fsys, int flags)
>  #endif
>  	sdram_t *sdram = (sdram_t *)(MMAP_SDRAM);
>  	pll_t *pll = (pll_t *)(MMAP_PLL);
> -	int fref, temp, fout, mfd;
> +	int fref, temp, mfd;
> +	int fout = 0;
>  	u32 i;

(haven't checked if similar reductions can be done)
 
>  	fref = FREF;
> diff --git a/arch/m68k/cpu/mcf5445x/speed.c b/arch/m68k/cpu/mcf5445x/speed.c
> index 4e363a4..a166dcc 100644
> --- a/arch/m68k/cpu/mcf5445x/speed.c
> +++ b/arch/m68k/cpu/mcf5445x/speed.c
> @@ -141,7 +141,7 @@ void setup_5445x_clocks(void)
>  #ifdef CONFIG_M54455EVB
>  	u8 *cpld = (u8 *)(CONFIG_SYS_CS2_BASE + 3);
>  #endif
> -	u8 bootmode;
> +	u8 bootmode = 0;

(ditto)
 
>  	/* To determine PCI is present or not */
>  	if (((in_be16(&ccm->ccr) & CCM_CCR_360_FBCONFIG_MASK) == 0x00e0) ||
> -- 
> 2.1.0.27.g96db324
> 
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot



Amicalement,
-- 
Albert.


More information about the U-Boot mailing list