Input: adp5588-keys - fix check on return code
authorNuno Sa <nuno.sa@analog.com>
Fri, 20 Sep 2024 07:22:52 +0000 (09:22 +0200)
committerDmitry Torokhov <dmitry.torokhov@gmail.com>
Fri, 20 Sep 2024 08:21:56 +0000 (01:21 -0700)
commiteb017f4ea13b1a5ad7f4332279f2e4c67b44bdea
tree205ee225611ee6388aab2ea0b4b95545d2f58eed
parent3870e2850b56306d1d1e435c5a1ccbccd7c59291
Input: adp5588-keys - fix check on return code

During adp5588_setup(), we read all the events to clear the event FIFO.
However, adp5588_read() just calls i2c_smbus_read_byte_data() which
returns the byte read in case everything goes well. Hence, we need to
explicitly check for a negative error code instead of checking for
something different than 0.

Fixes: e960309ce318 ("Input: adp5588-keys - bail out on returned error")
Cc: stable@vger.kernel.org
Signed-off-by: Nuno Sa <nuno.sa@analog.com>
Link: https://lore.kernel.org/r/20240920-fix-adp5588-err-check-v1-1-81f6e957ef24@analog.com
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
drivers/input/keyboard/adp5588-keys.c