Commit 01bb12e4 authored Sep 14, 2017 by Colin Ian King Committed by Tejun Heo Sep 18, 2017

ata: pata_artop: remove redundant initialization of pio



pio is initialized and the data is never read, instead it is almost
immediately being updated to a new value. Fix this by removing the
initialization.

Detected by scan-build:
"warning: Value stored to 'pio' during its initialization is never read"

Fixes: 669a5db4 ("[libata] Add a bunch of PATA drivers")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Tejun Heo <tj@kernel.org>

parent f4ac6476

drivers/ata/pata_artop.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -242,7 +242,7 @@ static void artop6210_set_dmamode (struct ata_port ap, struct ata_device adev)

		static void artop6260_set_dmamode (struct ata_port ap, struct ata_device adev)
		{
		unsigned int pio = adev->pio_mode - XFER_PIO_0;
		unsigned int pio;
		struct pci_dev *pdev = to_pci_dev(ap->host->dev);
		u8 ultra;

Admin message