[PATCH 1/2] dt/bindings: display: fix display-timing reference

[PATCH 1/2] dt/bindings: display: fix display-timing reference

[Baruch Siach]

Fixes: 14da3ed8dd08 ("devicetree/bindings: display: Document common panel properties")
Cc: Laurent Pinchart <laurent.pinchart+renesas-ryLnwIuWjnjg/C1BVhZhaw@public.gmane.org>
Cc: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
Signed-off-by: Baruch Siach <baruch-NswTu9S1W3P6gbPvEgmw2w@public.gmane.org>
---
 Documentation/devicetree/bindings/display/panel/panel-common.txt | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/display/panel/panel-common.txt b/Documentation/devicetree/bindings/display/panel/panel-common.txt
index ec52c472c845..0603af877155 100644
--- a/Documentation/devicetree/bindings/display/panel/panel-common.txt
+++ b/Documentation/devicetree/bindings/display/panel/panel-common.txt
@@ -38,7 +38,7 @@ Display Timings
   require specific display timings. The panel-timing subnode expresses those
   timings as specified in the timing subnode section of the display timing
   bindings defined in
-  Documentation/devicetree/bindings/display/display-timing.txt.
+  Documentation/devicetree/bindings/display/panel/display-timing.txt.
 
 
 Connectivity
-- 
2.11.0

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH 2/2] dt/bindings: display: move rotation into a common place

[Baruch Siach]

Commit b60c1be74741 (dt-bindings: display/panel: Add common rotation property)
added the rotation property description in a new file. We have a place for
common display panel properties already. Move there the rotation property.

Cc: Noralf Trønnes <noralf-L59+Z2yzLopAfugRpC6u6w@public.gmane.org>
Cc: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
Signed-off-by: Baruch Siach <baruch-NswTu9S1W3P6gbPvEgmw2w@public.gmane.org>
---
 Documentation/devicetree/bindings/display/panel/panel-common.txt | 6 ++++++
 Documentation/devicetree/bindings/display/panel/panel.txt        | 4 ----
 2 files changed, 6 insertions(+), 4 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/display/panel/panel.txt

diff --git a/Documentation/devicetree/bindings/display/panel/panel-common.txt b/Documentation/devicetree/bindings/display/panel/panel-common.txt
index 0603af877155..c92dcfd8c32f 100644
--- a/Documentation/devicetree/bindings/display/panel/panel-common.txt
+++ b/Documentation/devicetree/bindings/display/panel/panel-common.txt
@@ -89,3 +89,9 @@ backlight controller.
 
 - backlight: For panels whose backlight is controlled by an external backlight
   controller, this property contains a phandle that references the controller.
+
+
+Rotation
+--------
+
+- rotation: Display rotation in degrees counter clockwise (0,90,180,270)
diff --git a/Documentation/devicetree/bindings/display/panel/panel.txt b/Documentation/devicetree/bindings/display/panel/panel.txt
deleted file mode 100644
index e2e6867852b8..000000000000
--- a/Documentation/devicetree/bindings/display/panel/panel.txt
+++ /dev/null
@@ -1,4 +0,0 @@
-Common display properties
--------------------------
-
-- rotation:	Display rotation in degrees counter clockwise (0,90,180,270)
-- 
2.11.0

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 1/2] dt/bindings: display: fix display-timing reference

[Laurent Pinchart]

Hi Baruch,

Thank you for the patch.

On Tuesday 16 May 2017 11:19:04 Baruch Siach wrote:

A commit message body would be nice.

