Skip to contentSkip to conversation

ARM: Don't grab memory for LCD if FB address is defined

PATCH New Minkyu Kang 2 participants 2 messages Mar 3, 2011 Arm (32-bit)

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 ARM: Don't grab memory for LCD if FB address is defined

If FB address is defined as specific address then don't grab memory for LCD

---
 arch/arm/lib/board.c |    4 ++++
 1 files changed, 4 insertions(+), 0 deletions(-)
arch/arm/lib/board.c +4 -0
index c620d2c..5a4d2bd 100644
@@ -356,9 +356,13 @@ void board_init_f (ulong bootflag)
 #endif /* CONFIG_VFD */
 
 #ifdef CONFIG_LCD
+#ifdef CONFIG_FB_ADDR
+	gd->fb_base = CONFIG_FB_ADDR;
+#else
 	/* reserve memory for LCD display (always full pages) */
 	addr = lcd_setmem (addr);
 	gd->fb_base = addr;
+#endif /* CONFIG_FB_ADDR */
 #endif /* CONFIG_LCD */
 
 	/*
Signature · 1 line
1.7.1
#1 lore raw

PATCH 1/1 ARM: Don't grab memory for LCD if FB address is defined

Dear Minkyu Kang,

Quoted 6 lines · In message <[email protected]> you wrote:

If FB address is defined as specific address then don't grab memory for LCD

---
 arch/arm/lib/board.c |    4 ++++
 1 files changed, 4 insertions(+), 0 deletions(-)

Sorry, but NAK.

You cannot make this change for ARM only. It needs to be implemented for all architectures.

Best regards,

Wolfgang Denk

Signature · 7 lines
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
There comes to all races an ultimate crisis which  you  have  yet  to
face  ....  One  day  our  minds became so powerful we dared think of
ourselves as gods.
	-- Sargon, "Return to Tomorrow", stardate 4768.3