diff for duplicates of <20150911175555.GF18779@x1> diff --git a/a/1.txt b/N1/1.txt index e0ee31c..80b7732 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -5,7 +5,7 @@ On Fri, 11 Sep 2015, Peter Griffin wrote: > > > These nodes are required to get the fdma driver working > > > on STiH407 based silicon. > > > -> > > Signed-off-by: Peter Griffin <peter.griffin@linaro.org> +> > > Signed-off-by: Peter Griffin <peter.griffin-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> > > > --- > > > arch/arm/boot/dts/stih407-family.dtsi | 51 +++++++++++++++++++++++++++++++++++ > > > 1 file changed, 51 insertions(+) @@ -19,12 +19,12 @@ On Fri, 11 Sep 2015, Peter Griffin wrote: > > > }; > > > }; > > > + -> > > + fdma0: fdma0-audio at 8e20000 { +> > > + fdma0: fdma0-audio@8e20000 { > > > > I'm not familiar with the FDMA driver, so can't comment knowledgeably, > > but the <dev> part of <dev>@<base_address> should only describe the > > type of hardware. I believe in this case it should just be -> > dma at 08e20000. Also notice the leading zero in the address, which I +> > dma@08e20000. Also notice the leading zero in the address, which I > > believe mitigates possible confusion. Then you be more specific with > > the label, so something like 'fdma-audio' seems appropriate here. > @@ -74,5 +74,9 @@ Then you can have a different compatible for each. -- Lee Jones Linaro STMicroelectronics Landing Team Lead -Linaro.org ? Open source software for ARM SoCs +Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog +-- +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 a104dff..a64b916 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -2,10 +2,19 @@ "ref\01441980871-24475-8-git-send-email-peter.griffin@linaro.org\0" "ref\020150911162749.GD18779@x1\0" "ref\020150911164822.GA1150@griffinp-ThinkPad-X1-Carbon-2nd\0" - "From\0lee.jones@linaro.org (Lee Jones)\0" - "Subject\0[PATCH v2 7/9] ARM: STi: DT: STiH407: Add FDMA driver dt nodes.\0" + "From\0Lee Jones <lee.jones-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>\0" + "Subject\0Re: [PATCH v2 7/9] ARM: STi: DT: STiH407: Add FDMA driver dt nodes.\0" "Date\0Fri, 11 Sep 2015 18:55:55 +0100\0" - "To\0linux-arm-kernel@lists.infradead.org\0" + "To\0Peter Griffin <peter.griffin-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>\0" + "Cc\0linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org" + linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org + srinivas.kandagatla-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org + maxime.coquelin-qxv4g6HH51o@public.gmane.org + patrice.chotard-qxv4g6HH51o@public.gmane.org + vinod.koul-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org + robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org + dmaengine-u79uwXL29TY76Z2rM5mHXA@public.gmane.org + " devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org\0" "\00:1\0" "b\0" "On Fri, 11 Sep 2015, Peter Griffin wrote:\n" @@ -15,7 +24,7 @@ "> > > These nodes are required to get the fdma driver working\n" "> > > on STiH407 based silicon.\n" "> > > \n" - "> > > Signed-off-by: Peter Griffin <peter.griffin@linaro.org>\n" + "> > > Signed-off-by: Peter Griffin <peter.griffin-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>\n" "> > > ---\n" "> > > arch/arm/boot/dts/stih407-family.dtsi | 51 +++++++++++++++++++++++++++++++++++\n" "> > > 1 file changed, 51 insertions(+)\n" @@ -29,12 +38,12 @@ "> > > \t\t\t};\n" "> > > \t\t};\n" "> > > +\n" - "> > > +\t\tfdma0: fdma0-audio at 8e20000 {\n" + "> > > +\t\tfdma0: fdma0-audio@8e20000 {\n" "> > \n" "> > I'm not familiar with the FDMA driver, so can't comment knowledgeably,\n" "> > but the <dev> part of <dev>@<base_address> should only describe the\n" "> > type of hardware. I believe in this case it should just be\n" - "> > dma at 08e20000. Also notice the leading zero in the address, which I\n" + "> > dma@08e20000. Also notice the leading zero in the address, which I\n" "> > believe mitigates possible confusion. Then you be more specific with\n" "> > the label, so something like 'fdma-audio' seems appropriate here.\n" "> \n" @@ -84,7 +93,11 @@ "-- \n" "Lee Jones\n" "Linaro STMicroelectronics Landing Team Lead\n" - "Linaro.org ? Open source software for ARM SoCs\n" - Follow Linaro: Facebook | Twitter | Blog + "Linaro.org \342\224\202 Open source software for ARM SoCs\n" + "Follow Linaro: Facebook | Twitter | Blog\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 -a6d7039e136dce184d794d2626718d5ade9c4d25988e8a429484e5a60a416064 +f536cb51285e3994e9a74a19dd4cbd8c7409d32e6f38ede2da721262f0d2c1a4
diff --git a/a/1.txt b/N2/1.txt index e0ee31c..b0fc5fe 100644 --- a/a/1.txt +++ b/N2/1.txt @@ -19,12 +19,12 @@ On Fri, 11 Sep 2015, Peter Griffin wrote: > > > }; > > > }; > > > + -> > > + fdma0: fdma0-audio at 8e20000 { +> > > + fdma0: fdma0-audio@8e20000 { > > > > I'm not familiar with the FDMA driver, so can't comment knowledgeably, > > but the <dev> part of <dev>@<base_address> should only describe the > > type of hardware. I believe in this case it should just be -> > dma at 08e20000. Also notice the leading zero in the address, which I +> > dma@08e20000. Also notice the leading zero in the address, which I > > believe mitigates possible confusion. Then you be more specific with > > the label, so something like 'fdma-audio' seems appropriate here. > @@ -74,5 +74,5 @@ Then you can have a different compatible for each. -- Lee Jones Linaro STMicroelectronics Landing Team Lead -Linaro.org ? Open source software for ARM SoCs +Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog diff --git a/a/content_digest b/N2/content_digest index a104dff..b4081be 100644 --- a/a/content_digest +++ b/N2/content_digest @@ -2,10 +2,19 @@ "ref\01441980871-24475-8-git-send-email-peter.griffin@linaro.org\0" "ref\020150911162749.GD18779@x1\0" "ref\020150911164822.GA1150@griffinp-ThinkPad-X1-Carbon-2nd\0" - "From\0lee.jones@linaro.org (Lee Jones)\0" - "Subject\0[PATCH v2 7/9] ARM: STi: DT: STiH407: Add FDMA driver dt nodes.\0" + "From\0Lee Jones <lee.jones@linaro.org>\0" + "Subject\0Re: [PATCH v2 7/9] ARM: STi: DT: STiH407: Add FDMA driver dt nodes.\0" "Date\0Fri, 11 Sep 2015 18:55:55 +0100\0" - "To\0linux-arm-kernel@lists.infradead.org\0" + "To\0Peter Griffin <peter.griffin@linaro.org>\0" + "Cc\0linux-arm-kernel@lists.infradead.org" + linux-kernel@vger.kernel.org + srinivas.kandagatla@gmail.com + maxime.coquelin@st.com + patrice.chotard@st.com + vinod.koul@intel.com + robh+dt@kernel.org + dmaengine@vger.kernel.org + " devicetree@vger.kernel.org\0" "\00:1\0" "b\0" "On Fri, 11 Sep 2015, Peter Griffin wrote:\n" @@ -29,12 +38,12 @@ "> > > \t\t\t};\n" "> > > \t\t};\n" "> > > +\n" - "> > > +\t\tfdma0: fdma0-audio at 8e20000 {\n" + "> > > +\t\tfdma0: fdma0-audio@8e20000 {\n" "> > \n" "> > I'm not familiar with the FDMA driver, so can't comment knowledgeably,\n" "> > but the <dev> part of <dev>@<base_address> should only describe the\n" "> > type of hardware. I believe in this case it should just be\n" - "> > dma at 08e20000. Also notice the leading zero in the address, which I\n" + "> > dma@08e20000. Also notice the leading zero in the address, which I\n" "> > believe mitigates possible confusion. Then you be more specific with\n" "> > the label, so something like 'fdma-audio' seems appropriate here.\n" "> \n" @@ -84,7 +93,7 @@ "-- \n" "Lee Jones\n" "Linaro STMicroelectronics Landing Team Lead\n" - "Linaro.org ? Open source software for ARM SoCs\n" + "Linaro.org \342\224\202 Open source software for ARM SoCs\n" Follow Linaro: Facebook | Twitter | Blog -a6d7039e136dce184d794d2626718d5ade9c4d25988e8a429484e5a60a416064 +e9e7eef81dcd318111d88b79631fbfeda5913d3bd6d3eb7255c7155de6adbf2c
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.