I will try to read your GitHub diffs sometime this week.
On Sat, Sep 17, 2016 at 6:34 PM, Pavel Pisa <ppisa4li...@pikron.com> wrote:
> 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
> 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
> 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
> Best wishes,
devel mailing list