diff options
author | Dong Aisheng <dong.aisheng@linaro.org> | 2012-05-14 07:06:36 -0400 |
---|---|---|
committer | Linus Walleij <linus.walleij@linaro.org> | 2012-05-15 03:43:18 -0400 |
commit | c95df2db2cb5a147b42374f391858232517c0576 (patch) | |
tree | eacbb643afd225a991f538690462e9b6e7e52c81 /drivers/pinctrl/pinconf.c | |
parent | 4851680491f3629cd53b3a1283841deadd6db88c (diff) |
pinctrl: a minor error checking improvement for pinconf
Also checking invalid num_configs when validate the pinconf map.
Acked-by: Stephen Warren <swarren@wwwdotorg.org>
Signed-off-by: Dong Aisheng <dong.aisheng@linaro.org>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
Diffstat (limited to 'drivers/pinctrl/pinconf.c')
-rw-r--r-- | drivers/pinctrl/pinconf.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/pinctrl/pinconf.c b/drivers/pinctrl/pinconf.c index 7ce139ef7e64..43f474cdc110 100644 --- a/drivers/pinctrl/pinconf.c +++ b/drivers/pinctrl/pinconf.c | |||
@@ -50,9 +50,9 @@ int pinconf_validate_map(struct pinctrl_map const *map, int i) | |||
50 | return -EINVAL; | 50 | return -EINVAL; |
51 | } | 51 | } |
52 | 52 | ||
53 | if (map->data.configs.num_configs && | 53 | if (!map->data.configs.num_configs || |
54 | !map->data.configs.configs) { | 54 | !map->data.configs.configs) { |
55 | pr_err("failed to register map %s (%d): no configs ptr given\n", | 55 | pr_err("failed to register map %s (%d): no configs given\n", |
56 | map->name, i); | 56 | map->name, i); |
57 | return -EINVAL; | 57 | return -EINVAL; |
58 | } | 58 | } |