On Thu Mar 26 16:07:41 2026 +0100, Jacopo Mondi wrote:
> The rzg2l_cru_irq() irq handler uses a local variable to store the
> handler return value.
>
> Simplify it by using IRQ_NONE and IRQ_HANDLED.
>
> Signed-off-by: Jacopo Mondi <[email protected]>
> Reviewed-by: Tommaso Merciai <[email protected]>
> Reviewed-by: Daniel Scally <[email protected]>
> Signed-off-by: Hans Verkuil <[email protected]>
Patch committed.
Thanks,
Hans Verkuil
drivers/media/platform/renesas/rzg2l-cru/rzg2l-video.c | 7 ++-----
1 file changed, 2 insertions(+), 5 deletions(-)
---
diff --git a/drivers/media/platform/renesas/rzg2l-cru/rzg2l-video.c
b/drivers/media/platform/renesas/rzg2l-cru/rzg2l-video.c
index b6990466165e..5185a547461d 100644
--- a/drivers/media/platform/renesas/rzg2l-cru/rzg2l-video.c
+++ b/drivers/media/platform/renesas/rzg2l-cru/rzg2l-video.c
@@ -591,16 +591,13 @@ static void rzg2l_cru_stop_streaming(struct rzg2l_cru_dev
*cru)
irqreturn_t rzg2l_cru_irq(int irq, void *data)
{
struct rzg2l_cru_dev *cru = data;
- unsigned int handled = 0;
u32 irq_status;
u32 amnmbs;
int slot;
irq_status = rzg2l_cru_read(cru, CRUnINTS);
if (!irq_status)
- return IRQ_RETVAL(handled);
-
- handled = 1;
+ return IRQ_NONE;
rzg2l_cru_write(cru, CRUnINTS, rzg2l_cru_read(cru, CRUnINTS));
@@ -634,7 +631,7 @@ irqreturn_t rzg2l_cru_irq(int irq, void *data)
/* Prepare for next frame */
rzg2l_cru_fill_hw_slot(cru, slot);
- return IRQ_RETVAL(handled);
+ return IRQ_HANDLED;
}
irqreturn_t rzg3e_cru_irq(int irq, void *data)
_______________________________________________
linuxtv-commits mailing list -- [email protected]
To unsubscribe send an email to [email protected]
