Re: [PATCH v2 0/7] media: rkvdec: Add HEVC backend
From: Jonas Karlman <jonas@kwiboo.se>
Date: 2025-08-12 17:31:49
Also in:
linux-devicetree, linux-media, linux-rockchip, lkml
On 8/12/2025 2:44 PM, Nicolas Dufresne wrote:
I forgot, Le mardi 12 août 2025 à 08:38 -0400, Nicolas Dufresne a écrit :quoted
quoted
JCT-VC-HEVC_V1 on GStreamer-H.265-V4L2SL-Gst1.0: - DBLK_D_VIXS_2 (fail) - DSLICE_A_HHI_5 (fail) - EXT_A_ericsson_4 (fail) - PICSIZE_A_Bossen_1 (error) - PICSIZE_B_Bossen_1 (error) - PICSIZE_C_Bossen_1 (error) - PICSIZE_D_Bossen_1 (error) - SAODBLK_A_MainConcept_4 (fail) - SAODBLK_B_MainConcept_4 (fail) - TSUNEQBD_A_MAIN10_Technicolor_2 (error)I'me getting the same result if I force a single job in fluster. The test I posted was with 2 jobs. Detlev found that the iommu reset is required in more cases on RK3588/3576, perhaps the HEVC decoder in older hardware needs the same, I will try and report.
Vendor kernel [1] check following bits from RKVDEC_REG_INTERRUPT reg to decide if a full HW reset should be done. err_mask = RKVDEC_BUF_EMPTY_STA | RKVDEC_BUS_STA | RKVDEC_COLMV_REF_ERR_STA | RKVDEC_ERR_STA | RKVDEC_TIMEOUT_STA; Adding proper reset support can be rather involved and main reason why this series does not handle it, better suited for a separate future series. Proper HW reset will require e.g. dt-bindings, DT updates, pmu idle request integration and for rk3328 vendor even moved VPU reset to TF-A. Doing the iommu detach/attach dance not only on RKVDEC_SOFTRESET_RDY could possible improve some cases, until full reset can be implemented. [1] https://github.com/Kwiboo/linux-rockchip/blob/linux-6.1-stan-rkr6.1/drivers/video/rockchip/mpp/mpp_rkvdec.c#L924-L931 Regards, Jonas
Nicolas