* [PATCH] dt-bindings: drm: rcar-du: Document optional reset properties
@ 2017-03-03 13:30 Geert Uytterhoeven
2017-03-03 14:41 ` Laurent Pinchart
0 siblings, 1 reply; 7+ messages in thread
From: Geert Uytterhoeven @ 2017-03-03 13:30 UTC (permalink / raw)
To: Laurent Pinchart, David Airlie
Cc: Rob Herring, Mark Rutland, dri-devel, linux-renesas-soc,
devicetree, Geert Uytterhoeven
Document the optional properties for describing module resets, to
support resetting display channels and LVDS encoders on R-Car Gen2 and
Gen3.
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
Documentation/devicetree/bindings/display/renesas,du.txt | 10 ++++++++++
1 file changed, 10 insertions(+)
diff --git a/Documentation/devicetree/bindings/display/renesas,du.txt b/Documentation/devicetree/bindings/display/renesas,du.txt
index 1a02f099a0ff0a3a..7873d53764a297c2 100644
--- a/Documentation/devicetree/bindings/display/renesas,du.txt
+++ b/Documentation/devicetree/bindings/display/renesas,du.txt
@@ -36,6 +36,16 @@ Required Properties:
When supplied they must be named "dclkin.x" with "x" being the input
clock numerical index.
+Optional properties:
+ - resets: A list of phandles + reset-specifier pairs, one for each entry in
+ the reset-names property.
+ - reset-names: Names of the resets. This property is model-dependent.
+ - R8A779[0123456] use one reset for a group of one or more successive
+ channels, and one reset per LVDS encoder (if available). The resets
+ must be named "du.x" with "x" being the numerical index of the lowest
+ channel in the group. The LVDS resets must be named "lvds.x" with "x"
+ being the LVDS encoder numerical index.
+
Required nodes:
The connections to the DU output video ports are modeled using the OF graph
--
2.7.4
^ permalink raw reply related [flat|nested] 7+ messages in thread* Re: [PATCH] dt-bindings: drm: rcar-du: Document optional reset properties
2017-03-03 13:30 [PATCH] dt-bindings: drm: rcar-du: Document optional reset properties Geert Uytterhoeven
@ 2017-03-03 14:41 ` Laurent Pinchart
2017-03-03 19:03 ` Geert Uytterhoeven
0 siblings, 1 reply; 7+ messages in thread
From: Laurent Pinchart @ 2017-03-03 14:41 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Mark Rutland, devicetree, dri-devel, linux-renesas-soc,
Rob Herring
Hi Geert,
Thank you for the patch.
On Friday 03 Mar 2017 14:30:35 Geert Uytterhoeven wrote:
> Document the optional properties for describing module resets, to
> support resetting display channels and LVDS encoders on R-Car Gen2 and
> Gen3.
>
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> ---
> Documentation/devicetree/bindings/display/renesas,du.txt | 10 ++++++++++
> 1 file changed, 10 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/display/renesas,du.txt
> b/Documentation/devicetree/bindings/display/renesas,du.txt index
> 1a02f099a0ff0a3a..7873d53764a297c2 100644
> --- a/Documentation/devicetree/bindings/display/renesas,du.txt
> +++ b/Documentation/devicetree/bindings/display/renesas,du.txt
> @@ -36,6 +36,16 @@ Required Properties:
> When supplied they must be named "dclkin.x" with "x" being the input
> clock numerical index.
>
> +Optional properties:
> + - resets: A list of phandles + reset-specifier pairs, one for each entry
> in
s/phandlers/phandle/
> + the reset-names property.
What's in the reset specifier ?
> + - reset-names: Names of the resets. This property is model-dependent.
> + - R8A779[0123456] use one reset for a group of one or more successive
> + channels, and one reset per LVDS encoder (if available). The resets
> + must be named "du.x" with "x" being the numerical index of the lowest
> + channel in the group. The LVDS resets must be named "lvds.x" with
> "x" + being the LVDS encoder numerical index.
> +
> Required nodes:
>
> The connections to the DU output video ports are modeled using the OF graph
--
Regards,
Laurent Pinchart
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] dt-bindings: drm: rcar-du: Document optional reset properties
2017-03-03 14:41 ` Laurent Pinchart
@ 2017-03-03 19:03 ` Geert Uytterhoeven
2017-03-03 19:04 ` Geert Uytterhoeven
2017-03-03 19:17 ` Laurent Pinchart
0 siblings, 2 replies; 7+ messages in thread
From: Geert Uytterhoeven @ 2017-03-03 19:03 UTC (permalink / raw)
To: Laurent Pinchart
Cc: Mark Rutland, devicetree@vger.kernel.org, Geert Uytterhoeven,
DRI Development, Linux-Renesas, Rob Herring
Hi Laurent,
On Fri, Mar 3, 2017 at 3:41 PM, Laurent Pinchart
<laurent.pinchart@ideasonboard.com> wrote:
> On Friday 03 Mar 2017 14:30:35 Geert Uytterhoeven wrote:
>> Document the optional properties for describing module resets, to
>> support resetting display channels and LVDS encoders on R-Car Gen2 and
>> Gen3.
>>
>> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
>> ---
>> Documentation/devicetree/bindings/display/renesas,du.txt | 10 ++++++++++
>> 1 file changed, 10 insertions(+)
>>
>> diff --git a/Documentation/devicetree/bindings/display/renesas,du.txt
>> b/Documentation/devicetree/bindings/display/renesas,du.txt index
>> 1a02f099a0ff0a3a..7873d53764a297c2 100644
>> --- a/Documentation/devicetree/bindings/display/renesas,du.txt
>> +++ b/Documentation/devicetree/bindings/display/renesas,du.txt
>> @@ -36,6 +36,16 @@ Required Properties:
>> When supplied they must be named "dclkin.x" with "x" being the input
>> clock numerical index.
>>
>> +Optional properties:
>> + - resets: A list of phandles + reset-specifier pairs, one for each entry
>> in
>
> s/phandlers/phandle/
You're seeing typos that do not exist ;-)
>> + the reset-names property.
>
> What's in the reset specifier ?
That depends on the reset provider.
See "[v2,1/4] dt-bindings: clock: renesas: cpg-mssr: Document reset control
support" (https://patchwork.kernel.org/patch/9536627/) for the Renesas
CPG/MSSR case.
E.g. "resets = <&cpg 310>;"
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH] dt-bindings: drm: rcar-du: Document optional reset properties
2017-03-03 19:03 ` Geert Uytterhoeven
@ 2017-03-03 19:04 ` Geert Uytterhoeven
2017-03-03 19:15 ` Laurent Pinchart
2017-03-03 19:17 ` Laurent Pinchart
1 sibling, 1 reply; 7+ messages in thread
From: Geert Uytterhoeven @ 2017-03-03 19:04 UTC (permalink / raw)
To: Laurent Pinchart
Cc: Mark Rutland, devicetree@vger.kernel.org, Geert Uytterhoeven,
DRI Development, Linux-Renesas, Rob Herring
On Fri, Mar 3, 2017 at 8:03 PM, Geert Uytterhoeven <geert@linux-m68k.org> wrote:
>>> +Optional properties:
>>> + - resets: A list of phandles + reset-specifier pairs, one for each entry
>>> in
>>
>> s/phandlers/phandle/
>
> You're seeing typos that do not exist ;-)
Ah, you mean plural vs. singular?
You're right, but I just copied that from the "clocks" description...
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH] dt-bindings: drm: rcar-du: Document optional reset properties
2017-03-03 19:04 ` Geert Uytterhoeven
@ 2017-03-03 19:15 ` Laurent Pinchart
0 siblings, 0 replies; 7+ messages in thread
From: Laurent Pinchart @ 2017-03-03 19:15 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Mark Rutland, devicetree@vger.kernel.org, Geert Uytterhoeven,
DRI Development, Linux-Renesas, Rob Herring
Hi Geert,
On Friday 03 Mar 2017 20:04:26 Geert Uytterhoeven wrote:
> On Fri, Mar 3, 2017 at 8:03 PM, Geert Uytterhoeven wrote:
> >>> +Optional properties:
> >>> + - resets: A list of phandles + reset-specifier pairs, one for each
> >>> entry in
> >>
> >> s/phandlers/phandle/
> >
> > You're seeing typos that do not exist ;-)
>
> Ah, you mean plural vs. singular?
Yes that's what I meant.
> You're right, but I just copied that from the "clocks" description...
Oops :-)
--
Regards,
Laurent Pinchart
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] dt-bindings: drm: rcar-du: Document optional reset properties
2017-03-03 19:03 ` Geert Uytterhoeven
2017-03-03 19:04 ` Geert Uytterhoeven
@ 2017-03-03 19:17 ` Laurent Pinchart
2017-03-03 19:23 ` Geert Uytterhoeven
1 sibling, 1 reply; 7+ messages in thread
From: Laurent Pinchart @ 2017-03-03 19:17 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Mark Rutland, devicetree@vger.kernel.org, Geert Uytterhoeven,
DRI Development, Linux-Renesas, Rob Herring
Hi Geert,
On Friday 03 Mar 2017 20:03:09 Geert Uytterhoeven wrote:
> On Fri, Mar 3, 2017 at 3:41 PM, Laurent Pinchart wrote:
> > On Friday 03 Mar 2017 14:30:35 Geert Uytterhoeven wrote:
> >> Document the optional properties for describing module resets, to
> >> support resetting display channels and LVDS encoders on R-Car Gen2 and
> >> Gen3.
> >>
> >> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> >> ---
> >>
> >> Documentation/devicetree/bindings/display/renesas,du.txt | 10 ++++++++++
> >> 1 file changed, 10 insertions(+)
> >>
> >> diff --git a/Documentation/devicetree/bindings/display/renesas,du.txt
> >> b/Documentation/devicetree/bindings/display/renesas,du.txt index
> >> 1a02f099a0ff0a3a..7873d53764a297c2 100644
> >> --- a/Documentation/devicetree/bindings/display/renesas,du.txt
> >> +++ b/Documentation/devicetree/bindings/display/renesas,du.txt
> >>
> >> @@ -36,6 +36,16 @@ Required Properties:
> >> When supplied they must be named "dclkin.x" with "x" being the
> >> input clock numerical index.
> >>
> >> +Optional properties:
> >> + - resets: A list of phandles + reset-specifier pairs, one for each
> >> entry in
> >
> > s/phandlers/phandle/
>
> You're seeing typos that do not exist ;-)
>
> >> + the reset-names property.
> >
> > What's in the reset specifier ?
>
> That depends on the reset provider.
>
> See "[v2,1/4] dt-bindings: clock: renesas: cpg-mssr: Document reset control
> support" (https://patchwork.kernel.org/patch/9536627/) for the Renesas
> CPG/MSSR case.
>
> E.g. "resets = <&cpg 310>;"
Thanks. I never know whether the specified is the combination of phandle +
data, or is only the cells following the phandle.
--
Regards,
Laurent Pinchart
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] dt-bindings: drm: rcar-du: Document optional reset properties
2017-03-03 19:17 ` Laurent Pinchart
@ 2017-03-03 19:23 ` Geert Uytterhoeven
0 siblings, 0 replies; 7+ messages in thread
From: Geert Uytterhoeven @ 2017-03-03 19:23 UTC (permalink / raw)
To: Laurent Pinchart
Cc: Mark Rutland, devicetree@vger.kernel.org, Geert Uytterhoeven,
DRI Development, Linux-Renesas, Rob Herring
Hi Laurent,
On Fri, Mar 3, 2017 at 8:17 PM, Laurent Pinchart
<laurent.pinchart@ideasonboard.com> wrote:
> On Friday 03 Mar 2017 20:03:09 Geert Uytterhoeven wrote:
>> On Fri, Mar 3, 2017 at 3:41 PM, Laurent Pinchart wrote:
>> > What's in the reset specifier ?
>>
>> That depends on the reset provider.
>>
>> See "[v2,1/4] dt-bindings: clock: renesas: cpg-mssr: Document reset control
>> support" (https://patchwork.kernel.org/patch/9536627/) for the Renesas
>> CPG/MSSR case.
>>
>> E.g. "resets = <&cpg 310>;"
>
> Thanks. I never know whether the specified is the combination of phandle +
> data, or is only the cells following the phandle.
Trick to remember: the #<foo>-cells property tells how many cells there
are in the specifier.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2017-03-03 19:23 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-03-03 13:30 [PATCH] dt-bindings: drm: rcar-du: Document optional reset properties Geert Uytterhoeven
2017-03-03 14:41 ` Laurent Pinchart
2017-03-03 19:03 ` Geert Uytterhoeven
2017-03-03 19:04 ` Geert Uytterhoeven
2017-03-03 19:15 ` Laurent Pinchart
2017-03-03 19:17 ` Laurent Pinchart
2017-03-03 19:23 ` Geert Uytterhoeven
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).