[PATCH 4/6] drm/msm/dp: Use function arguments for aux writes

Dmitry Baryshkov dmitry.baryshkov at linaro.org
Tue Mar 26 17:31:33 UTC 2024


On Tue, 26 Mar 2024 at 17:06, Bjorn Andersson <andersson at kernel.org> wrote:
>
> From: Bjorn Andersson <quic_bjorande at quicinc.com>
>
> The dp_aux write operations takes the data to be operated on through a
> member of struct dp_catalog, rather than as an argument to the function.
>
> No state is maintained other than across the calling of the functions,
> so replace this member with a function argument.

Definitely yes, thank you!

Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov at linaro.org>

>
> Signed-off-by: Bjorn Andersson <quic_bjorande at quicinc.com>
> ---
>  drivers/gpu/drm/msm/dp/dp_aux.c     | 9 +++------
>  drivers/gpu/drm/msm/dp/dp_catalog.c | 8 ++++----
>  drivers/gpu/drm/msm/dp/dp_catalog.h | 5 ++---
>  3 files changed, 9 insertions(+), 13 deletions(-)

-- 
With best wishes
Dmitry


More information about the Freedreno mailing list