diff options
author | Rickard Strandqvist <rickard_strandqvist@spectrumdigital.se> | 2014-06-26 11:28:16 (GMT) |
---|---|---|
committer | Linus Walleij <linus.walleij@linaro.org> | 2014-07-11 12:08:32 (GMT) |
commit | 9eedfd688cb267d0fee06f68cbbddaf94423482e (patch) | |
tree | 9266fe2a8d1bf374a8f8c44fca05b5932c61718a /drivers/pinctrl/pinctrl-imx.c | |
parent | ca7162add9dc0b361cbf51084591c64db37199d8 (diff) | |
download | linux-9eedfd688cb267d0fee06f68cbbddaf94423482e.tar.xz |
pinctrl: pinctrl-imx.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-imx.c')
-rw-r--r-- | drivers/pinctrl/pinctrl-imx.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/pinctrl/pinctrl-imx.c b/drivers/pinctrl/pinctrl-imx.c index a24448e..946d594 100644 --- a/drivers/pinctrl/pinctrl-imx.c +++ b/drivers/pinctrl/pinctrl-imx.c @@ -515,7 +515,7 @@ static int imx_pinctrl_parse_functions(struct device_node *np, /* Initialise function */ func->name = np->name; func->num_groups = of_get_child_count(np); - if (func->num_groups <= 0) { + if (func->num_groups == 0) { dev_err(info->dev, "no groups defined in %s\n", np->full_name); return -EINVAL; } |