Re: [PATCH v2] staging: wilc1000: Remove unnecessary braces {} around single statement block

From: Claudiu Beznea
Date: Thu Apr 12 2018 - 04:17:01 EST




On 12.04.2018 10:59, Eyal Ilsar wrote:
> Remove unnecessary braces {} around an 'if' statement block with a single
> statement. Issue found by checkpatch.
>
> Signed-off-by: Eyal Ilsar <edilsar@xxxxxxxxx>

Reviewed-by: Claudiu Beznea <claudiu.beznea@xxxxxxxxxxxxx>

> ---
> Added an empty line before the 'Signed-off-by' line and a space between the
> name and e-mail address within that line.
>
> drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/drivers/staging/wilc1000/wilc_wfi_cfgoperations.c b/drivers/staging/wilc1000/wilc_wfi_cfgoperations.c
> index 205304c..325afe1 100644
> --- a/drivers/staging/wilc1000/wilc_wfi_cfgoperations.c
> +++ b/drivers/staging/wilc1000/wilc_wfi_cfgoperations.c
> @@ -284,9 +284,8 @@ static void remove_network_from_shadow(struct timer_list *unused)
> }
> }
>
> - if (last_scanned_cnt != 0) {
> + if (last_scanned_cnt != 0)
> mod_timer(&hAgingTimer, jiffies + msecs_to_jiffies(AGING_TIME));
> - }
> }
>
> static void clear_duringIP(struct timer_list *unused)
>