commit 59193a72f1a77ff8712e72fb1c255b2d721ce4a3
author Marek Behún <kabel@kernel.org> Tue May 21 09:13:27 2024 +0200
committer Tom Rini <trini@konsulko.com> Fri Jun 07 10:47:59 2024 -0600
tree 3dcf8ba1445956b67c9f7402dee7ddcb4cfbf64c
parent 246db7530a49866bcb7169b886d657f6a6b5b5dd

common: eeprom_field: Fix updating binary field

The __eeprom_field_update_bin() function is expected to parse a hex
string into bytes (potentially in reverse order), but the
simple_strtoul() function is given 0 as base. This does not work since
the string does not contain '0x' prefix. Add explicit base 16.

Signed-off-by: Marek Behún <kabel@kernel.org>

common/eeprom/eeprom_field.c

diff --git a/common/eeprom/eeprom_field.c b/common/eeprom/eeprom_field.c
index f56eebe..9b83141 100644
--- a/common/eeprom/eeprom_field.c
+++ b/common/eeprom/eeprom_field.c
@@ -55,7 +55,7 @@
 			tmp[k] = value[reverse ? i - 1 + k : i + k];
 		}
 
-		byte = simple_strtoul(tmp, &endptr, 0);
+		byte = simple_strtoul(tmp, &endptr, 16);
 		if (*endptr != '\0' || byte < 0)
 			return -1;