On Wed, Apr 7, 2010 at 3:40 AM, Kim Phillips <kim.phill...@freescale.com> wrote: > On Tue, 6 Apr 2010 18:29:08 +0300 > Michael Zaidman <michael.zaid...@gmail.com> wrote: > >> >> Rewriting code to use accessors API added 56 bytes or 14 opcodes. It >> >> doubled the routine's execution time. The routine is called in the >> >> loop per each ecc word, that significantly increased execution time of >> >> the ECC POST test and consequently board startup time. >> > >> > If this is really a problem, you can modify the code to perform the >> > writes without the I/O accessors, and then terminate the block with a >> > sync. >> >> I thought that is what I actually do in this patch... >> >> > You should add a comment explaining why you're not using I/O >> > accessors. >> >> Ok, I will add the explanation. > > use raw i/o accessors please. > > oh, and don't forget to sign off your work > > Thanks, > > Kim >
Thanks for your feedback. The version #2 of the patch is located here http://lists.denx.de/pipermail/u-boot/2010-April/069579.html -michael _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot