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@xxxxxxxxx> > >> --- > >> > >> 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 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html