]> git.baikalelectronics.ru Git - kernel.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)
commit6547e6f078c848aed703840dc2ef5ba39bf7059e
treec032969f77fe05ab371ff83d3ce14c410a63c87c
parent210fad26a9ad68d4438fef39d6e3848b468e919b
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