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

York Sun yorksun at freescale.com
Mon Mar 14 19:26:38 CET 2011


On Mon, 2011-03-14 at 12:38 -0400, 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(-)
> 

Ack.

York





More information about the U-Boot mailing list