diff for duplicates of <1457362392.2932.49.camel@synopsys.com> diff --git a/a/1.txt b/N1/1.txt index a8fabd7..b2cb702 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,33 +1,33 @@ Hi Rob, -On Fri, 2016-03-04@22:30 -0600, Rob Herring wrote: -> On Thu, Mar 03, 2016@05:39:14PM +0300, Alexey Brodkin wrote: +On Fri, 2016-03-04 at 22:30 -0600, Rob Herring wrote: +> On Thu, Mar 03, 2016 at 05:39:14PM +0300, Alexey Brodkin wrote: > > > > This add DT bindings documentation for ARC PGU display controller. > > -> > Signed-off-by: Alexey Brodkin <abrodkin at synopsys.com> -> > Cc: Rob Herring <robh+dt at kernel.org> -> > Cc: Pawel Moll <pawel.moll at arm.com> -> > Cc: Mark Rutland <mark.rutland at arm.com> -> > Cc: Ian Campbell <ijc+devicetree at hellion.org.uk> -> > Cc: Kumar Gala <galak at codeaurora.org> -> > Cc: devicetree at vger.kernel.org -> > Cc: linux-snps-arc at lists.infradead.org +> > Signed-off-by: Alexey Brodkin <abrodkin@synopsys.com> +> > Cc: Rob Herring <robh+dt@kernel.org> +> > Cc: Pawel Moll <pawel.moll@arm.com> +> > Cc: Mark Rutland <mark.rutland@arm.com> +> > Cc: Ian Campbell <ijc+devicetree@hellion.org.uk> +> > Cc: Kumar Gala <galak@codeaurora.org> +> > Cc: devicetree@vger.kernel.org +> > Cc: linux-snps-arc@lists.infradead.org > > --- > > > > Changes v1 -> v2: -> > ?* Clean-up -> Not really useful. What we like to see is what changed. Maintainers have? -> short memories and don't remember what they said previously (unless? +> > * Clean-up +> Not really useful. What we like to see is what changed. Maintainers have +> short memories and don't remember what they said previously (unless > comments are ignored). That's understood :) > > > > -> > ?.../devicetree/bindings/display/snps,arcpgu.txt????| 33 ++++++++++++++++++++++ -> > ?1 file changed, 33 insertions(+) -> > ?create mode 100644 Documentation/devicetree/bindings/display/snps,arcpgu.txt +> > .../devicetree/bindings/display/snps,arcpgu.txt | 33 ++++++++++++++++++++++ +> > 1 file changed, 33 insertions(+) +> > create mode 100644 Documentation/devicetree/bindings/display/snps,arcpgu.txt > > > > diff --git a/Documentation/devicetree/bindings/display/snps,arcpgu.txt > > b/Documentation/devicetree/bindings/display/snps,arcpgu.txt @@ -43,24 +43,24 @@ That's understood :) > > +framebuffer and sends it to a single digital encoder (usually HDMI). > > + > > +Required properties: -> > +??- compatible: "snps,arcpgu" -> > +??- reg: Physical base address and length of the controller's registers. -> > +??- clocks: A list of phandle + clock-specifier pairs, one for each -> > +????entry in 'clock-names'. -> > +??- clock-names: A list of clock names. For ARC PGU it should contain: -> > +??????- "pxlclk" for the clock feeding the output PLL of the controller. +> > + - compatible: "snps,arcpgu" +> > + - reg: Physical base address and length of the controller's registers. +> > + - clocks: A list of phandle + clock-specifier pairs, one for each +> > + entry in 'clock-names'. +> > + - clock-names: A list of clock names. For ARC PGU it should contain: +> > + - "pxlclk" for the clock feeding the output PLL of the controller. > > + > > +Required sub-nodes: -> > +??- port: The PGU connection to an encoder chip. The connection is modelled -> > +????using the OF graph bindings specified in -> > +????Documentation/devicetree/bindings/graph.txt. +> > + - port: The PGU connection to an encoder chip. The connection is modelled +> > + using the OF graph bindings specified in +> > + Documentation/devicetree/bindings/graph.txt. > > + > > +Example: > > + > > +/ { > > + ... > > + -> > + pgu at XXXXXXXX { +> > + pgu@XXXXXXXX { > > + compatible = "snps,arcpgu"; > > + reg = <0xXXXXXXXX 0x400>; > > + clocks = <&clock_node>; @@ -76,4 +76,7 @@ In particular "Required subnodes" section that still lists "port". And frankly now I'm a bit lost with what should I put in that binding description and what should not. Any comments here are much appreciated. --Alexey +-Alexey-- +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 diff --git a/a/content_digest b/N1/content_digest index 9aba66f..1beb01b 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,42 +1,49 @@ "ref\01457015956-30708-1-git-send-email-abrodkin@synopsys.com\0" "ref\01457015956-30708-3-git-send-email-abrodkin@synopsys.com\0" "ref\020160305043001.GB13525@rob-hp-laptop\0" - "From\0Alexey.Brodkin@synopsys.com (Alexey Brodkin)\0" - "Subject\0[PATCH 2/4 v2] drm: Add DT bindings documentation for ARC PGU display controller\0" + "From\0Alexey Brodkin <Alexey.Brodkin-HKixBCOQz3hWk0Htik3J/w@public.gmane.org>\0" + "Subject\0Re: [PATCH 2/4 v2] drm: Add DT bindings documentation for ARC PGU display controller\0" "Date\0Mon, 7 Mar 2016 14:53:12 +0000\0" - "To\0linux-snps-arc@lists.infradead.org\0" + "To\0robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>\0" + "Cc\0ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org <ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org>" + linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org <linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org> + mark.rutland-5wv7dgnIgG8@public.gmane.org <mark.rutland-5wv7dgnIgG8@public.gmane.org> + galak-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org <galak-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org> + pawel.moll-5wv7dgnIgG8@public.gmane.org <pawel.moll-5wv7dgnIgG8@public.gmane.org> + linux-snps-arc-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org <linux-snps-arc-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org> + " devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org <devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>\0" "\00:1\0" "b\0" "Hi Rob,\n" "\n" - "On Fri, 2016-03-04@22:30 -0600, Rob Herring wrote:\n" - "> On Thu, Mar 03, 2016@05:39:14PM +0300, Alexey Brodkin wrote:\n" + "On Fri, 2016-03-04 at 22:30 -0600, Rob Herring wrote:\n" + "> On Thu, Mar 03, 2016 at 05:39:14PM +0300, Alexey Brodkin wrote:\n" "> > \n" "> > This add DT bindings documentation for ARC PGU display controller.\n" "> > \n" - "> > Signed-off-by: Alexey Brodkin <abrodkin at synopsys.com>\n" - "> > Cc: Rob Herring <robh+dt at kernel.org>\n" - "> > Cc: Pawel Moll <pawel.moll at arm.com>\n" - "> > Cc: Mark Rutland <mark.rutland at arm.com>\n" - "> > Cc: Ian Campbell <ijc+devicetree at hellion.org.uk>\n" - "> > Cc: Kumar Gala <galak at codeaurora.org>\n" - "> > Cc: devicetree at vger.kernel.org\n" - "> > Cc: linux-snps-arc at lists.infradead.org\n" + "> > Signed-off-by: Alexey Brodkin <abrodkin@synopsys.com>\n" + "> > Cc: Rob Herring <robh+dt@kernel.org>\n" + "> > Cc: Pawel Moll <pawel.moll@arm.com>\n" + "> > Cc: Mark Rutland <mark.rutland@arm.com>\n" + "> > Cc: Ian Campbell <ijc+devicetree@hellion.org.uk>\n" + "> > Cc: Kumar Gala <galak@codeaurora.org>\n" + "> > Cc: devicetree@vger.kernel.org\n" + "> > Cc: linux-snps-arc@lists.infradead.org\n" "> > ---\n" "> > \n" "> > Changes v1 -> v2:\n" - "> > ?* Clean-up\n" - "> Not really useful. What we like to see is what changed. Maintainers have?\n" - "> short memories and don't remember what they said previously (unless?\n" + "> > \302\240* Clean-up\n" + "> Not really useful. What we like to see is what changed. Maintainers have\302\240\n" + "> short memories and don't remember what they said previously (unless\302\240\n" "> comments are ignored).\n" "\n" "That's understood :)\n" "\n" "> > \n" "> > \n" - "> > ?.../devicetree/bindings/display/snps,arcpgu.txt????| 33 ++++++++++++++++++++++\n" - "> > ?1 file changed, 33 insertions(+)\n" - "> > ?create mode 100644 Documentation/devicetree/bindings/display/snps,arcpgu.txt\n" + "> > \302\240.../devicetree/bindings/display/snps,arcpgu.txt\302\240\302\240\302\240\302\240| 33 ++++++++++++++++++++++\n" + "> > \302\2401 file changed, 33 insertions(+)\n" + "> > \302\240create mode 100644 Documentation/devicetree/bindings/display/snps,arcpgu.txt\n" "> > \n" "> > diff --git a/Documentation/devicetree/bindings/display/snps,arcpgu.txt\n" "> > b/Documentation/devicetree/bindings/display/snps,arcpgu.txt\n" @@ -52,24 +59,24 @@ "> > +framebuffer and sends it to a single digital encoder (usually HDMI).\n" "> > +\n" "> > +Required properties:\n" - "> > +??- compatible: \"snps,arcpgu\"\n" - "> > +??- reg: Physical base address and length of the controller's registers.\n" - "> > +??- clocks: A list of phandle + clock-specifier pairs, one for each\n" - "> > +????entry in 'clock-names'.\n" - "> > +??- clock-names: A list of clock names. For ARC PGU it should contain:\n" - "> > +??????- \"pxlclk\" for the clock feeding the output PLL of the controller.\n" + "> > +\302\240\302\240- compatible: \"snps,arcpgu\"\n" + "> > +\302\240\302\240- reg: Physical base address and length of the controller's registers.\n" + "> > +\302\240\302\240- clocks: A list of phandle + clock-specifier pairs, one for each\n" + "> > +\302\240\302\240\302\240\302\240entry in 'clock-names'.\n" + "> > +\302\240\302\240- clock-names: A list of clock names. For ARC PGU it should contain:\n" + "> > +\302\240\302\240\302\240\302\240\302\240\302\240- \"pxlclk\" for the clock feeding the output PLL of the controller.\n" "> > +\n" "> > +Required sub-nodes:\n" - "> > +??- port: The PGU connection to an encoder chip. The connection is modelled\n" - "> > +????using the OF graph bindings specified in\n" - "> > +????Documentation/devicetree/bindings/graph.txt.\n" + "> > +\302\240\302\240- port: The PGU connection to an encoder chip. The connection is modelled\n" + "> > +\302\240\302\240\302\240\302\240using the OF graph bindings specified in\n" + "> > +\302\240\302\240\302\240\302\240Documentation/devicetree/bindings/graph.txt.\n" "> > +\n" "> > +Example:\n" "> > +\n" "> > +/ {\n" "> > +\t...\n" "> > +\n" - "> > +\tpgu at XXXXXXXX {\n" + "> > +\tpgu@XXXXXXXX {\n" "> > +\t\tcompatible = \"snps,arcpgu\";\n" "> > +\t\treg = <0xXXXXXXXX 0x400>;\n" "> > +\t\tclocks = <&clock_node>;\n" @@ -85,6 +92,9 @@ "And frankly now I'm a bit lost with what should I put in that binding\n" "description and what should not. Any comments here are much appreciated.\n" "\n" - -Alexey + "-Alexey--\n" + "To unsubscribe from this list: send the line \"unsubscribe devicetree\" in\n" + "the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org\n" + More majordomo info at http://vger.kernel.org/majordomo-info.html -accc3f0162e78f179e16532f65160d2790e4c01213f7eccf8ce17ab26f86faf3 +914e3f703319b4955c090cfd2340983b9e185bcaea81d442c8e317e18e521c3c
diff --git a/a/1.txt b/N2/1.txt index a8fabd7..15b42af 100644 --- a/a/1.txt +++ b/N2/1.txt @@ -1,33 +1,33 @@ Hi Rob, -On Fri, 2016-03-04@22:30 -0600, Rob Herring wrote: -> On Thu, Mar 03, 2016@05:39:14PM +0300, Alexey Brodkin wrote: +On Fri, 2016-03-04 at 22:30 -0600, Rob Herring wrote: +> On Thu, Mar 03, 2016 at 05:39:14PM +0300, Alexey Brodkin wrote: > > > > This add DT bindings documentation for ARC PGU display controller. > > -> > Signed-off-by: Alexey Brodkin <abrodkin at synopsys.com> -> > Cc: Rob Herring <robh+dt at kernel.org> -> > Cc: Pawel Moll <pawel.moll at arm.com> -> > Cc: Mark Rutland <mark.rutland at arm.com> -> > Cc: Ian Campbell <ijc+devicetree at hellion.org.uk> -> > Cc: Kumar Gala <galak at codeaurora.org> -> > Cc: devicetree at vger.kernel.org -> > Cc: linux-snps-arc at lists.infradead.org +> > Signed-off-by: Alexey Brodkin <abrodkin@synopsys.com> +> > Cc: Rob Herring <robh+dt@kernel.org> +> > Cc: Pawel Moll <pawel.moll@arm.com> +> > Cc: Mark Rutland <mark.rutland@arm.com> +> > Cc: Ian Campbell <ijc+devicetree@hellion.org.uk> +> > Cc: Kumar Gala <galak@codeaurora.org> +> > Cc: devicetree@vger.kernel.org +> > Cc: linux-snps-arc@lists.infradead.org > > --- > > > > Changes v1 -> v2: -> > ?* Clean-up -> Not really useful. What we like to see is what changed. Maintainers have? -> short memories and don't remember what they said previously (unless? +> > * Clean-up +> Not really useful. What we like to see is what changed. Maintainers have +> short memories and don't remember what they said previously (unless > comments are ignored). That's understood :) > > > > -> > ?.../devicetree/bindings/display/snps,arcpgu.txt????| 33 ++++++++++++++++++++++ -> > ?1 file changed, 33 insertions(+) -> > ?create mode 100644 Documentation/devicetree/bindings/display/snps,arcpgu.txt +> > .../devicetree/bindings/display/snps,arcpgu.txt | 33 ++++++++++++++++++++++ +> > 1 file changed, 33 insertions(+) +> > create mode 100644 Documentation/devicetree/bindings/display/snps,arcpgu.txt > > > > diff --git a/Documentation/devicetree/bindings/display/snps,arcpgu.txt > > b/Documentation/devicetree/bindings/display/snps,arcpgu.txt @@ -43,24 +43,24 @@ That's understood :) > > +framebuffer and sends it to a single digital encoder (usually HDMI). > > + > > +Required properties: -> > +??- compatible: "snps,arcpgu" -> > +??- reg: Physical base address and length of the controller's registers. -> > +??- clocks: A list of phandle + clock-specifier pairs, one for each -> > +????entry in 'clock-names'. -> > +??- clock-names: A list of clock names. For ARC PGU it should contain: -> > +??????- "pxlclk" for the clock feeding the output PLL of the controller. +> > + - compatible: "snps,arcpgu" +> > + - reg: Physical base address and length of the controller's registers. +> > + - clocks: A list of phandle + clock-specifier pairs, one for each +> > + entry in 'clock-names'. +> > + - clock-names: A list of clock names. For ARC PGU it should contain: +> > + - "pxlclk" for the clock feeding the output PLL of the controller. > > + > > +Required sub-nodes: -> > +??- port: The PGU connection to an encoder chip. The connection is modelled -> > +????using the OF graph bindings specified in -> > +????Documentation/devicetree/bindings/graph.txt. +> > + - port: The PGU connection to an encoder chip. The connection is modelled +> > + using the OF graph bindings specified in +> > + Documentation/devicetree/bindings/graph.txt. > > + > > +Example: > > + > > +/ { > > + ... > > + -> > + pgu at XXXXXXXX { +> > + pgu@XXXXXXXX { > > + compatible = "snps,arcpgu"; > > + reg = <0xXXXXXXXX 0x400>; > > + clocks = <&clock_node>; diff --git a/a/content_digest b/N2/content_digest index 9aba66f..3b694c5 100644 --- a/a/content_digest +++ b/N2/content_digest @@ -1,42 +1,49 @@ "ref\01457015956-30708-1-git-send-email-abrodkin@synopsys.com\0" "ref\01457015956-30708-3-git-send-email-abrodkin@synopsys.com\0" "ref\020160305043001.GB13525@rob-hp-laptop\0" - "From\0Alexey.Brodkin@synopsys.com (Alexey Brodkin)\0" - "Subject\0[PATCH 2/4 v2] drm: Add DT bindings documentation for ARC PGU display controller\0" + "From\0Alexey Brodkin <Alexey.Brodkin@synopsys.com>\0" + "Subject\0Re: [PATCH 2/4 v2] drm: Add DT bindings documentation for ARC PGU display controller\0" "Date\0Mon, 7 Mar 2016 14:53:12 +0000\0" - "To\0linux-snps-arc@lists.infradead.org\0" + "To\0robh@kernel.org <robh@kernel.org>\0" + "Cc\0ijc+devicetree@hellion.org.uk <ijc+devicetree@hellion.org.uk>" + linux-kernel@vger.kernel.org <linux-kernel@vger.kernel.org> + mark.rutland@arm.com <mark.rutland@arm.com> + galak@codeaurora.org <galak@codeaurora.org> + pawel.moll@arm.com <pawel.moll@arm.com> + linux-snps-arc@lists.infradead.org <linux-snps-arc@lists.infradead.org> + " devicetree@vger.kernel.org <devicetree@vger.kernel.org>\0" "\00:1\0" "b\0" "Hi Rob,\n" "\n" - "On Fri, 2016-03-04@22:30 -0600, Rob Herring wrote:\n" - "> On Thu, Mar 03, 2016@05:39:14PM +0300, Alexey Brodkin wrote:\n" + "On Fri, 2016-03-04 at 22:30 -0600, Rob Herring wrote:\n" + "> On Thu, Mar 03, 2016 at 05:39:14PM +0300, Alexey Brodkin wrote:\n" "> > \n" "> > This add DT bindings documentation for ARC PGU display controller.\n" "> > \n" - "> > Signed-off-by: Alexey Brodkin <abrodkin at synopsys.com>\n" - "> > Cc: Rob Herring <robh+dt at kernel.org>\n" - "> > Cc: Pawel Moll <pawel.moll at arm.com>\n" - "> > Cc: Mark Rutland <mark.rutland at arm.com>\n" - "> > Cc: Ian Campbell <ijc+devicetree at hellion.org.uk>\n" - "> > Cc: Kumar Gala <galak at codeaurora.org>\n" - "> > Cc: devicetree at vger.kernel.org\n" - "> > Cc: linux-snps-arc at lists.infradead.org\n" + "> > Signed-off-by: Alexey Brodkin <abrodkin@synopsys.com>\n" + "> > Cc: Rob Herring <robh+dt@kernel.org>\n" + "> > Cc: Pawel Moll <pawel.moll@arm.com>\n" + "> > Cc: Mark Rutland <mark.rutland@arm.com>\n" + "> > Cc: Ian Campbell <ijc+devicetree@hellion.org.uk>\n" + "> > Cc: Kumar Gala <galak@codeaurora.org>\n" + "> > Cc: devicetree@vger.kernel.org\n" + "> > Cc: linux-snps-arc@lists.infradead.org\n" "> > ---\n" "> > \n" "> > Changes v1 -> v2:\n" - "> > ?* Clean-up\n" - "> Not really useful. What we like to see is what changed. Maintainers have?\n" - "> short memories and don't remember what they said previously (unless?\n" + "> > \302\240* Clean-up\n" + "> Not really useful. What we like to see is what changed. Maintainers have\302\240\n" + "> short memories and don't remember what they said previously (unless\302\240\n" "> comments are ignored).\n" "\n" "That's understood :)\n" "\n" "> > \n" "> > \n" - "> > ?.../devicetree/bindings/display/snps,arcpgu.txt????| 33 ++++++++++++++++++++++\n" - "> > ?1 file changed, 33 insertions(+)\n" - "> > ?create mode 100644 Documentation/devicetree/bindings/display/snps,arcpgu.txt\n" + "> > \302\240.../devicetree/bindings/display/snps,arcpgu.txt\302\240\302\240\302\240\302\240| 33 ++++++++++++++++++++++\n" + "> > \302\2401 file changed, 33 insertions(+)\n" + "> > \302\240create mode 100644 Documentation/devicetree/bindings/display/snps,arcpgu.txt\n" "> > \n" "> > diff --git a/Documentation/devicetree/bindings/display/snps,arcpgu.txt\n" "> > b/Documentation/devicetree/bindings/display/snps,arcpgu.txt\n" @@ -52,24 +59,24 @@ "> > +framebuffer and sends it to a single digital encoder (usually HDMI).\n" "> > +\n" "> > +Required properties:\n" - "> > +??- compatible: \"snps,arcpgu\"\n" - "> > +??- reg: Physical base address and length of the controller's registers.\n" - "> > +??- clocks: A list of phandle + clock-specifier pairs, one for each\n" - "> > +????entry in 'clock-names'.\n" - "> > +??- clock-names: A list of clock names. For ARC PGU it should contain:\n" - "> > +??????- \"pxlclk\" for the clock feeding the output PLL of the controller.\n" + "> > +\302\240\302\240- compatible: \"snps,arcpgu\"\n" + "> > +\302\240\302\240- reg: Physical base address and length of the controller's registers.\n" + "> > +\302\240\302\240- clocks: A list of phandle + clock-specifier pairs, one for each\n" + "> > +\302\240\302\240\302\240\302\240entry in 'clock-names'.\n" + "> > +\302\240\302\240- clock-names: A list of clock names. For ARC PGU it should contain:\n" + "> > +\302\240\302\240\302\240\302\240\302\240\302\240- \"pxlclk\" for the clock feeding the output PLL of the controller.\n" "> > +\n" "> > +Required sub-nodes:\n" - "> > +??- port: The PGU connection to an encoder chip. The connection is modelled\n" - "> > +????using the OF graph bindings specified in\n" - "> > +????Documentation/devicetree/bindings/graph.txt.\n" + "> > +\302\240\302\240- port: The PGU connection to an encoder chip. The connection is modelled\n" + "> > +\302\240\302\240\302\240\302\240using the OF graph bindings specified in\n" + "> > +\302\240\302\240\302\240\302\240Documentation/devicetree/bindings/graph.txt.\n" "> > +\n" "> > +Example:\n" "> > +\n" "> > +/ {\n" "> > +\t...\n" "> > +\n" - "> > +\tpgu at XXXXXXXX {\n" + "> > +\tpgu@XXXXXXXX {\n" "> > +\t\tcompatible = \"snps,arcpgu\";\n" "> > +\t\treg = <0xXXXXXXXX 0x400>;\n" "> > +\t\tclocks = <&clock_node>;\n" @@ -87,4 +94,4 @@ "\n" -Alexey -accc3f0162e78f179e16532f65160d2790e4c01213f7eccf8ce17ab26f86faf3 +71eb947f05ea5dbd5ec96379ce08b133fc39b448d4cc75c3e2b4b6efad99e5fe
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.