diff options
author | Masahiro Yamada <yamada.masahiro@socionext.com> | 2016-05-25 07:14:13 -0400 |
---|---|---|
committer | Linus Walleij <linus.walleij@linaro.org> | 2016-05-31 04:33:00 -0400 |
commit | 1fb1f0540deae634d454f917e3d45c27b69521a8 (patch) | |
tree | 626f4315ec6d90a083ce3ab72edfce3ea390aba9 /drivers/pinctrl/pinmux.c | |
parent | 8b2b3dcb343bf8cb62efc1bd6a9dbe850005e2ac (diff) |
pinctrl: return -ENOMEM instead of -EINVAL for kasprintf() failure
-ENOMEM is more suitable error code because kasprintf() fails
in case of memory shortage.
Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
Diffstat (limited to 'drivers/pinctrl/pinmux.c')
-rw-r--r-- | drivers/pinctrl/pinmux.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/pinctrl/pinmux.c b/drivers/pinctrl/pinmux.c index d94d76ca5651..ece702881946 100644 --- a/drivers/pinctrl/pinmux.c +++ b/drivers/pinctrl/pinmux.c | |||
@@ -256,7 +256,7 @@ int pinmux_request_gpio(struct pinctrl_dev *pctldev, | |||
256 | /* Conjure some name stating what chip and pin this is taken by */ | 256 | /* Conjure some name stating what chip and pin this is taken by */ |
257 | owner = kasprintf(GFP_KERNEL, "%s:%d", range->name, gpio); | 257 | owner = kasprintf(GFP_KERNEL, "%s:%d", range->name, gpio); |
258 | if (!owner) | 258 | if (!owner) |
259 | return -EINVAL; | 259 | return -ENOMEM; |
260 | 260 | ||
261 | ret = pin_request(pctldev, pin, owner, range); | 261 | ret = pin_request(pctldev, pin, owner, range); |
262 | if (ret < 0) | 262 | if (ret < 0) |