Re: [PATCH v1 3/3] arm64: defconfig: Enable imx8mq-librem5-devkit display stack

[Guido Günther]

Hi Fabio,
On Thu, Aug 20, 2020 at 01:11:28AM -0300, Fabio Estevam wrote:
> Hi Guido,
> 
> On Wed, Jul 29, 2020 at 9:19 AM Guido Günther <agx@xxxxxxxxxxx> wrote:
> >
> > Enable the panel, NWL DSI host controller and dphy. This
> > also needs the reset controller.
> >
> > Signed-off-by: Guido Günther <agx@xxxxxxxxxxx>
> > ---
> >  arch/arm64/configs/defconfig | 6 +++++-
> >  1 file changed, 5 insertions(+), 1 deletion(-)
> >
> > diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
> > index e0f33826819f..608019f6408d 100644
> > --- a/arch/arm64/configs/defconfig
> > +++ b/arch/arm64/configs/defconfig
> > @@ -640,10 +640,12 @@ CONFIG_DRM_MSM=m
> >  CONFIG_DRM_TEGRA=m
> >  CONFIG_DRM_PANEL_LVDS=m
> >  CONFIG_DRM_PANEL_SIMPLE=m
> > -CONFIG_DRM_SIMPLE_BRIDGE=m
> > +CONFIG_DRM_PANEL_SITRONIX_ST7703=m
> >  CONFIG_DRM_PANEL_TRULY_NT35597_WQXGA=m
> >  CONFIG_DRM_DISPLAY_CONNECTOR=m
> > +CONFIG_DRM_NWL_MIPI_DSI=m
> >  CONFIG_DRM_SII902X=m
> > +CONFIG_DRM_SIMPLE_BRIDGE=m
> 
> One nit: I suppose the CONFIG_DRM_SIMPLE_BRIDGE=m move comes from a
> defconfig re-sync.
> 
> Maybe this re-sync could be part of a separate patch?

I've moved re-sync of the drm related bits to a separate patch in v2. I
didn't do a full resync since this varies between kernel version but i
can do that too - maybe against 5.9-rc1 if that helps.

> Thanks for adding these options to the defconfig so that we could have
> MIPI DSI working by default on i.MX8M:
> 
> Reviewed-by: Fabio Estevam <festevam@xxxxxxxxx>
> 

Thanks!
 -- Guido