Re: [Outreachy kernel] [PATCH] Staging: rtl8188eu: Add blank line after declarations

[Jes Sorensen <jes.sorensen@gmail.com>]

On 02/23/15 17:32, Melike Yurtoglu wrote:
> WARNING: "Missing a blank line after declarations"
> That was found by running checkpatch
> 
> Signed-off-by: Melike Yurtoglu <aysemelikeyurtoglu@gmail.com>
> ---
>  drivers/staging/rtl8188eu/hal/Hal8188ERateAdaptive.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/staging/rtl8188eu/hal/Hal8188ERateAdaptive.c b/drivers/staging/rtl8188eu/hal/Hal8188ERateAdaptive.c
> index 3c651d5..2c23aa9 100644
> --- a/drivers/staging/rtl8188eu/hal/Hal8188ERateAdaptive.c
> +++ b/drivers/staging/rtl8188eu/hal/Hal8188ERateAdaptive.c
> @@ -538,6 +538,7 @@ int ODM_RAInfo_Init(struct odm_dm_struct *dm_odm, u8 macid)
>  	struct odm_ra_info *pRaInfo = &dm_odm->RAInfo[macid];
>  	u8 WirelessMode = 0xFF; /* invalid value */
>  	u8 max_rate_idx = 0x13; /* MCS7 */
> +
>  	if (dm_odm->pWirelessMode != NULL)
>  		WirelessMode = *(dm_odm->pWirelessMode);
>  
> 

Patch looks fine - I as always prefer focusing on a slightly more
personal commit message, and less so on the checkpatch warning, but I
think this one is fine. I don't think you need to re-post this patch.

Jes