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. > On a side note, even though it is hard to see, it seems you use two > spaces instead of a tab above, and that you do the same with other > defines That I can confirm. > (unless the mailer replaced tabs with spaces). > > In general, the widely used form for defines is > > #define<space>MY_DEFINE<tab>definition<tab>/* this is a comment */ I have changed my .emacs config and will test its behavior. > Also, I think > > +static struct i2c_driver isl12057_driver; > > is unnecessary Right. Cheers, a+ -- 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