Hello Gedare,
I have tried to update code according to your review.
I have added even enabled/uncommented and added implementation
for some more test. Especially ECC integrated TCRAM single errors
correction and reporting test. I have commented why double errors
abort exception based test is commented out because it would require
significantly more work. Have left original commits unchanged for now
1a98178 arm/tms570: define base addresses of all TMS570LS3137 SPI interfaces.
9d06f3b arm/tms570: include hardware initialization and selftest based on Ti
HalCoGen generated files.
1ef345e arm/tms570: include TMS570_USE_HWINIT_STARTUP option to select bare
metal startup and selftest.
74b20a1 arm/tms570: document BSP setup with included hardware initialization.
b38ecb7 arm/tms570: update bootstrap generated preinstall.am
I think that even for you it is easier to read diffs against reviewed state for
now.
I plan to collapse/rearrange commits then. I can send updated series
to mainling list then if the V2 complete review is preferred.
The new updates
472a561 arm/tms570: include HalCoGen generated MPU initialization.
used to test MPU and SDRAM or real HW, this is not mainline material for
this round
b092a83 bsps/arm: Export bsp_start_hook_0_done symbol from ARM start.S.
b204502 arm/tms570: Enable more self-tests including integrated TCRAM and its
ECC functionality testing.
this is a little dancing between eggs, but works
aed1d28 arm/tms570: Add data barriers to external SDRAM setup code.
there are still some problems with first configuration/reset cycle after
power on reset
c4cbfe8 arm/tms570: Update comments, document functions and add references to
the corresponding HalCoGen functions.
45fda72 arm/tms570: readme and configure help extended a little.
9f83ada arm/tms570: change ESM parity selftests related functions/types
prefix from tms570_partest_ to tms570_selftest_par_ .
The commits, diffs can be read at GitHub
https://github.com/AoLaD/rtems/commits/tms570-bsp
Best wishes,
Pavel
_______________________________________________
devel mailing list
[email protected]
http://lists.rtems.org/mailman/listinfo/devel
