From mboxrd@z Thu Jan 1 00:00:00 1970 From: Anatolij Gustschin Date: Mon, 27 Oct 2008 10:19:34 +0100 Subject: [U-Boot] [PATCH 5/5] lcd: Let the board code show board-specific info In-Reply-To: <1220278882-26029-5-git-send-email-haavard.skinnemoen@atmel.com> References: <1220278882-26029-1-git-send-email-haavard.skinnemoen@atmel.com> <1220278882-26029-2-git-send-email-haavard.skinnemoen@atmel.com> <1220278882-26029-3-git-send-email-haavard.skinnemoen@atmel.com> <1220278882-26029-4-git-send-email-haavard.skinnemoen@atmel.com> <1220278882-26029-5-git-send-email-haavard.skinnemoen@atmel.com> Message-ID: <490587A6.4010005@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Haavard Skinnemoen wrote: > The information displayed when CONFIG_LCD_INFO is set is inherently > board-specific, so it should be done by the board code. The current code > dealing with this only handles two cases, and is already a horrible mess > of #ifdeffery. > > Yes, this duplicates some code, but it also allows boards to print more > board-specific information; this used to be very difficult. > > Signed-off-by: Haavard Skinnemoen > --- > board/atmel/at91cap9adk/at91cap9adk.c | 29 ++++++++++ > board/atmel/at91sam9261ek/at91sam9261ek.c | 29 ++++++++++ > board/atmel/at91sam9263ek/at91sam9263ek.c | 29 ++++++++++ > board/atmel/at91sam9rlek/at91sam9rlek.c | 29 ++++++++++ > board/lwmon/lwmon.c | 29 ++++++++++ > board/tqc/tqm8xx/tqm8xx.c | 26 +++++++++ > common/lcd.c | 84 ++--------------------------- > include/lcd.h | 2 + > 8 files changed, 178 insertions(+), 79 deletions(-) Applied to u-boot-video/master repo. Thanks! Also fixed by s/CFG_MAX_NAND_DEVICE/CONFIG_SYS_MAX_NAND_DEVICE as it is needed since commit 6d0f6bcf337c5261c08fabe12982178c2c489d76. Best regards, Anatolij -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: office at denx.de