diff options
| author | Rickard Strandqvist <rickard_strandqvist@spectrumdigital.se> | 2014-06-26 07:29:49 -0400 |
|---|---|---|
| committer | Linus Walleij <linus.walleij@linaro.org> | 2014-07-11 08:08:33 -0400 |
| commit | 5aa498b95496204d3d9918b5d9287f0a4910a2a1 (patch) | |
| tree | 5857230e6075ed858ce28a8e60a20613ad37cfba | |
| parent | 9eedfd688cb267d0fee06f68cbbddaf94423482e (diff) | |
pinctrl: pinctrl-imx1-core.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>
| -rw-r--r-- | drivers/pinctrl/pinctrl-imx1-core.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/pinctrl/pinctrl-imx1-core.c b/drivers/pinctrl/pinctrl-imx1-core.c index 815384b377b5..483420757c9f 100644 --- a/drivers/pinctrl/pinctrl-imx1-core.c +++ b/drivers/pinctrl/pinctrl-imx1-core.c | |||
| @@ -526,7 +526,7 @@ static int imx1_pinctrl_parse_functions(struct device_node *np, | |||
| 526 | /* Initialise function */ | 526 | /* Initialise function */ |
| 527 | func->name = np->name; | 527 | func->name = np->name; |
| 528 | func->num_groups = of_get_child_count(np); | 528 | func->num_groups = of_get_child_count(np); |
| 529 | if (func->num_groups <= 0) | 529 | if (func->num_groups == 0) |
| 530 | return -EINVAL; | 530 | return -EINVAL; |
| 531 | 531 | ||
| 532 | func->groups = devm_kzalloc(info->dev, | 532 | func->groups = devm_kzalloc(info->dev, |