On Tue, 26 Mar 2024 at 17:06, Bjorn Andersson <andersson@xxxxxxxxxx> wrote: > > From: Bjorn Andersson <quic_bjorande@xxxxxxxxxxx> > > dp_catalog_panel_timing_cfg() takes 4 arguments, which are passed from > the calling function through members of struct dp_catalog. > > No state is maintained other than across this call, so switch to > function arguments to clean up the code. > > Signed-off-by: Bjorn Andersson <quic_bjorande@xxxxxxxxxxx> > --- > drivers/gpu/drm/msm/dp/dp_catalog.c | 14 ++++++-------- > drivers/gpu/drm/msm/dp/dp_catalog.h | 7 ++----- > drivers/gpu/drm/msm/dp/dp_panel.c | 14 +++++++++----- > 3 files changed, 17 insertions(+), 18 deletions(-) > Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx> -- With best wishes Dmitry