[PATCH 04/11] 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 --- 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;