Comparing barebox' commit [1] with dt-utils' commit [2] it is obvious that
adjusting the return value has been forgotten in barebox - fix that.
(As the actual code that is changed is removed by the preprocesser when
compiling barebox this patch is only of cosmetic nature to keep both code bases
in sync).

[1] 9d6d91931afb ("state: Remove -EUCLEAN check from userspace tool")
[2] 791a2404116d ("state: Remove -EUCLEAN check from userspace tool")

Signed-off-by: Ulrich Ölmann <[email protected]>
---
 common/state/backend_bucket_circular.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/common/state/backend_bucket_circular.c 
b/common/state/backend_bucket_circular.c
index 47970b79f36a..612fa0cd8a11 100644
--- a/common/state/backend_bucket_circular.c
+++ b/common/state/backend_bucket_circular.c
@@ -179,9 +179,10 @@ static int state_mtd_peb_read(struct 
state_backend_storage_bucket_circular *circ
                dev_err(circ->dev, "Failed to read circular storage len %d, 
%d\n",
                        len, ret);
                free(buf);
+               return ret;
        }
 
-       return ret;
+       return 0;
 }
 
 static int state_mtd_peb_write(struct state_backend_storage_bucket_circular 
*circ,
-- 
2.23.0


_______________________________________________
barebox mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/barebox

Reply via email to