[PATCHv4] rtc: Add support for Intersil ISL12057 I2C RTC chip

[Arnaud Ebalard]

Hi Guenter,

Guenter Roeck <linux at roeck-us.net> 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 at natisbad.org>
>
> 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+