Simplify the implementation of this function by using the phC format
specifier instead of using explicit for-loops.

Cc: Himanshu Madhani <hmadh...@marvell.com>
Cc: Giridhar Malavali <gmalav...@marvell.com>
Signed-off-by: Bart Van Assche <bvanass...@acm.org>
---
 drivers/scsi/qla2xxx/qla_target.c | 18 +++---------------
 1 file changed, 3 insertions(+), 15 deletions(-)

diff --git a/drivers/scsi/qla2xxx/qla_target.c 
b/drivers/scsi/qla2xxx/qla_target.c
index 01c9bbbfa256..153f78aeb4b0 100644
--- a/drivers/scsi/qla2xxx/qla_target.c
+++ b/drivers/scsi/qla2xxx/qla_target.c
@@ -6507,22 +6507,10 @@ void qlt_remove_target_resources(struct qla_hw_data *ha)
 static void qlt_lport_dump(struct scsi_qla_host *vha, u64 wwpn,
        unsigned char *b)
 {
-       int i;
-
-       pr_debug("qla2xxx HW vha->node_name: ");
-       for (i = 0; i < WWN_SIZE; i++)
-               pr_debug("%02x ", vha->node_name[i]);
-       pr_debug("\n");
-       pr_debug("qla2xxx HW vha->port_name: ");
-       for (i = 0; i < WWN_SIZE; i++)
-               pr_debug("%02x ", vha->port_name[i]);
-       pr_debug("\n");
-
-       pr_debug("qla2xxx passed configfs WWPN: ");
+       pr_debug("qla2xxx HW vha->node_name: %8phC\n", vha->node_name);
+       pr_debug("qla2xxx HW vha->port_name: %8phC\n", vha->port_name);
        put_unaligned_be64(wwpn, b);
-       for (i = 0; i < WWN_SIZE; i++)
-               pr_debug("%02x ", b[i]);
-       pr_debug("\n");
+       pr_debug("qla2xxx passed configfs WWPN: %8phC\n", b);
 }
 
 /**
-- 
2.22.0.rc1

Reply via email to