[PATCH v2 0/4] media: wave5: fix runtime PM error handling

[Jiale Yao <yaojiale02@163.com>]

The Wave5 decoder, encoder, and close paths assume that
pm_runtime_resume_and_get() always succeeds. Runtime resume can fail while
enabling the VPU clocks, after which firmware commands can access unavailable
registers and cleanup paths can drop references that were not acquired.

Check runtime resume results before hardware access. On errors from callbacks
that cannot return an error, return queued buffers and complete the active
job. When closing an instance, still release host-side resources and return
-EIO so the common release path completes instance cleanup.

media-next already checks the decoder destination queue resume result, but
uses vb2_buffer_done() instead of the mem2mem completion helper. This series
keeps the remaining decoder resume fixes separate from that interface fix.

Jiale Yao (4):
  media: wave5: handle remaining decoder runtime resume failures
  media: wave5: use m2m buffer completion helper
  media: wave5: handle encoder runtime resume failures
  media: wave5: handle runtime resume failure when closing

---
Changes in v2:
  - Rebase on media-next.
  - Split the decoder changes: fix the remaining unchecked resume results
    separately from the existing destination-buffer completion path.
  - Use v4l2_m2m_buf_done() for the destination-buffer completion path.
  - Stop the encoder and fail both queues on a runtime resume failure to
    prevent the mem2mem scheduler from repeatedly running a failed job.
  - Rebase the close-path cleanup on the already accepted runtime PM
    reference leak fix.
  - Drop v1 patch 4 because its runtime PM reference leak fix is already
    accepted upstream.

 .../chips-media/wave5/wave5-vpu-dec.c         | 49 ++++++++++++++++---
 .../chips-media/wave5/wave5-vpu-enc.c         | 33 +++++++++++--
 .../platform/chips-media/wave5/wave5-vpuapi.c | 18 ++++++-
 3 files changed, 89 insertions(+), 11 deletions(-)

-- 
2.34.1