[PATCH 04/11] common: eeprom_field: Drop unnecessary comparison
Marek Behún
kabel at kernel.org
Tue May 21 09:13:28 CEST 2024
The byte variable is of type unsigned char, it is never less than zero.
The error case is handled by *endptr, so drop the comparison altogether.
Signed-off-by: Marek Behún <kabel at kernel.org>
---
common/eeprom/eeprom_field.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/common/eeprom/eeprom_field.c b/common/eeprom/eeprom_field.c
index 9b831414a4..26f6041e54 100644
--- a/common/eeprom/eeprom_field.c
+++ b/common/eeprom/eeprom_field.c
@@ -56,7 +56,7 @@ static int __eeprom_field_update_bin(struct eeprom_field *field,
}
byte = simple_strtoul(tmp, &endptr, 16);
- if (*endptr != '\0' || byte < 0)
+ if (*endptr != '\0')
return -1;
field->buf[j] = byte;
--
2.44.1
More information about the U-Boot
mailing list