Re: [PATCH 2/7] drm/msm/gem: Remove submit_unlock_unpin_bo()

From: Dmitry Baryshkov
Date: Fri Oct 27 2023 - 20:39:22 EST


On Fri, 27 Oct 2023 at 19:59, Rob Clark <robdclark@xxxxxxxxx> wrote:
>
> From: Rob Clark <robdclark@xxxxxxxxxxxx>
>
> The only point it is called is before pinning objects, so the "unpin"
> part of the name is fiction. Just remove call submit_cleanup_bo()

Nit: 'remove it and call'

Other than that:

Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx>

> directly.
>
> Signed-off-by: Rob Clark <robdclark@xxxxxxxxxxxx>
> ---
> drivers/gpu/drm/msm/msm_gem_submit.c | 10 ++--------
> 1 file changed, 2 insertions(+), 8 deletions(-)


--
With best wishes
Dmitry