On Fri, 4 Nov 2011 01:47:04 -0500 Kumar Gala <ga...@kernel.crashing.org> wrote:
> On Nov 4, 2011, at 1:44 AM, Kumar Gala wrote: > > On Nov 3, 2011, at 2:09 PM, Wolfgang Denk wrote: > >> In message <1319407132-6590-1-git-send-email...@denx.de> you wrote: > >>> Commit 1272592 "powerpc: Use getenv_ulong() in place of getenv(), > >>> strtoul" instroduced a build warning for some PPC systems: > >>> > >>> board.c: In function 'board_init_r': > >>> board.c:626: warning: unused variable 's' > >>> > >>> Fix it. > >>> > >>> Signed-off-by: Wolfgang Denk <w...@denx.de> > >>> --- > >>> arch/powerpc/lib/board.c | 5 +++-- > >>> 1 files changed, 3 insertions(+), 2 deletions(-) > >> > >> Applied, thanks. > > > > I'm still seeing this warning when building MPC8536DS_config me too, on most 83xx boards. > I think we need to revert this now. > > I don't see why you'd need it. yes, all code fragments that utilize a variable 's' declare one local to their code body. Kim _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot