diff options
| author | Nathan Chancellor <natechancellor@gmail.com> | 2018-09-15 02:36:45 -0400 |
|---|---|---|
| committer | Martin K. Petersen <martin.petersen@oracle.com> | 2018-09-28 02:13:33 -0400 |
| commit | 90ded4e2005b1195a5e781009be991e1cd049c10 (patch) | |
| tree | f938fbb7ec74d9cd181441a7bfdb62a630ddff5d /drivers/message/fusion | |
| parent | eec73c2ec111164b85a41fa83aa7d37e9eb89e3f (diff) | |
scsi: mptfusion: Remove unnecessary parentheses
Clang warns when multiple pairs of parentheses are used for a single
conditional statement.
drivers/message/fusion/mptbase.c:338:11: warning: equality comparison
with extraneous parentheses [-Wparentheses-equality]
if ((ioc == NULL))
~~~~^~~~~~~
drivers/message/fusion/mptbase.c:338:11: note: remove extraneous
parentheses around the comparison to silence this warning
if ((ioc == NULL))
~ ^ ~
drivers/message/fusion/mptbase.c:338:11: note: use '=' to turn this
equality comparison into an assignment
if ((ioc == NULL))
^~
=
drivers/message/fusion/mptbase.c:342:12: warning: equality comparison
with extraneous parentheses [-Wparentheses-equality]
if ((pdev == NULL))
~~~~~^~~~~~~
drivers/message/fusion/mptbase.c:342:12: note: remove extraneous
parentheses around the comparison to silence this warning
if ((pdev == NULL))
~ ^ ~
drivers/message/fusion/mptbase.c:342:12: note: use '=' to turn this
equality comparison into an assignment
if ((pdev == NULL))
^~
=
2 warnings generated.
Remove them and while we're at it, simplify the NULL checks as '!var' is
used more than 'var == NULL'.
Reported-by: Nick Desaulniers <ndesaulniers@google.com>
Signed-off-by: Nathan Chancellor <natechancellor@gmail.com>
Reviewed-by: Nick Desaulniers <ndesaulniers@google.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
Diffstat (limited to 'drivers/message/fusion')
| -rw-r--r-- | drivers/message/fusion/mptbase.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/message/fusion/mptbase.c b/drivers/message/fusion/mptbase.c index dc1e43a02599..ba551d8dfba4 100644 --- a/drivers/message/fusion/mptbase.c +++ b/drivers/message/fusion/mptbase.c | |||
| @@ -335,11 +335,11 @@ static int mpt_remove_dead_ioc_func(void *arg) | |||
| 335 | MPT_ADAPTER *ioc = (MPT_ADAPTER *)arg; | 335 | MPT_ADAPTER *ioc = (MPT_ADAPTER *)arg; |
| 336 | struct pci_dev *pdev; | 336 | struct pci_dev *pdev; |
| 337 | 337 | ||
| 338 | if ((ioc == NULL)) | 338 | if (!ioc) |
| 339 | return -1; | 339 | return -1; |
| 340 | 340 | ||
| 341 | pdev = ioc->pcidev; | 341 | pdev = ioc->pcidev; |
| 342 | if ((pdev == NULL)) | 342 | if (!pdev) |
| 343 | return -1; | 343 | return -1; |
| 344 | 344 | ||
| 345 | pci_stop_and_remove_bus_device_locked(pdev); | 345 | pci_stop_and_remove_bus_device_locked(pdev); |
