Message ID | 20180702073022.10000-1-colin.king@canonical.com |
---|---|
State | Not Applicable |
Delegated to: | David Miller |
Headers | show |
Series | sata_nv: remove redundant pointers sdev0 and sdev1 | expand |
On Mon, Jul 02, 2018 at 08:30:22AM +0100, Colin King wrote: > From: Colin Ian King <colin.king@canonical.com> > > 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: Colin Ian King <colin.king@canonical.com> Applied to libata/for-4.18-fixes. Thanks.
diff --git a/drivers/ata/sata_nv.c b/drivers/ata/sata_nv.c index 10ae11aa1926..72c9b922a77b 100644 --- a/drivers/ata/sata_nv.c +++ b/drivers/ata/sata_nv.c @@ -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)) { /*