diff for duplicates of <1418371705.423.31.camel@mtksdaap41> diff --git a/a/1.txt b/N1/1.txt index 939e448..ba41070 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -24,7 +24,7 @@ Not yet > > + > > + memory { > -> Nit: should be memory at 40000000 (and you'll need to add device_type = +> Nit: should be memory@40000000 (and you'll need to add device_type = > "memory"). > > > + reg = <0 0x40000000 0 0x40000000>; @@ -32,7 +32,7 @@ Not yet skeleton.dtsi already has /memory node with address-cells=2, size-cells=1, which will cause build warning if I change to use -memory at 40000000, because we use size-cells=2. I will not include +memory@40000000, because we use size-cells=2. I will not include skeleton.dtsi and follow your suggestion in next version. > > + @@ -118,7 +118,7 @@ Yes, I remove and test ok in my platform > > + compatible = "simple-bus"; > > + ranges; > > + -> > + sysirq: intpol-controller at 10200620 { +> > + sysirq: intpol-controller@10200620 { > > + compatible = "mediatek,mt8173-sysirq", "mediatek,mt6577-sysirq"; > > + interrupt-controller; > > + #interrupt-cells = <3>; @@ -126,7 +126,7 @@ Yes, I remove and test ok in my platform > > + reg = <0 0x10200620 0 0x20>; > > + }; > > + -> > + gic: interrupt-controller at 10220000 { +> > + gic: interrupt-controller@10220000 { > > + compatible = "arm,cortex-a15-gic", "arm,cortex-a9-gic"; > > Surely this should be "arm,gic-400"? diff --git a/a/content_digest b/N1/content_digest index 5e96e26..343d28e 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,10 +1,29 @@ "ref\01418208602-35584-1-git-send-email-eddie.huang@mediatek.com\0" "ref\01418208602-35584-4-git-send-email-eddie.huang@mediatek.com\0" "ref\020141211180245.GE28150@leverpostej\0" - "From\0eddie.huang@mediatek.com (Eddie Huang)\0" - "Subject\0[PATCH 3/4] arm64: dts: Add mediatek MT8173 SoC and evaluation board dts and Makefile\0" + "From\0Eddie Huang <eddie.huang@mediatek.com>\0" + "Subject\0Re: [PATCH 3/4] arm64: dts: Add mediatek MT8173 SoC and evaluation board dts and Makefile\0" "Date\0Fri, 12 Dec 2014 16:08:25 +0800\0" - "To\0linux-arm-kernel@lists.infradead.org\0" + "To\0Mark Rutland <mark.rutland@arm.com>\0" + "Cc\0devicetree@vger.kernel.org <devicetree@vger.kernel.org>" + Jason Cooper <jason@lakedaemon.net> + Pawel Moll <Pawel.Moll@arm.com> + Ian Campbell <ijc+devicetree@hellion.org.uk> + Catalin Marinas <Catalin.Marinas@arm.com> + Olof Johansson <olof@lixom.net> + Will Deacon <Will.Deacon@arm.com> + linux-kernel@vger.kernel.org <linux-kernel@vger.kernel.org> + Robert Richter <rrichter@cavium.com> + srv_heupstream@mediatek.com <srv_heupstream@mediatek.com> + Rob Herring <robh+dt@kernel.org> + Sascha Hauer <kernel@pengutronix.de> + Kumar Gala <galak@codeaurora.org> + Matthias Brugger <matthias.bgg@gmail.com> + Joe.C <yingjoe.chen@mediatek.com> + Thomas Gleixner <tglx@linutronix.de> + Mark Brown <broonie@linaro.org> + linux-arm-kernel@lists.infradead.org <linux-arm-kernel@lists.infradead.org> + " yh.chen@mediatek.com <yh.chen@mediatek.com>\0" "\00:1\0" "b\0" "Hi Mark,\n" @@ -33,7 +52,7 @@ "> > +\n" "> > +\tmemory {\n" "> \n" - "> Nit: should be memory at 40000000 (and you'll need to add device_type =\n" + "> Nit: should be memory@40000000 (and you'll need to add device_type =\n" "> \"memory\").\n" "> \n" "> > +\t\treg = <0 0x40000000 0 0x40000000>;\n" @@ -41,7 +60,7 @@ "\n" "skeleton.dtsi already has /memory node with address-cells=2,\n" "size-cells=1, which will cause build warning if I change to use\n" - "memory at 40000000, because we use size-cells=2. I will not include\n" + "memory@40000000, because we use size-cells=2. I will not include\n" "skeleton.dtsi and follow your suggestion in next version.\n" "\n" "> > +\n" @@ -127,7 +146,7 @@ "> > +\t\tcompatible = \"simple-bus\";\n" "> > +\t\tranges;\n" "> > +\n" - "> > +\t\tsysirq: intpol-controller at 10200620 {\n" + "> > +\t\tsysirq: intpol-controller@10200620 {\n" "> > +\t\t\tcompatible = \"mediatek,mt8173-sysirq\", \"mediatek,mt6577-sysirq\";\n" "> > +\t\t\tinterrupt-controller;\n" "> > +\t\t\t#interrupt-cells = <3>;\n" @@ -135,7 +154,7 @@ "> > +\t\t\treg = <0 0x10200620 0 0x20>;\n" "> > +\t\t};\n" "> > +\n" - "> > +\t\tgic: interrupt-controller at 10220000 {\n" + "> > +\t\tgic: interrupt-controller@10220000 {\n" "> > +\t\t\tcompatible = \"arm,cortex-a15-gic\", \"arm,cortex-a9-gic\";\n" "> \n" "> Surely this should be \"arm,gic-400\"?\n" @@ -159,4 +178,4 @@ "\n" I will fix in next version -096ae028ad9cd1e2280f0a702da59333967cc5e5dc591c79444503a256144f77 +e629f434da52b0dbb1ce06440d263fecbfb0d77a6515d55e4811bf2781b08aa3
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.