aboutsummaryrefslogtreecommitdiffstats
path: root/arch/riscv/mm/ioremap.c
diff options
context:
space:
mode:
authorOlof Johansson <olof@lixom.net>2017-11-29 17:55:14 -0800
committerPalmer Dabbelt <palmer@sifive.com>2017-11-30 10:01:10 -0800
commitfe2726af9fdc93ecf2469b7c512fc1a8936e128c (patch)
tree0c745c6770c3ca0a4c0db509cd43387f58d0fb8c /arch/riscv/mm/ioremap.c
parentRISC-V: use RISCV_{INT,SHORT} instead of {INT,SHORT} for asm macros (diff)
downloadlinux-dev-fe2726af9fdc93ecf2469b7c512fc1a8936e128c.tar.xz
linux-dev-fe2726af9fdc93ecf2469b7c512fc1a8936e128c.zip
RISC-V: io.h: type fixes for warnings
include <linux/types.h> for __iomem definition. Also, add volatile to iounmap() like other architectures have it to avoid "discarding volatile" warnings from some drivers. Finally, explicitly promote the base address for INB/OUTB functions to avoid some old legacy drivers complaining about int-to-ptr promotions. The drivers are unlikely to work but they're included in allmodconfig so the warnings are noisy. Fixes, among other warnings, these with allmodconfig: ../arch/riscv/include/asm/io.h:24:21: error: expected '=', ',', ';', 'asm' or '__attribute__' before '*' token extern void __iomem *ioremap(phys_addr_t offset, unsigned long size); sound/pci/echoaudio/echoaudio.c: In function 'snd_echo_free': sound/pci/echoaudio/echoaudio.c:1879:10: warning: passing argument 1 of 'iounmap' discards 'volatile' qualifier from pointer target type [-Wdiscarded-qualifiers] Signed-off-by: Olof Johansson <olof@lixom.net> Signed-off-by: Palmer Dabbelt <palmer@sifive.com>
Diffstat (limited to '')
-rw-r--r--arch/riscv/mm/ioremap.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/riscv/mm/ioremap.c b/arch/riscv/mm/ioremap.c
index e99194a4077e..70ef2724cdf6 100644
--- a/arch/riscv/mm/ioremap.c
+++ b/arch/riscv/mm/ioremap.c
@@ -85,7 +85,7 @@ EXPORT_SYMBOL(ioremap);
*
* Caller must ensure there is only one unmapping for the same pointer.
*/
-void iounmap(void __iomem *addr)
+void iounmap(volatile void __iomem *addr)
{
vunmap((void *)((unsigned long)addr & PAGE_MASK));
}