On Mon, Dec 16, 2013 at 11:24:01PM +0100, Arnaud Ebalard wrote: > Hi Guenter, > > Guenter Roeck <linux@xxxxxxxxxxxx> writes: > > > On Mon, Dec 16, 2013 at 10:17:47PM +0100, Arnaud Ebalard wrote: > >> > >> Intersil ISL12057 I2C RTC chip also supports two alarms. This patch > >> only adds support for basic RTC functionalities (i.e. getting and > >> setting time). Tests have been performed on NETGEAR ReadyNAS 102 w/ > >> startup/shutdown scripts, hwclock, ntpdate and openntpd. > >> > >> Signed-off-by: Arnaud Ebalard <arno@xxxxxxxxxxxx> > > > > Here is a snippet from the saved patch: > > > > obj-$(CONFIG_RTC_DRV_IMXDI) +=3D rtc-imxdi.o > > obj-$(CONFIG_RTC_DRV_ISL1208) +=3D rtc-isl1208.o > > obj-$(CONFIG_RTC_DRV_ISL12022) +=3D rtc-isl12022.o > > +obj-$(CONFIG_RTC_DRV_ISL12057) +=3D rtc-isl12057.o > > obj-$(CONFIG_RTC_DRV_JZ4740) +=3D rtc-jz4740.o > > obj-$(CONFIG_RTC_DRV_LP8788) +=3D rtc-lp8788.o > > obj-$(CONFIG_RTC_DRV_LPC32XX) +=3D rtc-lpc32xx.o > > > > Something is adding those '3D' after each '=' into the patch. > > Well, I simply use git-imap-send to send the patch to my IMAP draft > folder and then sent it from my mailer (Gnus/5.13 for Emacs/24.3). > > What is very weird is that if I simply save the mail I received from > LAKML (I am registered to LAKML which I have put in Cc:), checkptach > gives me the following (my checkpatch.pl is the one in 3.13.0.rc4): > > $ ./scripts/checkpatch.pl /tmp/v4.patch ERROR: Do not include the > paragraph about writing to the Free Software Foundation's mailing > address from the sample GPL notice. The FSF has changed addresses in the > past, and may do so again. Linux already includes a copy of the GPL. > #165: FILE: drivers/rtc/rtc-isl12057.c:23: > + * You should have received a copy of the GNU General Public License$ > > ERROR: Do not include the paragraph about writing to the Free Software > Foundation's mailing address from the sample GPL notice. The FSF has > changed addresses in the past, and may do so again. Linux already > includes a copy of the GPL. > #166: FILE: drivers/rtc/rtc-isl12057.c:24: > + * along with this program; if not, write to the Free Software$ > > total: 2 errors, 0 warnings, 396 lines checked > > /tmp/v4.patch has style problems, please review. > > If any of these errors are false positives, please report > them to the maintainer, see CHECKPATCH in MAINTAINERS. > > Those false/positive are the only error checkptach reports on my side > for the patch. This is the same if I copy/paste the content of the > <pre></pre> with the patch in the following page: > > http://www.spinics.net/lists/arm-kernel/msg294759.html. > > I do not say you are wrong but I do not have a clue were the problem > may come from. > Oddly enough me not either. Let's assume the problem was on my side. I sent you another round of comments a bit ago. Thanks, Guenter -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html