On Thu, 28 Mar 2024 at 16:35, Bjorn Andersson <andersson@xxxxxxxxxx> wrote: > > From: Bjorn Andersson <quic_bjorande@xxxxxxxxxxx> > > The members of struct dp_debug are no longer used, so the only purpose > of this struct is as a type of the return value of dp_debug_get(), to > signal success/error. > > Drop the struct in favor of signalling the result of initialization > using an int, then merge dp_debug_get() with dp_debug_init() to avoid > the unnecessar boilerplate code. > > Signed-off-by: Bjorn Andersson <quic_bjorande@xxxxxxxxxxx> > --- > drivers/gpu/drm/msm/dp/dp_debug.c | 59 +++++++++++-------------------------- > drivers/gpu/drm/msm/dp/dp_debug.h | 38 +++++++----------------- > drivers/gpu/drm/msm/dp/dp_display.c | 10 ++----- > 3 files changed, 31 insertions(+), 76 deletions(-) Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx> -- With best wishes Dmitry