From d4509a6cd43010c6d28a45ba5bd676fd85df963c Mon Sep 17 00:00:00 2001
From: Julian Oes <julian@oes.ch>
Date: Sat, 1 Mar 2025 14:21:09 +1300
Subject: [PATCH] flashfs32: fix result handling (#24371)

We need to translate return values here, otherwise this complains being
unsuccessful when it was actually ok.
---
 src/lib/parameters/flashparams/flashfs32.c | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/src/lib/parameters/flashparams/flashfs32.c b/src/lib/parameters/flashparams/flashfs32.c
index 848623b261..25e97d2563 100644
--- a/src/lib/parameters/flashparams/flashfs32.c
+++ b/src/lib/parameters/flashparams/flashfs32.c
@@ -1128,6 +1128,11 @@ int parameter_flashfs_init(sector_descriptor_t *fconfig, uint8_t *buffer, uint16
 	if (pf == NULL) {
 		// Parameters can't be found, assume sector is corrupt or empty
 		rv = parameter_flashfs_erase();
+
+		// A positive return value means flash space has been erased successfully.
+		if (rv > 0) {
+			rv = 0;
+		}
 	}
 
 	return rv;