Commit f1fda895 authored by Al Viro's avatar Al Viro Committed by Linus Torvalds

[PATCH] powerpc: celleb trivial endianness and iomem annotations

Signed-off-by: default avatarAl Viro <viro@zeniv.linux.org.uk>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 95389b86
...@@ -65,13 +65,13 @@ static inline u8 celleb_fake_config_readb(void *addr) ...@@ -65,13 +65,13 @@ static inline u8 celleb_fake_config_readb(void *addr)
static inline u16 celleb_fake_config_readw(void *addr) static inline u16 celleb_fake_config_readw(void *addr)
{ {
u16 *p = addr; __le16 *p = addr;
return le16_to_cpu(*p); return le16_to_cpu(*p);
} }
static inline u32 celleb_fake_config_readl(void *addr) static inline u32 celleb_fake_config_readl(void *addr)
{ {
u32 *p = addr; __le32 *p = addr;
return le32_to_cpu(*p); return le32_to_cpu(*p);
} }
...@@ -83,16 +83,16 @@ static inline void celleb_fake_config_writeb(u32 val, void *addr) ...@@ -83,16 +83,16 @@ static inline void celleb_fake_config_writeb(u32 val, void *addr)
static inline void celleb_fake_config_writew(u32 val, void *addr) static inline void celleb_fake_config_writew(u32 val, void *addr)
{ {
u16 val16; __le16 val16;
u16 *p = addr; __le16 *p = addr;
val16 = cpu_to_le16(val); val16 = cpu_to_le16(val);
*p = val16; *p = val16;
} }
static inline void celleb_fake_config_writel(u32 val, void *addr) static inline void celleb_fake_config_writel(u32 val, void *addr)
{ {
u32 val32; __le32 val32;
u32 *p = addr; __le32 *p = addr;
val32 = cpu_to_le32(val); val32 = cpu_to_le32(val);
*p = val32; *p = val32;
} }
......
...@@ -47,7 +47,7 @@ ...@@ -47,7 +47,7 @@
#if 0 /* test code for epci dummy read */ #if 0 /* test code for epci dummy read */
static void celleb_epci_dummy_read(struct pci_dev *dev) static void celleb_epci_dummy_read(struct pci_dev *dev)
{ {
void *epci_base; void __iomem *epci_base;
struct device_node *node; struct device_node *node;
struct pci_controller *hose; struct pci_controller *hose;
u32 val; u32 val;
...@@ -58,7 +58,7 @@ static void celleb_epci_dummy_read(struct pci_dev *dev) ...@@ -58,7 +58,7 @@ static void celleb_epci_dummy_read(struct pci_dev *dev)
if (!hose) if (!hose)
return; return;
epci_base = (void *)hose->cfg_addr; epci_base = hose->cfg_addr;
val = in_be32(epci_base + SCC_EPCI_WATRP); val = in_be32(epci_base + SCC_EPCI_WATRP);
iosync(); iosync();
...@@ -71,18 +71,18 @@ static inline void clear_and_disable_master_abort_interrupt( ...@@ -71,18 +71,18 @@ static inline void clear_and_disable_master_abort_interrupt(
struct pci_controller *hose) struct pci_controller *hose)
{ {
void __iomem *addr; void __iomem *addr;
addr = (void *)hose->cfg_addr + PCI_COMMAND; addr = hose->cfg_addr + PCI_COMMAND;
out_be32(addr, in_be32(addr) | (PCI_STATUS_REC_MASTER_ABORT << 16)); out_be32(addr, in_be32(addr) | (PCI_STATUS_REC_MASTER_ABORT << 16));
} }
static int celleb_epci_check_abort(struct pci_controller *hose, static int celleb_epci_check_abort(struct pci_controller *hose,
unsigned long addr) void __iomem *addr)
{ {
void __iomem *reg, *epci_base; void __iomem *reg, *epci_base;
u32 val; u32 val;
iob(); iob();
epci_base = (void *)hose->cfg_addr; epci_base = hose->cfg_addr;
reg = epci_base + PCI_COMMAND; reg = epci_base + PCI_COMMAND;
val = in_be32(reg); val = in_be32(reg);
...@@ -108,23 +108,23 @@ static int celleb_epci_check_abort(struct pci_controller *hose, ...@@ -108,23 +108,23 @@ static int celleb_epci_check_abort(struct pci_controller *hose,
return PCIBIOS_SUCCESSFUL; return PCIBIOS_SUCCESSFUL;
} }
static unsigned long celleb_epci_make_config_addr(struct pci_controller *hose, static void __iomem *celleb_epci_make_config_addr(struct pci_controller *hose,
unsigned int devfn, int where) unsigned int devfn, int where)
{ {
unsigned long addr; void __iomem *addr;
struct pci_bus *bus = hose->bus; struct pci_bus *bus = hose->bus;
if (bus->self) if (bus->self)
addr = (unsigned long)hose->cfg_data + addr = hose->cfg_data +
(((bus->number & 0xff) << 16) (((bus->number & 0xff) << 16)
| ((devfn & 0xff) << 8) | ((devfn & 0xff) << 8)
| (where & 0xff) | (where & 0xff)
| 0x01000000); | 0x01000000);
else else
addr = (unsigned long)hose->cfg_data + addr = hose->cfg_data +
(((devfn & 0xff) << 8) | (where & 0xff)); (((devfn & 0xff) << 8) | (where & 0xff));
pr_debug("EPCI: config_addr = 0x%016lx\n", addr); pr_debug("EPCI: config_addr = 0x%p\n", addr);
return addr; return addr;
} }
...@@ -132,7 +132,7 @@ static unsigned long celleb_epci_make_config_addr(struct pci_controller *hose, ...@@ -132,7 +132,7 @@ static unsigned long celleb_epci_make_config_addr(struct pci_controller *hose,
static int celleb_epci_read_config(struct pci_bus *bus, static int celleb_epci_read_config(struct pci_bus *bus,
unsigned int devfn, int where, int size, u32 * val) unsigned int devfn, int where, int size, u32 * val)
{ {
unsigned long addr; void __iomem *addr;
struct device_node *node; struct device_node *node;
struct pci_controller *hose; struct pci_controller *hose;
...@@ -148,17 +148,17 @@ static int celleb_epci_read_config(struct pci_bus *bus, ...@@ -148,17 +148,17 @@ static int celleb_epci_read_config(struct pci_bus *bus,
if (bus->number == hose->first_busno && devfn == 0) { if (bus->number == hose->first_busno && devfn == 0) {
/* EPCI controller self */ /* EPCI controller self */
addr = (unsigned long)hose->cfg_addr + where; addr = hose->cfg_addr + where;
switch (size) { switch (size) {
case 1: case 1:
*val = in_8((u8 *)addr); *val = in_8(addr);
break; break;
case 2: case 2:
*val = in_be16((u16 *)addr); *val = in_be16(addr);
break; break;
case 4: case 4:
*val = in_be32((u32 *)addr); *val = in_be32(addr);
break; break;
default: default:
return PCIBIOS_DEVICE_NOT_FOUND; return PCIBIOS_DEVICE_NOT_FOUND;
...@@ -171,13 +171,13 @@ static int celleb_epci_read_config(struct pci_bus *bus, ...@@ -171,13 +171,13 @@ static int celleb_epci_read_config(struct pci_bus *bus,
switch (size) { switch (size) {
case 1: case 1:
*val = in_8((u8 *)addr); *val = in_8(addr);
break; break;
case 2: case 2:
*val = in_le16((u16 *)addr); *val = in_le16(addr);
break; break;
case 4: case 4:
*val = in_le32((u32 *)addr); *val = in_le32(addr);
break; break;
default: default:
return PCIBIOS_DEVICE_NOT_FOUND; return PCIBIOS_DEVICE_NOT_FOUND;
...@@ -188,13 +188,13 @@ static int celleb_epci_read_config(struct pci_bus *bus, ...@@ -188,13 +188,13 @@ static int celleb_epci_read_config(struct pci_bus *bus,
"addr=0x%lx, devfn=0x%x, where=0x%x, size=0x%x, val=0x%x\n", "addr=0x%lx, devfn=0x%x, where=0x%x, size=0x%x, val=0x%x\n",
addr, devfn, where, size, *val); addr, devfn, where, size, *val);
return celleb_epci_check_abort(hose, 0); return celleb_epci_check_abort(hose, NULL);
} }
static int celleb_epci_write_config(struct pci_bus *bus, static int celleb_epci_write_config(struct pci_bus *bus,
unsigned int devfn, int where, int size, u32 val) unsigned int devfn, int where, int size, u32 val)
{ {
unsigned long addr; void __iomem *addr;
struct device_node *node; struct device_node *node;
struct pci_controller *hose; struct pci_controller *hose;
...@@ -210,17 +210,17 @@ static int celleb_epci_write_config(struct pci_bus *bus, ...@@ -210,17 +210,17 @@ static int celleb_epci_write_config(struct pci_bus *bus,
if (bus->number == hose->first_busno && devfn == 0) { if (bus->number == hose->first_busno && devfn == 0) {
/* EPCI controller self */ /* EPCI controller self */
addr = (unsigned long)hose->cfg_addr + where; addr = hose->cfg_addr + where;
switch (size) { switch (size) {
case 1: case 1:
out_8((u8 *)addr, val); out_8(addr, val);
break; break;
case 2: case 2:
out_be16((u16 *)addr, val); out_be16(addr, val);
break; break;
case 4: case 4:
out_be32((u32 *)addr, val); out_be32(addr, val);
break; break;
default: default:
return PCIBIOS_DEVICE_NOT_FOUND; return PCIBIOS_DEVICE_NOT_FOUND;
...@@ -233,13 +233,13 @@ static int celleb_epci_write_config(struct pci_bus *bus, ...@@ -233,13 +233,13 @@ static int celleb_epci_write_config(struct pci_bus *bus,
switch (size) { switch (size) {
case 1: case 1:
out_8((u8 *)addr, val); out_8(addr, val);
break; break;
case 2: case 2:
out_le16((u16 *)addr, val); out_le16(addr, val);
break; break;
case 4: case 4:
out_le32((u32 *)addr, val); out_le32(addr, val);
break; break;
default: default:
return PCIBIOS_DEVICE_NOT_FOUND; return PCIBIOS_DEVICE_NOT_FOUND;
...@@ -261,7 +261,7 @@ static int __devinit celleb_epci_init(struct pci_controller *hose) ...@@ -261,7 +261,7 @@ static int __devinit celleb_epci_init(struct pci_controller *hose)
void __iomem *reg, *epci_base; void __iomem *reg, *epci_base;
int hwres = 0; int hwres = 0;
epci_base = (void *)hose->cfg_addr; epci_base = hose->cfg_addr;
/* PCI core reset(Internal bus and PCI clock) */ /* PCI core reset(Internal bus and PCI clock) */
reg = epci_base + SCC_EPCI_CKCTRL; reg = epci_base + SCC_EPCI_CKCTRL;
......
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