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 when the actually DDR clock is faster than the slowest DIMM > >> > can support. > >> > > >> > York > > Did you guys agree on wording? Is the patch ok or needs changing? > > I haven't seen an updated patch from York. I'd like to see the addition of > text > like, "It happens when the actually DDR clock is faster than the slowest DIMM > can support." People need to know what to fix, not just what's wrong. > > Also, %d is for signed integers, not unsigned. >
I will update the patch according to Timur's suggestion. York _______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot
