diff mbox series

dt-bindings: display: add binding for Innolux ee101ia-01d panel

Message ID 20181113124205.29319-1-heiko@sntech.de (mailing list archive)
State New, archived
Headers show
Series dt-bindings: display: add binding for Innolux ee101ia-01d panel | expand

Commit Message

Heiko Stübner Nov. 13, 2018, 12:42 p.m. UTC
From: Heiko Stuebner <heiko.stuebner@bq.com>

This is a panel handled through the generic lvds-panel binding,
so only needs its additional compatible specified.

Signed-off-by: Heiko Stuebner <heiko.stuebner@bq.com>
---
 .../bindings/display/panel/innolux,ee101ia-01d.txt         | 7 +++++++
 1 file changed, 7 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/display/panel/innolux,ee101ia-01d.txt

Comments

Rob Herring (Arm) Dec. 4, 2018, 8:15 p.m. UTC | #1
On Tue, 13 Nov 2018 13:42:05 +0100, Heiko Stuebner wrote:
> From: Heiko Stuebner <heiko.stuebner@bq.com>
> 
> This is a panel handled through the generic lvds-panel binding,
> so only needs its additional compatible specified.
> 
> Signed-off-by: Heiko Stuebner <heiko.stuebner@bq.com>
> ---
>  .../bindings/display/panel/innolux,ee101ia-01d.txt         | 7 +++++++
>  1 file changed, 7 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/display/panel/innolux,ee101ia-01d.txt
> 

Reviewed-by: Rob Herring <robh@kernel.org>
Heiko Stübner Jan. 29, 2019, 1:37 p.m. UTC | #2
Hi Thierry,

Am Dienstag, 13. November 2018, 13:42:05 CET schrieb Heiko Stuebner:
> From: Heiko Stuebner <heiko.stuebner@bq.com>
> 
> This is a panel handled through the generic lvds-panel binding,
> so only needs its additional compatible specified.
> 
> Signed-off-by: Heiko Stuebner <heiko.stuebner@bq.com>

just pulling this pending patch out of the woodworks again too :-)

Thanks in advance for considering it.
Heiko

> ---
>  .../bindings/display/panel/innolux,ee101ia-01d.txt         | 7 +++++++
>  1 file changed, 7 insertions(+)
>  create mode 100644
> Documentation/devicetree/bindings/display/panel/innolux,ee101ia-01d.txt
> 
> diff --git
> a/Documentation/devicetree/bindings/display/panel/innolux,ee101ia-01d.txt
> b/Documentation/devicetree/bindings/display/panel/innolux,ee101ia-01d.txt
> new file mode 100644
> index 000000000000..e5ca4ccd55ed
> --- /dev/null
> +++
> b/Documentation/devicetree/bindings/display/panel/innolux,ee101ia-01d.txt
> @@ -0,0 +1,7 @@
> +Innolux Corporation 10.1" EE101IA-01D WXGA (1280x800) LVDS panel
> +
> +Required properties:
> +- compatible: should be "innolux,ee101ia-01d"
> +
> +This binding is compatible with the lvds-panel binding, which is specified
> +in panel-lvds.txt in this directory.
Daniel Vetter Jan. 29, 2019, 4:45 p.m. UTC | #3
On Tue, Jan 29, 2019 at 02:37:23PM +0100, Heiko Stübner wrote:
> Hi Thierry,
> 
> Am Dienstag, 13. November 2018, 13:42:05 CET schrieb Heiko Stuebner:
> > From: Heiko Stuebner <heiko.stuebner@bq.com>
> > 
> > This is a panel handled through the generic lvds-panel binding,
> > so only needs its additional compatible specified.
> > 
> > Signed-off-by: Heiko Stuebner <heiko.stuebner@bq.com>
> 
> just pulling this pending patch out of the woodworks again too :-)
> 
> Thanks in advance for considering it.

It's a panel patch, you have the review from Rob, you have commit rights
to drm-misc and panel is maintained in drm-misc. There's nothing blocking
you from pushing it, just do it :-)

Cheers, Daniel

> Heiko
> 
> > ---
> >  .../bindings/display/panel/innolux,ee101ia-01d.txt         | 7 +++++++
> >  1 file changed, 7 insertions(+)
> >  create mode 100644
> > Documentation/devicetree/bindings/display/panel/innolux,ee101ia-01d.txt
> > 
> > diff --git
> > a/Documentation/devicetree/bindings/display/panel/innolux,ee101ia-01d.txt
> > b/Documentation/devicetree/bindings/display/panel/innolux,ee101ia-01d.txt
> > new file mode 100644
> > index 000000000000..e5ca4ccd55ed
> > --- /dev/null
> > +++
> > b/Documentation/devicetree/bindings/display/panel/innolux,ee101ia-01d.txt
> > @@ -0,0 +1,7 @@
> > +Innolux Corporation 10.1" EE101IA-01D WXGA (1280x800) LVDS panel
> > +
> > +Required properties:
> > +- compatible: should be "innolux,ee101ia-01d"
> > +
> > +This binding is compatible with the lvds-panel binding, which is specified
> > +in panel-lvds.txt in this directory.
> 
> 
> 
> 
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
Heiko Stübner Feb. 3, 2019, 9:26 a.m. UTC | #4
Am Dienstag, 29. Januar 2019, 17:45:50 CET schrieb Daniel Vetter:
> On Tue, Jan 29, 2019 at 02:37:23PM +0100, Heiko Stübner wrote:
> > Hi Thierry,
> > 
> > Am Dienstag, 13. November 2018, 13:42:05 CET schrieb Heiko Stuebner:
> > > From: Heiko Stuebner <heiko.stuebner@bq.com>
> > > 
> > > This is a panel handled through the generic lvds-panel binding,
> > > so only needs its additional compatible specified.
> > > 
> > > Signed-off-by: Heiko Stuebner <heiko.stuebner@bq.com>
> > 
> > just pulling this pending patch out of the woodworks again too :-)
> > 
> > Thanks in advance for considering it.
> 
> It's a panel patch, you have the review from Rob, you have commit rights
> to drm-misc and panel is maintained in drm-misc. There's nothing blocking
> you from pushing it, just do it :-)

yeah, at some point I'll probably internalize the drm-misc concept
without feeling of stepping on someone's toes :-D

Anyway, applied and pushed now.
Heiko
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/display/panel/innolux,ee101ia-01d.txt b/Documentation/devicetree/bindings/display/panel/innolux,ee101ia-01d.txt
new file mode 100644
index 000000000000..e5ca4ccd55ed
--- /dev/null
+++ b/Documentation/devicetree/bindings/display/panel/innolux,ee101ia-01d.txt
@@ -0,0 +1,7 @@ 
+Innolux Corporation 10.1" EE101IA-01D WXGA (1280x800) LVDS panel
+
+Required properties:
+- compatible: should be "innolux,ee101ia-01d"
+
+This binding is compatible with the lvds-panel binding, which is specified
+in panel-lvds.txt in this directory.