2019-09-12 09:09:37

by Jitao Shi

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

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

--
2.21.0

2019-09-16 09:13:47

by Nicolas Boichat

Subject: Re: [PATCH 0/3] add panel driver for innolux,p097pfg with ssd2825 bridge

Hi Jitao,

On Thu, Sep 12, 2019 at 5:04 PM Jitao Shi <[email protected]> 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 <[email protected]>

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
>