# HG changeset patch
# User sergeig
# Date 1332349745 0
# Node ID 7a634ca81afe68a0f8d9d98c49c6dc6561652335
# Parent  75c92b704b8ae24e90a230ba2cd9ed66eca2cfba
* src/strata_aux.c (strata_hw_cfi): Fix compiler warnings that
variable is set but not used. [ Bugzilla 1001541 ]

diff --git a/packages/devs/flash/intel/stratav2/current/ChangeLog b/packages/devs/flash/intel/stratav2/current/ChangeLog
--- 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
diff --git a/packages/devs/flash/intel/stratav2/current/src/strata_aux.c b/packages/devs/flash/intel/stratav2/current/src/strata_aux.c
--- 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);