From mboxrd@z Thu Jan 1 00:00:00 1970 From: Heiko Schocher Date: Wed, 11 Mar 2009 09:33:04 +0100 Subject: [U-Boot] [PATCH] 83xx: correctly set encryption and I2C bus 0 clock In-Reply-To: <49B767DC.7020205@aimvalley.nl> References: <49B767DC.7020205@aimvalley.nl> Message-ID: <49B77740.7010400@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 Norbert, added Kim Phillips to cc as he is the 83xx maintainer Norbert van Bolhuis wrote: > This patch makes sure the correct mask is applied when setting > the encryption and I2C bus 0 clock in SCCR. > Failing to do so may lead to ENCCM being 0 in which case I2C bus 0 > won't function. > > > --- cpu/mpc83xx/cpu_init.c.org 2009-03-11 07:56:57.000000000 +0100 > +++ cpu/mpc83xx/cpu_init.c 2009-03-11 07:57:37.000000000 +0100 > @@ -106,7 +106,7 @@ void cpu_init_f (volatile immap_t * im) > #ifdef CONFIG_SYS_SCCR_ENCCM > /* Encryption clock mode */ > im->clk.sccr = (im->clk.sccr & ~SCCR_ENCCM) | > - (CONFIG_SYS_SCCR_ENCCM << SCCR_PCICM_SHIFT); > + (CONFIG_SYS_SCCR_ENCCM << SCCR_ENCCM_SHIFT); > #endif > > #ifdef CONFIG_SYS_SCCR_PCICM > > > Signed-off-by: Norbert van Bolhuis > Can you please use git-format-patch for making a patch? Otherwise your patch seems okay to me, thanks Heiko -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany