diff for duplicates of <20150508072527.GI16220@x1> diff --git a/a/1.txt b/N1/1.txt index 16d8143..f549a88 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,65 +1,49 @@ On Fri, 08 May 2015, Peter Griffin wrote: > On Fri, 08 May 2015, Lee Jones wrote: -> > > > > > =20 -> > > > > > -- compatible : Must be one of: "st,stih407-lpc" "st,stih416-l= -pc" +> > > > > > +> > > > > > -- compatible : Must be one of: "st,stih407-lpc" "st,stih416-lpc" > > > > > > - "st,stih415-lpc" "st,stid127-lpc" > > > > > > +- compatible : Must be one of: "st,stih407-lpc" -> > > > >=20 -> > > > > The same comment as the RTC DT patch, you are removing the compat= -ibles +> > > > > +> > > > > The same comment as the RTC DT patch, you are removing the compatibles > > > > > documentation for the other supported platforms like stih416-lpc. -> > > > > AFAIK they are required in the driver to get the correct sysconfi= -g register. -> > > >=20 +> > > > > AFAIK they are required in the driver to get the correct sysconfig register. +> > > > > > > > That's intentional. I haven't yet tested any of this IP on > > > > STiH41{5,6} & STiH127. Due to lack of documentation, I'm not even > > > > sure if this IP even exists on some of the other platforms. I will > > > > add them back when support is added to both driver and DTB and I've > > > > been able to test them. -> > >=20 -> > > That was kind of my point, the driver code AFAIK already contains sup= -port +> > > +> > > That was kind of my point, the driver code AFAIK already contains support > > > for these SoC's. -> > >=20 -> > > I would either expect the patch to remove support from the DT docs AN= -D the +> > > +> > > I would either expect the patch to remove support from the DT docs AND the > > > driver, or leave it as is. -> > >=20 +> > > > > > It seems odd to only change the DT docs, and become unaligned to the -> > > code (this assumes I'm looking at the latest patchset here=20 -> > > https://lkml.org/lkml/2015/3/4/1088 which includes support for these = -SoCs). -> >=20 +> > > code (this assumes I'm looking at the latest patchset here +> > > https://lkml.org/lkml/2015/3/4/1088 which includes support for these SoCs). +> > > > The decision to remove these 'supported' platforms was made on the RTC > > side, where there is only support for "st,stih407-lpc" in the driver. ->=20 +> > The change makes sense in the RTC case then. ->=20 +> > > I thought it best to mirror that thought over to the Watchdog LPC > > bindings, ->=20 -> Why? This is the bit I disagree with, the wdt DT docs IMHO should match t= -he wdt +> +> Why? This is the bit I disagree with, the wdt DT docs IMHO should match the wdt > driver code. Fair enough. I'll add the 'supported' compatibles back in. ---=20 +-- Lee Jones Linaro STMicroelectronics Landing Team Lead -Linaro.org =E2=94=82 Open source software for ARM SoCs +Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog - ---=20 ---=20 -You received this message because you are subscribed to "rtc-linux". -Membership options at http://groups.google.com/group/rtc-linux . -Please read http://groups.google.com/group/rtc-linux/web/checklist -before submitting a driver. ----=20 -You received this message because you are subscribed to the Google Groups "= -rtc-linux" group. -To unsubscribe from this group and stop receiving emails from it, send an e= -mail to rtc-linux+unsubscribe@googlegroups.com. -For more options, visit https://groups.google.com/d/optout. +-- +To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in +the body of a message to majordomo@vger.kernel.org +More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/a/content_digest b/N1/content_digest index 491c106..6417e09 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -6,7 +6,7 @@ "ref\020150508064918.GF16220@x1\0" "ref\020150508071604.GA2232@griffinp-ThinkPad-X1-Carbon-2nd\0" "From\0Lee Jones <lee.jones@linaro.org>\0" - "Subject\0[rtc-linux] Re: [STLinux Kernel] [PATCH 06/12] watchdog: bindings: Supply knowledge of a third supported device - clocksource\0" + "Subject\0Re: [STLinux Kernel] [PATCH 06/12] watchdog: bindings: Supply knowledge of a third supported device - clocksource\0" "Date\0Fri, 8 May 2015 08:25:27 +0100\0" "To\0Peter Griffin <peter.griffin@linaro.org>\0" "Cc\0linux-arm-kernel@lists.infradead.org" @@ -24,68 +24,52 @@ "b\0" "On Fri, 08 May 2015, Peter Griffin wrote:\n" "> On Fri, 08 May 2015, Lee Jones wrote:\n" - "> > > > > > =20\n" - "> > > > > > -- compatible \t: Must be one of: \"st,stih407-lpc\" \"st,stih416-l=\n" - "pc\"\n" + "> > > > > > \n" + "> > > > > > -- compatible \t: Must be one of: \"st,stih407-lpc\" \"st,stih416-lpc\"\n" "> > > > > > -\t\t\t\t \"st,stih415-lpc\" \"st,stid127-lpc\"\n" "> > > > > > +- compatible \t: Must be one of: \"st,stih407-lpc\"\n" - "> > > > >=20\n" - "> > > > > The same comment as the RTC DT patch, you are removing the compat=\n" - "ibles\n" + "> > > > > \n" + "> > > > > The same comment as the RTC DT patch, you are removing the compatibles\n" "> > > > > documentation for the other supported platforms like stih416-lpc.\n" - "> > > > > AFAIK they are required in the driver to get the correct sysconfi=\n" - "g register.\n" - "> > > >=20\n" + "> > > > > AFAIK they are required in the driver to get the correct sysconfig register.\n" + "> > > > \n" "> > > > That's intentional. I haven't yet tested any of this IP on\n" "> > > > STiH41{5,6} & STiH127. Due to lack of documentation, I'm not even\n" "> > > > sure if this IP even exists on some of the other platforms. I will\n" "> > > > add them back when support is added to both driver and DTB and I've\n" "> > > > been able to test them.\n" - "> > >=20\n" - "> > > That was kind of my point, the driver code AFAIK already contains sup=\n" - "port\n" + "> > > \n" + "> > > That was kind of my point, the driver code AFAIK already contains support\n" "> > > for these SoC's.\n" - "> > >=20\n" - "> > > I would either expect the patch to remove support from the DT docs AN=\n" - "D the\n" + "> > > \n" + "> > > I would either expect the patch to remove support from the DT docs AND the\n" "> > > driver, or leave it as is.\n" - "> > >=20\n" + "> > > \n" "> > > It seems odd to only change the DT docs, and become unaligned to the\n" - "> > > code (this assumes I'm looking at the latest patchset here=20\n" - "> > > https://lkml.org/lkml/2015/3/4/1088 which includes support for these =\n" - "SoCs).\n" - "> >=20\n" + "> > > code (this assumes I'm looking at the latest patchset here \n" + "> > > https://lkml.org/lkml/2015/3/4/1088 which includes support for these SoCs).\n" + "> > \n" "> > The decision to remove these 'supported' platforms was made on the RTC\n" "> > side, where there is only support for \"st,stih407-lpc\" in the driver.\n" - ">=20\n" + "> \n" "> The change makes sense in the RTC case then.\n" - ">=20\n" + "> \n" "> > I thought it best to mirror that thought over to the Watchdog LPC\n" "> > bindings,\n" - ">=20\n" - "> Why? This is the bit I disagree with, the wdt DT docs IMHO should match t=\n" - "he wdt\n" + "> \n" + "> Why? This is the bit I disagree with, the wdt DT docs IMHO should match the wdt\n" "> driver code.\n" "\n" "Fair enough. I'll add the 'supported' compatibles back in.\n" "\n" - "--=20\n" + "-- \n" "Lee Jones\n" "Linaro STMicroelectronics Landing Team Lead\n" - "Linaro.org =E2=94=82 Open source software for ARM SoCs\n" + "Linaro.org \342\224\202 Open source software for ARM SoCs\n" "Follow Linaro: Facebook | Twitter | Blog\n" - "\n" - "--=20\n" - "--=20\n" - "You received this message because you are subscribed to \"rtc-linux\".\n" - "Membership options at http://groups.google.com/group/rtc-linux .\n" - "Please read http://groups.google.com/group/rtc-linux/web/checklist\n" - "before submitting a driver.\n" - "---=20\n" - "You received this message because you are subscribed to the Google Groups \"=\n" - "rtc-linux\" group.\n" - "To unsubscribe from this group and stop receiving emails from it, send an e=\n" - "mail to rtc-linux+unsubscribe@googlegroups.com.\n" - For more options, visit https://groups.google.com/d/optout. + "--\n" + "To unsubscribe from this list: send the line \"unsubscribe linux-watchdog\" in\n" + "the body of a message to majordomo@vger.kernel.org\n" + More majordomo info at http://vger.kernel.org/majordomo-info.html -1598512223918cee6cc272c328e26189454fb64d04b8373d4316e8446e51a65d +da27e6e3aa25aa0b7f2e812e548b1f7aeca15a21b35ca17bce30c754b85fec0e
diff --git a/a/1.txt b/N2/1.txt index 16d8143..6500ff4 100644 --- a/a/1.txt +++ b/N2/1.txt @@ -1,65 +1,45 @@ On Fri, 08 May 2015, Peter Griffin wrote: > On Fri, 08 May 2015, Lee Jones wrote: -> > > > > > =20 -> > > > > > -- compatible : Must be one of: "st,stih407-lpc" "st,stih416-l= -pc" +> > > > > > +> > > > > > -- compatible : Must be one of: "st,stih407-lpc" "st,stih416-lpc" > > > > > > - "st,stih415-lpc" "st,stid127-lpc" > > > > > > +- compatible : Must be one of: "st,stih407-lpc" -> > > > >=20 -> > > > > The same comment as the RTC DT patch, you are removing the compat= -ibles +> > > > > +> > > > > The same comment as the RTC DT patch, you are removing the compatibles > > > > > documentation for the other supported platforms like stih416-lpc. -> > > > > AFAIK they are required in the driver to get the correct sysconfi= -g register. -> > > >=20 +> > > > > AFAIK they are required in the driver to get the correct sysconfig register. +> > > > > > > > That's intentional. I haven't yet tested any of this IP on > > > > STiH41{5,6} & STiH127. Due to lack of documentation, I'm not even > > > > sure if this IP even exists on some of the other platforms. I will > > > > add them back when support is added to both driver and DTB and I've > > > > been able to test them. -> > >=20 -> > > That was kind of my point, the driver code AFAIK already contains sup= -port +> > > +> > > That was kind of my point, the driver code AFAIK already contains support > > > for these SoC's. -> > >=20 -> > > I would either expect the patch to remove support from the DT docs AN= -D the +> > > +> > > I would either expect the patch to remove support from the DT docs AND the > > > driver, or leave it as is. -> > >=20 +> > > > > > It seems odd to only change the DT docs, and become unaligned to the -> > > code (this assumes I'm looking at the latest patchset here=20 -> > > https://lkml.org/lkml/2015/3/4/1088 which includes support for these = -SoCs). -> >=20 +> > > code (this assumes I'm looking at the latest patchset here +> > > https://lkml.org/lkml/2015/3/4/1088 which includes support for these SoCs). +> > > > The decision to remove these 'supported' platforms was made on the RTC > > side, where there is only support for "st,stih407-lpc" in the driver. ->=20 +> > The change makes sense in the RTC case then. ->=20 +> > > I thought it best to mirror that thought over to the Watchdog LPC > > bindings, ->=20 -> Why? This is the bit I disagree with, the wdt DT docs IMHO should match t= -he wdt +> +> Why? This is the bit I disagree with, the wdt DT docs IMHO should match the wdt > driver code. Fair enough. I'll add the 'supported' compatibles back in. ---=20 +-- Lee Jones Linaro STMicroelectronics Landing Team Lead -Linaro.org =E2=94=82 Open source software for ARM SoCs +Linaro.org ? Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog - ---=20 ---=20 -You received this message because you are subscribed to "rtc-linux". -Membership options at http://groups.google.com/group/rtc-linux . -Please read http://groups.google.com/group/rtc-linux/web/checklist -before submitting a driver. ----=20 -You received this message because you are subscribed to the Google Groups "= -rtc-linux" group. -To unsubscribe from this group and stop receiving emails from it, send an e= -mail to rtc-linux+unsubscribe@googlegroups.com. -For more options, visit https://groups.google.com/d/optout. diff --git a/a/content_digest b/N2/content_digest index 491c106..7d95a34 100644 --- a/a/content_digest +++ b/N2/content_digest @@ -5,87 +5,56 @@ "ref\020150507195529.GA28554@griffinp-ThinkPad-X1-Carbon-2nd\0" "ref\020150508064918.GF16220@x1\0" "ref\020150508071604.GA2232@griffinp-ThinkPad-X1-Carbon-2nd\0" - "From\0Lee Jones <lee.jones@linaro.org>\0" - "Subject\0[rtc-linux] Re: [STLinux Kernel] [PATCH 06/12] watchdog: bindings: Supply knowledge of a third supported device - clocksource\0" + "From\0lee.jones@linaro.org (Lee Jones)\0" + "Subject\0[STLinux Kernel] [PATCH 06/12] watchdog: bindings: Supply knowledge of a third supported device - clocksource\0" "Date\0Fri, 8 May 2015 08:25:27 +0100\0" - "To\0Peter Griffin <peter.griffin@linaro.org>\0" - "Cc\0linux-arm-kernel@lists.infradead.org" - linux-kernel@vger.kernel.org - devicetree@vger.kernel.org - kernel@stlinux.com - rtc-linux@googlegroups.com - a.zummo@towertech.it - daniel.lezcano@linaro.org - wim@iguana.be - tglx@linutronix.de - linux@roeck-us.net - " linux-watchdog@vger.kernel.org\0" + "To\0linux-arm-kernel@lists.infradead.org\0" "\00:1\0" "b\0" "On Fri, 08 May 2015, Peter Griffin wrote:\n" "> On Fri, 08 May 2015, Lee Jones wrote:\n" - "> > > > > > =20\n" - "> > > > > > -- compatible \t: Must be one of: \"st,stih407-lpc\" \"st,stih416-l=\n" - "pc\"\n" + "> > > > > > \n" + "> > > > > > -- compatible \t: Must be one of: \"st,stih407-lpc\" \"st,stih416-lpc\"\n" "> > > > > > -\t\t\t\t \"st,stih415-lpc\" \"st,stid127-lpc\"\n" "> > > > > > +- compatible \t: Must be one of: \"st,stih407-lpc\"\n" - "> > > > >=20\n" - "> > > > > The same comment as the RTC DT patch, you are removing the compat=\n" - "ibles\n" + "> > > > > \n" + "> > > > > The same comment as the RTC DT patch, you are removing the compatibles\n" "> > > > > documentation for the other supported platforms like stih416-lpc.\n" - "> > > > > AFAIK they are required in the driver to get the correct sysconfi=\n" - "g register.\n" - "> > > >=20\n" + "> > > > > AFAIK they are required in the driver to get the correct sysconfig register.\n" + "> > > > \n" "> > > > That's intentional. I haven't yet tested any of this IP on\n" "> > > > STiH41{5,6} & STiH127. Due to lack of documentation, I'm not even\n" "> > > > sure if this IP even exists on some of the other platforms. I will\n" "> > > > add them back when support is added to both driver and DTB and I've\n" "> > > > been able to test them.\n" - "> > >=20\n" - "> > > That was kind of my point, the driver code AFAIK already contains sup=\n" - "port\n" + "> > > \n" + "> > > That was kind of my point, the driver code AFAIK already contains support\n" "> > > for these SoC's.\n" - "> > >=20\n" - "> > > I would either expect the patch to remove support from the DT docs AN=\n" - "D the\n" + "> > > \n" + "> > > I would either expect the patch to remove support from the DT docs AND the\n" "> > > driver, or leave it as is.\n" - "> > >=20\n" + "> > > \n" "> > > It seems odd to only change the DT docs, and become unaligned to the\n" - "> > > code (this assumes I'm looking at the latest patchset here=20\n" - "> > > https://lkml.org/lkml/2015/3/4/1088 which includes support for these =\n" - "SoCs).\n" - "> >=20\n" + "> > > code (this assumes I'm looking at the latest patchset here \n" + "> > > https://lkml.org/lkml/2015/3/4/1088 which includes support for these SoCs).\n" + "> > \n" "> > The decision to remove these 'supported' platforms was made on the RTC\n" "> > side, where there is only support for \"st,stih407-lpc\" in the driver.\n" - ">=20\n" + "> \n" "> The change makes sense in the RTC case then.\n" - ">=20\n" + "> \n" "> > I thought it best to mirror that thought over to the Watchdog LPC\n" "> > bindings,\n" - ">=20\n" - "> Why? This is the bit I disagree with, the wdt DT docs IMHO should match t=\n" - "he wdt\n" + "> \n" + "> Why? This is the bit I disagree with, the wdt DT docs IMHO should match the wdt\n" "> driver code.\n" "\n" "Fair enough. I'll add the 'supported' compatibles back in.\n" "\n" - "--=20\n" + "-- \n" "Lee Jones\n" "Linaro STMicroelectronics Landing Team Lead\n" - "Linaro.org =E2=94=82 Open source software for ARM SoCs\n" - "Follow Linaro: Facebook | Twitter | Blog\n" - "\n" - "--=20\n" - "--=20\n" - "You received this message because you are subscribed to \"rtc-linux\".\n" - "Membership options at http://groups.google.com/group/rtc-linux .\n" - "Please read http://groups.google.com/group/rtc-linux/web/checklist\n" - "before submitting a driver.\n" - "---=20\n" - "You received this message because you are subscribed to the Google Groups \"=\n" - "rtc-linux\" group.\n" - "To unsubscribe from this group and stop receiving emails from it, send an e=\n" - "mail to rtc-linux+unsubscribe@googlegroups.com.\n" - For more options, visit https://groups.google.com/d/optout. + "Linaro.org ? Open source software for ARM SoCs\n" + Follow Linaro: Facebook | Twitter | Blog -1598512223918cee6cc272c328e26189454fb64d04b8373d4316e8446e51a65d +bf2b8998cd33c9360413061279bdc19d647c6f2d49513370ab7785545e43b321
diff --git a/a/1.txt b/N3/1.txt index 16d8143..124ad1c 100644 --- a/a/1.txt +++ b/N3/1.txt @@ -1,65 +1,49 @@ On Fri, 08 May 2015, Peter Griffin wrote: > On Fri, 08 May 2015, Lee Jones wrote: -> > > > > > =20 -> > > > > > -- compatible : Must be one of: "st,stih407-lpc" "st,stih416-l= -pc" +> > > > > > +> > > > > > -- compatible : Must be one of: "st,stih407-lpc" "st,stih416-lpc" > > > > > > - "st,stih415-lpc" "st,stid127-lpc" > > > > > > +- compatible : Must be one of: "st,stih407-lpc" -> > > > >=20 -> > > > > The same comment as the RTC DT patch, you are removing the compat= -ibles +> > > > > +> > > > > The same comment as the RTC DT patch, you are removing the compatibles > > > > > documentation for the other supported platforms like stih416-lpc. -> > > > > AFAIK they are required in the driver to get the correct sysconfi= -g register. -> > > >=20 +> > > > > AFAIK they are required in the driver to get the correct sysconfig register. +> > > > > > > > That's intentional. I haven't yet tested any of this IP on > > > > STiH41{5,6} & STiH127. Due to lack of documentation, I'm not even > > > > sure if this IP even exists on some of the other platforms. I will > > > > add them back when support is added to both driver and DTB and I've > > > > been able to test them. -> > >=20 -> > > That was kind of my point, the driver code AFAIK already contains sup= -port +> > > +> > > That was kind of my point, the driver code AFAIK already contains support > > > for these SoC's. -> > >=20 -> > > I would either expect the patch to remove support from the DT docs AN= -D the +> > > +> > > I would either expect the patch to remove support from the DT docs AND the > > > driver, or leave it as is. -> > >=20 +> > > > > > It seems odd to only change the DT docs, and become unaligned to the -> > > code (this assumes I'm looking at the latest patchset here=20 -> > > https://lkml.org/lkml/2015/3/4/1088 which includes support for these = -SoCs). -> >=20 +> > > code (this assumes I'm looking at the latest patchset here +> > > https://lkml.org/lkml/2015/3/4/1088 which includes support for these SoCs). +> > > > The decision to remove these 'supported' platforms was made on the RTC > > side, where there is only support for "st,stih407-lpc" in the driver. ->=20 +> > The change makes sense in the RTC case then. ->=20 +> > > I thought it best to mirror that thought over to the Watchdog LPC > > bindings, ->=20 -> Why? This is the bit I disagree with, the wdt DT docs IMHO should match t= -he wdt +> +> Why? This is the bit I disagree with, the wdt DT docs IMHO should match the wdt > driver code. Fair enough. I'll add the 'supported' compatibles back in. ---=20 +-- Lee Jones Linaro STMicroelectronics Landing Team Lead -Linaro.org =E2=94=82 Open source software for ARM SoCs +Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog - ---=20 ---=20 -You received this message because you are subscribed to "rtc-linux". -Membership options at http://groups.google.com/group/rtc-linux . -Please read http://groups.google.com/group/rtc-linux/web/checklist -before submitting a driver. ----=20 -You received this message because you are subscribed to the Google Groups "= -rtc-linux" group. -To unsubscribe from this group and stop receiving emails from it, send an e= -mail to rtc-linux+unsubscribe@googlegroups.com. -For more options, visit https://groups.google.com/d/optout. +-- +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/N3/content_digest index 491c106..a9e9506 100644 --- a/a/content_digest +++ b/N3/content_digest @@ -5,87 +5,71 @@ "ref\020150507195529.GA28554@griffinp-ThinkPad-X1-Carbon-2nd\0" "ref\020150508064918.GF16220@x1\0" "ref\020150508071604.GA2232@griffinp-ThinkPad-X1-Carbon-2nd\0" - "From\0Lee Jones <lee.jones@linaro.org>\0" - "Subject\0[rtc-linux] Re: [STLinux Kernel] [PATCH 06/12] watchdog: bindings: Supply knowledge of a third supported device - clocksource\0" + "From\0Lee Jones <lee.jones-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>\0" + "Subject\0Re: [STLinux Kernel] [PATCH 06/12] watchdog: bindings: Supply knowledge of a third supported device - clocksource\0" "Date\0Fri, 8 May 2015 08:25:27 +0100\0" - "To\0Peter Griffin <peter.griffin@linaro.org>\0" - "Cc\0linux-arm-kernel@lists.infradead.org" - linux-kernel@vger.kernel.org - devicetree@vger.kernel.org - kernel@stlinux.com - rtc-linux@googlegroups.com - a.zummo@towertech.it - daniel.lezcano@linaro.org - wim@iguana.be - tglx@linutronix.de - linux@roeck-us.net - " linux-watchdog@vger.kernel.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 + devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org + kernel-F5mvAk5X5gdBDgjK7y7TUQ@public.gmane.org + rtc-linux-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org + a.zummo-BfzFCNDTiLLj+vYz1yj4TQ@public.gmane.org + daniel.lezcano-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org + wim-IQzOog9fTRqzQB+pC5nmwQ@public.gmane.org + tglx-hfZtesqFncYOwBW4kG4KsQ@public.gmane.org + linux-0h96xk9xTtrk1uMJSBkQmQ@public.gmane.org + " linux-watchdog-u79uwXL29TY76Z2rM5mHXA@public.gmane.org\0" "\00:1\0" "b\0" "On Fri, 08 May 2015, Peter Griffin wrote:\n" "> On Fri, 08 May 2015, Lee Jones wrote:\n" - "> > > > > > =20\n" - "> > > > > > -- compatible \t: Must be one of: \"st,stih407-lpc\" \"st,stih416-l=\n" - "pc\"\n" + "> > > > > > \n" + "> > > > > > -- compatible \t: Must be one of: \"st,stih407-lpc\" \"st,stih416-lpc\"\n" "> > > > > > -\t\t\t\t \"st,stih415-lpc\" \"st,stid127-lpc\"\n" "> > > > > > +- compatible \t: Must be one of: \"st,stih407-lpc\"\n" - "> > > > >=20\n" - "> > > > > The same comment as the RTC DT patch, you are removing the compat=\n" - "ibles\n" + "> > > > > \n" + "> > > > > The same comment as the RTC DT patch, you are removing the compatibles\n" "> > > > > documentation for the other supported platforms like stih416-lpc.\n" - "> > > > > AFAIK they are required in the driver to get the correct sysconfi=\n" - "g register.\n" - "> > > >=20\n" + "> > > > > AFAIK they are required in the driver to get the correct sysconfig register.\n" + "> > > > \n" "> > > > That's intentional. I haven't yet tested any of this IP on\n" "> > > > STiH41{5,6} & STiH127. Due to lack of documentation, I'm not even\n" "> > > > sure if this IP even exists on some of the other platforms. I will\n" "> > > > add them back when support is added to both driver and DTB and I've\n" "> > > > been able to test them.\n" - "> > >=20\n" - "> > > That was kind of my point, the driver code AFAIK already contains sup=\n" - "port\n" + "> > > \n" + "> > > That was kind of my point, the driver code AFAIK already contains support\n" "> > > for these SoC's.\n" - "> > >=20\n" - "> > > I would either expect the patch to remove support from the DT docs AN=\n" - "D the\n" + "> > > \n" + "> > > I would either expect the patch to remove support from the DT docs AND the\n" "> > > driver, or leave it as is.\n" - "> > >=20\n" + "> > > \n" "> > > It seems odd to only change the DT docs, and become unaligned to the\n" - "> > > code (this assumes I'm looking at the latest patchset here=20\n" - "> > > https://lkml.org/lkml/2015/3/4/1088 which includes support for these =\n" - "SoCs).\n" - "> >=20\n" + "> > > code (this assumes I'm looking at the latest patchset here \n" + "> > > https://lkml.org/lkml/2015/3/4/1088 which includes support for these SoCs).\n" + "> > \n" "> > The decision to remove these 'supported' platforms was made on the RTC\n" "> > side, where there is only support for \"st,stih407-lpc\" in the driver.\n" - ">=20\n" + "> \n" "> The change makes sense in the RTC case then.\n" - ">=20\n" + "> \n" "> > I thought it best to mirror that thought over to the Watchdog LPC\n" "> > bindings,\n" - ">=20\n" - "> Why? This is the bit I disagree with, the wdt DT docs IMHO should match t=\n" - "he wdt\n" + "> \n" + "> Why? This is the bit I disagree with, the wdt DT docs IMHO should match the wdt\n" "> driver code.\n" "\n" "Fair enough. I'll add the 'supported' compatibles back in.\n" "\n" - "--=20\n" + "-- \n" "Lee Jones\n" "Linaro STMicroelectronics Landing Team Lead\n" - "Linaro.org =E2=94=82 Open source software for ARM SoCs\n" + "Linaro.org \342\224\202 Open source software for ARM SoCs\n" "Follow Linaro: Facebook | Twitter | Blog\n" - "\n" - "--=20\n" - "--=20\n" - "You received this message because you are subscribed to \"rtc-linux\".\n" - "Membership options at http://groups.google.com/group/rtc-linux .\n" - "Please read http://groups.google.com/group/rtc-linux/web/checklist\n" - "before submitting a driver.\n" - "---=20\n" - "You received this message because you are subscribed to the Google Groups \"=\n" - "rtc-linux\" group.\n" - "To unsubscribe from this group and stop receiving emails from it, send an e=\n" - "mail to rtc-linux+unsubscribe@googlegroups.com.\n" - For more options, visit https://groups.google.com/d/optout. + "--\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 -1598512223918cee6cc272c328e26189454fb64d04b8373d4316e8446e51a65d +ffdd20f95bf16942b3d4f3cbcd6e8c1d2871be8bef9dfecb2f10509690e6c15e
diff --git a/a/1.txt b/N4/1.txt index 16d8143..63be3db 100644 --- a/a/1.txt +++ b/N4/1.txt @@ -1,65 +1,45 @@ On Fri, 08 May 2015, Peter Griffin wrote: > On Fri, 08 May 2015, Lee Jones wrote: -> > > > > > =20 -> > > > > > -- compatible : Must be one of: "st,stih407-lpc" "st,stih416-l= -pc" +> > > > > > +> > > > > > -- compatible : Must be one of: "st,stih407-lpc" "st,stih416-lpc" > > > > > > - "st,stih415-lpc" "st,stid127-lpc" > > > > > > +- compatible : Must be one of: "st,stih407-lpc" -> > > > >=20 -> > > > > The same comment as the RTC DT patch, you are removing the compat= -ibles +> > > > > +> > > > > The same comment as the RTC DT patch, you are removing the compatibles > > > > > documentation for the other supported platforms like stih416-lpc. -> > > > > AFAIK they are required in the driver to get the correct sysconfi= -g register. -> > > >=20 +> > > > > AFAIK they are required in the driver to get the correct sysconfig register. +> > > > > > > > That's intentional. I haven't yet tested any of this IP on > > > > STiH41{5,6} & STiH127. Due to lack of documentation, I'm not even > > > > sure if this IP even exists on some of the other platforms. I will > > > > add them back when support is added to both driver and DTB and I've > > > > been able to test them. -> > >=20 -> > > That was kind of my point, the driver code AFAIK already contains sup= -port +> > > +> > > That was kind of my point, the driver code AFAIK already contains support > > > for these SoC's. -> > >=20 -> > > I would either expect the patch to remove support from the DT docs AN= -D the +> > > +> > > I would either expect the patch to remove support from the DT docs AND the > > > driver, or leave it as is. -> > >=20 +> > > > > > It seems odd to only change the DT docs, and become unaligned to the -> > > code (this assumes I'm looking at the latest patchset here=20 -> > > https://lkml.org/lkml/2015/3/4/1088 which includes support for these = -SoCs). -> >=20 +> > > code (this assumes I'm looking at the latest patchset here +> > > https://lkml.org/lkml/2015/3/4/1088 which includes support for these SoCs). +> > > > The decision to remove these 'supported' platforms was made on the RTC > > side, where there is only support for "st,stih407-lpc" in the driver. ->=20 +> > The change makes sense in the RTC case then. ->=20 +> > > I thought it best to mirror that thought over to the Watchdog LPC > > bindings, ->=20 -> Why? This is the bit I disagree with, the wdt DT docs IMHO should match t= -he wdt +> +> Why? This is the bit I disagree with, the wdt DT docs IMHO should match the wdt > driver code. Fair enough. I'll add the 'supported' compatibles back in. ---=20 +-- Lee Jones Linaro STMicroelectronics Landing Team Lead -Linaro.org =E2=94=82 Open source software for ARM SoCs +Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog - ---=20 ---=20 -You received this message because you are subscribed to "rtc-linux". -Membership options at http://groups.google.com/group/rtc-linux . -Please read http://groups.google.com/group/rtc-linux/web/checklist -before submitting a driver. ----=20 -You received this message because you are subscribed to the Google Groups "= -rtc-linux" group. -To unsubscribe from this group and stop receiving emails from it, send an e= -mail to rtc-linux+unsubscribe@googlegroups.com. -For more options, visit https://groups.google.com/d/optout. diff --git a/a/content_digest b/N4/content_digest index 491c106..f8a431c 100644 --- a/a/content_digest +++ b/N4/content_digest @@ -6,7 +6,7 @@ "ref\020150508064918.GF16220@x1\0" "ref\020150508071604.GA2232@griffinp-ThinkPad-X1-Carbon-2nd\0" "From\0Lee Jones <lee.jones@linaro.org>\0" - "Subject\0[rtc-linux] Re: [STLinux Kernel] [PATCH 06/12] watchdog: bindings: Supply knowledge of a third supported device - clocksource\0" + "Subject\0Re: [STLinux Kernel] [PATCH 06/12] watchdog: bindings: Supply knowledge of a third supported device - clocksource\0" "Date\0Fri, 8 May 2015 08:25:27 +0100\0" "To\0Peter Griffin <peter.griffin@linaro.org>\0" "Cc\0linux-arm-kernel@lists.infradead.org" @@ -24,68 +24,48 @@ "b\0" "On Fri, 08 May 2015, Peter Griffin wrote:\n" "> On Fri, 08 May 2015, Lee Jones wrote:\n" - "> > > > > > =20\n" - "> > > > > > -- compatible \t: Must be one of: \"st,stih407-lpc\" \"st,stih416-l=\n" - "pc\"\n" + "> > > > > > \n" + "> > > > > > -- compatible \t: Must be one of: \"st,stih407-lpc\" \"st,stih416-lpc\"\n" "> > > > > > -\t\t\t\t \"st,stih415-lpc\" \"st,stid127-lpc\"\n" "> > > > > > +- compatible \t: Must be one of: \"st,stih407-lpc\"\n" - "> > > > >=20\n" - "> > > > > The same comment as the RTC DT patch, you are removing the compat=\n" - "ibles\n" + "> > > > > \n" + "> > > > > The same comment as the RTC DT patch, you are removing the compatibles\n" "> > > > > documentation for the other supported platforms like stih416-lpc.\n" - "> > > > > AFAIK they are required in the driver to get the correct sysconfi=\n" - "g register.\n" - "> > > >=20\n" + "> > > > > AFAIK they are required in the driver to get the correct sysconfig register.\n" + "> > > > \n" "> > > > That's intentional. I haven't yet tested any of this IP on\n" "> > > > STiH41{5,6} & STiH127. Due to lack of documentation, I'm not even\n" "> > > > sure if this IP even exists on some of the other platforms. I will\n" "> > > > add them back when support is added to both driver and DTB and I've\n" "> > > > been able to test them.\n" - "> > >=20\n" - "> > > That was kind of my point, the driver code AFAIK already contains sup=\n" - "port\n" + "> > > \n" + "> > > That was kind of my point, the driver code AFAIK already contains support\n" "> > > for these SoC's.\n" - "> > >=20\n" - "> > > I would either expect the patch to remove support from the DT docs AN=\n" - "D the\n" + "> > > \n" + "> > > I would either expect the patch to remove support from the DT docs AND the\n" "> > > driver, or leave it as is.\n" - "> > >=20\n" + "> > > \n" "> > > It seems odd to only change the DT docs, and become unaligned to the\n" - "> > > code (this assumes I'm looking at the latest patchset here=20\n" - "> > > https://lkml.org/lkml/2015/3/4/1088 which includes support for these =\n" - "SoCs).\n" - "> >=20\n" + "> > > code (this assumes I'm looking at the latest patchset here \n" + "> > > https://lkml.org/lkml/2015/3/4/1088 which includes support for these SoCs).\n" + "> > \n" "> > The decision to remove these 'supported' platforms was made on the RTC\n" "> > side, where there is only support for \"st,stih407-lpc\" in the driver.\n" - ">=20\n" + "> \n" "> The change makes sense in the RTC case then.\n" - ">=20\n" + "> \n" "> > I thought it best to mirror that thought over to the Watchdog LPC\n" "> > bindings,\n" - ">=20\n" - "> Why? This is the bit I disagree with, the wdt DT docs IMHO should match t=\n" - "he wdt\n" + "> \n" + "> Why? This is the bit I disagree with, the wdt DT docs IMHO should match the wdt\n" "> driver code.\n" "\n" "Fair enough. I'll add the 'supported' compatibles back in.\n" "\n" - "--=20\n" + "-- \n" "Lee Jones\n" "Linaro STMicroelectronics Landing Team Lead\n" - "Linaro.org =E2=94=82 Open source software for ARM SoCs\n" - "Follow Linaro: Facebook | Twitter | Blog\n" - "\n" - "--=20\n" - "--=20\n" - "You received this message because you are subscribed to \"rtc-linux\".\n" - "Membership options at http://groups.google.com/group/rtc-linux .\n" - "Please read http://groups.google.com/group/rtc-linux/web/checklist\n" - "before submitting a driver.\n" - "---=20\n" - "You received this message because you are subscribed to the Google Groups \"=\n" - "rtc-linux\" group.\n" - "To unsubscribe from this group and stop receiving emails from it, send an e=\n" - "mail to rtc-linux+unsubscribe@googlegroups.com.\n" - For more options, visit https://groups.google.com/d/optout. + "Linaro.org \342\224\202 Open source software for ARM SoCs\n" + Follow Linaro: Facebook | Twitter | Blog -1598512223918cee6cc272c328e26189454fb64d04b8373d4316e8446e51a65d +e0a5b5f849685d2239f42b8eb031f6bdc5b447074e199eb70e596cc20dcb75a5
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.