mbox series

[PATCHv4,0/3] media: venus: close() fixes

Message ID 20241025105104.723764-1-senozhatsky@chromium.org
Headers show
Series media: venus: close() fixes | expand

Message

Sergey Senozhatsky Oct. 25, 2024, 10:50 a.m. UTC 
A couple of fixes for venus driver close() handling
(both enc and dec).

v3->v4:
-- removed vdec_ctrl_deinit() (Dikshita)

Sergey Senozhatsky (3):
  media: venus: fix enc/dec destruction order
  media: venus: sync with threaded IRQ during inst destruction
  media: venus: factor out inst destruction routine

 drivers/media/platform/qcom/venus/core.c      | 25 +++++++++++++++++++
 drivers/media/platform/qcom/venus/core.h      |  2 ++
 drivers/media/platform/qcom/venus/vdec.c      | 13 ++--------
 drivers/media/platform/qcom/venus/vdec.h      |  1 -
 .../media/platform/qcom/venus/vdec_ctrls.c    |  6 -----
 drivers/media/platform/qcom/venus/venc.c      | 12 +--------
 6 files changed, 30 insertions(+), 29 deletions(-)

Comments

Sergey Senozhatsky Oct. 25, 2024, 1:01 p.m. UTC | #1 
On (24/10/25 12:43), Bryan O'Donoghue wrote:
> On 25/10/2024 11:50, Sergey Senozhatsky wrote:
> > -	venc_ctrl_deinit(inst);
> 
> Shouldn't you be dropping venc_ctrl_deinit as you have done with
> vdec_ctrl_deinit ?

Oh, yes, will send out v5 in a bit.