> Fixes: 14da3ed8dd08 ("devicetree/bindings: display: Document common panel
> properties")
> Cc: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
> Cc: Rob Herring <robh@kernel.org>
> Signed-off-by: Baruch Siach <baruch@tkos.co.il>
> ---
>  Documentation/devicetree/bindings/display/panel/panel-common.txt | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git
> a/Documentation/devicetree/bindings/display/panel/panel-common.txt
> b/Documentation/devicetree/bindings/display/panel/panel-common.txt index
> ec52c472c845..0603af877155 100644
> --- a/Documentation/devicetree/bindings/display/panel/panel-common.txt
> +++ b/Documentation/devicetree/bindings/display/panel/panel-common.txt
> @@ -38,7 +38,7 @@ Display Timings
>    require specific display timings. The panel-timing subnode expresses
> those timings as specified in the timing subnode section of the display
> timing bindings defined in
> -  Documentation/devicetree/bindings/display/display-timing.txt.
> +  Documentation/devicetree/bindings/display/panel/display-timing.txt.

My bad.

Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

> 
> 
>  Connectivity

-- 
Regards,

Laurent Pinchart

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [PATCH 2/2] dt/bindings: display: move rotation into a common place

[Noralf Trønnes]

Den 16.05.2017 10.19, skrev Baruch Siach:
> Commit b60c1be74741 (dt-bindings: display/panel: Add common rotation property)
> added the rotation property description in a new file. We have a place for
> common display panel properties already. Move there the rotation property.
>
> Cc: Noralf Trønnes <noralf-L59+Z2yzLopAfugRpC6u6w@public.gmane.org>
> Cc: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
> Signed-off-by: Baruch Siach <baruch-NswTu9S1W3P6gbPvEgmw2w@public.gmane.org>
> ---

Acked-by: Noralf Trønnes <noralf-L59+Z2yzLopAfugRpC6u6w@public.gmane.org>

>   Documentation/devicetree/bindings/display/panel/panel-common.txt | 6 ++++++
>   Documentation/devicetree/bindings/display/panel/panel.txt        | 4 ----
>   2 files changed, 6 insertions(+), 4 deletions(-)
>   delete mode 100644 Documentation/devicetree/bindings/display/panel/panel.txt
>
> diff --git a/Documentation/devicetree/bindings/display/panel/panel-common.txt b/Documentation/devicetree/bindings/display/panel/panel-common.txt
> index 0603af877155..c92dcfd8c32f 100644
> --- a/Documentation/devicetree/bindings/display/panel/panel-common.txt
> +++ b/Documentation/devicetree/bindings/display/panel/panel-common.txt
> @@ -89,3 +89,9 @@ backlight controller.
>   
>   - backlight: For panels whose backlight is controlled by an external backlight
>     controller, this property contains a phandle that references the controller.
> +
> +
> +Rotation
> +--------
> +
> +- rotation: Display rotation in degrees counter clockwise (0,90,180,270)
> diff --git a/Documentation/devicetree/bindings/display/panel/panel.txt b/Documentation/devicetree/bindings/display/panel/panel.txt
> deleted file mode 100644
> index e2e6867852b8..000000000000
> --- a/Documentation/devicetree/bindings/display/panel/panel.txt
> +++ /dev/null
> @@ -1,4 +0,0 @@
> -Common display properties
> --------------------------
> -
> -- rotation:	Display rotation in degrees counter clockwise (0,90,180,270)

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 1/2] dt/bindings: display: fix display-timing reference

[Rob Herring]

On Tue, May 16, 2017 at 11:19:04AM +0300, Baruch Siach wrote:
> Fixes: 14da3ed8dd08 ("devicetree/bindings: display: Document common panel properties")
> Cc: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
> Cc: Rob Herring <robh@kernel.org>
> Signed-off-by: Baruch Siach <baruch@tkos.co.il>
> ---
>  Documentation/devicetree/bindings/display/panel/panel-common.txt | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Acked-by: Rob Herring <robh@kernel.org>
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [PATCH 2/2] dt/bindings: display: move rotation into a common place

[Rob Herring]

On Tue, May 16, 2017 at 11:19:05AM +0300, Baruch Siach wrote:
> Commit b60c1be74741 (dt-bindings: display/panel: Add common rotation property)
> added the rotation property description in a new file. We have a place for
> common display panel properties already. Move there the rotation property.
> 
> Cc: Noralf Trønnes <noralf@tronnes.org>
> Cc: Rob Herring <robh@kernel.org>
> Signed-off-by: Baruch Siach <baruch@tkos.co.il>
> ---
>  Documentation/devicetree/bindings/display/panel/panel-common.txt | 6 ++++++
>  Documentation/devicetree/bindings/display/panel/panel.txt        | 4 ----
>  2 files changed, 6 insertions(+), 4 deletions(-)
>  delete mode 100644 Documentation/devicetree/bindings/display/panel/panel.txt

Acked-by: Rob Herring <robh@kernel.org>
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [PATCH 1/2] dt/bindings: display: fix display-timing reference

[Daniel Vetter]

On Tue, May 23, 2017 at 09:47:56AM -0500, Rob Herring wrote:
> On Tue, May 16, 2017 at 11:19:04AM +0300, Baruch Siach wrote:
> > Fixes: 14da3ed8dd08 ("devicetree/bindings: display: Document common panel properties")
> > Cc: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
> > Cc: Rob Herring <robh@kernel.org>
> > Signed-off-by: Baruch Siach <baruch@tkos.co.il>
> > ---
> >  Documentation/devicetree/bindings/display/panel/panel-common.txt | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> Acked-by: Rob Herring <robh@kernel.org>

We now have a pile of acks on these patches, but no one to push them.
Seems like it's for the panel tree, but Thierry isn't on cc afaics.

