Re: [PATCH 12/19] drm/panel: jadard-jd9365da-h3: support variable DSI configuration

[Dmitry Baryshkov]

On Wed, Apr 08, 2026 at 10:15:13AM +0200, Linus Walleij wrote:
> Hi Dmitry,
> 
> thanks for your patch!
> 
> On Wed, Apr 1, 2026 at 9:27 AM Dmitry Baryshkov
> <dmitry.baryshkov@xxxxxxxxxxxxxxxx> wrote:
> 
> > Several panels support attachment either using 4 DSI lanes or just 2. In
> > some cases, this requires a different panel mode to fulfill clock
> > requirements. Extend the driver to handle such cases by letting the
> > panel description to omit lanes specification and parsing number of
> > lanes from the DT.
> >
> > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxxxxxxxx>
> (...)
> 
> > +       if (dsi_ctx->dsi->lanes == 2)
> > +               mipi_dsi_dcs_write_seq_multi(dsi_ctx, 0x80, 0x01);
> > +       else
> > +               mipi_dsi_dcs_write_seq_multi(dsi_ctx, 0x80, 0x03);
> 
> If lanes 2 we do unexplicable magic A else we do unexplicable magic B?
> 
> Do we know more about what is actually going on here?
> 
> Can you check the datasheet?

The datasheet (you can find it) is pretty useless here. Unfortunately I
don't have a second source of information. As far as I know, this
value maps directly to the number of lanes and nothing else.

> 
> The patch is nice anyway and no big deal so, so +/- that fixup:
> Reviewed-by: Linus Walleij <linusw@xxxxxxxxxx>
> 
> Yours,
> Linus Walleij

-- 
With best wishes
Dmitry