staging/lustre: Remove unneeded {} in lprocfs_stats_unlock()
authorOleg Drokin <green@linuxhacker.ru>
Fri, 26 Feb 2016 06:50:12 +0000 (01:50 -0500)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 2 Mar 2016 03:17:41 +0000 (19:17 -0800)
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 <green@linuxhacker.ru>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/lustre/lustre/include/lprocfs_status.h

index 89052b2..4146c9c 100644 (file)
@@ -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;
        }