gpio: omap: return error if requested debounce time is not possible
authorDavid Rivshin <DRivshin@allworx.com>
Mon, 24 Apr 2017 22:56:50 +0000 (18:56 -0400)
committerLinus Walleij <linus.walleij@linaro.org>
Fri, 28 Apr 2017 07:50:22 +0000 (09:50 +0200)
commit83977443938122baeed28dc9f078db3da9855f7c
treed2a5106491740b2c8330a8de9a3d3c6badfe6cda
parent9384793036afb7529c1c564e839ef4356271d68e
gpio: omap: return error if requested debounce time is not possible

omap_gpio_debounce() does not validate that the requested debounce
is within a range it can handle. Instead it lets the register value
wrap silently, and always returns success.

This can lead to all sorts of unexpected behavior, such as gpio_keys
asking for a too-long debounce, but getting a very short debounce in
practice.

Fix this by returning -EINVAL if the requested value does not fit into
the register field. If there is no debounce clock available at all,
return -ENOTSUPP.

Fixes: e85ec6c3047b ("gpio: omap: fix omap2_set_gpio_debounce")
Cc: <stable@vger.kernel.org> # 4.3+
Signed-off-by: David Rivshin <drivshin@allworx.com>
Acked-by: Grygorii Strashko <grygorii.strashko@ti.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/gpio/gpio-omap.c