From: Oleg Drokin Date: Fri, 26 Feb 2016 06:50:12 +0000 (-0500) Subject: staging/lustre: Remove unneeded {} in lprocfs_stats_unlock() X-Git-Tag: next-20160307~21^2~78 X-Git-Url: https://git.karo-electronics.de/?a=commitdiff_plain;h=da24e7715f0c663156df7c492d73f1346edaff7f;p=karo-tx-linux.git staging/lustre: Remove unneeded {} in lprocfs_stats_unlock() Since there's only one call in those if () else branches, the braces are not really necessary. Highlighted by checkpatch. Signed-off-by: Oleg Drokin Signed-off-by: Greg Kroah-Hartman --- diff --git a/drivers/staging/lustre/lustre/include/lprocfs_status.h b/drivers/staging/lustre/lustre/include/lprocfs_status.h index 89052b209762..4146c9c3999f 100644 --- a/drivers/staging/lustre/lustre/include/lprocfs_status.h +++ b/drivers/staging/lustre/lustre/include/lprocfs_status.h @@ -426,11 +426,10 @@ static inline void lprocfs_stats_unlock(struct lprocfs_stats *stats, int opc, case LPROCFS_GET_SMP_ID: if (stats->ls_flags & LPROCFS_STATS_FLAG_NOPERCPU) { - if (stats->ls_flags & LPROCFS_STATS_FLAG_IRQ_SAFE) { + if (stats->ls_flags & LPROCFS_STATS_FLAG_IRQ_SAFE) spin_unlock_irqrestore(&stats->ls_lock, *flags); - } else { + else spin_unlock(&stats->ls_lock); - } } else { put_cpu(); } @@ -438,11 +437,10 @@ static inline void lprocfs_stats_unlock(struct lprocfs_stats *stats, int opc, case LPROCFS_GET_NUM_CPU: if (stats->ls_flags & LPROCFS_STATS_FLAG_NOPERCPU) { - if (stats->ls_flags & LPROCFS_STATS_FLAG_IRQ_SAFE) { + if (stats->ls_flags & LPROCFS_STATS_FLAG_IRQ_SAFE) spin_unlock_irqrestore(&stats->ls_lock, *flags); - } else { + else spin_unlock(&stats->ls_lock); - } } return; }