diff options
author | Arnd Bergmann <arnd@arndb.de> | 2016-09-09 11:26:50 +0200 |
---|---|---|
committer | Linus Walleij <linus.walleij@linaro.org> | 2016-09-13 10:41:49 +0200 |
commit | 55956035266d8f5d825f30add0d7af3850cca0ad (patch) | |
tree | 6247ce5809160d1e0616bbd79784a5528ce81949 /drivers/pinctrl/meson | |
parent | 03e9888f955cc0950a15713a93af8946c56a37bb (diff) |
pinctrl: aspeed: fix regmap error handling
The newly added aspeed driver tries to check for a negative return
value from a pinctrl function, but stores the intermediate value in
a 'bool' variable, which cannot work:
drivers/pinctrl/aspeed/pinctrl-aspeed.c: In function 'aspeed_sig_expr_set':
drivers/pinctrl/aspeed/pinctrl-aspeed.c:192:11: error: comparison of constant '0' with boolean expression is always false [-Werror=bool-compare]
This slightly reworks the logic to use an explicit comparison with zero
before assigning to the temporary variable.
Reported-by: Colin King <colin.king@canonical.com>
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Acked-by: Joel Stanley <joel@jms.id.au>
Reviewed-by: Andrew Jeffery <andrew@aj.id.au>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
Diffstat (limited to 'drivers/pinctrl/meson')
0 files changed, 0 insertions, 0 deletions