diff for duplicates of <1463014882.20290.203.camel@kernel.crashing.org> diff --git a/a/1.txt b/N1/1.txt index 89d8fca..24e7ee4 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,7 +1,7 @@ On Wed, 2016-05-11 at 09:33 -0500, Rob Herring wrote: > > +- interrupt-controller : Identifies the node as an interrupt controller > > +- #interrupt-cells : Specifies the number of cells needed to encode an -> > +? interrupt source. The value shall be 1. +> > + interrupt source. The value shall be 1. > > No need for level vs. edge flags? @@ -12,16 +12,16 @@ make it part of the binding I suppose. > > +- valid-sources : bitmask of valid irq sources > -> Drop this. Either all interrupt controllers need this or none of? +> Drop this. Either all interrupt controllers need this or none of > them do. The valid sources are the ones described in the DT. Looking at the code (I wrote that ages ago), this is only used for: -? - Failing map on an unsupported source, so we could drop it + - Failing map on an unsupported source, so we could drop it -? - Counting the number of sources in order to optimize the -? ? revmap size allocation. We could unconditionally allocate -? ? 64, I don't see a big deal here. + - Counting the number of sources in order to optimize the + revmap size allocation. We could unconditionally allocate + 64, I don't see a big deal here. So yes Joel, feel free to just ditch this. @@ -32,16 +32,20 @@ So yes Joel, feel free to just ditch this. > > Needs a unit address. > -> > +????? compatible = "aspeed,ast2400-vic"; -> > +????? interrupt-controller; -> > +????? #interrupt-cells = <1>; -> > +????? valid-sources = < 0xffffffff 0x0007ffff>; -> > +????? reg = <0x1e6c0080 0x80>; +> > + compatible = "aspeed,ast2400-vic"; +> > + interrupt-controller; +> > + #interrupt-cells = <1>; +> > + valid-sources = < 0xffffffff 0x0007ffff>; +> > + reg = <0x1e6c0080 0x80>; > > + }; -> > --? +> > -- > > 2.8.1 -> >? +> > > > -- > > To unsubscribe from this list: send the line "unsubscribe devicetree" in -> > the body of a message to majordomo at vger.kernel.org -> > More majordomo info at? http://vger.kernel.org/majordomo-info.html +> > the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org +> > More majordomo info at http://vger.kernel.org/majordomo-info.html +-- +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 aaa8db3..cf574fb 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -2,16 +2,25 @@ "ref\01462802317-27086-1-git-send-email-joel@jms.id.au\0" "ref\01462802317-27086-2-git-send-email-joel@jms.id.au\0" "ref\020160511143313.GA7426@rob-hp-laptop\0" - "From\0benh@kernel.crashing.org (Benjamin Herrenschmidt)\0" - "Subject\0[PATCH v2 1/2] doc/devicetree: Add Aspeed VIC bindings\0" + "From\0Benjamin Herrenschmidt <benh-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org>\0" + "Subject\0Re: [PATCH v2 1/2] doc/devicetree: Add Aspeed VIC bindings\0" "Date\0Thu, 12 May 2016 11:01:22 +1000\0" - "To\0linux-arm-kernel@lists.infradead.org\0" + "To\0Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>" + " Joel Stanley <joel-U3u1mxZcP9KHXe+LvDLADg@public.gmane.org>\0" + "Cc\0tglx-hfZtesqFncYOwBW4kG4KsQ@public.gmane.org" + jason-NLaQJdtUoK4Be96aLqz0jA@public.gmane.org + marc.zyngier-5wv7dgnIgG8@public.gmane.org + devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org + linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org + jk-mnsaURCQ41sdnm+yROfE0A@public.gmane.org + arnd-r2nGTMty4D4@public.gmane.org + " baruch-NswTu9S1W3P6gbPvEgmw2w@public.gmane.org\0" "\00:1\0" "b\0" "On Wed, 2016-05-11 at 09:33 -0500, Rob Herring wrote:\n" "> > +- interrupt-controller : Identifies the node as an interrupt controller\n" "> > +- #interrupt-cells : Specifies the number of cells needed to encode an\n" - "> > +? interrupt source. The value shall be 1.\n" + "> > +\302\240 interrupt source. The value shall be 1.\n" "> \n" "> No need for level vs. edge flags?\n" "\n" @@ -22,16 +31,16 @@ "\n" "> > +- valid-sources : bitmask of valid irq sources\n" "> \n" - "> Drop this. Either all interrupt controllers need this or none of?\n" + "> Drop this. Either all interrupt controllers need this or none of\302\240\n" "> them do. The valid sources are the ones described in the DT.\n" "\n" "Looking at the code (I wrote that ages ago), this is only used for:\n" "\n" - "? - Failing map on an unsupported source, so we could drop it\n" + "\302\240 - Failing map on an unsupported source, so we could drop it\n" "\n" - "? - Counting the number of sources in order to optimize the\n" - "? ? revmap size allocation. We could unconditionally allocate\n" - "? ? 64, I don't see a big deal here.\n" + "\302\240 - Counting the number of sources in order to optimize the\n" + "\302\240 \302\240 revmap size allocation. We could unconditionally allocate\n" + "\302\240 \302\240 64, I don't see a big deal here.\n" "\n" "So yes Joel, feel free to just ditch this.\n" "\n" @@ -42,18 +51,22 @@ "> \n" "> Needs a unit address.\n" "> \n" - "> > +????? compatible = \"aspeed,ast2400-vic\";\n" - "> > +????? interrupt-controller;\n" - "> > +????? #interrupt-cells = <1>;\n" - "> > +????? valid-sources = < 0xffffffff 0x0007ffff>;\n" - "> > +????? reg = <0x1e6c0080 0x80>;\n" + "> > +\302\240\302\240\302\240\302\240\302\240 compatible = \"aspeed,ast2400-vic\";\n" + "> > +\302\240\302\240\302\240\302\240\302\240 interrupt-controller;\n" + "> > +\302\240\302\240\302\240\302\240\302\240 #interrupt-cells = <1>;\n" + "> > +\302\240\302\240\302\240\302\240\302\240 valid-sources = < 0xffffffff 0x0007ffff>;\n" + "> > +\302\240\302\240\302\240\302\240\302\240 reg = <0x1e6c0080 0x80>;\n" "> > + };\n" - "> > --?\n" + "> > --\302\240\n" "> > 2.8.1\n" - "> >?\n" + "> >\302\240\n" "> > --\n" "> > To unsubscribe from this list: send the line \"unsubscribe devicetree\" in\n" - "> > the body of a message to majordomo at vger.kernel.org\n" - > > More majordomo info at? http://vger.kernel.org/majordomo-info.html + "> > the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org\n" + "> > More majordomo info at\302\240 http://vger.kernel.org/majordomo-info.html\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 -0c7780f9f82fe359f2d64b7235e208fec49e28b9751a6ec57e2a6bfb5a1d4891 +e2ae46c65f8242a3d37cd948e4d759c8d70dbd3f8b1378c7eaa36a3b74aa2822
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.