changeset 3134:7a634ca81afe

* src/strata_aux.c (strata_hw_cfi): Fix compiler warnings that variable is set but not used. [ Bugzilla 1001541 ]
author sergeig
date Wed, 21 Mar 2012 17:09:05 +0000
parents 75c92b704b8a
children ccd24398a051
files packages/devs/flash/intel/stratav2/current/ChangeLog packages/devs/flash/intel/stratav2/current/src/strata_aux.c
diffstat 2 files changed, 8 insertions(+), 2 deletions(-) [+]
line wrap: on
line diff
--- a/packages/devs/flash/intel/stratav2/current/ChangeLog
+++ b/packages/devs/flash/intel/stratav2/current/ChangeLog
@@ -1,3 +1,9 @@
+2012-03-21  Sergei Gavrikov  <sergei.gavrikov@gmail.com>
+            Jonathan Larmour  <jifl@eCosCentric.com>
+
+	* src/strata_aux.c (strata_hw_cfi): Fix compiler warnings that
+	variable is set but not used. [ Bugzilla 1001541 ]
+
 2009-04-21  Bart Veer  <bartv@ecoscentric.com>
 
 	* src/strata.c: Added set of parentheses around macro parameters
--- a/packages/devs/flash/intel/stratav2/current/src/strata_aux.c
+++ b/packages/devs/flash/intel/stratav2/current/src/strata_aux.c
@@ -163,7 +163,7 @@ STRATA_FNNAME(strata_hw_cfi)(struct cyg_
     int     dev_size;
     int     i;
     int     erase_regions;
-    cyg_uint8 writebuffer_lsb, writebuffer_msb;
+    cyg_uint8 writebuffer_lsb;
     cyg_uint32 writebuffer;
 
     sf_diag("addr %08x %d cmd %08x\n",addr, sizeof(STRATA_TYPE), STRATA_COMMAND_CFI);
@@ -210,7 +210,7 @@ STRATA_FNNAME(strata_hw_cfi)(struct cyg_
     }
 
     writebuffer_lsb = STRATA_UNSWAP(addr[STRATA_OFFSET_CFI_WRITE_BUFFER_LSB]) & 0x00FF;
-    writebuffer_msb = STRATA_UNSWAP(addr[STRATA_OFFSET_CFI_WRITE_BUFFER_MSB]) & 0x00FF;
+    /* No need to include MSB, it would never be as large as 2^256 ! */
     writebuffer = 1 << writebuffer_lsb;
 
     strata_dev->bufsize = writebuffer / sizeof(STRATA_TYPE);