Re: [PATCH] Style Fix: Blank lines after declaration

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Dec 10, 2015 at 06:00:14AM -0700, BryanSPaul wrote:
> WARNING: Missing a blank line after declarations lines added
> 
> Signed-off-by: Bryan Paul <bryan.paul@xxxxxxxxx>
> ---
Your name in the From: header and Signedoff-by name should match.

>  drivers/staging/rtl8188eu/core/rtw_efuse.c | 13 +++++++++++++
>  1 file changed, 13 insertions(+)
> 
> diff --git a/drivers/staging/rtl8188eu/core/rtw_efuse.c b/drivers/staging/rtl8188eu/core/rtw_efuse.c
> index eb89423..5f71088 100644
> --- a/drivers/staging/rtl8188eu/core/rtw_efuse.c
> +++ b/drivers/staging/rtl8188eu/core/rtw_efuse.c
> @@ -220,6 +220,7 @@ static void efuse_read_phymap_from_txpktbuf(
>  	struct adapter  *adapter,
>  	int bcnhead,	/* beacon head, where FW store len(2-byte) and efuse physical map. */
>  	u8 *content,	/* buffer to store efuse physical map */
> +
>  	u16 *size	/* for efuse content: the max byte to read. will update to byte read */

Why you will need a blank line here? These are part of function
arguments.

regards
sudip
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel



[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux