[git:media_stage/master] media: adv7180: Only request valids IRQs

[Hans Verkuil]

This is an automatic generated email to let you know that the following patch 
were queued:

Subject: media: adv7180: Only request valids IRQs
Author:  Ricardo Ribalda <riba...@chromium.org>
Date:    Wed Apr 10 21:54:42 2024 +0000

i2c_device_probe(), seems to assume that irq = 0 means that there is no
irq to request.

The driver also believes that on the clean path. So lets be consistent
here.

Also make smatch happy.

Fix:
drivers/media/i2c/adv7180.c:1526 adv7180_probe() warn: 'client->irq' from 
request_threaded_irq() not released on lines: 1526

Signed-off-by: Ricardo Ribalda <riba...@chromium.org>
Signed-off-by: Hans Verkuil <hverkuil-ci...@xs4all.nl>

 drivers/media/i2c/adv7180.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

---

diff --git a/drivers/media/i2c/adv7180.c b/drivers/media/i2c/adv7180.c
index 4829cbe32419..819ff9f7c90f 100644
--- a/drivers/media/i2c/adv7180.c
+++ b/drivers/media/i2c/adv7180.c
@@ -1486,7 +1486,7 @@ static int adv7180_probe(struct i2c_client *client)
        if (ret)
                goto err_media_entity_cleanup;
 
-       if (state->irq) {
+       if (state->irq > 0) {
                ret = request_threaded_irq(client->irq, NULL, adv7180_irq,
                                           IRQF_ONESHOT | IRQF_TRIGGER_FALLING,
                                           KBUILD_MODNAME, state);