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

Commit 8740fa6f authored by Colin Ian King's avatar Colin Ian King Committed by Tejun Heo
Browse files

sata_nv: remove redundant pointers sdev0 and sdev1 



Pointers sdev0 and sdev1 are being assigned but are never used hence they
are redundant and can be removed.

Cleans up clang warnings:
warning: variable 'sdev0' set but not used [-Wunused-but-set-variable]
warning: variable 'sdev1' set but not used [-Wunused-but-set-variable]

Signed-off-by: default avatarColin Ian King <colin.king@canonical.com>
Signed-off-by: default avatarTejun Heo <tj@kernel.org>
parent eb36333d

drivers/ata/sata_nv.c

+0 −3
Original line number Diff line number Diff line
@@ -675,7 +675,6 @@ static int nv_adma_slave_config(struct scsi_device *sdev) 
	struct ata_port *ap = ata_shost_to_port(sdev->host);

	struct nv_adma_port_priv *pp = ap->private_data;

	struct nv_adma_port_priv *port0, *port1;

	struct scsi_device *sdev0, *sdev1;

	struct pci_dev *pdev = to_pci_dev(ap->host->dev);

	unsigned long segment_boundary, flags;

	unsigned short sg_tablesize;
@@ -736,8 +735,6 @@ static int nv_adma_slave_config(struct scsi_device *sdev) 


	port0 = ap->host->ports[0]->private_data;

	port1 = ap->host->ports[1]->private_data;

	sdev0 = ap->host->ports[0]->link.device[0].sdev;

	sdev1 = ap->host->ports[1]->link.device[0].sdev;

	if ((port0->flags & NV_ADMA_ATAPI_SETUP_COMPLETE) ||

	    (port1->flags & NV_ADMA_ATAPI_SETUP_COMPLETE)) {

		/*