On Tue 28 Aug 15:39 PDT 2018, Abhinav Kumar wrote:
From: "abhinavk@codeaurora.org" abhinavk@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@codeaurora.org Signed-off-by: Abhinav Kumar abhinavk@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.
Regards, Bjorn