[flashrom-gerrit] [XS] Change in flashrom[main]: erasure_layout: Fix double invocation of erasers

[Anastasia Klimchuk (Code Review)]

Anastasia Klimchuk has submitted this change. ( https://review.coreboot.org/c/flashrom/+/77747?usp=email )

Change subject: erasure_layout: Fix double invocation of erasers
......................................................................

erasure_layout: Fix double invocation of erasers

Erasefn was invoked over the same block of memory twice.
This patch removes the second redundant invokation. It was
accidentally introduced during earlier refactoring of the code.

Change-Id: I92351eba0fd29114ce98b4a839358e92d176af28
Signed-off-by: Anastasia Klimchuk 
Reviewed-on: https://review.coreboot.org/c/flashrom/+/77747
Reviewed-by: Alexander Goncharov 
Tested-by: build bot (Jenkins) 
Reviewed-by: Peter Marheine 
---
M erasure_layout.c
1 file changed, 0 insertions(+), 9 deletions(-)

Approvals:
  Peter Marheine: Looks good to me, approved
  build bot (Jenkins): Verified
  Alexander Goncharov: Looks good to me, approved

diff --git a/erasure_layout.c b/erasure_layout.c
index c9ac44b..b43e52e 100644
--- a/erasure_layout.c
+++ b/erasure_layout.c
@@ -339,15 +339,6 @@
    			}
    		}

-			ret = erasefn(flashctx, start_addr, block_len);
-			if (ret) {
-				msg_cerr("Failed to execute erase command "
-					"for offset %#"PRIx32" to %#"PRIx32".\n",
-					start_addr, start_addr + block_len);
-				ret = -1;
-				goto _end;
-			}
-
    		// adjust curcontents
    		memset(curcontents+start_addr, erased_value, block_len);
    		// after erase make it unselected again