scsi: ufs: non functional macro fix
authorTomohiro Kusumi <tkusumi@tuxera.com>
Tue, 28 Mar 2017 13:49:26 +0000 (16:49 +0300)
committerMartin K. Petersen <martin.petersen@oracle.com>
Thu, 30 Mar 2017 02:42:21 +0000 (22:42 -0400)
Not having () isn't likely to do any harm in this case, but all the
other macros below do have it. Also add "are" in a comment.

Signed-off-by: Tomohiro Kusumi <tkusumi@tuxera.com>
Reviewed-by: Subhash Jadavani <subhashj@codeaurora.org>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/ufs/ufshcd.c

index dd4625919f2283f86d8a0f6cdf8574935d76678f..089b76f43ad963fd98dfa5c1d568745b37655434 100644 (file)
@@ -151,11 +151,11 @@ enum {
 };
 
 #define ufshcd_set_eh_in_progress(h) \
-       (h->eh_flags |= UFSHCD_EH_IN_PROGRESS)
+       ((h)->eh_flags |= UFSHCD_EH_IN_PROGRESS)
 #define ufshcd_eh_in_progress(h) \
-       (h->eh_flags & UFSHCD_EH_IN_PROGRESS)
+       ((h)->eh_flags & UFSHCD_EH_IN_PROGRESS)
 #define ufshcd_clear_eh_in_progress(h) \
-       (h->eh_flags &= ~UFSHCD_EH_IN_PROGRESS)
+       ((h)->eh_flags &= ~UFSHCD_EH_IN_PROGRESS)
 
 #define ufshcd_set_ufs_dev_active(h) \
        ((h)->curr_dev_pwr_mode = UFS_ACTIVE_PWR_MODE)
@@ -1491,7 +1491,7 @@ start:
                        break;
                }
                /*
-                * If we here, it means gating work is either done or
+                * If we are here, it means gating work is either done or
                 * currently running. Hence, fall through to cancel gating
                 * work and to enable clocks.
                 */