diff options
| author | Tom Rini <trini@konsulko.com> | 2017-05-10 15:20:15 -0400 |
|---|---|---|
| committer | Tom Rini <trini@konsulko.com> | 2017-05-12 08:37:39 -0400 |
| commit | 758979101d161704dc9c1e42db98a728724f59d0 (patch) | |
| tree | 5a4b0bc1593df007d5444e7b5d54955af1b462e4 /drivers/gpio | |
| parent | 62f733b396cbe39315e9f28ea348300c6a72caa9 (diff) | |
gpio-uclass.c: Fix comparison of unsigned expression warning
We declare that gpio_base (which is the base for counting gpios, not an
address) is unsigned. Therefore the comparison with >= 0 is always
true. As the desire is to allow for this base number to be 0, we can
just drop this check. Reported by clang-3.8.
Cc: Simon Glass <sjg@chromium.org>
Signed-off-by: Tom Rini <trini@konsulko.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
Diffstat (limited to 'drivers/gpio')
| -rw-r--r-- | drivers/gpio/gpio-uclass.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpio/gpio-uclass.c b/drivers/gpio/gpio-uclass.c index 9ab9df4ce7..ba4804083d 100644 --- a/drivers/gpio/gpio-uclass.c +++ b/drivers/gpio/gpio-uclass.c @@ -68,7 +68,7 @@ int dm_gpio_lookup_name(const char *name, struct gpio_desc *desc) if (numeric != -1) { offset = numeric - uc_priv->gpio_base; /* Allow GPIOs to be numbered from 0 */ - if (offset >= 0 && offset < uc_priv->gpio_count) + if (offset < uc_priv->gpio_count) break; } |