scsi: qedi: Remove additional char from boot target iqnname
authorManish Rangankar <mrangankar@marvell.com>
Wed, 8 Apr 2020 06:43:27 +0000 (23:43 -0700)
committerMartin K. Petersen <martin.petersen@oracle.com>
Tue, 14 Apr 2020 01:44:19 +0000 (21:44 -0400)
While parsing the iSCSI TLV data to MFW request, a newline was added to the
firmware boot target iqnname string. Because of this, we were getting the
following error even after the boot target was successfully logged in:

"[qedi_get_protocol_tlv_data:1197]:1: Boot target not set"

Remove the trailing newline.

[mkp: clarified commit desc]

Link: https://lore.kernel.org/r/20200408064332.19377-2-mrangankar@marvell.com
Reviewed-by: Lee Duncan <lduncan@suse.com>
Signed-off-by: Manish Rangankar <mrangankar@marvell.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/qedi/qedi_main.c

index b995b19865ca9b4bd508ff8e35c9dbc762d63fd0..2fe11407445584137140f0e75325a382fdf149d7 100644 (file)
@@ -921,7 +921,7 @@ static void qedi_get_boot_tgt_info(struct nvm_iscsi_block *block,
        ipv6_en = !!(block->generic.ctrl_flags &
                     NVM_ISCSI_CFG_GEN_IPV6_ENABLED);
 
-       snprintf(tgt->iscsi_name, sizeof(tgt->iscsi_name), "%s\n",
+       snprintf(tgt->iscsi_name, sizeof(tgt->iscsi_name), "%s",
                 block->target[index].target_name.byte);
 
        tgt->ipv6_en = ipv6_en;