mbox series

[0/3] add panel driver for innolux,p097pfg with ssd2825 bridge

Message ID 20190912090404.89822-1-jitao.shi@mediatek.com (mailing list archive)
Headers show
Series add panel driver for innolux,p097pfg with ssd2825 bridge | expand

Message

Jitao Shi Sept. 12, 2019, 9:04 a.m. UTC
Add driver to support panel innolux,p097pfg with bridge ssd2858.
SSD2858 can spilt dsi 4 lanes to 8 lanes.

Jitao Shi (3):
  drm/panel: panel-innolux: Allow 2 reset pins for panel
  dt-bindings: display: Add documentation for innolux,p097pfg_ssd2858
    panel
  drm/panel: panel-innolux: Add support for P097PFZ behind SSD2858

 .../display/panel/innolux,p097pfg_ssd2858.txt |  39 +++++
 drivers/gpu/drm/panel/panel-innolux-p079zca.c | 140 ++++++++++++++++--
 2 files changed, 164 insertions(+), 15 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/display/panel/innolux,p097pfg_ssd2858.txt

Comments

Nicolas Boichat Sept. 16, 2019, 4:27 a.m. UTC | #1
Hi Jitao,

On Thu, Sep 12, 2019 at 5:04 PM Jitao Shi <jitao.shi@mediatek.com> wrote:
>
> Add driver to support panel innolux,p097pfg with bridge ssd2858.
> SSD2858 can spilt dsi 4 lanes to 8 lanes.
>
> Jitao Shi (3):
>   drm/panel: panel-innolux: Allow 2 reset pins for panel
>   dt-bindings: display: Add documentation for innolux,p097pfg_ssd2858
>     panel
>   drm/panel: panel-innolux: Add support for P097PFZ behind SSD2858

I'm the author of at least the first patch, and at least part of the
third one, so it would have been better if you clarified the origin of
the patches.

Also, I do not believe that this is the right approach (this was
always a temporary hack that we had no intent to upstream as-is), we'd
probably want to add a proper bridge driver for SSD2858, and allow
arbitrary panels behind it.

Please consider this series
Nacked-by: Nicolas Boichat <drinkcat@chromium.org>

Thanks,

>  .../display/panel/innolux,p097pfg_ssd2858.txt |  39 +++++
>  drivers/gpu/drm/panel/panel-innolux-p079zca.c | 140 ++++++++++++++++--
>  2 files changed, 164 insertions(+), 15 deletions(-)
>  create mode 100644 Documentation/devicetree/bindings/display/panel/innolux,p097pfg_ssd2858.txt
>
> --
> 2.21.0
>