diff options
| author |   Geordan Neukum <gneukum1@gmail.com> | 2019-06-05 01:09:12 +0000 |
|---|---|---|
| committer |   Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2019-06-06 15:05:02 +0200 |
| commit | fbf476ea29cc7294586dcf0b89f22593458deb0d (patch) | |
| tree | 2795b01e7bf41724dd933f96c22e58603a7dd8ca /drivers/staging/kpc2000 | |
| parent | staging: kpc2000: kpc_spi: remove unnecessary struct member chip_select (diff) | |
| download | linux-dev-fbf476ea29cc7294586dcf0b89f22593458deb0d.tar.xz linux-dev-fbf476ea29cc7294586dcf0b89f22593458deb0d.zip | |
staging: kpc2000: kpc_spi: remove unnecessary ulong repr of i/o addr
The kpc_spi driver stashes off an unsigned long representation of the
i/o mapping returned by devm_ioremap_nocache(). This is unnecessary, as
the only use of the unsigned long repr is to eventually be re-cast to
an (u64 __iomem *). Instead of casting the (void __iomem *) to an
(unsigned long) then a (u64 __iomem *), just remove this intermediate
step. As this intermediary is no longer used, also remove it from its
structure.
Signed-off-by: Geordan Neukum <gneukum1@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/kpc2000')
| -rw-r--r-- | drivers/staging/kpc2000/kpc2000_spi.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/staging/kpc2000/kpc2000_spi.c b/drivers/staging/kpc2000/kpc2000_spi.c index 07b0327d8bef..4f517afc6239 100644 --- a/drivers/staging/kpc2000/kpc2000_spi.c +++ b/drivers/staging/kpc2000/kpc2000_spi.c @@ -103,7 +103,6 @@ static struct spi_board_info p2kr0_board_info[] = { struct kp_spi { struct spi_master *master; u64 __iomem *base; - unsigned long phys; struct device *dev; }; @@ -462,9 +461,8 @@ kp_spi_probe(struct platform_device *pldev) goto free_master; } - kpspi->phys = (unsigned long)devm_ioremap_nocache(&pldev->dev, r->start, - resource_size(r)); - kpspi->base = (u64 __iomem *)kpspi->phys; + kpspi->base = devm_ioremap_nocache(&pldev->dev, r->start, + resource_size(r)); status = spi_register_master(master); if (status < 0) { |