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 
> 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

Reply via email to