video: fbdev: controlfb: Fix set but not used warnings
authorSam Ravnborg <sam@ravnborg.org>
Sun, 6 Dec 2020 19:02:46 +0000 (20:02 +0100)
committerSam Ravnborg <sam@ravnborg.org>
Tue, 8 Dec 2020 17:36:36 +0000 (18:36 +0100)
The controlfb driver has a number of dummy defines for IO operations.
They were introduced in commit a07a63b0e24d
("video: fbdev: controlfb: add COMPILE_TEST support").

The write variants did not use their value parameter in the
dummy versions, resulting in set but not used warnings.
Fix this by adding "(void)val" to silence the compiler.

Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
Acked-by: Thomas Zimmermann <tzimmermann@suse.de>
Cc: Sam Ravnborg <sam@ravnborg.org>
Cc: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
Cc: "Gustavo A. R. Silva" <gustavoars@kernel.org>
Cc: Michael Ellerman <mpe@ellerman.id.au>
Link: https://patchwork.freedesktop.org/patch/msgid/20201206190247.1861316-13-sam@ravnborg.org
drivers/video/fbdev/controlfb.c

index 2df56bd..5093114 100644 (file)
@@ -64,9 +64,9 @@
 #undef in_le32
 #undef out_le32
 #define in_8(addr)             0
-#define out_8(addr, val)
+#define out_8(addr, val)       (void)(val)
 #define in_le32(addr)          0
-#define out_le32(addr, val)
+#define out_le32(addr, val)    (void)(val)
 #define pgprot_cached_wthru(prot) (prot)
 #else
 static void invalid_vram_cache(void __force *addr)