diff for duplicates of <1457013497.3005.99.camel@synopsys.com> diff --git a/a/1.txt b/N1/1.txt index d70ccc3..302f0b2 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,22 +1,22 @@ Hi Rob, -On Tue, 2016-02-23@14:38 -0600, Rob Herring wrote: -> On Fri, Feb 19, 2016@04:03:52PM +0300, Alexey Brodkin wrote: +On Tue, 2016-02-23 at 14:38 -0600, Rob Herring wrote: +> On Fri, Feb 19, 2016 at 04:03:52PM +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 > > --- -> > ?.../devicetree/bindings/display/snps,arcpgu.txt????| 74 ++++++++++++++++++++++ -> > ?1 file changed, 74 insertions(+) -> > ?create mode 100644 Documentation/devicetree/bindings/display/snps,arcpgu.txt +> > .../devicetree/bindings/display/snps,arcpgu.txt | 74 ++++++++++++++++++++++ +> > 1 file changed, 74 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 @@ -32,20 +32,20 @@ On Tue, 2016-02-23@14:38 -0600, Rob Herring wrote: > > +framebuffer and sends it to a single digital encoder (usually HDMI). > > + > > +Required properties: -> > +??- compatible: "snps,arcpgu" -> Seems like this should be more specific. Is there some sort or? +> > + - compatible: "snps,arcpgu" +> Seems like this should be more specific. Is there some sort or > versioning with ARC blocks? Well as of today there's only one and only version of PGU. So is there a real need for "snps,arcpgu-1.0"? > > -> > +??- 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. -> > +??- encoder-slave: Phandle of encoder chip. +> > + - 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. +> > + - encoder-slave: Phandle of encoder chip. > This is unnecessary with the OF graph. Do you mean I may drop "encoder-slave" from bindings description? @@ -58,19 +58,23 @@ display. > > +/ { > > + ... > > + -> > + pgu at 0xXXXXXXXX { +> > + pgu@0xXXXXXXXX { > Unit address should not have '0x' Correct, let me fix it. > > + > > + /* HDMI encoder on I2C bus */ -> > + i2c at 0xXXXXXXXX { +> > + i2c@0xXXXXXXXX { > > + compatible = "..."; > > + -> > + encoder_node:encoder_node at 0xXXXXXXXX{ +> > + encoder_node:encoder_node@0xXXXXXXXX{ > I2C addresses would be 8-bit only. Indeed - copy-paste is evil :) -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 5a79281..4722ca0 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,31 +1,38 @@ "ref\01455887034-7739-1-git-send-email-abrodkin@synopsys.com\0" "ref\01455887034-7739-3-git-send-email-abrodkin@synopsys.com\0" "ref\020160223203841.GA15744@rob-hp-laptop\0" - "From\0Alexey.Brodkin@synopsys.com (Alexey Brodkin)\0" - "Subject\0[PATCH 2/4] 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] drm: Add DT bindings documentation for ARC PGU display controller\0" "Date\0Thu, 3 Mar 2016 13:58:18 +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 Tue, 2016-02-23@14:38 -0600, Rob Herring wrote:\n" - "> On Fri, Feb 19, 2016@04:03:52PM +0300, Alexey Brodkin wrote:\n" + "On Tue, 2016-02-23 at 14:38 -0600, Rob Herring wrote:\n" + "> On Fri, Feb 19, 2016 at 04:03:52PM +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" - "> > ?.../devicetree/bindings/display/snps,arcpgu.txt????| 74 ++++++++++++++++++++++\n" - "> > ?1 file changed, 74 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| 74 ++++++++++++++++++++++\n" + "> > \302\2401 file changed, 74 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" @@ -41,20 +48,20 @@ "> > +framebuffer and sends it to a single digital encoder (usually HDMI).\n" "> > +\n" "> > +Required properties:\n" - "> > +??- compatible: \"snps,arcpgu\"\n" - "> Seems like this should be more specific. Is there some sort or?\n" + "> > +\302\240\302\240- compatible: \"snps,arcpgu\"\n" + "> Seems like this should be more specific. Is there some sort or\302\240\n" "> versioning with ARC blocks?\n" "\n" "Well as of today there's only one and only version of PGU.\n" "So is there a real need for \"snps,arcpgu-1.0\"?\n" "\n" "> > \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" - "> > +??- encoder-slave: Phandle of encoder chip.\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" + "> > +\302\240\302\240- encoder-slave: Phandle of encoder chip.\n" "> This is unnecessary with the OF graph.\n" "\n" "Do you mean I may drop \"encoder-slave\" from bindings description?\n" @@ -67,21 +74,25 @@ "> > +/ {\n" "> > +\t...\n" "> > +\n" - "> > +\tpgu at 0xXXXXXXXX {\n" + "> > +\tpgu@0xXXXXXXXX {\n" "> Unit address should not have '0x'\n" "\n" "Correct, let me fix it.\n" "\n" "> > +\n" "> > +\t/* HDMI encoder on I2C bus */\n" - "> > +\ti2c at 0xXXXXXXXX {\n" + "> > +\ti2c@0xXXXXXXXX {\n" "> > +\t\tcompatible = \"...\";\n" "> > +\n" - "> > +\t\tencoder_node:encoder_node at 0xXXXXXXXX{\n" + "> > +\t\tencoder_node:encoder_node@0xXXXXXXXX{\n" "> I2C addresses would be 8-bit only.\n" "\n" "Indeed - copy-paste is evil :)\n" "\n" - -Alexey + "-Alexey\n" + "--\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 -960067393cba3d4aaa8aaeb672d5089c02060d895d48f702ef8edc20b453b5b0 +f9e156980d4c96582689b596ac09f23dabcc9a2692d2a82dfc647827d765e909
diff --git a/a/1.txt b/N2/1.txt index d70ccc3..08a7077 100644 --- a/a/1.txt +++ b/N2/1.txt @@ -1,22 +1,22 @@ Hi Rob, -On Tue, 2016-02-23@14:38 -0600, Rob Herring wrote: -> On Fri, Feb 19, 2016@04:03:52PM +0300, Alexey Brodkin wrote: +On Tue, 2016-02-23 at 14:38 -0600, Rob Herring wrote: +> On Fri, Feb 19, 2016 at 04:03:52PM +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 > > --- -> > ?.../devicetree/bindings/display/snps,arcpgu.txt????| 74 ++++++++++++++++++++++ -> > ?1 file changed, 74 insertions(+) -> > ?create mode 100644 Documentation/devicetree/bindings/display/snps,arcpgu.txt +> > .../devicetree/bindings/display/snps,arcpgu.txt | 74 ++++++++++++++++++++++ +> > 1 file changed, 74 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 @@ -32,20 +32,20 @@ On Tue, 2016-02-23@14:38 -0600, Rob Herring wrote: > > +framebuffer and sends it to a single digital encoder (usually HDMI). > > + > > +Required properties: -> > +??- compatible: "snps,arcpgu" -> Seems like this should be more specific. Is there some sort or? +> > + - compatible: "snps,arcpgu" +> Seems like this should be more specific. Is there some sort or > versioning with ARC blocks? Well as of today there's only one and only version of PGU. So is there a real need for "snps,arcpgu-1.0"? > > -> > +??- 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. -> > +??- encoder-slave: Phandle of encoder chip. +> > + - 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. +> > + - encoder-slave: Phandle of encoder chip. > This is unnecessary with the OF graph. Do you mean I may drop "encoder-slave" from bindings description? @@ -58,17 +58,17 @@ display. > > +/ { > > + ... > > + -> > + pgu at 0xXXXXXXXX { +> > + pgu@0xXXXXXXXX { > Unit address should not have '0x' Correct, let me fix it. > > + > > + /* HDMI encoder on I2C bus */ -> > + i2c at 0xXXXXXXXX { +> > + i2c@0xXXXXXXXX { > > + compatible = "..."; > > + -> > + encoder_node:encoder_node at 0xXXXXXXXX{ +> > + encoder_node:encoder_node@0xXXXXXXXX{ > I2C addresses would be 8-bit only. Indeed - copy-paste is evil :) diff --git a/a/content_digest b/N2/content_digest index 5a79281..e21c697 100644 --- a/a/content_digest +++ b/N2/content_digest @@ -1,31 +1,38 @@ "ref\01455887034-7739-1-git-send-email-abrodkin@synopsys.com\0" "ref\01455887034-7739-3-git-send-email-abrodkin@synopsys.com\0" "ref\020160223203841.GA15744@rob-hp-laptop\0" - "From\0Alexey.Brodkin@synopsys.com (Alexey Brodkin)\0" - "Subject\0[PATCH 2/4] drm: Add DT bindings documentation for ARC PGU display controller\0" + "From\0Alexey Brodkin <Alexey.Brodkin@synopsys.com>\0" + "Subject\0Re: [PATCH 2/4] drm: Add DT bindings documentation for ARC PGU display controller\0" "Date\0Thu, 3 Mar 2016 13:58:18 +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 Tue, 2016-02-23@14:38 -0600, Rob Herring wrote:\n" - "> On Fri, Feb 19, 2016@04:03:52PM +0300, Alexey Brodkin wrote:\n" + "On Tue, 2016-02-23 at 14:38 -0600, Rob Herring wrote:\n" + "> On Fri, Feb 19, 2016 at 04:03:52PM +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" - "> > ?.../devicetree/bindings/display/snps,arcpgu.txt????| 74 ++++++++++++++++++++++\n" - "> > ?1 file changed, 74 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| 74 ++++++++++++++++++++++\n" + "> > \302\2401 file changed, 74 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" @@ -41,20 +48,20 @@ "> > +framebuffer and sends it to a single digital encoder (usually HDMI).\n" "> > +\n" "> > +Required properties:\n" - "> > +??- compatible: \"snps,arcpgu\"\n" - "> Seems like this should be more specific. Is there some sort or?\n" + "> > +\302\240\302\240- compatible: \"snps,arcpgu\"\n" + "> Seems like this should be more specific. Is there some sort or\302\240\n" "> versioning with ARC blocks?\n" "\n" "Well as of today there's only one and only version of PGU.\n" "So is there a real need for \"snps,arcpgu-1.0\"?\n" "\n" "> > \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" - "> > +??- encoder-slave: Phandle of encoder chip.\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" + "> > +\302\240\302\240- encoder-slave: Phandle of encoder chip.\n" "> This is unnecessary with the OF graph.\n" "\n" "Do you mean I may drop \"encoder-slave\" from bindings description?\n" @@ -67,21 +74,21 @@ "> > +/ {\n" "> > +\t...\n" "> > +\n" - "> > +\tpgu at 0xXXXXXXXX {\n" + "> > +\tpgu@0xXXXXXXXX {\n" "> Unit address should not have '0x'\n" "\n" "Correct, let me fix it.\n" "\n" "> > +\n" "> > +\t/* HDMI encoder on I2C bus */\n" - "> > +\ti2c at 0xXXXXXXXX {\n" + "> > +\ti2c@0xXXXXXXXX {\n" "> > +\t\tcompatible = \"...\";\n" "> > +\n" - "> > +\t\tencoder_node:encoder_node at 0xXXXXXXXX{\n" + "> > +\t\tencoder_node:encoder_node@0xXXXXXXXX{\n" "> I2C addresses would be 8-bit only.\n" "\n" "Indeed - copy-paste is evil :)\n" "\n" -Alexey -960067393cba3d4aaa8aaeb672d5089c02060d895d48f702ef8edc20b453b5b0 +3acd3f650ed07a4da058462092fbb7da593ce09bee60977f3fbd2e9c19946d01
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.