Commit 2ce6e200 authored by Joe Perches's avatar Joe Perches Committed by Martin K. Petersen

scsi: pm8001: Convert pm8001_printk() to pm8001_info()

Use the more common logging style.

[mkp: fixed a few conflicts]

Link: https://lore.kernel.org/r/69dc34ff63adfa60b3f203ed2d58143b5692af57.1606192458.git.joe@perches.comAcked-by: default avatarJack Wang <jinpu.wang@cloud.ionos.com>
Signed-off-by: default avatarJoe Perches <joe@perches.com>
Signed-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
parent ac85cca3
...@@ -1290,7 +1290,7 @@ static int __maybe_unused pm8001_pci_suspend(struct device *dev) ...@@ -1290,7 +1290,7 @@ static int __maybe_unused pm8001_pci_suspend(struct device *dev)
for (j = 0; j < PM8001_MAX_MSIX_VEC; j++) for (j = 0; j < PM8001_MAX_MSIX_VEC; j++)
tasklet_kill(&pm8001_ha->tasklet[j]); tasklet_kill(&pm8001_ha->tasklet[j]);
#endif #endif
pm8001_printk(pm8001_ha, "pdev=0x%p, slot=%s, entering " pm8001_info(pm8001_ha, "pdev=0x%p, slot=%s, entering "
"suspended state\n", pdev, "suspended state\n", pdev,
pm8001_ha->name); pm8001_ha->name);
return 0; return 0;
...@@ -1314,7 +1314,7 @@ static int __maybe_unused pm8001_pci_resume(struct device *dev) ...@@ -1314,7 +1314,7 @@ static int __maybe_unused pm8001_pci_resume(struct device *dev)
pm8001_ha = sha->lldd_ha; pm8001_ha = sha->lldd_ha;
device_state = pdev->current_state; device_state = pdev->current_state;
pm8001_printk(pm8001_ha, "pdev=0x%p, slot=%s, resuming from previous operating state [D%d]\n", pm8001_info(pm8001_ha, "pdev=0x%p, slot=%s, resuming from previous operating state [D%d]\n",
pdev, pm8001_ha->name, device_state); pdev, pm8001_ha->name, device_state);
rc = pci_go_44(pdev); rc = pci_go_44(pdev);
......
...@@ -1191,7 +1191,7 @@ int pm8001_abort_task(struct sas_task *task) ...@@ -1191,7 +1191,7 @@ int pm8001_abort_task(struct sas_task *task)
phy_id = pm8001_dev->attached_phy; phy_id = pm8001_dev->attached_phy;
ret = pm8001_find_tag(task, &tag); ret = pm8001_find_tag(task, &tag);
if (ret == 0) { if (ret == 0) {
pm8001_printk(pm8001_ha, "no tag for task:%p\n", task); pm8001_info(pm8001_ha, "no tag for task:%p\n", task);
return TMF_RESP_FUNC_FAILED; return TMF_RESP_FUNC_FAILED;
} }
spin_lock_irqsave(&task->task_state_lock, flags); spin_lock_irqsave(&task->task_state_lock, flags);
...@@ -1313,7 +1313,7 @@ int pm8001_abort_task(struct sas_task *task) ...@@ -1313,7 +1313,7 @@ int pm8001_abort_task(struct sas_task *task)
task->slow_task = NULL; task->slow_task = NULL;
spin_unlock_irqrestore(&task->task_state_lock, flags); spin_unlock_irqrestore(&task->task_state_lock, flags);
if (rc != TMF_RESP_FUNC_COMPLETE) if (rc != TMF_RESP_FUNC_COMPLETE)
pm8001_printk(pm8001_ha, "rc= %d\n", rc); pm8001_info(pm8001_ha, "rc= %d\n", rc);
return rc; return rc;
} }
......
...@@ -70,14 +70,14 @@ ...@@ -70,14 +70,14 @@
#define PM8001_DEVIO_LOGGING 0x100 /* development io message logging */ #define PM8001_DEVIO_LOGGING 0x100 /* development io message logging */
#define PM8001_IOERR_LOGGING 0x200 /* development io err message logging */ #define PM8001_IOERR_LOGGING 0x200 /* development io err message logging */
#define pm8001_printk(HBA, fmt, ...) \ #define pm8001_info(HBA, fmt, ...) \
pr_info("%s:: %s %d:" fmt, \ pr_info("%s:: %s %d:" fmt, \
(HBA)->name, __func__, __LINE__, ##__VA_ARGS__) (HBA)->name, __func__, __LINE__, ##__VA_ARGS__)
#define pm8001_dbg(HBA, level, fmt, ...) \ #define pm8001_dbg(HBA, level, fmt, ...) \
do { \ do { \
if (unlikely((HBA)->logging_level & PM8001_##level##_LOGGING)) \ if (unlikely((HBA)->logging_level & PM8001_##level##_LOGGING)) \
pm8001_printk(HBA, fmt, ##__VA_ARGS__); \ pm8001_info(HBA, fmt, ##__VA_ARGS__); \
} while (0) } while (0)
#define PM8001_USE_TASKLET #define PM8001_USE_TASKLET
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment