diff options
| author | Arnd Bergmann <arnd@arndb.de> | 2016-06-10 10:51:04 +0200 |
|---|---|---|
| committer | Arnd Bergmann <arnd@arndb.de> | 2017-07-27 22:57:53 +0200 |
| commit | c4caa8db4c90a6d0e2258e34a203961e07903c05 (patch) | |
| tree | c032969f77fe05ab371ff83d3ce14c410a63c87c /ipc | |
| parent | cd5bad4135e8e72144a19429d359c36fd4172a88 (diff) | |
| download | linux-c4caa8db4c90a6d0e2258e34a203961e07903c05.tar.bz2 | |
ARM: ixp4xx: fix ioport_unmap definition
An empty macro definition can cause unexpected behavior, in
case of the ixp4xx ioport_unmap, we get two warnings:
drivers/net/wireless/marvell/libertas/if_cs.c: In function 'if_cs_release':
drivers/net/wireless/marvell/libertas/if_cs.c:826:3: error: suggest braces around empty body in an 'if' statement [-Werror=empty-body]
ioport_unmap(card->iobase);
drivers/vfio/pci/vfio_pci_rdwr.c: In function 'vfio_pci_vga_rw':
drivers/vfio/pci/vfio_pci_rdwr.c:230:15: error: the omitted middle operand in ?: will always be 'true', suggest explicit middle operand [-Werror=parentheses]
is_ioport ? ioport_unmap(iomem) : iounmap(iomem);
This uses an inline function to define the macro in a safer way.
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Acked-by: Krzysztof Halasa <khalasa@piap.pl>
Diffstat (limited to 'ipc')
0 files changed, 0 insertions, 0 deletions