common: eeprom_field: Drop unnecessary comparison

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@kernel.org>

diff --git a/common/eeprom/eeprom_field.c b/common/eeprom/eeprom_field.c
index 9b831414a4ba146f8ad469b212d2994757b6adc4..26f6041e54883a4dec931e6105c4bd2c1e151db9 100644 (file)
--- 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;