diff for duplicates of <871tmxrsj8.fsf@natisbad.org> diff --git a/a/1.txt b/N1/1.txt index 5c8a904..6485b86 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,6 +1,6 @@ Hi Gregory, -Gregory CLEMENT <gregory.clement@free-electrons.com> writes: +Gregory CLEMENT <gregory.clement-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org> writes: > The new mvebu SoCs come with a new RTC driver. This patch adds the > support for this new IP which is currently found in the Armada 38x @@ -9,7 +9,7 @@ Gregory CLEMENT <gregory.clement@free-electrons.com> writes: > This RTC provides two alarms, but only the first one is used in the > driver. The RTC also allows using periodic interrupts. > -> Signed-off-by: Gregory CLEMENT <gregory.clement@free-electrons.com> +> Signed-off-by: Gregory CLEMENT <gregory.clement-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org> > --- > drivers/rtc/Kconfig | 10 ++ > drivers/rtc/Makefile | 1 + @@ -61,7 +61,7 @@ Gregory CLEMENT <gregory.clement@free-electrons.com> writes: > + * > + * Copyright (C) 2015 Marvell > + * -> + * Gregory Clement <gregory.clement@free-electrons.com> +> + * Gregory Clement <gregory.clement-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org> > + * > + * This program is free software; you can redistribute it and/or > + * modify it under the terms of the GNU General Public License as @@ -112,7 +112,7 @@ Gregory CLEMENT <gregory.clement@free-electrons.com> writes: > +} The thing I do not get is how you can guarantee that an undelayed -writel() in a function will not be followed less than 5?s later by +writel() in a function will not be followed less than 5µs later by another writel() in another function. For instance: 1) a call to set_alarm() followed by a call to alarm_irq_enable(). @@ -138,7 +138,7 @@ Considering the description of the bug, why not doing the following: everywhere. All writes being under the protection of your spinlock, this will -guarantee the 5?s delay in all cases. +guarantee the 5µs delay in all cases. > +static int armada38x_rtc_read_time(struct device *dev, struct rtc_time *tm) @@ -207,3 +207,7 @@ final writel() w/o having enforced the expected 100ms delay. > [SNIP] > +-- +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 4e5607e..2448a62 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,14 +1,29 @@ "ref\01421257155-3379-1-git-send-email-gregory.clement@free-electrons.com\0" "ref\01421257155-3379-3-git-send-email-gregory.clement@free-electrons.com\0" - "From\0arno@natisbad.org (Arnaud Ebalard)\0" - "Subject\0[PATCH v2 2/5] drivers/rtc/rtc-armada38x: Add a new RTC driver for recent mvebu SoCs\0" + "From\0arno-LkuqDEemtHBg9hUCZPvPmw@public.gmane.org (Arnaud Ebalard)\0" + "Subject\0Re: [PATCH v2 2/5] drivers/rtc/rtc-armada38x: Add a new RTC driver for recent mvebu SoCs\0" "Date\0Wed, 14 Jan 2015 21:55:39 +0100\0" - "To\0linux-arm-kernel@lists.infradead.org\0" + "To\0Gregory CLEMENT <gregory.clement-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>\0" + "Cc\0Alessandro Zummo <a.zummo-BfzFCNDTiLLj+vYz1yj4TQ@public.gmane.org>" + rtc-linux-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org + Jason Cooper <jason-NLaQJdtUoK4Be96aLqz0jA@public.gmane.org> + Andrew Lunn <andrew-g2DYL2Zd6BY@public.gmane.org> + Sebastian Hesselbarth <sebastian.hesselbarth-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> + Thomas Petazzoni <thomas.petazzoni-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org> + Ezequiel Garcia <ezequiel.garcia-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org> + linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org + Maxime Ripard <maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org> + Boris BREZILLON <boris.brezillon-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org> + Lior Amsalem <alior-eYqpPyKDWXRBDgjK7y7TUQ@public.gmane.org> + Tawfik Bayouk <tawfik-eYqpPyKDWXRBDgjK7y7TUQ@public.gmane.org> + Nadav Haklai <nadavh-eYqpPyKDWXRBDgjK7y7TUQ@public.gmane.org> + Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org> + " devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org\0" "\00:1\0" "b\0" "Hi Gregory,\n" "\n" - "Gregory CLEMENT <gregory.clement@free-electrons.com> writes:\n" + "Gregory CLEMENT <gregory.clement-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org> writes:\n" "\n" "> The new mvebu SoCs come with a new RTC driver. This patch adds the\n" "> support for this new IP which is currently found in the Armada 38x\n" @@ -17,7 +32,7 @@ "> This RTC provides two alarms, but only the first one is used in the\n" "> driver. The RTC also allows using periodic interrupts.\n" ">\n" - "> Signed-off-by: Gregory CLEMENT <gregory.clement@free-electrons.com>\n" + "> Signed-off-by: Gregory CLEMENT <gregory.clement-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>\n" "> ---\n" "> drivers/rtc/Kconfig | 10 ++\n" "> drivers/rtc/Makefile | 1 +\n" @@ -69,7 +84,7 @@ "> + *\n" "> + * Copyright (C) 2015 Marvell\n" "> + *\n" - "> + * Gregory Clement <gregory.clement@free-electrons.com>\n" + "> + * Gregory Clement <gregory.clement-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>\n" "> + *\n" "> + * This program is free software; you can redistribute it and/or\n" "> + * modify it under the terms of the GNU General Public License as\n" @@ -120,7 +135,7 @@ "> +}\n" "\n" "The thing I do not get is how you can guarantee that an undelayed\n" - "writel() in a function will not be followed less than 5?s later by\n" + "writel() in a function will not be followed less than 5\302\265s later by\n" "another writel() in another function. For instance:\n" "\n" " 1) a call to set_alarm() followed by a call to alarm_irq_enable().\n" @@ -146,7 +161,7 @@ " everywhere.\n" "\n" "All writes being under the protection of your spinlock, this will\n" - "guarantee the 5?s delay in all cases.\n" + "guarantee the 5\302\265s delay in all cases.\n" "\n" "\n" "> +static int armada38x_rtc_read_time(struct device *dev, struct rtc_time *tm)\n" @@ -214,6 +229,10 @@ "\n" "\n" "> [SNIP]\n" - > + ">\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 -167d8729910aa70b22e89355509236fa3ea14370aaf2921aa7afbf3652ae46bf +42a04ae136432eea2d6ec72e1e09b107c1f927102d891c921108e0d83a7da682
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.