Re: [PATCH] media: verisilicon: vp9: do not arm watchdog on preparation failure

Tharit Tangkijwanichakul <[email protected]> Fri, 24 Jul 2026 08:56:35 +0700

Newsgroups	dev.linux.lists.linux-kernel-mentees,org.infradead.lists.linux-rockchip,org.kernel.vger.linux-kernel,org.kernel.vger.linux-media
Message-ID	<CACak8wPMt9Qp6xSs6AATLRjH3pHgFvvT0xxGHQY3q_WQWNO_BA@mail.gmail.com>

> Nack.
> First it schedules a software timer not an hardware timer, which led after
> timeout to call hantro_job_finish_no_pm() with VB2_BUF_STATE_ERROR parameter.
> We need this call to complete v4l2_m2m job.
>
> If you want to stop the task before the timeout you need to add something like
> hantro_end_prepare_run_with_error() where you do the correct sequence.
>
> Regards,
> Benjamin

Hi Benjamin,

Understood. Thanks for the review and pointing out my misunderstanding.

Best,
Tharit