Wolfgang,

Without progress indicator, slowe test on memory takes minutes and it looks like hanging. You probably don't want to run it every time the board boots up.

York Sun ----- Original Message ----- From:"Wolfgang Denk" wd@denx.de To:"York Sun" yorksun@freescale.com Cc:"u-boot@lists.denx.de" u-boot@lists.denx.de Sent:8/29/2010 3:56 AM Subject:Re: [U-Boot] [PATCH 2/7] Expand POST memory test to support arch-depended implementation.

Dear York Sun,

In message 1282944356-4020-2-git-send-email-yorksun@freescale.com you wrote:

Add progress indicator for slow test. It is useful when the testing takes too longer to finish. The indicator is reused from flash programming.

Hwconfig is used to turn on slow test when not enabled by flag.

NAK.

POST is supposed to be an automatic, unmonitored functionality. Results are suposed to be reported through a mechanism compatible to Linux' syslog system.

There is no place for progress indicators here.

Please note that your subject "support arch-depended implementation" and the commit message "Add progress indicator" are seriously out of sync as well. Another reason for a NAK.

Best regards,

Wolfgang Denk