[PATCH RESEND v6 1/2] drm/panel: Add support for Truly NT35597 panel driver

[Linus Walleij]

On Mon, Sep 3, 2018 at 9:50 PM Bjorn Andersson
<bjorn.andersson at linaro.org> wrote:
> On Tue 28 Aug 15:39 PDT 2018, Abhinav Kumar wrote:
> > From: "abhinavk at codeaurora.org" <abhinavk at codeaurora.org>
> >
> > Add support for Truly NT35597 panel driver used
> > in MSM reference platforms.
> >
> > This panel driver supports both single DSI and dual DSI
> > modes.
> >
> > However, this patch series adds support only for
> > dual DSI mode.
> >
> > Changes in v6:
> > - Introduce panel config to store panel specific
> >   details
> > - Bring back the size member for the panel command
> >   structure to make the design more scalable
> > - Move the display mode from the DT to driver
> > - Change the compatible string to indicate which
> >   which board and panel it will be used for
> > - Rename the functions to match the panel driver
> > - Have a data member for each compatible string
> > - Remove the panel commands split as its not
> >   required for the panel init functionality
> >
> > Signed-off-by: Archit Taneja <architt at codeaurora.org>
> > Signed-off-by: Abhinav Kumar <abhinavk at codeaurora.org>
> [..]
> > diff --git a/drivers/gpu/drm/panel/panel-truly-nt35597.c b/drivers/gpu/drm/panel/panel-truly-nt35597.c
> [..]
> > +     {       .compatible = "qcom,sdm845-mtp-2K-display",
>
> I do not agree with Linus here.
>
> If I understand correctly the variability in the video timing comes from
> the fact that the LCD Driver IC (NT35597) can be paired with different
> LCD panels. As such I strongly believe that the compatible should
> describe this combo, not which phone/devkit you mounted a panel using
> this Driver IC on.

I agree with that.

I just saw "it's some qualcomm string, not just the name of the LCD
driver so OK".

The inability to parse the "qcom,*" string is just my ignorance, sorry
for that.

Yours,
Linus Walleij