Re: [PATCH] Staging: rtl8192e: Fix warning exceeding characters per line

From	Greg KH <gregkh@linuxfoundation.org>
Newsgroups	linux.kernel
Subject	Re: [PATCH] Staging: rtl8192e: Fix warning exceeding characters per line
Date	2016-03-22 03:10 +0100
Message-ID	<rfjVE-67W-9@gated-at.bofh.it> (permalink)
References	<rbTl0-7Va-5@gated-at.bofh.it> <rbTl0-7Va-3@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On Sat, Mar 12, 2016 at 04:01:03PM +0100, Adonis Settouf wrote:
> This patch fixes the checkpatch.pl warning regarding the exceeding 80
> characters on line 36. This patch add a newline after function statement
> on line 52

You did 2 different things in the same patch, that should be 2 patches.

> 
> Signed-off-by: Adonis Settouf <adonis.settouf@gmail.com>
> ---
>  drivers/staging/rtl8192e/dot11d.h | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/staging/rtl8192e/dot11d.h b/drivers/staging/rtl8192e/dot11d.h
> index 735a199..5cffc8f 100644
> --- a/drivers/staging/rtl8192e/dot11d.h
> +++ b/drivers/staging/rtl8192e/dot11d.h
> @@ -30,8 +30,8 @@ enum dot11d_state {
>  };
>  
>  /**
> - * struct rt_dot11d_info * @CountryIeLen: value greater than 0 if @CountryIeBuf contains
> - *		  valid country information element.
> + * struct rt_dot11d_info * @CountryIeLen: value greater than 0 if
> + * @CountryIeBuf contains valid country information element.

You forgot to indent like the rest of this comment block did :(

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

Thread

Re: [PATCH] Staging: rtl8192e: Fix warning exceeding characters per  line Greg KH <gregkh@linuxfoundation.org> - 2016-03-22 03:10 +0100

csiph-web