diff options
| author |   Rickard Strandqvist <rickard_strandqvist@spectrumdigital.se> | 2014-06-26 13:26:45 +0200 |
|---|---|---|
| committer |   Linus Walleij <linus.walleij@linaro.org> | 2014-07-11 14:08:32 +0200 |
| commit | ca7162add9dc0b361cbf51084591c64db37199d8 (patch) | |
| tree | ae351aac7822c93127a2b5691583351a194b576d /drivers/pinctrl/pinctrl-at91.c | |
| parent | pinctrl: st: add IRQCHIP_SKIP_SET_WAKE flag (diff) | |
| download | linux-dev-ca7162add9dc0b361cbf51084591c64db37199d8.tar.xz linux-dev-ca7162add9dc0b361cbf51084591c64db37199d8.zip | |
pinctrl: pinctrl-at91.c: Cleaning up if unsigned is less than zero
Remove checking if a unsigned is less than zero
This was found using a static code analysis program called cppcheck.
Signed-off-by: Rickard Strandqvist <rickard_strandqvist@spectrumdigital.se>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
Diffstat (limited to 'drivers/pinctrl/pinctrl-at91.c')
| -rw-r--r-- | drivers/pinctrl/pinctrl-at91.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/pinctrl/pinctrl-at91.c b/drivers/pinctrl/pinctrl-at91.c index bd57ab514aa4..ec9744f33403 100644 --- a/drivers/pinctrl/pinctrl-at91.c +++ b/drivers/pinctrl/pinctrl-at91.c @@ -924,7 +924,7 @@ static int at91_pinctrl_parse_functions(struct device_node *np, /* Initialise function */ func->name = np->name; func->ngroups = of_get_child_count(np); - if (func->ngroups <= 0) { + if (func->ngroups == 0) { dev_err(info->dev, "no groups defined\n"); return -EINVAL; } |