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


Groups > linux.kernel > #1559044

Re: [PATCH] drivers: staging: rtl8188eu: include: Fix semicolon after macro definition

From Greg KH <gregkh@linuxfoundation.org>
Newsgroups linux.kernel
Subject Re: [PATCH] drivers: staging: rtl8188eu: include: Fix semicolon after macro definition
Date 2017-01-14 19:40 +0100
Message-ID <sZBp8-Qr-19@gated-at.bofh.it> (permalink)
References <sZB5L-Kf-7@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On Sat, Jan 14, 2017 at 11:47:56PM +0530, Kartikey Singh wrote:
> fixed a trailing semicolon after macro definition
> 
> Signed-off-by: Kartikey Singh <dr.kartikeynrc@gmail.com>
> ---
>  drivers/staging/rtl8188eu/include/basic_types.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/staging/rtl8188eu/include/basic_types.h b/drivers/staging/rtl8188eu/include/basic_types.h
> index 69c4d49..cbd6f92 100644
> --- a/drivers/staging/rtl8188eu/include/basic_types.h
> +++ b/drivers/staging/rtl8188eu/include/basic_types.h
> @@ -128,7 +128,7 @@ value to host byte ordering.*/
>  		(						\
>  		LE_BITS_CLEARED_TO_2BYTE(__pstart, __bitoffset, __bitlen) | \
>  		((((u16)__val) & BIT_LEN_MASK_16(__bitlen)) << (__bitoffset)) \
> -		);
> +		)

This macro, and others in this file, are also never used, can you just
remove them too?

thanks,

greg k-h

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


Thread

[PATCH] drivers: staging: rtl8188eu: include: Fix semicolon after  macro definition Kartikey Singh <dr.kartikeynrc@gmail.com> - 2017-01-14 19:20 +0100
  Re: [PATCH] drivers: staging: rtl8188eu: include: Fix semicolon  after macro definition Greg KH <gregkh@linuxfoundation.org> - 2017-01-14 19:40 +0100

csiph-web