* [PATCH] OMAP: Fix build failure on MEMBLOCK_REAL_LIMIT
@ 2010-11-01 16:30 Namhyung Kim
2010-11-02 9:43 ` Tomi Valkeinen
2010-11-07 22:44 ` Felipe Contreras
0 siblings, 2 replies; 3+ messages in thread
From: Namhyung Kim @ 2010-11-01 16:30 UTC (permalink / raw)
To: Tony Lindgren, Tomi Valkeinen, linux-omap
The MEMBLOCK_REAL_LIMIT symbol was removed by the commit e63075a3c937
(memblock: Introduce default allocation limit and use it to replace
explicit ones) so it breaks beagle board build as follows:
CC drivers/video/omap2/vram.o
drivers/video/omap2/vram.c: In function 'omap_vram_reserve_sdram_memblock':
drivers/video/omap2/vram.c:573: error: 'MEMBLOCK_REAL_LIMIT' undeclared (first use in this function)
drivers/video/omap2/vram.c:573: error: (Each undeclared identifier is reported only once
drivers/video/omap2/vram.c:573: error: for each function it appears in.)
make[5]: *** [drivers/video/omap2/vram.o] Error 1
make[4]: *** [drivers/video/omap2] Error 2
make[3]: *** [drivers/video] Error 2
make[2]: *** [drivers] Error 2
make[1]: *** [sub-make] Error 2
make: *** [all] Error 2
Signed-off-by: Namhyung Kim <namhyung@gmail.com>
---
drivers/video/omap2/vram.c | 3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)
diff --git a/drivers/video/omap2/vram.c b/drivers/video/omap2/vram.c
index fed2a72..39b85f7 100644
--- a/drivers/video/omap2/vram.c
+++ b/drivers/video/omap2/vram.c
@@ -570,7 +570,8 @@ void __init omap_vram_reserve_sdram_memblock(void)
return;
}
} else {
- paddr = memblock_alloc_base(size, PAGE_SIZE, MEMBLOCK_REAL_LIMIT);
+ paddr = memblock_alloc_base(size, PAGE_SIZE,
+ MEMBLOCK_ALLOC_ACCESSIBLE);
}
omap_vram_add_region(paddr, size);
--
1.7.0.4
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH] OMAP: Fix build failure on MEMBLOCK_REAL_LIMIT
2010-11-01 16:30 [PATCH] OMAP: Fix build failure on MEMBLOCK_REAL_LIMIT Namhyung Kim
@ 2010-11-02 9:43 ` Tomi Valkeinen
2010-11-07 22:44 ` Felipe Contreras
1 sibling, 0 replies; 3+ messages in thread
From: Tomi Valkeinen @ 2010-11-02 9:43 UTC (permalink / raw)
To: ext Namhyung Kim; +Cc: Tony Lindgren, linux-omap@vger.kernel.org
On Mon, 2010-11-01 at 17:30 +0100, ext Namhyung Kim wrote:
> The MEMBLOCK_REAL_LIMIT symbol was removed by the commit e63075a3c937
> (memblock: Introduce default allocation limit and use it to replace
> explicit ones) so it breaks beagle board build as follows:
>
> CC drivers/video/omap2/vram.o
> drivers/video/omap2/vram.c: In function 'omap_vram_reserve_sdram_memblock':
> drivers/video/omap2/vram.c:573: error: 'MEMBLOCK_REAL_LIMIT' undeclared (first use in this function)
> drivers/video/omap2/vram.c:573: error: (Each undeclared identifier is reported only once
> drivers/video/omap2/vram.c:573: error: for each function it appears in.)
> make[5]: *** [drivers/video/omap2/vram.o] Error 1
> make[4]: *** [drivers/video/omap2] Error 2
> make[3]: *** [drivers/video] Error 2
> make[2]: *** [drivers] Error 2
> make[1]: *** [sub-make] Error 2
> make: *** [all] Error 2
Thanks, applied to my tree.
Tomi
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] OMAP: Fix build failure on MEMBLOCK_REAL_LIMIT
2010-11-01 16:30 [PATCH] OMAP: Fix build failure on MEMBLOCK_REAL_LIMIT Namhyung Kim
2010-11-02 9:43 ` Tomi Valkeinen
@ 2010-11-07 22:44 ` Felipe Contreras
1 sibling, 0 replies; 3+ messages in thread
From: Felipe Contreras @ 2010-11-07 22:44 UTC (permalink / raw)
To: Namhyung Kim; +Cc: Tony Lindgren, Tomi Valkeinen, linux-omap
On Mon, Nov 1, 2010 at 6:30 PM, Namhyung Kim <namhyung@gmail.com> wrote:
> --- a/drivers/video/omap2/vram.c
> +++ b/drivers/video/omap2/vram.c
> @@ -570,7 +570,8 @@ void __init omap_vram_reserve_sdram_memblock(void)
> return;
> }
> } else {
> - paddr = memblock_alloc_base(size, PAGE_SIZE, MEMBLOCK_REAL_LIMIT);
> + paddr = memblock_alloc_base(size, PAGE_SIZE,
> + MEMBLOCK_ALLOC_ACCESSIBLE);
This is exactly the same as:
paddr = memblock_alloc(size, PAGE_SIZE);
Let's use that instead.
--
Felipe Contreras
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2010-11-07 22:44 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-11-01 16:30 [PATCH] OMAP: Fix build failure on MEMBLOCK_REAL_LIMIT Namhyung Kim
2010-11-02 9:43 ` Tomi Valkeinen
2010-11-07 22:44 ` Felipe Contreras
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox