[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <37a906eb-4cd1-4034-9bf6-2f9f4560b9e3@suse.de>
Date: Tue, 5 Nov 2024 14:04:29 +0200
From: Stanimir Varbanov <svarbanov@...e.de>
To: Sergey Senozhatsky <senozhatsky@...omium.org>,
Stanimir Varbanov <stanimir.k.varbanov@...il.com>,
Vikash Garodia <quic_vgarodia@...cinc.com>
Cc: Dikshita Agarwal <quic_dikshita@...cinc.com>,
Bryan O'Donoghue <bryan.odonoghue@...aro.org>, linux-media@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCHv6 0/3] media: venus: close() fixes
Hi Sergey,
Thank you for the patch!
On 10/25/24 19:56, Sergey Senozhatsky wrote:
> A couple of fixes for venus driver close() handling
> (both enc and dec).
>
> v5->v6:
> -- added kfree() backtrace to 0002
>
> 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
Could you please combine 1/3 and 2/3 commit bodies into 3/3 body and
resend the new 3/3 only. I do not see a reason to apply 1/3 and 2/3.
Also, on what platform this was tested?
~Stan
>
> 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 | 5 ----
> drivers/media/platform/qcom/venus/venc.c | 14 ++---------
> drivers/media/platform/qcom/venus/venc.h | 1 -
> .../media/platform/qcom/venus/venc_ctrls.c | 5 ----
> 8 files changed, 31 insertions(+), 35 deletions(-)
>
Powered by blists - more mailing lists