s390/qdio: use dev_info() in qdio_print_subchannel_info()
authorJulian Wiedmann <jwi@linux.ibm.com>
Mon, 12 Jul 2021 07:39:57 +0000 (09:39 +0200)
committerVasily Gorbik <gor@linux.ibm.com>
Wed, 18 Aug 2021 08:01:28 +0000 (10:01 +0200)
Prefer dev_info() over a raw printk. This also adds the device and
driver names into the output, so that we have:

Before:
    qdio: 0.0.17c0 ZFCP on SC 17 using [...]

After:
    zfcp 0.0.17c0: qdio: ZFCP on SC 17 using [...]

Signed-off-by: Julian Wiedmann <jwi@linux.ibm.com>
Reviewed-by: Benjamin Block <bblock@linux.ibm.com>
Signed-off-by: Vasily Gorbik <gor@linux.ibm.com>
drivers/s390/cio/qdio_setup.c

index ebefe82..8a3c260 100644 (file)
@@ -479,10 +479,8 @@ void qdio_shutdown_irq(struct qdio_irq *irq)
 
 void qdio_print_subchannel_info(struct qdio_irq *irq_ptr)
 {
-       char s[80];
-
-       snprintf(s, 80, "qdio: %s %s on SC %x using AI:%d QEBSM:%d PRI:%d TDD:%d SIGA:%s%s%s\n",
-                dev_name(&irq_ptr->cdev->dev),
+       dev_info(&irq_ptr->cdev->dev,
+                "qdio: %s on SC %x using AI:%d QEBSM:%d PRI:%d TDD:%d SIGA:%s%s%s\n",
                 (irq_ptr->qib.qfmt == QDIO_QETH_QFMT) ? "OSA" :
                        ((irq_ptr->qib.qfmt == QDIO_ZFCP_QFMT) ? "ZFCP" : "HS"),
                 irq_ptr->schid.sch_no,
@@ -493,7 +491,6 @@ void qdio_print_subchannel_info(struct qdio_irq *irq_ptr)
                 qdio_need_siga_in(irq_ptr) ? "R" : " ",
                 qdio_need_siga_out(irq_ptr) ? "W" : " ",
                 qdio_need_siga_sync(irq_ptr) ? "S" : " ");
-       printk(KERN_INFO "%s", s);
 }
 
 int __init qdio_setup_init(void)