arm: mach-orion5x: use IOMEM() for base address definitions
authorThomas Petazzoni <thomas.petazzoni@free-electrons.com>
Tue, 11 Sep 2012 12:27:21 +0000 (14:27 +0200)
committerJason Cooper <jason@lakedaemon.net>
Fri, 21 Sep 2012 18:03:56 +0000 (18:03 +0000)
We now define all virtual base address constants using IOMEM() so that
those are naturally typed as void __iomem pointers, and we do the
necessary adjustements in the mach-orion5x code.

Note that we introduce a few temporary additional "unsigned long"
casts when calling into plat-orion functions. Those are removed by
followup patches converting plat-orion functions to void __iomem
pointers as well.

Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Acked-by: Arnd Bergmann <arnd@arndb.de>
Tested-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: Jason Cooper <jason@lakedaemon.net>
arch/arm/mach-orion5x/addr-map.c
arch/arm/mach-orion5x/common.c
arch/arm/mach-orion5x/dns323-setup.c
arch/arm/mach-orion5x/include/mach/orion5x.h
arch/arm/mach-orion5x/irq.c
arch/arm/mach-orion5x/mpp.c
arch/arm/mach-orion5x/pci.c

index eaac83d1df6f2207a4106276ce8d95c75b686589..d309f53b7a0cb228088c27d896e327a9d303972d 100644 (file)
@@ -79,7 +79,7 @@ static int __init cpu_win_can_remap(const struct orion_addr_map_cfg *cfg,
 static struct orion_addr_map_cfg addr_map_cfg __initdata = {
        .num_wins = 8,
        .cpu_win_can_remap = cpu_win_can_remap,
-       .bridge_virt_base = ORION5X_BRIDGE_VIRT_BASE,
+       .bridge_virt_base = (unsigned long) ORION5X_BRIDGE_VIRT_BASE,
 };
 
 static const struct __initdata orion_addr_map_info addr_map_info[] = {
index 410291c676668befcfe5b43723af071ba153e5db..a890cb0eb83d9328572501749b58f294314677b2 100644 (file)
  ****************************************************************************/
 static struct map_desc orion5x_io_desc[] __initdata = {
        {
-               .virtual        = ORION5X_REGS_VIRT_BASE,
+               .virtual        = (unsigned long) ORION5X_REGS_VIRT_BASE,
                .pfn            = __phys_to_pfn(ORION5X_REGS_PHYS_BASE),
                .length         = ORION5X_REGS_SIZE,
                .type           = MT_DEVICE,
        }, {
-               .virtual        = ORION5X_PCIE_IO_VIRT_BASE,
+               .virtual        = (unsigned long) ORION5X_PCIE_IO_VIRT_BASE,
                .pfn            = __phys_to_pfn(ORION5X_PCIE_IO_PHYS_BASE),
                .length         = ORION5X_PCIE_IO_SIZE,
                .type           = MT_DEVICE,
        }, {
-               .virtual        = ORION5X_PCI_IO_VIRT_BASE,
+               .virtual        = (unsigned long) ORION5X_PCI_IO_VIRT_BASE,
                .pfn            = __phys_to_pfn(ORION5X_PCI_IO_PHYS_BASE),
                .length         = ORION5X_PCI_IO_SIZE,
                .type           = MT_DEVICE,
        }, {
-               .virtual        = ORION5X_PCIE_WA_VIRT_BASE,
+               .virtual        = (unsigned long) ORION5X_PCIE_WA_VIRT_BASE,
                .pfn            = __phys_to_pfn(ORION5X_PCIE_WA_PHYS_BASE),
                .length         = ORION5X_PCIE_WA_SIZE,
                .type           = MT_DEVICE,
@@ -156,7 +156,8 @@ void __init orion5x_spi_init()
  ****************************************************************************/
 void __init orion5x_uart0_init(void)
 {
-       orion_uart0_init(UART0_VIRT_BASE, UART0_PHYS_BASE,
+       orion_uart0_init((unsigned long) UART0_VIRT_BASE,
+                        UART0_PHYS_BASE,
                         IRQ_ORION5X_UART0, tclk);
 }
 
@@ -165,7 +166,8 @@ void __init orion5x_uart0_init(void)
  ****************************************************************************/
 void __init orion5x_uart1_init(void)
 {
-       orion_uart1_init(UART1_VIRT_BASE, UART1_PHYS_BASE,
+       orion_uart1_init((unsigned long) UART1_VIRT_BASE,
+                        UART1_PHYS_BASE,
                         IRQ_ORION5X_UART1, tclk);
 }
 
@@ -203,7 +205,7 @@ void __init orion5x_wdt_init(void)
  ****************************************************************************/
 void __init orion5x_init_early(void)
 {
-       orion_time_set_base(TIMER_VIRT_BASE);
+       orion_time_set_base((unsigned long) TIMER_VIRT_BASE);
 }
 
 int orion5x_tclk;
@@ -224,7 +226,8 @@ static void __init orion5x_timer_init(void)
 {
        orion5x_tclk = orion5x_find_tclk();
 
-       orion_time_init(ORION5X_BRIDGE_VIRT_BASE, BRIDGE_INT_TIMER1_CLR,
+       orion_time_init((unsigned long) ORION5X_BRIDGE_VIRT_BASE,
+                       BRIDGE_INT_TIMER1_CLR,
                        IRQ_ORION5X_BRIDGE, orion5x_tclk);
 }
 
index d470864b4e42b9ba0a86eb62789ab2820edc9898..3a5e6fc197f79fcba55119de62c7200ead5cceaa 100644 (file)
@@ -700,7 +700,7 @@ static void __init dns323_init(void)
                 * Note: AFAIK, rev B1 needs the same treatement but I'll let
                 * somebody else test it.
                 */
-               writel(0x5, ORION5X_SATA_VIRT_BASE | 0x2c);
+               writel(0x5, ORION5X_SATA_VIRT_BASE + 0x2c);
                break;
        }
 }
index 87bd378b820322ff02ab6d68e63c2886d683e479..6fd38ab5e267c4c57ce07548d8cb7ddbbec9cc5c 100644 (file)
  * fe000000    f0000000        16M     PCIe WA space (Orion-1/Orion-NAS only)
  ****************************************************************************/
 #define ORION5X_REGS_PHYS_BASE         0xf1000000
-#define ORION5X_REGS_VIRT_BASE         0xfdd00000
+#define ORION5X_REGS_VIRT_BASE         IOMEM(0xfdd00000)
 #define ORION5X_REGS_SIZE              SZ_1M
 
 #define ORION5X_PCIE_IO_PHYS_BASE      0xf2000000
-#define ORION5X_PCIE_IO_VIRT_BASE      0xfde00000
+#define ORION5X_PCIE_IO_VIRT_BASE      IOMEM(0xfde00000)
 #define ORION5X_PCIE_IO_BUS_BASE       0x00000000
 #define ORION5X_PCIE_IO_SIZE           SZ_1M
 
 #define ORION5X_PCI_IO_PHYS_BASE       0xf2100000
-#define ORION5X_PCI_IO_VIRT_BASE       0xfdf00000
+#define ORION5X_PCI_IO_VIRT_BASE       IOMEM(0xfdf00000)
 #define ORION5X_PCI_IO_BUS_BASE                0x00100000
 #define ORION5X_PCI_IO_SIZE            SZ_1M
 
@@ -55,7 +55,7 @@
 
 /* Relevant only for Orion-1/Orion-NAS */
 #define ORION5X_PCIE_WA_PHYS_BASE      0xf0000000
-#define ORION5X_PCIE_WA_VIRT_BASE      0xfe000000
+#define ORION5X_PCIE_WA_VIRT_BASE      IOMEM(0xfe000000)
 #define ORION5X_PCIE_WA_SIZE           SZ_16M
 
 #define ORION5X_PCIE_MEM_PHYS_BASE     0xe0000000
index 17da7091d310e4e321b9e104b3ad1b7832ebd9be..bf9ff4f008188272b0637e28c104501a69771cb6 100644 (file)
@@ -12,6 +12,7 @@
 #include <linux/gpio.h>
 #include <linux/kernel.h>
 #include <linux/irq.h>
+#include <linux/io.h>
 #include <mach/bridge-regs.h>
 #include <plat/irq.h>
 
@@ -24,11 +25,11 @@ static int __initdata gpio0_irqs[4] = {
 
 void __init orion5x_init_irq(void)
 {
-       orion_irq_init(0, (void __iomem *)MAIN_IRQ_MASK);
+       orion_irq_init(0, MAIN_IRQ_MASK);
 
        /*
         * Initialize gpiolib for GPIOs 0-31.
         */
-       orion_gpio_init(NULL, 0, 32, (void __iomem *)GPIO_VIRT_BASE, 0,
+       orion_gpio_init(NULL, 0, 32, GPIO_VIRT_BASE, 0,
                        IRQ_ORION5X_GPIO_START, gpio0_irqs);
 }
index 5b70026f478c6be00032e353d72ebd14d049d925..30f5ef6e2ec269a2f592b660bc6a218a550b2fb5 100644 (file)
@@ -40,5 +40,6 @@ static unsigned int __init orion5x_variant(void)
 void __init orion5x_mpp_conf(unsigned int *mpp_list)
 {
        orion_mpp_conf(mpp_list, orion5x_variant(),
-                      MPP_MAX, ORION5X_DEV_BUS_VIRT_BASE);
+                      MPP_MAX,
+                      (unsigned long) ORION5X_DEV_BUS_VIRT_BASE);
 }
index b36f928d017c3165302ddb844d9c3e316f90beb1..b0822a864e0f281707354a45d5c552cfd26c69c8 100644 (file)
@@ -38,7 +38,7 @@
 /*****************************************************************************
  * PCIe controller
  ****************************************************************************/
-#define PCIE_BASE      ((void __iomem *)ORION5X_PCIE_VIRT_BASE)
+#define PCIE_BASE      (ORION5X_PCIE_VIRT_BASE)
 
 void __init orion5x_pcie_id(u32 *dev, u32 *rev)
 {
@@ -111,7 +111,7 @@ static int pcie_rd_conf_wa(struct pci_bus *bus, u32 devfn,
                return PCIBIOS_DEVICE_NOT_FOUND;
        }
 
-       ret = orion_pcie_rd_conf_wa((void __iomem *)ORION5X_PCIE_WA_VIRT_BASE,
+       ret = orion_pcie_rd_conf_wa(ORION5X_PCIE_WA_VIRT_BASE,
                                    bus, devfn, where, size, val);
 
        return ret;