gpio: exar: Fix bad handling for ida_simple_get error path
authorTakashi Iwai <tiwai@suse.de>
Wed, 29 Apr 2020 13:56:54 +0000 (15:56 +0200)
committerBartosz Golaszewski <bgolaszewski@baylibre.com>
Tue, 5 May 2020 16:24:56 +0000 (18:24 +0200)
commit333830aa149a87cabeb5d30fbcf12eecc8040d2c
tree1667bbad3bc34a8148738fd16d26f0bce8dd8226
parent19c26d90ff4ca08ef2a2fef23cc9c13cfbfd891e
gpio: exar: Fix bad handling for ida_simple_get error path

The commit 7ecced0934e5 ("gpio: exar: add a check for the return value
of ida_simple_get fails") added a goto jump to the common error
handler for ida_simple_get() error, but this is wrong in two ways:
it doesn't set the proper return code and, more badly, it invokes
ida_simple_remove() with a negative index that shall lead to a kernel
panic via BUG_ON().

This patch addresses those two issues.

Fixes: 7ecced0934e5 ("gpio: exar: add a check for the return value of ida_simple_get fails")
Cc: <stable@vger.kernel.org>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
drivers/gpio/gpio-exar.c