Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1345694

Re: [PATCH] staging: rtl8192e: rtllib_softmac.c: Remove multiple blank lines

Show all headers | View raw

On Sunday 28 February 2016 08:18 PM, Tapan Prakash T wrote:
> Fixed blank lines issues reported by checkpatch.pl
Hi,

I can see that you have sent 3 patches for the same file. But they are
sent as separate patches. So, this might be confusing for the maintainer.

So, in this case these are some points you need to keep in mind:

1. You can send a single patch for the same warnings/errors in the same
file or directory covering all occurrences.

2. If you want to send patches for different errors in a single file then you
need to send a patch in the form of a patchset.

I think this should help you with understanding my point in a better way:

https://www.kernel.org/doc/Documentation/SubmittingPatches

> Signed-off-by: Tapan Prakash T <tapanprakasht@gmail.com>
> ---
>  drivers/staging/rtl8192e/rtllib_softmac.c | 2 --
>  1 file changed, 2 deletions(-)
>
> diff --git a/drivers/staging/rtl8192e/rtllib_softmac.c b/drivers/staging/rtl8192e/rtllib_softmac.c
> index 25b5b5e..b40298b 100644
> --- a/drivers/staging/rtl8192e/rtllib_softmac.c
> +++ b/drivers/staging/rtl8192e/rtllib_softmac.c
> @@ -13,7 +13,6 @@
>   * released under the GPL
>   */
>  
> -
>  #include "rtllib.h"
>  
>  #include <linux/random.h>
> @@ -25,7 +24,6 @@
>  
>  static void rtllib_sta_wakeup(struct rtllib_device *ieee, short nl);
>  
> -
>  static short rtllib_is_54g(struct rtllib_network *net)
>  {
>  	return (net->rates_ex_len > 0) || (net->rates_len > 4);

-- 
Vaishali

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

[PATCH] staging: rtl8192e: rtllib_softmac.c: Remove multiple blank lines Tapan Prakash T <tapanprakasht@gmail.com> - 2016-02-28 15:50 +0100
  Re: [PATCH] staging: rtl8192e: rtllib_softmac.c: Remove multiple  blank lines Vaishali Thakkar <vaishali.thakkar@oracle.com> - 2016-02-29 12:20 +0100

csiph-web