From mboxrd@z Thu Jan 1 00:00:00 1970 From: Heiko Schocher Date: Wed, 11 Mar 2009 13:37:18 +0100 Subject: [U-Boot] [PATCH 3/3] 8xx: add support for new keymile kmsupx4 board. In-Reply-To: <20090311114029.08FCA832E8B8@gemini.denx.de> References: <49B770A2.9020302@denx.de> <20090311114029.08FCA832E8B8@gemini.denx.de> Message-ID: <49B7B07E.1080606@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 <49B770A2.9020302@denx.de> you wrote: >> This patch adds support for the kmsupx4 board from Keymile, >> based on a Freescale MPC852T CPU > ... >> --- a/cpu/mpc8xx/start.S >> +++ b/cpu/mpc8xx/start.S >> @@ -142,7 +142,8 @@ boot_warm: >> lis r3, IDC_DISABLE at h /* Disable data cache */ >> mtspr DC_CST, r3 >> >> -#if !(defined(CONFIG_IP860) || defined(CONFIG_PCU_E) || defined (CONFIG_FLAGADM)) >> +#if !(defined(CONFIG_IP860) || defined(CONFIG_PCU_E) || \ >> + defined (CONFIG_FLAGADM) || defined(CONFIG_KMSUPX4)) >> /* On IP860 and PCU E, >> * we cannot enable IC yet >> */ > > Why exactly is this necessary? > > I would like to avoid new #ifdef's in common code in general, but > here this should most likely be not needed either (IP860 and PCU E > had very special hardware issues; I don't know about FLAGADM) - are > you absolutely sure you need that? Hmm.. when I enable the IC here the CPU is dead ... but you are right, there should be no such special thing for this port ... I try to find out more. >> diff --git a/drivers/i2c/soft_i2c.c b/drivers/i2c/soft_i2c.c >> index da6cec1..f273c67 100644 >> --- a/drivers/i2c/soft_i2c.c >> +++ b/drivers/i2c/soft_i2c.c >> @@ -43,6 +43,9 @@ >> #ifdef CONFIG_MPC866 /* only valid for MPC866 */ >> #include >> #endif >> +#ifdef CONFIG_MPC852T /* only valid for MPC852T */ >> +#include >> +#endif > > Please change into > > #if defined(CONFIG_MPC852T) || defined(CONFIG_MPC866) OK thanks Heiko -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany