On Sep 1, 2009, at 5:17 PM, Anton Vorontsov wrote: > The warning is bogus, so silence it by initializing the 'ret' > variable. > > Signed-off-by: Anton Vorontsov <[email protected]> > --- > > On Tue, Sep 01, 2009 at 09:27:37PM +0200, Wolfgang Denk wrote: >> Dear Anton Vorontsov, >> >> In message <[email protected]> you wrote: >>> >>> We can fix the warnings by assigning some value to a variable >>> at declaration, but the advantage of 'x = x' trick is that it >>> doesn't generate any code. >> >> Argh... what a clev^H^H^H^Hdirty trick. >> >> Thanks for the explanation. >> >> However, in this case it seems to make sense to me to explicitly >> initialize the return code with zero. > > Well, I don't see any advantages of this, but here it is anyway. > > Thanks. > > p.s. Timur, since the patch has changed, I couldn't preserve your > previous Ack. > > board/freescale/common/sys_eeprom.c | 3 ++- > 1 files changed, 2 insertions(+), 1 deletions(-)
applied to 85xx. - k _______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot
