Messages in this thread | | | Subject | Re: [PATCH v2] staging: wilc1000: Remove unnecessary braces {} around single statement block | From | Claudiu Beznea <> | Date | Thu, 12 Apr 2018 11:16:55 +0300 |
| |
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@gmail.com>
Reviewed-by: Claudiu Beznea <claudiu.beznea@microchip.com>
> --- > 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) >
| |