[U-Boot] [PATCH v3 6/8] POST: add new memory regions test

Wolfgang Denk wd at denx.de
Thu Oct 6 23:42:18 CEST 2011


Dear Valentin Longchamp,

In message <1314975550-15766-7-git-send-email-valentin.longchamp at keymile.com> you wrote:
> This test is similar to the actual POST memory test but quicker and
> far less complete. It checks the address and data lines and then only
> tests some regularly placed sub regions of the RAM.
> This can be useful when we want to test the RAM but we do not have enough
> time to run the full memory test.
> 
> The POST memory test code was rearranged in order to avoid code duplication
> between the two tests but the memory test functionnality remains the same.
> 
> Signed-off-by: Valentin Longchamp <valentin.longchamp at keymile.com>
> Signed-off-by: Holger Brunck <holger.brunck at keymile.com>
> Cc: Mike Frysinger <vapier at gentoo.org>
> ---
> Changes for v3:
>  - cosmetic changes in the test list
> ---
>  include/post.h        |    2 +
>  post/drivers/memory.c |   81 +++++++++++++++++++++++++++++++++++++-----------
>  post/tests.c          |   15 ++++++++-
>  3 files changed, 78 insertions(+), 20 deletions(-)

Checkpatch says:

total: 0 errors, 4 warnings, 158 lines checked

Please clean up and resubmit.  Thanks.

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: wd at denx.de
The evolution of the human race will not be accomplished in  the  ten
thousand  years  of  tame  animals,  but in the million years of wild
animals, because man is and will always be a wild animal.
                                              - Charles Galton Darwin


More information about the U-Boot mailing list