Re: [PATCH v4 07/16] drm/msm/dp: pull I/O data out of msm_dp_catalog_private()

From: Abhinav Kumar
Date: Mon Dec 16 2024 - 14:48:36 EST




On 12/15/2024 2:44 PM, Dmitry Baryshkov wrote:
Having I/O regions inside a msm_dp_catalog_private() results in extra
layers of one-line wrappers for accessing the data. Move I/O region base
and size to the globally visible struct msm_dp_catalog.

Reviewed-by: Stephen Boyd <swboyd@xxxxxxxxxxxx>
Tested-by: Stephen Boyd <swboyd@xxxxxxxxxxxx> # sc7180-trogdor
Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx>
---
drivers/gpu/drm/msm/dp/dp_catalog.c | 542 +++++++++++++++---------------------
drivers/gpu/drm/msm/dp/dp_catalog.h | 12 +
2 files changed, 232 insertions(+), 322 deletions(-)


For the reminder of this series, please wait for the discussion at [1] to complete.


[1] : https://patchwork.freedesktop.org/patch/628449/?series=141074&rev=3