Skip to content
Snippets Groups Projects
Commit 8703ef3f authored by Simon Glass's avatar Simon Glass
Browse files

common: Move LCD and video memory reservation together


These two functions are conceptually the same. Move them together in the
pre-relocation init.

Signed-off-by: default avatarSimon Glass <sjg@chromium.org>
Acked-by: default avatarAnatolij Gustschin <agust@denx.de>
parent b5146b28
No related branches found
No related tags found
No related merge requests found
...@@ -451,22 +451,10 @@ static int reserve_lcd(void) ...@@ -451,22 +451,10 @@ static int reserve_lcd(void)
} }
#endif /* CONFIG_LCD */ #endif /* CONFIG_LCD */
static int reserve_trace(void)
{
#ifdef CONFIG_TRACE
gd->relocaddr -= CONFIG_TRACE_BUFFER_SIZE;
gd->trace_buff = map_sysmem(gd->relocaddr, CONFIG_TRACE_BUFFER_SIZE);
debug("Reserving %dk for trace data at: %08lx\n",
CONFIG_TRACE_BUFFER_SIZE >> 10, gd->relocaddr);
#endif
return 0;
}
#if defined(CONFIG_VIDEO) && (!defined(CONFIG_PPC) || defined(CONFIG_8xx)) && \ #if defined(CONFIG_VIDEO) && (!defined(CONFIG_PPC) || defined(CONFIG_8xx)) && \
!defined(CONFIG_ARM) && !defined(CONFIG_X86) && \ !defined(CONFIG_ARM) && !defined(CONFIG_X86) && \
!defined(CONFIG_BLACKFIN) && !defined(CONFIG_M68K) !defined(CONFIG_BLACKFIN) && !defined(CONFIG_M68K)
static int reserve_video(void) static int reserve_legacy_video(void)
{ {
/* reserve memory for video display (always full pages) */ /* reserve memory for video display (always full pages) */
gd->relocaddr = video_setmem(gd->relocaddr); gd->relocaddr = video_setmem(gd->relocaddr);
...@@ -476,6 +464,18 @@ static int reserve_video(void) ...@@ -476,6 +464,18 @@ static int reserve_video(void)
} }
#endif #endif
static int reserve_trace(void)
{
#ifdef CONFIG_TRACE
gd->relocaddr -= CONFIG_TRACE_BUFFER_SIZE;
gd->trace_buff = map_sysmem(gd->relocaddr, CONFIG_TRACE_BUFFER_SIZE);
debug("Reserving %dk for trace data at: %08lx\n",
CONFIG_TRACE_BUFFER_SIZE >> 10, gd->relocaddr);
#endif
return 0;
}
static int reserve_uboot(void) static int reserve_uboot(void)
{ {
/* /*
...@@ -960,13 +960,13 @@ static init_fnc_t init_sequence_f[] = { ...@@ -960,13 +960,13 @@ static init_fnc_t init_sequence_f[] = {
#ifdef CONFIG_LCD #ifdef CONFIG_LCD
reserve_lcd, reserve_lcd,
#endif #endif
reserve_trace,
/* TODO: Why the dependency on CONFIG_8xx? */ /* TODO: Why the dependency on CONFIG_8xx? */
#if defined(CONFIG_VIDEO) && (!defined(CONFIG_PPC) || defined(CONFIG_8xx)) && \ #if defined(CONFIG_VIDEO) && (!defined(CONFIG_PPC) || defined(CONFIG_8xx)) && \
!defined(CONFIG_ARM) && !defined(CONFIG_X86) && \ !defined(CONFIG_ARM) && !defined(CONFIG_X86) && \
!defined(CONFIG_BLACKFIN) && !defined(CONFIG_M68K) !defined(CONFIG_BLACKFIN) && !defined(CONFIG_M68K)
reserve_video, reserve_video,
#endif #endif
reserve_trace,
#if !defined(CONFIG_BLACKFIN) #if !defined(CONFIG_BLACKFIN)
reserve_uboot, reserve_uboot,
#endif #endif
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment