From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kumar Gala Date: Fri, 3 Jun 2011 02:46:39 -0500 Subject: [U-Boot] [PATCH 6/7] powerpc/mpc8xxx: Adding fallback to raw timing on supported boards In-Reply-To: <1306452353-11611-6-git-send-email-yorksun@freescale.com> References: <1306452353-11611-1-git-send-email-yorksun@freescale.com> <1306452353-11611-6-git-send-email-yorksun@freescale.com> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On May 26, 2011, at 6:25 PM, York Sun wrote: > In case of empty SPD or checksum error, fallback to raw timing on > supported boards. > > Signed-off-by: York Sun > --- > arch/powerpc/cpu/mpc8xxx/ddr/main.c | 8 ++++++++ > 1 files changed, 8 insertions(+), 0 deletions(-) > > diff --git a/arch/powerpc/cpu/mpc8xxx/ddr/main.c b/arch/powerpc/cpu/mpc8xxx/ddr/main.c > index db7cf11..2e43da6 100644 > --- a/arch/powerpc/cpu/mpc8xxx/ddr/main.c > +++ b/arch/powerpc/cpu/mpc8xxx/ddr/main.c > @@ -343,12 +343,20 @@ fsl_ddr_compute(fsl_ddr_info_t *pinfo, unsigned int start_step, > &(pinfo->dimm_params[i][j]); > > retval = compute_dimm_parameters(spd, pdimm, i); > +#ifdef CONFIG_DDR_RAW_TIMING Should this be CONFIG_SYS_DDR_RAW_TIMING?? We need this added to README at a minimum for what DDR_RAW_TIMING means. > + if (retval != 0) { > + printf("SPD error! Trying fallback to " > + "raw timing calculation\n"); > + fsl_ddr_get_dimm_params(pdimm, i, j); > + } > +#else > if (retval == 2) { > printf("Error: compute_dimm_parameters" > " non-zero returned FATAL value " > "for memctl=%u dimm=%u\n", i, j); > return 0; > } > +#endif > if (retval) { > debug("Warning: compute_dimm_parameters" > " non-zero return value for memctl=%u " > -- > 1.7.0.4 > > > _______________________________________________ > U-Boot mailing list > U-Boot at lists.denx.de > http://lists.denx.de/mailman/listinfo/u-boot