Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 89576205 authored by Matthew R. Ochs's avatar Matthew R. Ochs Committed by James Bottomley
Browse files

cxlflash: Remove unused variable from queuecommand 



The queuecommand routine has a local dev pointer used for the
dev_* prints. The two prints that currently exist are tucked
under a debug define and thus can be left out. Use the actual
location instead of a local to avoid this warning.

This patch is intended to be applied after the "CXL Flash Error
Recovery and Superpipe" series.

Signed-off-by: default avatarMatthew R. Ochs <mrochs@linux.vnet.ibm.com>
Signed-off-by: default avatarManoj N. Kumar <manoj@linux.vnet.ibm.com>
Reported-by: default avatarkbuild test robot <fengguang.wu@intel.com>
Signed-off-by: default avatarJames Bottomley <JBottomley@Odin.com>
parent 4da74db0

drivers/scsi/cxlflash/main.c

+4 −3
Original line number Diff line number Diff line
@@ -354,7 +354,6 @@ static int cxlflash_queuecommand(struct Scsi_Host *host, struct scsi_cmnd *scp) 
	struct cxlflash_cfg *cfg = (struct cxlflash_cfg *)host->hostdata;

	struct afu *afu = cfg->afu;

	struct pci_dev *pdev = cfg->dev;

	struct device *dev = &cfg->dev->dev;

	struct afu_cmd *cmd;

	u32 port_sel = scp->device->channel + 1;

	int nseg, i, ncount;
@@ -384,11 +383,13 @@ static int cxlflash_queuecommand(struct Scsi_Host *host, struct scsi_cmnd *scp) 


	switch (cfg->state) {

	case STATE_LIMBO:

		dev_dbg_ratelimited(dev, "%s: device in limbo!\n", __func__);

		dev_dbg_ratelimited(&cfg->dev->dev, "%s: device in limbo!\n",

				    __func__);

		rc = SCSI_MLQUEUE_HOST_BUSY;

		goto out;

	case STATE_FAILTERM:

		dev_dbg_ratelimited(dev, "%s: device has failed!\n", __func__);

		dev_dbg_ratelimited(&cfg->dev->dev, "%s: device has failed!\n",

				    __func__);

		scp->result = (DID_NO_CONNECT << 16);

		scp->scsi_done(scp);

		rc = 0;