[U-Boot] [PATCH] env_sf: remove warning introduced with last patch

Stefano Babic sbabic at denx.de
Fri Oct 29 11:14:48 CEST 2010


On 10/28/2010 07:47 PM, Mike Frysinger wrote:

> if you're going to start at 1, shouldnt you then drop the redundant
> assignment to 1 ?
>         if (!saved_buffer) {
>             ret = 1;
>             goto done;
>         }
> -mike
> 

Yes, it is redundant - I'll drop it.

Stefano

-- 
=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: office at denx.de
=====================================================================


More information about the U-Boot mailing list