Dear Aldo Cedillo, In message <1283212736-32760-1-git-send-email-aldocedi...@ti.com> you wrote: > From: Aldo Brett Cedillo Martinez <aldo.cedi...@ti.com> > > Zoom2 and Zoom2 used to hang with "md" command. It was due to a problem > with a buffer size in print_buffer() function. A macro was introduced > to use a different buffer size in case of compiling for Zoom2 and Zoom3.
NAK to this patch. Forst, I thinkt he problem is not with buffer size, but with buffer alignemnt as observed elasewhere, so most probably Albert's "display_buffer: fix misaligned buffer" patch will fix this issue, too. > #define DEFAULT_LINE_LENGTH_BYTES (16) > int print_buffer (ulong addr, void* data, uint width, uint count, uint > linelen) > { > +#if defined(CONFIG_OMAP3_ZOOM2) || defined(CONFIG_OMAP3_ZOOM3) > + uint8_t linebuf[DEFAULT_LINE_LENGTH_BYTES]; > +#else > uint8_t linebuf[MAX_LINE_LENGTH_BYTES + 1]; > +#endif /* CONFIG_OMAP3 */ Second, it is broken to use DEFAULT_LINE_LENGTH_BYTES in the declaration of the buffer, but then use MAX_LINE_LENGTH_BYTES+1 when accessing it. [Actually I think we should kill one of these defines.] Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de Overflow on /dev/null, please empty the bit bucket. _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot