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
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 resetsmust be named "du.x" with "x" being the numerical index of the lowestchannel 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
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
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
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 :-)
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.
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@lists.freedesktop.org
-
Geert Uytterhoeven -
Geert Uytterhoeven -
Laurent Pinchart