Someone needs to take care of this (and no it's no going to be me).
-Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [PATCH 1/2] dt/bindings: display: fix display-timing reference

[Baruch Siach]

Hi Daniel,

On Wed, May 24, 2017 at 02:23:22PM +0200, Daniel Vetter wrote:
> On Tue, May 23, 2017 at 09:47:56AM -0500, Rob Herring wrote:
> > On Tue, May 16, 2017 at 11:19:04AM +0300, Baruch Siach wrote:
> > > Fixes: 14da3ed8dd08 ("devicetree/bindings: display: Document common panel properties")
> > > Cc: Laurent Pinchart <laurent.pinchart+renesas-ryLnwIuWjnjg/C1BVhZhaw@public.gmane.org>
> > > Cc: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
> > > Signed-off-by: Baruch Siach <baruch-NswTu9S1W3P6gbPvEgmw2w@public.gmane.org>
> > > ---
> > >  Documentation/devicetree/bindings/display/panel/panel-common.txt | 2 +-
> > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > Acked-by: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
> 
> We now have a pile of acks on these patches, but no one to push them.
> Seems like it's for the panel tree, but Thierry isn't on cc afaics.
> 
> Someone needs to take care of this (and no it's no going to be me).

Thanks for looking into this.

Just for the record, Thierry has been on To/Cc of all messages in this thread, 
accept yours.

baruch

-- 
     http://baruch.siach.name/blog/                  ~. .~   Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
   - baruch-NswTu9S1W3P6gbPvEgmw2w@public.gmane.org - tel: +972.2.679.5364, http://www.tkos.co.il -
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 1/2] dt/bindings: display: fix display-timing reference

[Daniel Vetter]

On Wed, May 24, 2017 at 03:37:36PM +0300, Baruch Siach wrote:
> Hi Daniel,
> 
> On Wed, May 24, 2017 at 02:23:22PM +0200, Daniel Vetter wrote:
> > On Tue, May 23, 2017 at 09:47:56AM -0500, Rob Herring wrote:
> > > On Tue, May 16, 2017 at 11:19:04AM +0300, Baruch Siach wrote:
> > > > Fixes: 14da3ed8dd08 ("devicetree/bindings: display: Document common panel properties")
> > > > Cc: Laurent Pinchart <laurent.pinchart+renesas-ryLnwIuWjnjg/C1BVhZhaw@public.gmane.org>
> > > > Cc: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
> > > > Signed-off-by: Baruch Siach <baruch-NswTu9S1W3P6gbPvEgmw2w@public.gmane.org>
> > > > ---
> > > >  Documentation/devicetree/bindings/display/panel/panel-common.txt | 2 +-
> > > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > > 
> > > Acked-by: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
> > 
> > We now have a pile of acks on these patches, but no one to push them.
> > Seems like it's for the panel tree, but Thierry isn't on cc afaics.
> > 
> > Someone needs to take care of this (and no it's no going to be me).
> 
> Thanks for looking into this.
> 
> Just for the record, Thierry has been on To/Cc of all messages in this thread, 
> accept yours.

Ah, that might be his dri-devel settings, there's a mailman option to
remove the cc if you're subscribed. And I got the mails through the m-l.

I think this wasn't the first time this happened either ...
-Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 1/2] dt/bindings: display: fix display-timing reference

[Thierry Reding]

[-- Attachment #1: Type: text/plain, Size: 1782 bytes --]

On Wed, May 24, 2017 at 05:51:31PM +0200, Daniel Vetter wrote:
> On Wed, May 24, 2017 at 03:37:36PM +0300, Baruch Siach wrote:
> > Hi Daniel,
> > 
> > On Wed, May 24, 2017 at 02:23:22PM +0200, Daniel Vetter wrote:
> > > On Tue, May 23, 2017 at 09:47:56AM -0500, Rob Herring wrote:
> > > > On Tue, May 16, 2017 at 11:19:04AM +0300, Baruch Siach wrote:
> > > > > Fixes: 14da3ed8dd08 ("devicetree/bindings: display: Document common panel properties")
> > > > > Cc: Laurent Pinchart <laurent.pinchart+renesas-ryLnwIuWjnjg/C1BVhZhaw@public.gmane.org>
> > > > > Cc: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
> > > > > Signed-off-by: Baruch Siach <baruch-NswTu9S1W3P6gbPvEgmw2w@public.gmane.org>
> > > > > ---
> > > > >  Documentation/devicetree/bindings/display/panel/panel-common.txt | 2 +-
> > > > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > > > 
> > > > Acked-by: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
> > > 
> > > We now have a pile of acks on these patches, but no one to push them.
> > > Seems like it's for the panel tree, but Thierry isn't on cc afaics.
> > > 
> > > Someone needs to take care of this (and no it's no going to be me).
> > 
> > Thanks for looking into this.
> > 
> > Just for the record, Thierry has been on To/Cc of all messages in this thread, 
> > accept yours.
> 
> Ah, that might be his dri-devel settings, there's a mailman option to
> remove the cc if you're subscribed. And I got the mails through the m-l.
> 
> I think this wasn't the first time this happened either ...

Sorry, I've been busy moving house for the past few weeks. And yes,
indeed, this seems to have been the mailmain option. I've switched that
around now, so hopefully this won't happen again.

Thierry

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]