* [PATCH] fix a pile of OMAP issues
@ 2008-12-01 16:05 Russell King - ARM Linux
2008-12-01 17:01 ` Tony Lindgren
0 siblings, 1 reply; 2+ messages in thread
From: Russell King - ARM Linux @ 2008-12-01 16:05 UTC (permalink / raw)
To: Tony Lindgren; +Cc: linux-omap, linux-arm-kernel
This patch fixes a number of sillies, from missing 'const' to using
'return' in void functions, to functions with no arguments not even
'void' and a cast which isn't required.
Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
--
I want to merge this for the next ARM kernel pull by Linus. Please
ack the patch.
arch/arm/mach-omap1/io.c | 2 +-
arch/arm/plat-omap/include/mach/omapfb.h | 4 ++--
arch/arm/plat-omap/sram.c | 8 ++++----
drivers/video/omap/omapfb_main.c | 2 +-
4 files changed, 8 insertions(+), 8 deletions(-)
diff --git a/arch/arm/mach-omap1/io.c b/arch/arm/mach-omap1/io.c
index b3bd8ca..4c3e582 100644
--- a/arch/arm/mach-omap1/io.c
+++ b/arch/arm/mach-omap1/io.c
@@ -128,7 +128,7 @@ void __init omap1_map_common_io(void)
* Common low-level hardware init for omap1. This should only get called from
* board specific init.
*/
-void __init omap1_init_common_hw()
+void __init omap1_init_common_hw(void)
{
/* REVISIT: Refer to OMAP5910 Errata, Advisory SYS_1: "Timeout Abort
* on a Posted Write in the TIPB Bridge".
diff --git a/arch/arm/plat-omap/include/mach/omapfb.h b/arch/arm/plat-omap/include/mach/omapfb.h
index ec67fb4..7b74d12 100644
--- a/arch/arm/plat-omap/include/mach/omapfb.h
+++ b/arch/arm/plat-omap/include/mach/omapfb.h
@@ -353,8 +353,8 @@ struct omapfb_device {
u32 pseudo_palette[17];
struct lcd_panel *panel; /* LCD panel */
- struct lcd_ctrl *ctrl; /* LCD controller */
- struct lcd_ctrl *int_ctrl; /* internal LCD ctrl */
+ const struct lcd_ctrl *ctrl; /* LCD controller */
+ const struct lcd_ctrl *int_ctrl; /* internal LCD ctrl */
struct lcd_ctrl_extif *ext_if; /* LCD ctrl external
interface */
struct device *dev;
diff --git a/arch/arm/plat-omap/sram.c b/arch/arm/plat-omap/sram.c
index 9f9a921..dcd9d16 100644
--- a/arch/arm/plat-omap/sram.c
+++ b/arch/arm/plat-omap/sram.c
@@ -255,7 +255,7 @@ void omap_sram_reprogram_clock(u32 dpllctl, u32 ckctl)
if (!_omap_sram_reprogram_clock)
omap_sram_error();
- return _omap_sram_reprogram_clock(dpllctl, ckctl);
+ _omap_sram_reprogram_clock(dpllctl, ckctl);
}
int __init omap1_sram_init(void)
@@ -282,8 +282,8 @@ void omap2_sram_ddr_init(u32 *slow_dll_ctrl, u32 fast_dll_ctrl,
if (!_omap2_sram_ddr_init)
omap_sram_error();
- return _omap2_sram_ddr_init(slow_dll_ctrl, fast_dll_ctrl,
- base_cs, force_unlock);
+ _omap2_sram_ddr_init(slow_dll_ctrl, fast_dll_ctrl,
+ base_cs, force_unlock);
}
static void (*_omap2_sram_reprogram_sdrc)(u32 perf_level, u32 dll_val,
@@ -294,7 +294,7 @@ void omap2_sram_reprogram_sdrc(u32 perf_level, u32 dll_val, u32 mem_type)
if (!_omap2_sram_reprogram_sdrc)
omap_sram_error();
- return _omap2_sram_reprogram_sdrc(perf_level, dll_val, mem_type);
+ _omap2_sram_reprogram_sdrc(perf_level, dll_val, mem_type);
}
static u32 (*_omap2_set_prcm)(u32 dpll_ctrl_val, u32 sdrc_rfr_val, int bypass);
diff --git a/drivers/video/omap/omapfb_main.c b/drivers/video/omap/omapfb_main.c
index 5a5e407..1a49519 100644
--- a/drivers/video/omap/omapfb_main.c
+++ b/drivers/video/omap/omapfb_main.c
@@ -392,7 +392,7 @@ static void set_fb_fix(struct fb_info *fbi)
int bpp;
rg = &plane->fbdev->mem_desc.region[plane->idx];
- fbi->screen_base = (char __iomem *)rg->vaddr;
+ fbi->screen_base = rg->vaddr;
fix->smem_start = rg->paddr;
fix->smem_len = rg->size;
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] fix a pile of OMAP issues
2008-12-01 16:05 [PATCH] fix a pile of OMAP issues Russell King - ARM Linux
@ 2008-12-01 17:01 ` Tony Lindgren
0 siblings, 0 replies; 2+ messages in thread
From: Tony Lindgren @ 2008-12-01 17:01 UTC (permalink / raw)
To: Russell King - ARM Linux; +Cc: linux-omap, linux-arm-kernel
* Russell King - ARM Linux <linux@arm.linux.org.uk> [081201 08:05]:
> This patch fixes a number of sillies, from missing 'const' to using
> 'return' in void functions, to functions with no arguments not even
> 'void' and a cast which isn't required.
>
> Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
>
> --
> I want to merge this for the next ARM kernel pull by Linus. Please
> ack the patch.
Good stuff, here's my ack:
Acked-by: Tony Lindgren <tony@atomide.com>
Tony
> arch/arm/mach-omap1/io.c | 2 +-
> arch/arm/plat-omap/include/mach/omapfb.h | 4 ++--
> arch/arm/plat-omap/sram.c | 8 ++++----
> drivers/video/omap/omapfb_main.c | 2 +-
> 4 files changed, 8 insertions(+), 8 deletions(-)
>
> diff --git a/arch/arm/mach-omap1/io.c b/arch/arm/mach-omap1/io.c
> index b3bd8ca..4c3e582 100644
> --- a/arch/arm/mach-omap1/io.c
> +++ b/arch/arm/mach-omap1/io.c
> @@ -128,7 +128,7 @@ void __init omap1_map_common_io(void)
> * Common low-level hardware init for omap1. This should only get called from
> * board specific init.
> */
> -void __init omap1_init_common_hw()
> +void __init omap1_init_common_hw(void)
> {
> /* REVISIT: Refer to OMAP5910 Errata, Advisory SYS_1: "Timeout Abort
> * on a Posted Write in the TIPB Bridge".
> diff --git a/arch/arm/plat-omap/include/mach/omapfb.h b/arch/arm/plat-omap/include/mach/omapfb.h
> index ec67fb4..7b74d12 100644
> --- a/arch/arm/plat-omap/include/mach/omapfb.h
> +++ b/arch/arm/plat-omap/include/mach/omapfb.h
> @@ -353,8 +353,8 @@ struct omapfb_device {
> u32 pseudo_palette[17];
>
> struct lcd_panel *panel; /* LCD panel */
> - struct lcd_ctrl *ctrl; /* LCD controller */
> - struct lcd_ctrl *int_ctrl; /* internal LCD ctrl */
> + const struct lcd_ctrl *ctrl; /* LCD controller */
> + const struct lcd_ctrl *int_ctrl; /* internal LCD ctrl */
> struct lcd_ctrl_extif *ext_if; /* LCD ctrl external
> interface */
> struct device *dev;
> diff --git a/arch/arm/plat-omap/sram.c b/arch/arm/plat-omap/sram.c
> index 9f9a921..dcd9d16 100644
> --- a/arch/arm/plat-omap/sram.c
> +++ b/arch/arm/plat-omap/sram.c
> @@ -255,7 +255,7 @@ void omap_sram_reprogram_clock(u32 dpllctl, u32 ckctl)
> if (!_omap_sram_reprogram_clock)
> omap_sram_error();
>
> - return _omap_sram_reprogram_clock(dpllctl, ckctl);
> + _omap_sram_reprogram_clock(dpllctl, ckctl);
> }
>
> int __init omap1_sram_init(void)
> @@ -282,8 +282,8 @@ void omap2_sram_ddr_init(u32 *slow_dll_ctrl, u32 fast_dll_ctrl,
> if (!_omap2_sram_ddr_init)
> omap_sram_error();
>
> - return _omap2_sram_ddr_init(slow_dll_ctrl, fast_dll_ctrl,
> - base_cs, force_unlock);
> + _omap2_sram_ddr_init(slow_dll_ctrl, fast_dll_ctrl,
> + base_cs, force_unlock);
> }
>
> static void (*_omap2_sram_reprogram_sdrc)(u32 perf_level, u32 dll_val,
> @@ -294,7 +294,7 @@ void omap2_sram_reprogram_sdrc(u32 perf_level, u32 dll_val, u32 mem_type)
> if (!_omap2_sram_reprogram_sdrc)
> omap_sram_error();
>
> - return _omap2_sram_reprogram_sdrc(perf_level, dll_val, mem_type);
> + _omap2_sram_reprogram_sdrc(perf_level, dll_val, mem_type);
> }
>
> static u32 (*_omap2_set_prcm)(u32 dpll_ctrl_val, u32 sdrc_rfr_val, int bypass);
> diff --git a/drivers/video/omap/omapfb_main.c b/drivers/video/omap/omapfb_main.c
> index 5a5e407..1a49519 100644
> --- a/drivers/video/omap/omapfb_main.c
> +++ b/drivers/video/omap/omapfb_main.c
> @@ -392,7 +392,7 @@ static void set_fb_fix(struct fb_info *fbi)
> int bpp;
>
> rg = &plane->fbdev->mem_desc.region[plane->idx];
> - fbi->screen_base = (char __iomem *)rg->vaddr;
> + fbi->screen_base = rg->vaddr;
> fix->smem_start = rg->paddr;
> fix->smem_len = rg->size;
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2008-12-01 17:01 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-12-01 16:05 [PATCH] fix a pile of OMAP issues Russell King - ARM Linux
2008-12-01 17:01 ` Tony Lindgren
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox