[U-Boot] [PATCH] tegra20: initialize variable to avoid compiler warning

Simon Glass sjg at chromium.org
Fri Oct 26 18:38:01 CEST 2012


Hi,

On Sat, Oct 20, 2012 at 12:18 AM, Allen Martin <amartin at nvidia.com> wrote:
> Initialize this variable to avoid a compiler warning about possible
> use of uninitialized variable with gcc 4.4.6.
>
> Signed-off-by: Allen Martin <amartin at nvidia.com>

Acked-by: Simon Glass <sjg at chromium.org>
Tested-by: Simon Glass <sjg at chromium.org>

I suppose the compiler is confused by the 'return node' bit. It might
be better to set '*emcp = NULL' at the top of decode_emc(), but this
is fine too.

> ---
>  arch/arm/cpu/tegra20-common/emc.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/cpu/tegra20-common/emc.c b/arch/arm/cpu/tegra20-common/emc.c
> index 97420d7..90edf00 100644
> --- a/arch/arm/cpu/tegra20-common/emc.c
> +++ b/arch/arm/cpu/tegra20-common/emc.c
> @@ -257,7 +257,7 @@ static int decode_emc(const void *blob, unsigned rate, struct emc_ctlr **emcp,
>  int tegra_set_emc(const void *blob, unsigned rate)
>  {
>         struct emc_ctlr *emc;
> -       const u32 *table;
> +       const u32 *table = NULL;
>         int err, i;
>
>         err = decode_emc(blob, rate, &emc, &table);
> --
> 1.7.10.4
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot


More information about the U-Boot mailing list