media: verisilicon: Fix CPU stalls on G2 bus error

Browse source

commit 19c286b755 upstream.

In some seek stress tests, we are getting IRQ from the G2 decoder where
the dec_bus_int and the dec_e bits are high, meaning the decoder is
still running despite the error.

Fix this by reworking the IRQ handler to only finish the job once we
have reached completion and move the software reset to when our software
watchdog triggers.

This way, we let the hardware continue on errors when it did not self
reset and in worse case scenario the hardware timeout will
automatically stop it. The actual error will be fixed in a follow up
patch.

Fixes: 3385c514ec ("media: hantro: Convert imx8m_vpu_g2_irq to helper")
Cc: stable@vger.kernel.org
Reviewed-by: Benjamin Gaignard <benjamin.gaignard@collabora.com>
Signed-off-by: Nicolas Dufresne <nicolas.dufresne@collabora.com>
Signed-off-by: Hans Verkuil <hverkuil+cisco@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

...

This commit is contained in:

Nicolas Dufresne 2025-09-22 14:43:38 -04:00 • committed by Greg Kroah-Hartman

parent db7ab33238

commit add7da91cc

6 changed files with 85 additions and 23 deletions

Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL

Download patch file Download diff file Expand all files Collapse all files

84

drivers/media/platform/verisilicon/hantro_g2.c

View file

2

drivers/media/platform/verisilicon/hantro_g2_hevc_dec.c

View file

13

drivers/media/platform/verisilicon/hantro_g2_regs.h

View file

2

drivers/media/platform/verisilicon/hantro_g2_vp9_dec.c

View file

1

drivers/media/platform/verisilicon/hantro_hw.h

View file

2

drivers/media/platform/verisilicon/imx8m_vpu_hw.c

View file