diff options
| author | Corentin Labbe <clabbe@baylibre.com> | 2019-11-21 11:14:45 +0300 |
|---|---|---|
| committer | Dave Airlie <airlied@redhat.com> | 2019-12-04 06:36:16 +0300 |
| commit | 4673402ebf9fc1712fb41cce6a808923481495f3 (patch) | |
| tree | b902ee5d67a260b77c17f11139b51a3d1d16426f | |
| parent | 5f448266ce9632c5318a5dbbc024fc7951f089d7 (diff) | |
| download | linux-4673402ebf9fc1712fb41cce6a808923481495f3.tar.xz | |
ia64: agp: Replace empty define with do while
It's dangerous to use empty code define.
Furthermore it lead to the following warning:
drivers/char/agp/generic.c: In function « agp_generic_destroy_page »:
drivers/char/agp/generic.c:1266:28: attention : suggest braces around empty body in an « if » statement [-Wempty-body]
So let's replace emptyness by "do {} while(0)"
Signed-off-by: Corentin Labbe <clabbe@baylibre.com>
Acked-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Dave Airlie <airlied@redhat.com>
Link: https://patchwork.freedesktop.org/patch/msgid/1574324085-4338-6-git-send-email-clabbe@baylibre.com
| -rw-r--r-- | arch/ia64/include/asm/agp.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/arch/ia64/include/asm/agp.h b/arch/ia64/include/asm/agp.h index 2b451c4496da..0261507dc264 100644 --- a/arch/ia64/include/asm/agp.h +++ b/arch/ia64/include/asm/agp.h @@ -14,8 +14,8 @@ * in coherent mode, which lets us map the AGP memory as normal (write-back) memory * (unlike x86, where it gets mapped "write-coalescing"). */ -#define map_page_into_agp(page) /* nothing */ -#define unmap_page_from_agp(page) /* nothing */ +#define map_page_into_agp(page) do { } while (0) +#define unmap_page_from_agp(page) do { } while (0) #define flush_agp_cache() mb() /* GATT allocation. Returns/accepts GATT kernel virtual address. */ |