[U-Boot] [PATCH v5 1/3] mpc8xxx: DDR2/DDR3: Clean up DIMM-type switch statements

Kumar Gala kumar.gala at freescale.com
Tue Mar 15 08:55:38 CET 2011


On Mar 14, 2011, at 11:38 AM, Kyle Moffett wrote:

> The numeric constants in the switch statements are replaced by #defines
> added to the common ddr_spd.h header.  This dramatically improves the
> readability of the switch statments.
> 
> In addition, a few of the longer lines were cleaned up, and the DDR2
> type for an SO-RDIMM module was added to the DDR2 switch statement.
> 
> Signed-off-by: Kyle Moffett <Kyle.D.Moffett at boeing.com>
> Cc: Andy Fleming <afleming at gmail.com>
> Cc: Kumar Gala <kumar.gala at freescale.com>
> Cc: Kim Phillips <kim.phillips at freescale.com>
> ---
> arch/powerpc/cpu/mpc8xxx/ddr/ddr2_dimm_params.c |   23 +++++++-----
> arch/powerpc/cpu/mpc8xxx/ddr/ddr3_dimm_params.c |   43 ++++++++++------------
> common/ddr_spd.c                                |    2 +-
> include/ddr_spd.h                               |   28 ++++++++++-----
> 4 files changed, 53 insertions(+), 43 deletions(-)

Can you rebase on 'next' branch of git://git.denx.de/u-boot-mpc85xx.git

- k


More information about the U-Boot mailing list