Acked-by: Saurav Kashyap <saurav.kashyap@xxxxxxxxxx> >The value of the pointer called "nxt" is not used after the >"nxt = qla24xx_copy_eft(ha, nxt)" statement. Hence keep the function >call but remove the assignment. > >Signed-off-by: Bart Van Assche <bvanassche@xxxxxxx> >Cc: Chad Dupuis <chad.dupuis@xxxxxxxxxx> >Cc: Saurav Kashyap <saurav.kashyap@xxxxxxxxxx> >--- > drivers/scsi/qla2xxx/qla_dbg.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > >diff --git a/drivers/scsi/qla2xxx/qla_dbg.c >b/drivers/scsi/qla2xxx/qla_dbg.c >index cfa2a20..c612785 100644 >--- a/drivers/scsi/qla2xxx/qla_dbg.c >+++ b/drivers/scsi/qla2xxx/qla_dbg.c >@@ -1468,7 +1468,7 @@ qla25xx_fw_dump(scsi_qla_host_t *vha, int >hardware_locked) > > nxt = qla2xxx_copy_queues(ha, nxt); > >- nxt = qla24xx_copy_eft(ha, nxt); >+ qla24xx_copy_eft(ha, nxt); > > /* Chain entries -- started with MQ. */ > nxt_chain = qla25xx_copy_fce(ha, nxt_chain, &last_chain); >@@ -1787,7 +1787,7 @@ qla81xx_fw_dump(scsi_qla_host_t *vha, int >hardware_locked) > > nxt = qla2xxx_copy_queues(ha, nxt); > >- nxt = qla24xx_copy_eft(ha, nxt); >+ qla24xx_copy_eft(ha, nxt); > > /* Chain entries -- started with MQ. */ > nxt_chain = qla25xx_copy_fce(ha, nxt_chain, &last_chain); >@@ -2289,7 +2289,7 @@ qla83xx_fw_dump(scsi_qla_host_t *vha, int >hardware_locked) > copy_queue: > nxt = qla2xxx_copy_queues(ha, nxt); > >- nxt = qla24xx_copy_eft(ha, nxt); >+ qla24xx_copy_eft(ha, nxt); > > /* Chain entries -- started with MQ. */ > nxt_chain = qla25xx_copy_fce(ha, nxt_chain, &last_chain); >-- >1.7.10.4 > >
<<attachment: winmail.dat>>
