Re: [U-Boot] [PATCH] powerpc/mpc8xxx: reword max tCKmin message

2011-05-05 Thread Kumar Gala
On May 2, 2011, at 10:40 PM, York Sun wrote: On Mon, 2011-05-02 at 20:19 -0700, Tabi Timur-B04825 wrote: On Mon, May 2, 2011 at 8:51 PM, York Sun york...@freescale.com wrote: - printf(The DIMM max tCKmin is %d ps, + printf(The combined minimum tCKmin is %d ps,

Re: [U-Boot] [PATCH] powerpc/mpc8xxx: reword max tCKmin message

2011-05-05 Thread York Sun
On Thu, 2011-05-05 at 10:04 -0500, Timur Tabi wrote: Kumar Gala wrote: That still needs some work, IMHO. I think you might need the word which before doesn't. However, even with that, it's not clear what's wrong. Where does the bad value of mclk_ps come from? It happens

[U-Boot] [PATCH] powerpc/mpc8xxx: reword max tCKmin message

2011-05-02 Thread York Sun
Reword The DIMM max tCKmin is to The combined tCKmin is. It means the max tCKmin across all DIMMs on the same controller. Signed-off-by: York Sun york...@freescale.com --- .../cpu/mpc8xxx/ddr/lc_common_dimm_params.c|2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git

Re: [U-Boot] [PATCH] powerpc/mpc8xxx: reword max tCKmin message

2011-05-02 Thread Tabi Timur-B04825
On Mon, May 2, 2011 at 8:51 PM, York Sun york...@freescale.com wrote: -               printf(The DIMM max tCKmin is %d ps, +               printf(The combined minimum tCKmin is %d ps,                        doesn't support the MCLK cycle %d ps\n, That still needs some work, IMHO. I think you

Re: [U-Boot] [PATCH] powerpc/mpc8xxx: reword max tCKmin message

2011-05-02 Thread York Sun
On Mon, 2011-05-02 at 20:19 -0700, Tabi Timur-B04825 wrote: On Mon, May 2, 2011 at 8:51 PM, York Sun york...@freescale.com wrote: - printf(The DIMM max tCKmin is %d ps, + printf(The combined minimum tCKmin is %d ps, doesn't support the