[ Please do not top-post. ] On Fri, Jan 16, 2015 at 10:22:56AM +0530, Jamal Mohammad wrote: > I think you are write ... checkpatch.pl was giving the error at the > line so i added a blank line... i will send an updated patch.. You should not trust checkpatch.pl blindly, and not run it on in-kernel files either. Use it on your own patches as a way to find (possible) mistakes before submission. Johan -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH] gpio: gpio-dln2: Added a Blank line after declaration
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
- Subject: Re: [PATCH] gpio: gpio-dln2: Added a Blank line after declaration
- From: Johan Hovold <johan@xxxxxxxxxx>
- Date: Fri, 16 Jan 2015 09:51:42 +0100
- In-reply-to: <CAP8vtK+vHR_7h_SYWfBNDZkAqUbtEdSDqw+AteZWGqoY37tYTQ@mail.gmail.com>
- User-agent: Mutt/1.5.22 (2013-10-16)
- References:
- [PATCH] gpio: gpio-dln2: Added a Blank line after declaration
- From: Mohammad Jamal
- Re: [PATCH] gpio: gpio-dln2: Added a Blank line after declaration
- From: Linus Walleij
- Re: [PATCH] gpio: gpio-dln2: Added a Blank line after declaration
- From: Johan Hovold
- Re: [PATCH] gpio: gpio-dln2: Added a Blank line after declaration
- From: Jamal Mohammad
- [PATCH] gpio: gpio-dln2: Added a Blank line after declaration
- Prev by Date: [PATCH v3 3/3] arm64: dts: Add APM X-Gene standby GPIO controller DTS entries
- Next by Date: Re: [PATCH v6 2/3] gpio: Cygnus: add GPIO driver
- Previous by thread: Re: [PATCH] gpio: gpio-dln2: Added a Blank line after declaration
- Next by thread: [PATCH][v2] gpio-generic: add bgpio_set_multiple functions
- Index(es):
 |