From mboxrd@z Thu Jan 1 00:00:00 1970 From: Heiko Schocher Date: Mon, 18 Jan 2010 09:34:08 +0100 Subject: [U-Boot] arm: suen3, suen3_v1, mgcoge2_arm_p1a support In-Reply-To: <20100117235121.5E479C88AE@gemini.denx.de> References: <4B459395.2050301@denx.de> <20100117235121.5E479C88AE@gemini.denx.de> Message-ID: <4B541D00.6090806@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 Hello Wolfgang, Wolfgang Denk wrote: > In message <4B459395.2050301@denx.de> you wrote: >> This patch adds support for the Keymile SUEN3 board variants which >> are based on the Marvell Kirkwood (88F6281) SoC. All variants >> uses common code stored in board/keymile/km_arm/km_arm.c >> >> mgcoge2_arm_p1a board: >> This adds support for the ARM part of the mgcoge2. The suen3 >> target was moved to the correct suen3 p1b version. There is a >> difference between the GPIO configuration between suen3 and mgcoge2. > ... > > >> --- /dev/null >> +++ b/include/configs/km-arm.h > ... >> +#if 1 // test-only cramfs ... >> +#define CONFIG_SYS_MAX_FLASH_SECT 256 > > Please do not add dead code (the "#if 1" here), > and do not use C++ comments (please fix this globally). Ups, sorry. >> +#ifndef __ASSEMBLY__ >> +typedef struct { >> + unsigned long size; /* total bank size in bytes */ >> + unsigned short sector_count; /* number of erase units */ >> + unsigned long flash_id; /* combined device & manufacturer code */ >> + unsigned long start[CONFIG_SYS_MAX_FLASH_SECT]; /* virtual sector start address */ >> + unsigned char protect[CONFIG_SYS_MAX_FLASH_SECT]; /* sector protection status */ >> +} flash_info_t; > > Lines way too long. Please fix globally. > > Umm.. why cannot you use the struct flash_info_t as declared in > "include/flash.h" ?? Hmm.. I take a look at this point... Stefan? Maybe, you can help here, because this comes from your original port, I think. bye Heiko -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany