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

Kumar Gala kumar.gala at freescale.com
Thu Mar 31 10:30:42 CEST 2011


On Mar 15, 2011, at 12:04 PM, 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>
> 
> ---
> Changelog:
> v2: Moved the constants to include/ddr_spd.h and also fixed DDR3
> v3: No changes
> v4: Fixed up excessively long lines
> v5: No changes
> v6: Rebased onto the 'next' branch of git://git.denx.de/u-boot-mpc85xx.git
> 
> 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(-)

applied to 85xx next

- k


More information about the U-Boot mailing list