Re: [PATCH v4 07/10] drm/msm/dsi: Remove custom DSI config handling

[Marijn Suijten]

On 2023-03-14 14:44:06, Konrad Dybcio wrote:
> 
> 
> On 14.03.2023 14:05, Marijn Suijten wrote:
> > On 2023-03-14 13:13:45, Konrad Dybcio wrote:
> >> Now that the only user is handled by common code, remove the option to
> >> specify custom handlers through match data.
> >>
> >> This is effectively a revert of commit:
> >> 5ae15e76271 ("drm/msm/dsi: Allow to specify dsi config as pdata")
> >>
> >> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx>
> >> Reviewed-by: Marijn Suijten <marijn.suijten@xxxxxxxxxxxxxx>
> >> Signed-off-by: Konrad Dybcio <konrad.dybcio@xxxxxxxxxx>
> >> ---
> [...]
> >> diff --git a/drivers/gpu/drm/msm/dsi/dsi_cfg.h b/drivers/gpu/drm/msm/dsi/dsi_cfg.h
> >> index 8772a3631ac1..91bdaf50bb1a 100644
> >> --- a/drivers/gpu/drm/msm/dsi/dsi_cfg.h
> >> +++ b/drivers/gpu/drm/msm/dsi/dsi_cfg.h
> >> @@ -65,8 +65,5 @@ struct msm_dsi_cfg_handler {
> >>  
> >>  const struct msm_dsi_cfg_handler *msm_dsi_cfg_get(u32 major, u32 minor);
> >>  
> >> -/* Non autodetect configs */
> >> -extern const struct msm_dsi_cfg_handler qcm2290_dsi_cfg_handler;
> >> -
> > 
> > Probably the wrong `fixup!` commit: this should have been part of patch
> > 6 where the struct is removed, not patch 7 (this patch).
> Yeah, that's most likely what happened.. Does that warrant another resend?

For proper cleanliness it should have been... but maybe Dmitry can patch
this up while applying?  In that case, at least have my:

Reviewed-by: Marijn Suijten <marijn.suijten@xxxxxxxxxxxxxx>