Commit bfca9459 authored by John Bowler's avatar John Bowler Committed by Russell King

[ARM] 3083/1: include/asm-arm/arch-ixp4xx/io.h: eliminate warnings for pointer...

[ARM] 3083/1: include/asm-arm/arch-ixp4xx/io.h: eliminate warnings for pointer passed to integral function argument

Patch from John Bowler

Fix for a compiler warning, this wasn't apparent in 2.6.12, I
believe the compiler options have been changed (somewhere) so
that passing a (void*) to a (u32) argument is now warned.

This accounts for the majority of the warnings in my builds of
the 2.6.14 kernel for NSLU2.

The patch changes pointer parameters declared as u32 to be
declared as either, for read parameters:

const volatile void __iomem *

and for write parameters:

volatile void __iomem *
Signed-off-by: default avatarJohn Bowler <jbowler@acm.org>
Signed-off-by: default avatarDeepak Saxena <dsaxena@plexity.net>
Signed-off-by: default avatarRussell King <rmk+kernel@arm.linux.org.uk>
parent 9dc2d0f5
...@@ -80,9 +80,9 @@ __ixp4xx_iounmap(void __iomem *addr) ...@@ -80,9 +80,9 @@ __ixp4xx_iounmap(void __iomem *addr)
#define __arch_ioremap(a, s, f, x) __ixp4xx_ioremap(a, s, f, x) #define __arch_ioremap(a, s, f, x) __ixp4xx_ioremap(a, s, f, x)
#define __arch_iounmap(a) __ixp4xx_iounmap(a) #define __arch_iounmap(a) __ixp4xx_iounmap(a)
#define writeb(p, v) __ixp4xx_writeb(p, v) #define writeb(v, p) __ixp4xx_writeb(v, p)
#define writew(p, v) __ixp4xx_writew(p, v) #define writew(v, p) __ixp4xx_writew(v, p)
#define writel(p, v) __ixp4xx_writel(p, v) #define writel(v, p) __ixp4xx_writel(v, p)
#define writesb(p, v, l) __ixp4xx_writesb(p, v, l) #define writesb(p, v, l) __ixp4xx_writesb(p, v, l)
#define writesw(p, v, l) __ixp4xx_writesw(p, v, l) #define writesw(p, v, l) __ixp4xx_writesw(p, v, l)
...@@ -97,8 +97,9 @@ __ixp4xx_iounmap(void __iomem *addr) ...@@ -97,8 +97,9 @@ __ixp4xx_iounmap(void __iomem *addr)
#define readsl(p, v, l) __ixp4xx_readsl(p, v, l) #define readsl(p, v, l) __ixp4xx_readsl(p, v, l)
static inline void static inline void
__ixp4xx_writeb(u8 value, u32 addr) __ixp4xx_writeb(u8 value, volatile void __iomem *p)
{ {
u32 addr = (u32)p;
u32 n, byte_enables, data; u32 n, byte_enables, data;
if (addr >= VMALLOC_START) { if (addr >= VMALLOC_START) {
...@@ -113,15 +114,16 @@ __ixp4xx_writeb(u8 value, u32 addr) ...@@ -113,15 +114,16 @@ __ixp4xx_writeb(u8 value, u32 addr)
} }
static inline void static inline void
__ixp4xx_writesb(u32 bus_addr, const u8 *vaddr, int count) __ixp4xx_writesb(volatile void __iomem *bus_addr, const u8 *vaddr, int count)
{ {
while (count--) while (count--)
writeb(*vaddr++, bus_addr); writeb(*vaddr++, bus_addr);
} }
static inline void static inline void
__ixp4xx_writew(u16 value, u32 addr) __ixp4xx_writew(u16 value, volatile void __iomem *p)
{ {
u32 addr = (u32)p;
u32 n, byte_enables, data; u32 n, byte_enables, data;
if (addr >= VMALLOC_START) { if (addr >= VMALLOC_START) {
...@@ -136,15 +138,16 @@ __ixp4xx_writew(u16 value, u32 addr) ...@@ -136,15 +138,16 @@ __ixp4xx_writew(u16 value, u32 addr)
} }
static inline void static inline void
__ixp4xx_writesw(u32 bus_addr, const u16 *vaddr, int count) __ixp4xx_writesw(volatile void __iomem *bus_addr, const u16 *vaddr, int count)
{ {
while (count--) while (count--)
writew(*vaddr++, bus_addr); writew(*vaddr++, bus_addr);
} }
static inline void static inline void
__ixp4xx_writel(u32 value, u32 addr) __ixp4xx_writel(u32 value, volatile void __iomem *p)
{ {
u32 addr = (u32)p;
if (addr >= VMALLOC_START) { if (addr >= VMALLOC_START) {
__raw_writel(value, addr); __raw_writel(value, addr);
return; return;
...@@ -154,15 +157,16 @@ __ixp4xx_writel(u32 value, u32 addr) ...@@ -154,15 +157,16 @@ __ixp4xx_writel(u32 value, u32 addr)
} }
static inline void static inline void
__ixp4xx_writesl(u32 bus_addr, const u32 *vaddr, int count) __ixp4xx_writesl(volatile void __iomem *bus_addr, const u32 *vaddr, int count)
{ {
while (count--) while (count--)
writel(*vaddr++, bus_addr); writel(*vaddr++, bus_addr);
} }
static inline unsigned char static inline unsigned char
__ixp4xx_readb(u32 addr) __ixp4xx_readb(const volatile void __iomem *p)
{ {
u32 addr = (u32)p;
u32 n, byte_enables, data; u32 n, byte_enables, data;
if (addr >= VMALLOC_START) if (addr >= VMALLOC_START)
...@@ -177,15 +181,16 @@ __ixp4xx_readb(u32 addr) ...@@ -177,15 +181,16 @@ __ixp4xx_readb(u32 addr)
} }
static inline void static inline void
__ixp4xx_readsb(u32 bus_addr, u8 *vaddr, u32 count) __ixp4xx_readsb(const volatile void __iomem *bus_addr, u8 *vaddr, u32 count)
{ {
while (count--) while (count--)
*vaddr++ = readb(bus_addr); *vaddr++ = readb(bus_addr);
} }
static inline unsigned short static inline unsigned short
__ixp4xx_readw(u32 addr) __ixp4xx_readw(const volatile void __iomem *p)
{ {
u32 addr = (u32)p;
u32 n, byte_enables, data; u32 n, byte_enables, data;
if (addr >= VMALLOC_START) if (addr >= VMALLOC_START)
...@@ -200,15 +205,16 @@ __ixp4xx_readw(u32 addr) ...@@ -200,15 +205,16 @@ __ixp4xx_readw(u32 addr)
} }
static inline void static inline void
__ixp4xx_readsw(u32 bus_addr, u16 *vaddr, u32 count) __ixp4xx_readsw(const volatile void __iomem *bus_addr, u16 *vaddr, u32 count)
{ {
while (count--) while (count--)
*vaddr++ = readw(bus_addr); *vaddr++ = readw(bus_addr);
} }
static inline unsigned long static inline unsigned long
__ixp4xx_readl(u32 addr) __ixp4xx_readl(const volatile void __iomem *p)
{ {
u32 addr = (u32)p;
u32 data; u32 data;
if (addr >= VMALLOC_START) if (addr >= VMALLOC_START)
...@@ -221,7 +227,7 @@ __ixp4xx_readl(u32 addr) ...@@ -221,7 +227,7 @@ __ixp4xx_readl(u32 addr)
} }
static inline void static inline void
__ixp4xx_readsl(u32 bus_addr, u32 *vaddr, u32 count) __ixp4xx_readsl(const volatile void __iomem *bus_addr, u32 *vaddr, u32 count)
{ {
while (count--) while (count--)
*vaddr++ = readl(bus_addr); *vaddr++ = readl(bus_addr);
...@@ -239,7 +245,7 @@ __ixp4xx_readsl(u32 bus_addr, u32 *vaddr, u32 count) ...@@ -239,7 +245,7 @@ __ixp4xx_readsl(u32 bus_addr, u32 *vaddr, u32 count)
eth_copy_and_sum((s),__mem_pci(c),(l),(b)) eth_copy_and_sum((s),__mem_pci(c),(l),(b))
static inline int static inline int
check_signature(unsigned long bus_addr, const unsigned char *signature, check_signature(const unsigned char __iomem *bus_addr, const unsigned char *signature,
int length) int length)
{ {
int retval = 0; int retval = 0;
...@@ -389,7 +395,7 @@ __ixp4xx_insl(u32 io_addr, u32 *vaddr, u32 count) ...@@ -389,7 +395,7 @@ __ixp4xx_insl(u32 io_addr, u32 *vaddr, u32 count)
#define __is_io_address(p) (((unsigned long)p >= PIO_OFFSET) && \ #define __is_io_address(p) (((unsigned long)p >= PIO_OFFSET) && \
((unsigned long)p <= (PIO_MASK + PIO_OFFSET))) ((unsigned long)p <= (PIO_MASK + PIO_OFFSET)))
static inline unsigned int static inline unsigned int
__ixp4xx_ioread8(void __iomem *addr) __ixp4xx_ioread8(const void __iomem *addr)
{ {
unsigned long port = (unsigned long __force)addr; unsigned long port = (unsigned long __force)addr;
if (__is_io_address(port)) if (__is_io_address(port))
...@@ -398,12 +404,12 @@ __ixp4xx_ioread8(void __iomem *addr) ...@@ -398,12 +404,12 @@ __ixp4xx_ioread8(void __iomem *addr)
#ifndef CONFIG_IXP4XX_INDIRECT_PCI #ifndef CONFIG_IXP4XX_INDIRECT_PCI
return (unsigned int)__raw_readb(port); return (unsigned int)__raw_readb(port);
#else #else
return (unsigned int)__ixp4xx_readb(port); return (unsigned int)__ixp4xx_readb(addr);
#endif #endif
} }
static inline void static inline void
__ixp4xx_ioread8_rep(void __iomem *addr, void *vaddr, u32 count) __ixp4xx_ioread8_rep(const void __iomem *addr, void *vaddr, u32 count)
{ {
unsigned long port = (unsigned long __force)addr; unsigned long port = (unsigned long __force)addr;
if (__is_io_address(port)) if (__is_io_address(port))
...@@ -412,12 +418,12 @@ __ixp4xx_ioread8_rep(void __iomem *addr, void *vaddr, u32 count) ...@@ -412,12 +418,12 @@ __ixp4xx_ioread8_rep(void __iomem *addr, void *vaddr, u32 count)
#ifndef CONFIG_IXP4XX_INDIRECT_PCI #ifndef CONFIG_IXP4XX_INDIRECT_PCI
__raw_readsb(addr, vaddr, count); __raw_readsb(addr, vaddr, count);
#else #else
__ixp4xx_readsb(port, vaddr, count); __ixp4xx_readsb(addr, vaddr, count);
#endif #endif
} }
static inline unsigned int static inline unsigned int
__ixp4xx_ioread16(void __iomem *addr) __ixp4xx_ioread16(const void __iomem *addr)
{ {
unsigned long port = (unsigned long __force)addr; unsigned long port = (unsigned long __force)addr;
if (__is_io_address(port)) if (__is_io_address(port))
...@@ -426,12 +432,12 @@ __ixp4xx_ioread16(void __iomem *addr) ...@@ -426,12 +432,12 @@ __ixp4xx_ioread16(void __iomem *addr)
#ifndef CONFIG_IXP4XX_INDIRECT_PCI #ifndef CONFIG_IXP4XX_INDIRECT_PCI
return le16_to_cpu(__raw_readw((u32)port)); return le16_to_cpu(__raw_readw((u32)port));
#else #else
return (unsigned int)__ixp4xx_readw((u32)port); return (unsigned int)__ixp4xx_readw(addr);
#endif #endif
} }
static inline void static inline void
__ixp4xx_ioread16_rep(void __iomem *addr, void *vaddr, u32 count) __ixp4xx_ioread16_rep(const void __iomem *addr, void *vaddr, u32 count)
{ {
unsigned long port = (unsigned long __force)addr; unsigned long port = (unsigned long __force)addr;
if (__is_io_address(port)) if (__is_io_address(port))
...@@ -440,12 +446,12 @@ __ixp4xx_ioread16_rep(void __iomem *addr, void *vaddr, u32 count) ...@@ -440,12 +446,12 @@ __ixp4xx_ioread16_rep(void __iomem *addr, void *vaddr, u32 count)
#ifndef CONFIG_IXP4XX_INDIRECT_PCI #ifndef CONFIG_IXP4XX_INDIRECT_PCI
__raw_readsw(addr, vaddr, count); __raw_readsw(addr, vaddr, count);
#else #else
__ixp4xx_readsw(port, vaddr, count); __ixp4xx_readsw(addr, vaddr, count);
#endif #endif
} }
static inline unsigned int static inline unsigned int
__ixp4xx_ioread32(void __iomem *addr) __ixp4xx_ioread32(const void __iomem *addr)
{ {
unsigned long port = (unsigned long __force)addr; unsigned long port = (unsigned long __force)addr;
if (__is_io_address(port)) if (__is_io_address(port))
...@@ -454,13 +460,13 @@ __ixp4xx_ioread32(void __iomem *addr) ...@@ -454,13 +460,13 @@ __ixp4xx_ioread32(void __iomem *addr)
#ifndef CONFIG_IXP4XX_INDIRECT_PCI #ifndef CONFIG_IXP4XX_INDIRECT_PCI
return le32_to_cpu(__raw_readl((u32)port)); return le32_to_cpu(__raw_readl((u32)port));
#else #else
return (unsigned int)__ixp4xx_readl((u32)port); return (unsigned int)__ixp4xx_readl(addr);
#endif #endif
} }
} }
static inline void static inline void
__ixp4xx_ioread32_rep(void __iomem *addr, void *vaddr, u32 count) __ixp4xx_ioread32_rep(const void __iomem *addr, void *vaddr, u32 count)
{ {
unsigned long port = (unsigned long __force)addr; unsigned long port = (unsigned long __force)addr;
if (__is_io_address(port)) if (__is_io_address(port))
...@@ -469,7 +475,7 @@ __ixp4xx_ioread32_rep(void __iomem *addr, void *vaddr, u32 count) ...@@ -469,7 +475,7 @@ __ixp4xx_ioread32_rep(void __iomem *addr, void *vaddr, u32 count)
#ifndef CONFIG_IXP4XX_INDIRECT_PCI #ifndef CONFIG_IXP4XX_INDIRECT_PCI
__raw_readsl(addr, vaddr, count); __raw_readsl(addr, vaddr, count);
#else #else
__ixp4xx_readsl(port, vaddr, count); __ixp4xx_readsl(addr, vaddr, count);
#endif #endif
} }
...@@ -483,7 +489,7 @@ __ixp4xx_iowrite8(u8 value, void __iomem *addr) ...@@ -483,7 +489,7 @@ __ixp4xx_iowrite8(u8 value, void __iomem *addr)
#ifndef CONFIG_IXP4XX_INDIRECT_PCI #ifndef CONFIG_IXP4XX_INDIRECT_PCI
__raw_writeb(value, port); __raw_writeb(value, port);
#else #else
__ixp4xx_writeb(value, port); __ixp4xx_writeb(value, addr);
#endif #endif
} }
...@@ -497,7 +503,7 @@ __ixp4xx_iowrite8_rep(void __iomem *addr, const void *vaddr, u32 count) ...@@ -497,7 +503,7 @@ __ixp4xx_iowrite8_rep(void __iomem *addr, const void *vaddr, u32 count)
#ifndef CONFIG_IXP4XX_INDIRECT_PCI #ifndef CONFIG_IXP4XX_INDIRECT_PCI
__raw_writesb(addr, vaddr, count); __raw_writesb(addr, vaddr, count);
#else #else
__ixp4xx_writesb(port, vaddr, count); __ixp4xx_writesb(addr, vaddr, count);
#endif #endif
} }
...@@ -511,7 +517,7 @@ __ixp4xx_iowrite16(u16 value, void __iomem *addr) ...@@ -511,7 +517,7 @@ __ixp4xx_iowrite16(u16 value, void __iomem *addr)
#ifndef CONFIG_IXP4XX_INDIRECT_PCI #ifndef CONFIG_IXP4XX_INDIRECT_PCI
__raw_writew(cpu_to_le16(value), addr); __raw_writew(cpu_to_le16(value), addr);
#else #else
__ixp4xx_writew(value, port); __ixp4xx_writew(value, addr);
#endif #endif
} }
...@@ -525,7 +531,7 @@ __ixp4xx_iowrite16_rep(void __iomem *addr, const void *vaddr, u32 count) ...@@ -525,7 +531,7 @@ __ixp4xx_iowrite16_rep(void __iomem *addr, const void *vaddr, u32 count)
#ifndef CONFIG_IXP4XX_INDIRECT_PCI #ifndef CONFIG_IXP4XX_INDIRECT_PCI
__raw_writesw(addr, vaddr, count); __raw_writesw(addr, vaddr, count);
#else #else
__ixp4xx_writesw(port, vaddr, count); __ixp4xx_writesw(addr, vaddr, count);
#endif #endif
} }
...@@ -539,7 +545,7 @@ __ixp4xx_iowrite32(u32 value, void __iomem *addr) ...@@ -539,7 +545,7 @@ __ixp4xx_iowrite32(u32 value, void __iomem *addr)
#ifndef CONFIG_IXP4XX_INDIRECT_PCI #ifndef CONFIG_IXP4XX_INDIRECT_PCI
__raw_writel(cpu_to_le32(value), port); __raw_writel(cpu_to_le32(value), port);
#else #else
__ixp4xx_writel(value, port); __ixp4xx_writel(value, addr);
#endif #endif
} }
...@@ -553,7 +559,7 @@ __ixp4xx_iowrite32_rep(void __iomem *addr, const void *vaddr, u32 count) ...@@ -553,7 +559,7 @@ __ixp4xx_iowrite32_rep(void __iomem *addr, const void *vaddr, u32 count)
#ifndef CONFIG_IXP4XX_INDIRECT_PCI #ifndef CONFIG_IXP4XX_INDIRECT_PCI
__raw_writesl(addr, vaddr, count); __raw_writesl(addr, vaddr, count);
#else #else
__ixp4xx_writesl(port, vaddr, count); __ixp4xx_writesl(addr, vaddr, count);
#endif #endif
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment