serial: lantiq: store and compare return status correctly
authorMuhammad Usama Anjum <usama.anjum@collabora.com>
Tue, 21 Dec 2021 18:41:51 +0000 (23:41 +0500)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 30 Dec 2021 12:23:46 +0000 (13:23 +0100)
commitcb559bb974536d75c8385b2caa57ab5a3862c29a
tree301b9b1d09c92aedf8c37f917ebfb8fd929343be
parentb4a29b94804c4774f22555651296b838df6ec0e4
serial: lantiq: store and compare return status correctly

platform_get_irq() returns signed status. It should be stored and
compared as signed value before storing to unsigned variable. Implicit
conversion from signed to unsigned and then comparison with less than
zero is wrong as unsigned value can never be less than zero.

Fixes: f087f01ca2c5 ("serial: lantiq: Use platform_get_irq() to get the interrupt")
Acked-by: Rob Herring <robh@kernel.org>
Signed-off-by: Muhammad Usama Anjum <usama.anjum@collabora.com>
Link: https://lore.kernel.org/r/YcIf7+oSWWn34ND6@debian-BULLSEYE-live-builder-AMD64
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/tty/serial/lantiq.c