All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] qla2xxx: Remove an unused structure member
@ 2017-05-22 20:10 Bart Van Assche
  2017-05-23  7:20 ` Christoph Hellwig
  2017-05-24  2:05 ` Martin K. Petersen
  0 siblings, 2 replies; 3+ messages in thread
From: Bart Van Assche @ 2017-05-22 20:10 UTC (permalink / raw)
  To: Martin K . Petersen, James Bottomley
  Cc: linux-scsi, Bart Van Assche, Quinn Tran, Hannes Reinecke,
	Christoph Hellwig, Andy Grover, David Disseldorp

qla_tgt_cmd.free_work is not used by the qla2xxx driver. Hence
remove that member of struct qla_tgt_cmd.

Signed-off-by: Bart Van Assche <bart.vanassche@sandisk.com>
Acked-by: Himanshu Madhani <himanshu.madhani@cavium.com>
Cc: Quinn Tran <quinn.tran@cavium.com>
Cc: Hannes Reinecke <hare@suse.com>
Cc: Christoph Hellwig <hch@lst.de>
Cc: Andy Grover <agrover@redhat.com>
Cc: David Disseldorp <ddiss@suse.de>
---
 drivers/scsi/qla2xxx/qla_target.h | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/scsi/qla2xxx/qla_target.h b/drivers/scsi/qla2xxx/qla_target.h
index d64420251194..dae278859554 100644
--- a/drivers/scsi/qla2xxx/qla_target.h
+++ b/drivers/scsi/qla2xxx/qla_target.h
@@ -862,7 +862,6 @@ struct qla_tgt_cmd {
 	struct se_cmd se_cmd;
 	struct fc_port *sess;
 	int state;
-	struct work_struct free_work;
 	struct work_struct work;
 	/* Sense buffer that will be mapped into outgoing status */
 	unsigned char sense_buffer[TRANSPORT_SENSE_BUFFER];
-- 
2.12.2

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] qla2xxx: Remove an unused structure member
  2017-05-22 20:10 [PATCH] qla2xxx: Remove an unused structure member Bart Van Assche
@ 2017-05-23  7:20 ` Christoph Hellwig
  2017-05-24  2:05 ` Martin K. Petersen
  1 sibling, 0 replies; 3+ messages in thread
From: Christoph Hellwig @ 2017-05-23  7:20 UTC (permalink / raw)
  To: Bart Van Assche
  Cc: Martin K . Petersen, James Bottomley, linux-scsi, Quinn Tran,
	Hannes Reinecke, Christoph Hellwig, Andy Grover,
	David Disseldorp

On Mon, May 22, 2017 at 01:10:53PM -0700, Bart Van Assche wrote:
> qla_tgt_cmd.free_work is not used by the qla2xxx driver. Hence
> remove that member of struct qla_tgt_cmd.

Looks fine,

Reviewed-by: Christoph Hellwig <hch@lst.de>

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] qla2xxx: Remove an unused structure member
  2017-05-22 20:10 [PATCH] qla2xxx: Remove an unused structure member Bart Van Assche
  2017-05-23  7:20 ` Christoph Hellwig
@ 2017-05-24  2:05 ` Martin K. Petersen
  1 sibling, 0 replies; 3+ messages in thread
From: Martin K. Petersen @ 2017-05-24  2:05 UTC (permalink / raw)
  To: Bart Van Assche
  Cc: Martin K . Petersen, James Bottomley, linux-scsi, Quinn Tran,
	Hannes Reinecke, Christoph Hellwig, Andy Grover,
	David Disseldorp


Bart,

> qla_tgt_cmd.free_work is not used by the qla2xxx driver. Hence remove
> that member of struct qla_tgt_cmd.

Applied to 4.13/scsi-queue. Thanks, Bart!

-- 
Martin K. Petersen	Oracle Linux Engineering

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2017-05-24  2:08 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-05-22 20:10 [PATCH] qla2xxx: Remove an unused structure member Bart Van Assche
2017-05-23  7:20 ` Christoph Hellwig
2017-05-24  2:05 ` Martin K. Petersen

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.