]> git.kernelconcepts.de Git - karo-tx-linux.git/commit
ARM: ixp4xx: fix ioport_unmap definition
authorArnd Bergmann <arnd@arndb.de>
Fri, 10 Jun 2016 08:51:04 +0000 (10:51 +0200)
committerArnd Bergmann <arnd@arndb.de>
Thu, 27 Jul 2017 20:57:53 +0000 (22:57 +0200)
commitc4caa8db4c90a6d0e2258e34a203961e07903c05
treec032969f77fe05ab371ff83d3ce14c410a63c87c
parentcd5bad4135e8e72144a19429d359c36fd4172a88
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>
arch/arm/mach-ixp4xx/include/mach/io.h