[PATCH] Style Fix: Blank lines after declaration

Sudip Mukherjee sudipm.mukherjee at gmail.com
Fri Dec 11 05:38:56 UTC 2015


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 at yahoo.com>
> ---
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


More information about the devel mailing list