diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2017-11-08 03:38:01 -0500 |
---|---|---|
committer | Martin K. Petersen <martin.petersen@oracle.com> | 2017-11-08 18:16:38 -0500 |
commit | 6767aced2ff1d4f214b2f0632d04dce02e6b6b2a (patch) | |
tree | d47d16da69090bfe31b6967b89edb8b28cac36a0 | |
parent | 820f188659122602ab217dd80cfa32b3ac0c55c0 (diff) |
scsi: mpt3sas: cleanup _scsih_pcie_enumeration_event()
The indenting wasn't right, because the last two prints weren't indented
far enough. Also it used pr_info() where it was supposed to use
pr_cont(). I reversed the if statement and pulled the code in one tab
and did a couple other minor cleanups.
Fixes: 4318c7347847 ("scsi: mpt3sas: Handle NVMe PCIe device related events generated from firmware.")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by: Sathya Prakash Veerichetty <sathya.prakash@broadcom.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
-rw-r--r-- | drivers/scsi/mpt3sas/mpt3sas_scsih.c | 21 |
1 files changed, 11 insertions, 10 deletions
diff --git a/drivers/scsi/mpt3sas/mpt3sas_scsih.c b/drivers/scsi/mpt3sas/mpt3sas_scsih.c index 93c5fe062686..18d1804c674f 100644 --- a/drivers/scsi/mpt3sas/mpt3sas_scsih.c +++ b/drivers/scsi/mpt3sas/mpt3sas_scsih.c | |||
@@ -7700,17 +7700,18 @@ _scsih_pcie_enumeration_event(struct MPT3SAS_ADAPTER *ioc, | |||
7700 | Mpi26EventDataPCIeEnumeration_t *event_data = | 7700 | Mpi26EventDataPCIeEnumeration_t *event_data = |
7701 | (Mpi26EventDataPCIeEnumeration_t *)fw_event->event_data; | 7701 | (Mpi26EventDataPCIeEnumeration_t *)fw_event->event_data; |
7702 | 7702 | ||
7703 | if (ioc->logging_level & MPT_DEBUG_EVENT_WORK_TASK) { | 7703 | if (!(ioc->logging_level & MPT_DEBUG_EVENT_WORK_TASK)) |
7704 | pr_info(MPT3SAS_FMT "pcie enumeration event: (%s) Flag 0x%02x", | 7704 | return; |
7705 | ioc->name, | 7705 | |
7706 | ((event_data->ReasonCode == | 7706 | pr_info(MPT3SAS_FMT "pcie enumeration event: (%s) Flag 0x%02x", |
7707 | MPI26_EVENT_PCIE_ENUM_RC_STARTED) ? | 7707 | ioc->name, |
7708 | "started" : "completed"), event_data->Flags); | 7708 | (event_data->ReasonCode == MPI26_EVENT_PCIE_ENUM_RC_STARTED) ? |
7709 | "started" : "completed", | ||
7710 | event_data->Flags); | ||
7709 | if (event_data->EnumerationStatus) | 7711 | if (event_data->EnumerationStatus) |
7710 | pr_info("enumeration_status(0x%08x)", | 7712 | pr_cont("enumeration_status(0x%08x)", |
7711 | le32_to_cpu(event_data->EnumerationStatus)); | 7713 | le32_to_cpu(event_data->EnumerationStatus)); |
7712 | pr_info("\n"); | 7714 | pr_cont("\n"); |
7713 | } | ||
7714 | } | 7715 | } |
7715 | 7716 | ||
7716 | /** | 7717 | /** |