[U-Boot] [PATCH] MPC85xxCDS: Fix missing LCRR_DBYP bits for 66-133MHz LBC

Kumar Gala galak at kernel.crashing.org
Fri Jan 13 19:55:40 CET 2012


On Dec 16, 2011, at 4:31 PM, Paul Gortmaker wrote:

> These boards were meaning to deploy this value:
> 
>  #define LCRR_DBYP        0x80000000
> 
> but were missing a zero, and hence toggling a bit that
> lands in an area marked as reserved in the 8548 reference
> manual.
> 
> According to the documentation, LCRR_DBYP should be used as:
> 
>   PLL bypass. This bit should be set when using low bus
>   clock frequencies if the PLL is unable to lock.  When in
>   PLL bypass mode, incoming data is captured in the middle
>   of the bus clock cycle.  It is recommended that PLL bypass
>   mode be used at frequencies of 83 MHz or less.
> 
> So the impact would most likely be undefined behaviour for
> LBC peripherals on boards that were running below 83MHz LBC.
> Looking at the actual u-boot code, the missing DBYP bit was
> meant to be deployed as follows:
> 
>      Between 66 and 133, the DLL is enabled with an
>      override workaround.
> 
> In the future, we'll convert all boards to use the symbolic
> DBYP constant to avoid these "count the zeros" problems, but
> for now, just fix the impacted boards.
> 
> Signed-off-by: Paul Gortmaker <paul.gortmaker at windriver.com>

applied to 85xx

- k


More information about the U-Boot mailing list