diff options
author | Andy Shevchenko <andriy.shevchenko@linux.intel.com> | 2016-07-04 11:44:26 +0200 |
---|---|---|
committer | Mark Brown <broonie@kernel.org> | 2016-07-04 16:13:42 +0200 |
commit | 21ddba19ce851d322172a1f006d27e4f6ff3059d (patch) | |
tree | aa3afb677ebb16016776d511566d37208adf3c78 /drivers/spi | |
parent | spi: pxa2xx-pci: Enable SPI on Intel Merrifield (diff) | |
download | linux-21ddba19ce851d322172a1f006d27e4f6ff3059d.tar.xz linux-21ddba19ce851d322172a1f006d27e4f6ff3059d.zip |
spi: pxa2xx-pci: Remove unused code
pcim_iomap_table() can't fail when called after pcim_iomap_regions(). Moreover,
we already dereference returned value and kernel will crash if it is not
correct.
Remove obvious leftover of commit 0202775bc3a2 ("spi/pxa2xx-pci: switch to use
pcim_* interfaces").
Cc: Mika Westerberg <mika.westerberg@linux.intel.com>
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'drivers/spi')
-rw-r--r-- | drivers/spi/spi-pxa2xx-pci.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/drivers/spi/spi-pxa2xx-pci.c b/drivers/spi/spi-pxa2xx-pci.c index b025eaf14f0f..4e3c5a7d9a96 100644 --- a/drivers/spi/spi-pxa2xx-pci.c +++ b/drivers/spi/spi-pxa2xx-pci.c @@ -202,10 +202,6 @@ static int pxa2xx_spi_pci_probe(struct pci_dev *dev, ssp = &spi_pdata.ssp; ssp->phys_base = pci_resource_start(dev, 0); ssp->mmio_base = pcim_iomap_table(dev)[0]; - if (!ssp->mmio_base) { - dev_err(&dev->dev, "failed to ioremap() registers\n"); - return -EIO; - } ssp->irq = dev->irq; ssp->port_id = (c->port_id >= 0) ? c->port_id : dev->devfn; ssp->type = c->type; |