Commit 68fb659f authored by Paul Walmsley's avatar Paul Walmsley Committed by Tony Lindgren

mach-omap2: fix sparse warnings, some style issues for OMAP2 builds

This cleans up remaining sparse warnings for OMAP2 builds and fixes a
few CodingStyle issues.  This involves:

- removing casts where they don't belong,

- adding casts where they should be,

- marking internal functions as static,

- fixing some indentation in board-2430sdp-flash.c.

Patch should cause no change in kernel behavior.
Signed-off-by: default avatarPaul Walmsley <paul@pwsan.com>
Signed-off-by: default avatarTony Lindgren <tony@atomide.com>
parent a802243d
...@@ -125,16 +125,16 @@ static struct platform_device sdp_onenand_device = { ...@@ -125,16 +125,16 @@ static struct platform_device sdp_onenand_device = {
void __init sdp2430_flash_init(void) void __init sdp2430_flash_init(void)
{ {
unsigned long gpmc_base_add, gpmc_cs_base_add; void __iomem *gpmc_base_add, *gpmc_cs_base_add;
unsigned char cs = 0; unsigned char cs = 0;
gpmc_base_add = OMAP243X_GPMC_VIRT; gpmc_base_add = (__force void __iomem *)OMAP243X_GPMC_VIRT;
while (cs < GPMC_CS_NUM) { while (cs < GPMC_CS_NUM) {
int ret = 0; int ret = 0;
/* Each GPMC set for a single CS is at offset 0x30 */ /* Each GPMC set for a single CS is at offset 0x30 */
gpmc_cs_base_add = gpmc_cs_base_add = (gpmc_base_add + GPMC_OFF_CONFIG1_0 +
(gpmc_base_add + GPMC_OFF_CONFIG1_0 + (cs*0x30)); (cs*0x30));
/* xloader/Uboot would have programmed the NAND/oneNAND /* xloader/Uboot would have programmed the NAND/oneNAND
* base address for us This is a ugly hack. The proper * base address for us This is a ugly hack. The proper
...@@ -165,8 +165,8 @@ void __init sdp2430_flash_init(void) ...@@ -165,8 +165,8 @@ void __init sdp2430_flash_init(void)
if (flash_type == NAND) { if (flash_type == NAND) {
sdp_nand_data.cs = cs; sdp_nand_data.cs = cs;
sdp_nand_data.gpmc_cs_baseaddr = (void *) gpmc_cs_base_add; sdp_nand_data.gpmc_cs_baseaddr = gpmc_cs_base_add;
sdp_nand_data.gpmc_baseaddr = (void *) gpmc_base_add; sdp_nand_data.gpmc_baseaddr = gpmc_base_add;
if (platform_device_register(&sdp_nand_device) < 0) { if (platform_device_register(&sdp_nand_device) < 0) {
printk(KERN_ERR "Unable to register NAND device\n"); printk(KERN_ERR "Unable to register NAND device\n");
......
...@@ -54,11 +54,13 @@ static int eac_mux_disabled = 0; ...@@ -54,11 +54,13 @@ static int eac_mux_disabled = 0;
static int clkout2_mux_disabled = 0; static int clkout2_mux_disabled = 0;
static u32 saved_mux[2]; static u32 saved_mux[2];
#define MUX_EAC_IOP2V(x) (__force void __iomem *)io_p2v(x)
static void n800_enable_eac_mux(void) static void n800_enable_eac_mux(void)
{ {
if (!eac_mux_disabled) if (!eac_mux_disabled)
return; return;
__raw_writel(saved_mux[1], IO_ADDRESS(0x48000124)); __raw_writel(saved_mux[1], MUX_EAC_IOP2V(0x48000124));
eac_mux_disabled = 0; eac_mux_disabled = 0;
} }
...@@ -68,8 +70,8 @@ static void n800_disable_eac_mux(void) ...@@ -68,8 +70,8 @@ static void n800_disable_eac_mux(void)
WARN_ON(eac_mux_disabled); WARN_ON(eac_mux_disabled);
return; return;
} }
saved_mux[1] = __raw_readl(IO_ADDRESS(0x48000124)); saved_mux[1] = __raw_readl(MUX_EAC_IOP2V(0x48000124));
__raw_writel(0x1f1f1f1f, IO_ADDRESS(0x48000124)); __raw_writel(0x1f1f1f1f, MUX_EAC_IOP2V(0x48000124));
eac_mux_disabled = 1; eac_mux_disabled = 1;
} }
...@@ -77,7 +79,7 @@ static void n800_enable_clkout2_mux(void) ...@@ -77,7 +79,7 @@ static void n800_enable_clkout2_mux(void)
{ {
if (!clkout2_mux_disabled) if (!clkout2_mux_disabled)
return; return;
__raw_writel(saved_mux[0], IO_ADDRESS(0x480000e8)); __raw_writel(saved_mux[0], MUX_EAC_IOP2V(0x480000e8));
clkout2_mux_disabled = 0; clkout2_mux_disabled = 0;
} }
...@@ -89,10 +91,10 @@ static void n800_disable_clkout2_mux(void) ...@@ -89,10 +91,10 @@ static void n800_disable_clkout2_mux(void)
WARN_ON(clkout2_mux_disabled); WARN_ON(clkout2_mux_disabled);
return; return;
} }
saved_mux[0] = __raw_readl(IO_ADDRESS(0x480000e8)); saved_mux[0] = __raw_readl(MUX_EAC_IOP2V(0x480000e8));
l = saved_mux[0] & ~0xff; l = saved_mux[0] & ~0xff;
l |= 0x1f; l |= 0x1f;
__raw_writel(l, IO_ADDRESS(0x480000e8)); __raw_writel(l, MUX_EAC_IOP2V(0x480000e8));
clkout2_mux_disabled = 1; clkout2_mux_disabled = 1;
} }
......
...@@ -37,6 +37,8 @@ ...@@ -37,6 +37,8 @@
#include <../drivers/cbus/retu.h> #include <../drivers/cbus/retu.h>
#include <../drivers/media/video/tcm825x.h> #include <../drivers/media/video/tcm825x.h>
#include "board-n800.h"
#if defined (CONFIG_VIDEO_TCM825X) || defined (CONFIG_VIDEO_TCM825X_MODULE) #if defined (CONFIG_VIDEO_TCM825X) || defined (CONFIG_VIDEO_TCM825X_MODULE)
#define OMAP24XX_CAMERA_JAM_HACK #define OMAP24XX_CAMERA_JAM_HACK
......
...@@ -18,4 +18,6 @@ void __init nokia_n800_common_init(void); ...@@ -18,4 +18,6 @@ void __init nokia_n800_common_init(void);
void __init nokia_n800_map_io(void); void __init nokia_n800_map_io(void);
void __init nokia_n800_init_irq(void); void __init nokia_n800_init_irq(void);
extern const struct tcm825x_platform_data n800_tcm825x_platform_data;
#endif #endif
...@@ -530,7 +530,7 @@ static inline void omap2_clk_check_reg(u32 flags, void __iomem **reg) ...@@ -530,7 +530,7 @@ static inline void omap2_clk_check_reg(u32 flags, void __iomem **reg)
*reg = (__force void __iomem *)tmp; *reg = (__force void __iomem *)tmp;
} }
void __init omap2_clk_rewrite_base(struct clk *clk) static void __init omap2_clk_rewrite_base(struct clk *clk)
{ {
omap2_clk_check_reg(clk->flags, &clk->clksel_reg); omap2_clk_check_reg(clk->flags, &clk->clksel_reg);
omap2_clk_check_reg(clk->flags, &clk->enable_reg); omap2_clk_check_reg(clk->flags, &clk->enable_reg);
......
...@@ -75,12 +75,12 @@ static void omap2_mbox_enable_irq(struct omap_mbox *mbox, ...@@ -75,12 +75,12 @@ static void omap2_mbox_enable_irq(struct omap_mbox *mbox,
static inline unsigned int mbox_read_reg(unsigned int reg) static inline unsigned int mbox_read_reg(unsigned int reg)
{ {
return __raw_readl(mbox_base + reg); return __raw_readl((void __iomem *)(mbox_base + reg));
} }
static inline void mbox_write_reg(unsigned int val, unsigned int reg) static inline void mbox_write_reg(unsigned int val, unsigned int reg)
{ {
__raw_writel(val, mbox_base + reg); __raw_writel(val, (void __iomem *)(mbox_base + reg));
} }
/* Mailbox H/W preparations */ /* Mailbox H/W preparations */
......
...@@ -185,7 +185,7 @@ static struct omap_mcbsp_platform_data omap34xx_mcbsp_pdata[] = { ...@@ -185,7 +185,7 @@ static struct omap_mcbsp_platform_data omap34xx_mcbsp_pdata[] = {
#define OMAP34XX_MCBSP_PDATA_SZ 0 #define OMAP34XX_MCBSP_PDATA_SZ 0
#endif #endif
int __init omap2_mcbsp_init(void) static int __init omap2_mcbsp_init(void)
{ {
int i; int i;
......
...@@ -105,13 +105,13 @@ struct omap_mmu_tlb_entry { ...@@ -105,13 +105,13 @@ struct omap_mmu_tlb_entry {
static inline unsigned long static inline unsigned long
omap_mmu_read_reg(struct omap_mmu *mmu, unsigned long reg) omap_mmu_read_reg(struct omap_mmu *mmu, unsigned long reg)
{ {
return __raw_readl(mmu->base + reg); return __raw_readl((void __iomem *)(mmu->base + reg));
} }
static inline void omap_mmu_write_reg(struct omap_mmu *mmu, static inline void omap_mmu_write_reg(struct omap_mmu *mmu,
unsigned long val, unsigned long reg) unsigned long val, unsigned long reg)
{ {
__raw_writel(val, mmu->base + reg); __raw_writel(val, (void __iomem *)(mmu->base + reg));
} }
#endif /* __MACH_OMAP2_MMU_H */ #endif /* __MACH_OMAP2_MMU_H */
...@@ -408,7 +408,7 @@ static void __init_or_module omap2_cfg_debug(const struct pin_config *cfg, u16 r ...@@ -408,7 +408,7 @@ static void __init_or_module omap2_cfg_debug(const struct pin_config *cfg, u16 r
#endif #endif
#ifdef CONFIG_ARCH_OMAP24XX #ifdef CONFIG_ARCH_OMAP24XX
int __init_or_module omap24xx_cfg_reg(const struct pin_config *cfg) static int __init_or_module omap24xx_cfg_reg(const struct pin_config *cfg)
{ {
static DEFINE_SPINLOCK(mux_spin_lock); static DEFINE_SPINLOCK(mux_spin_lock);
unsigned long flags; unsigned long flags;
......
...@@ -28,6 +28,7 @@ ...@@ -28,6 +28,7 @@
#include <asm/mach/time.h> #include <asm/mach/time.h>
#include <asm/atomic.h> #include <asm/atomic.h>
#include <asm/arch/pm.h>
#include "pm.h" #include "pm.h"
unsigned short enable_dyn_sleep; unsigned short enable_dyn_sleep;
......
...@@ -32,11 +32,11 @@ extern void pm_init_serial_console(void); ...@@ -32,11 +32,11 @@ extern void pm_init_serial_console(void);
extern void serial_console_sleep(int enable); extern void serial_console_sleep(int enable);
extern int omap2_pm_debug; extern int omap2_pm_debug;
#else #else
#define omap2_read_32k_sync_counter() 0; #define omap2_read_32k_sync_counter() 0
#define serial_console_sleep(enable) do; while(0) #define serial_console_sleep(enable) do {} while (0);
#define pm_init_serial_console() do; while(0) #define pm_init_serial_console() do {} while (0);
#define omap2_pm_dump(mode,resume,us) do; while(0) #define omap2_pm_dump(mode, resume, us) do {} while (0);
#define serial_console_fclk_mask(f1,f2) do; while(0) #define serial_console_fclk_mask(f1, f2) do {} while (0);
#define omap2_pm_debug 0 #define omap2_pm_debug 0
#endif /* CONFIG_PM_DEBUG */ #endif /* CONFIG_PM_DEBUG */
#endif #endif
...@@ -61,6 +61,8 @@ struct omap_mbox { ...@@ -61,6 +61,8 @@ struct omap_mbox {
void (*err_notify)(void); void (*err_notify)(void);
}; };
extern struct omap_mbox mbox_dsp_info;
int omap_mbox_msg_send(struct omap_mbox *, mbox_msg_t msg, void *); int omap_mbox_msg_send(struct omap_mbox *, mbox_msg_t msg, void *);
void omap_mbox_init_seq(struct omap_mbox *); void omap_mbox_init_seq(struct omap_mbox *);
......
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