On Fri, Dec 9, 2011 at 4:14 AM, Wolfgang Denk <w...@denx.de> wrote: > Fix: > flash.c: In function 'flash_get_offsets': > flash.c:122:10: warning: variable 'pOrgDef' set but not used > [-Wunused-but-set-variable] > flash.c: In function 'flash_erase': > flash.c:263:6: warning: variable 'flag' set but not used > [-Wunused-but-set-variable] > flash.c: In function 'write_data': > flash.c:439:6: warning: variable 'flag' set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Wolfgang Denk <w...@denx.de> > Cc: Dave Peverley <dpever...@mpc-data.co.uk>
Acked-by: Tom Rini <tr...@ti.com> > --- > board/ti/omap730p2/flash.c | 19 ++++++++++++------- > 1 files changed, 12 insertions(+), 7 deletions(-) > > diff --git a/board/ti/omap730p2/flash.c b/board/ti/omap730p2/flash.c > index 185bc2d..a292627 100644 > --- a/board/ti/omap730p2/flash.c > +++ b/board/ti/omap730p2/flash.c > @@ -119,9 +119,7 @@ unsigned long flash_init (void) > static void flash_get_offsets (ulong base, flash_info_t * info) > { > int i; > - OrgDef *pOrgDef; > > - pOrgDef = OrgIntel_28F256L18T; > if (info->flash_id == FLASH_UNKNOWN) { > return; > } > @@ -335,6 +333,10 @@ int flash_erase (flash_info_t * info, int s_first, int > s_last) > printf (" done\n"); > } > } > + > + if (flag) > + enable_interrupts(); > + > return rcode; > } > > @@ -436,7 +438,7 @@ static int write_data (flash_info_t * info, ulong dest, > FPW data) > { > FPWV *addr = (FPWV *) dest; > ulong status; > - int flag; > + int flag, rc = 0; > ulong start; > > /* Check if Flash is (sufficiently) erased */ > @@ -456,12 +458,15 @@ static int write_data (flash_info_t * info, ulong dest, > FPW data) > /* wait while polling the status register */ > while (((status = *addr) & (FPW) 0x00800080) != (FPW) 0x00800080) { > if (get_timer(start) > CONFIG_SYS_FLASH_WRITE_TOUT) { > - *addr = (FPW) 0x00FF00FF; /* restore read mode > */ > - return (1); > + rc = 1; > + goto done; > } > } > - *addr = (FPW) 0x00FF00FF; /* restore read mode */ > - return (0); > +done: > + *addr = (FPW)0x00FF00FF; /* restore read mode */ > + if (flag) > + enable_interrupts(); > + return rc; > } > > void inline spin_wheel (void) > -- > 1.7.6.4 > > _______________________________________________ > U-Boot mailing list > U-Boot@lists.denx.de > http://lists.denx.de/mailman/listinfo/u-boot -- Tom _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot