Merge request https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1187 
was reviewed by Pavel Pisa

--
  
Pavel Pisa commented on a discussion: 
https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1187#note_148669


Thanks for guidance.

I would recheck that, because I have got incorrect results with the original 
setup which is in the fact what you propose. So I have switched to conservative 
`enabled-by` individual BSPs and it worked for me.

For clarification for the other case. If I configure with
```
--rtems-bsps=arm/tms570lc4357_hdk_sdram
```
Which results in
```
TMS570_USE_HWINIT_STARTUP = False
```
and I change the value in `config.ini` to
```
TMS570_USE_HWINIT_STARTUP = True
```
then you are convinced that
```
-enabled-by: TMS570_USE_HWINIT_STARTUP
```
should work, is that right?

I have tried such things and got incorrect results.

Anyway, the TMS570 BSP needs some care before 7.0 release. Because changes by 
Airbus, you and others introduced 4357 but the options has been flipped which 
resulted in the Flash builds (probably these whee safety matters) to disable 
SRAM check and even ECC on it and parity check on peripherals. On the other 
hand in SRAM build the test code wiped out out itself by destructive internal 
SRAM tests.

I have done some steps to make it work again but it needs this and probably 
some more changes. Problem is, that I have my older OpenOCD branch which 
flashes 3137, but my attempts to use my knowledge to make current OpenOCD 
attempts (based on other's code my and corrections) to wok with 4357 are not 
successful yet. But on the other hand I have colleagues with Ti tools on the 
computer at company where I provide consultations so I have some possibility to 
flash 4357 as well.

I have spoken with Ti representatives at Embedded World this year and they have 
informed me that they have their public OpenOCD for which works with their 
newer chips. But they did not respond to my followup e/mail to provide 
documentation for TMS570 Flash and possibly source library blobs code... But I 
try again when I have some time.


-- 
View it on GitLab: 
https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1187
You're receiving this email because of your account on gitlab.rtems.org.


_______________________________________________
bugs mailing list
[email protected]
http://lists.rtems.org/mailman/listinfo/bugs

Reply via email to