<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN" "http://www.w3.org/TR/REC-html40/loose.dtd">
<html lang="en" style="--code-editor-font: var(--default-mono-font, "GitLab Mono"), JetBrains Mono, Menlo, DejaVu Sans Mono, Liberation Mono, Consolas, Ubuntu Mono, Courier New, andale mono, lucida console, monospace;">
<head>
<meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
<title>
GitLab
</title>
<style data-premailer="ignore" type="text/css">
a { color: #1068bf; }
</style>
<style>img {
max-width: 100%; height: auto;
}
body {
font-size: .875rem;
}
body {
-webkit-text-shadow: rgba(255,255,255,.01) 0 0 1px;
}
body {
font-family: "GitLab Sans",-apple-system,BlinkMacSystemFont,"Segoe UI",Roboto,"Noto Sans",Ubuntu,Cantarell,"Helvetica Neue",sans-serif,"Apple Color Emoji","Segoe UI Emoji","Segoe UI Symbol","Noto Color Emoji"; font-size: inherit;
}
</style>
</head>
<body style="font-size: inherit; -webkit-text-shadow: rgba(255,255,255,.01) 0 0 1px; font-family: "GitLab Sans",-apple-system,BlinkMacSystemFont,"Segoe UI",Roboto,"Noto Sans",Ubuntu,Cantarell,"Helvetica Neue",sans-serif,"Apple Color Emoji","Segoe UI Emoji","Segoe UI Symbol","Noto Color Emoji";">
<div class="content">
<div style="color: #333333; border-bottom-width: 8px; border-bottom-color: #ededed; border-bottom-style: solid; font-weight: bold; line-height: 1.4; padding: 20px 0;">
Merge request <a href="https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1187">!1187</a> was reviewed by <a href="https://gitlab.rtems.org/ppisa">Pavel Pisa</a>
</div>
<p style="color: #777777;">
<a href="https://gitlab.rtems.org/ppisa">Pavel Pisa</a>
commented on a
discussion on <a href="https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1187#note_148169">bsps/arm/tms570/start/bspstarthooks-hwinit.c</a>:
</p>
<table class="code gl-mb-5" style="border-spacing: 0; margin-bottom: 1rem; border-collapse: collapse; width: auto; font-family: monospace; font-size: 90%;" bgcolor="#ffffff" width="100%" cellpadding="0" cellspacing="0">
<tr class="line_holder new" style="line-height: 1.6;">
<td class="old_line diff-line-num new" data-linenumber="204" style="width: 35px; color: rgba(5,5,6,.24); border-right-width: 1px; border-right-color: #c7f0d2; border-right-style: solid; padding: inherit;" align="right" bgcolor="#ddfbe6">
</td>
<td class="new_line diff-line-num new" data-linenumber="229" style="width: 35px; color: rgba(5,5,6,.24); border-right-width: 1px; border-right-color: #c7f0d2; border-right-style: solid; padding: inherit;" align="right" bgcolor="#ddfbe6">
229
</td>
<td class="line_content new" style="padding: inherit;" bgcolor="#ecfdf0"><pre style="display: block; font-size: 14px; color: #3a383f; position: relative; font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; word-break: break-all; word-wrap: break-word; background-color: inherit; border-radius: 2px; margin: 0; padding: 0; border: inherit solid #dcdcde;">+<span class="line" data-lang="c"><span class="cm" style="color: #999988; font-style: italic;"> */</span></span>
</pre></td>
</tr>
<tr class="line_holder new" style="line-height: 1.6;">
<td class="old_line diff-line-num new" data-linenumber="204" style="width: 35px; color: rgba(5,5,6,.24); border-right-width: 1px; border-right-color: #c7f0d2; border-right-style: solid; padding: inherit;" align="right" bgcolor="#ddfbe6">
</td>
<td class="new_line diff-line-num new" data-linenumber="230" style="width: 35px; color: rgba(5,5,6,.24); border-right-width: 1px; border-right-color: #c7f0d2; border-right-style: solid; padding: inherit;" align="right" bgcolor="#ddfbe6">
230
</td>
<td class="line_content new" style="padding: inherit;" bgcolor="#ecfdf0"><pre style="display: block; font-size: 14px; color: #3a383f; position: relative; font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; word-break: break-all; word-wrap: break-word; background-color: inherit; border-radius: 2px; margin: 0; padding: 0; border: inherit solid #dcdcde;">+<span class="line" data-lang="c"><span class="c" style="color: #999988; font-style: italic;">#if 0</span></span>
</pre></td>
</tr>
<tr class="line_holder new" style="line-height: 1.6;">
<td class="old_line diff-line-num new" data-linenumber="204" style="width: 35px; color: rgba(5,5,6,.24); border-right-width: 1px; border-right-color: #c7f0d2; border-right-style: solid; padding: inherit;" align="right" bgcolor="#ddfbe6">
</td>
<td class="new_line diff-line-num new" data-linenumber="231" style="width: 35px; color: rgba(5,5,6,.24); border-right-width: 1px; border-right-color: #c7f0d2; border-right-style: solid; padding: inherit;" align="right" bgcolor="#ddfbe6">
231
</td>
<td class="line_content new" style="padding: inherit;" bgcolor="#ecfdf0"><pre style="display: block; font-size: 14px; color: #3a383f; position: relative; font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; word-break: break-all; word-wrap: break-word; background-color: inherit; border-radius: 2px; margin: 0; padding: 0; border: inherit solid #dcdcde;">+<span class="line" data-lang="c"><span class="c" style="color: #999988; font-style: italic;"> /* Disabled for now, requires specific support in data abort handler */</span></span>
</pre></td>
</tr>
</table>
<div class="md" style="border-bottom-width: 4px; border-bottom-color: #ededed; border-bottom-style: solid; padding-bottom: 1em; position: relative; z-index: 1; color: #3a383f; word-wrap: break-word;">
<p dir="auto" style="color: #3a383f; margin: 0 0 1rem;" align="initial">I can add TODO. But it did not newer worked on RTEMS except for special scenario when some other bootloader on TMS570LS3137 has been used and the bootloader provided data-abort masking during test.</p>
<p dir="auto" style="color: #3a383f; margin: 0 0 1rem;" align="initial">There has been much more breakages in TMS570 BSP from the referenced commit introduced during TMS570LC4357 bringup because it flipped the logic by mistake and internal SRAM init and PBIST has never been run when run from Flash or SDRAM. And it has not been run for intram build (which would cause breakage anyway) and SDRAM because <code style="font-size: 90%; color: #18171d; word-wrap: break-word; background-color: #ececef; border-radius: .25rem; margin-top: 0; font-weight: inherit; overflow-wrap: break-word; white-space: break-spaces; font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; word-break: keep-all; padding: 0.125rem 0.25rem;">TMS570_USE_HWINIT_STARTUP</code> has been set only for Flash build.</p>
<p dir="auto" style="color: #3a383f; margin: 0 0 1rem;" align="initial">So if somebody used the BSP without some local modifications or introduction/update of safety startup from HalCoGen then system has had been run without ECC on SRAM and without guarantee that errors would be detected even on internal SRAM engine level because its PBIST testing internal logic and functionality of all SRAM cells has been skipped.</p>
<p dir="auto" style="color: #3a383f; margin: 0 0 1rem;" align="initial">In the fact, I am not sure if <code style="font-size: 90%; color: #18171d; word-wrap: break-word; background-color: #ececef; border-radius: .25rem; margin-top: 0; font-weight: inherit; overflow-wrap: break-word; white-space: break-spaces; font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; word-break: keep-all; padding: 0.125rem 0.25rem;">tms570_pbist_run_and_check</code> has been good idea. As I have described, it cannot be used for SRAM, because it has to be reinitialized after it and two levels of the functions nesting required to use SRAM for saved registers (i.e. <code style="font-size: 90%; color: #18171d; word-wrap: break-word; background-color: #ececef; border-radius: .25rem; font-weight: inherit; overflow-wrap: break-word; white-space: break-spaces; font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; word-break: keep-all; padding: 0.125rem 0.25rem;">lr</code>) when SRAM is not available.</p>
<p dir="auto" style="color: #3a383f; margin: 0 0 1rem;" align="initial">The preceding of <code style="font-size: 90%; color: #18171d; word-wrap: break-word; background-color: #ececef; border-radius: .25rem; margin-top: 0; font-weight: inherit; overflow-wrap: break-word; white-space: break-spaces; font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; word-break: keep-all; padding: 0.125rem 0.25rem;">tms570_pbist_run</code>, <code style="font-size: 90%; color: #18171d; word-wrap: break-word; background-color: #ececef; border-radius: .25rem; font-weight: inherit; overflow-wrap: break-word; white-space: break-spaces; font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; word-break: keep-all; padding: 0.125rem 0.25rem;">tms570_pbist_is_test_passed</code> and <code style="font-size: 90%; color: #18171d; word-wrap: break-word; background-color: #ececef; border-radius: .25rem; font-weight: inherit; overflow-wrap: break-word; white-space: break-spaces; font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; word-break: keep-all; padding: 0.125rem 0.25rem;">tms570_pbist_sto</code> sequences replaced by <code style="font-size: 90%; color: #18171d; word-wrap: break-word; background-color: #ececef; border-radius: .25rem; font-weight: inherit; overflow-wrap: break-word; white-space: break-spaces; font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; word-break: keep-all; padding: 0.125rem 0.25rem;">tms570_pbist_run_and_check</code> means that testing is defendant on internal SRAM which is not know to be usable/without errors and damage yet.</p>
<p dir="auto" style="color: #3a383f; margin: 0 0 1rem;" align="initial">Option is to move internal SRAM test to assembly part in <code style="font-size: 90%; color: #18171d; word-wrap: break-word; background-color: #ececef; border-radius: .25rem; margin-top: 0; font-weight: inherit; overflow-wrap: break-word; white-space: break-spaces; font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; word-break: keep-all; padding: 0.125rem 0.25rem;">bsp_start_hook_0</code> before all other code. But it has disadvantage, that SRAM would be tested before clock setup so low frequency can mask some problem and the testing could be much slower. Move of all calls of the functions till the comment</p>
<div class="gl-relative markdown-code-block js-markdown-code">
<pre class="code highlight js-syntax-highlight language-plaintext" v-pre="true" style="display: block; font-size: 90%; color: #3a383f; line-height: 1.6em; overflow-x: auto; border-radius: .25rem; position: relative; font-family: monospace; font-variant-ligatures: none; word-break: break-all; word-wrap: break-word; background-color: #ffffff; -premailer-cellpadding: 0; -premailer-cellspacing: 0; -premailer-width: 100%; margin: 0 0 1rem; padding: 12px; border: 1px solid #dcdcde;"><code style="font-size: inherit; color: inherit; word-wrap: normal; word-break: keep-all; background-color: inherit; border-radius: .25rem; white-space: pre; margin-top: 0; font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; overflow-wrap: normal; padding: unset;"><span id="LC1" class="line" lang="plaintext" style="margin-top: 0;">end of the code skipped for tms570_running_from_tcram()</span></code></pre>
<copy-code></copy-code><insert-code-snippet></insert-code-snippet>
</div>
<p dir="auto" style="color: #3a383f; margin: 0 0 1rem;" align="initial">to the assembly part of the hook could help but again there is probably (most sure) dependency on SRAM in <code style="font-size: 90%; color: #18171d; word-wrap: break-word; background-color: #ececef; border-radius: .25rem; margin-top: 0; font-weight: inherit; overflow-wrap: break-word; white-space: break-spaces; font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; word-break: keep-all; padding: 0.125rem 0.25rem;">tms570_system_hw_init</code> and code is quite complex there that its move to some assembly sequences interpreting some list of peripherals registers manipulation without need of SRAM would be really complex. On the other hand HaCoGen code uses similar sequence with SRAM use before full SRAM testing</p>
<p dir="auto" style="color: #3a383f; margin: 0 0 1rem;" align="initial"><code style="font-size: 90%; color: #18171d; word-wrap: break-word; background-color: #ececef; border-radius: .25rem; margin-top: 0; font-weight: inherit; overflow-wrap: break-word; white-space: break-spaces; font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; word-break: keep-all; padding: 0.125rem 0.25rem;">_c_int00</code> -> <code style="font-size: 90%; color: #18171d; word-wrap: break-word; background-color: #ececef; border-radius: .25rem; font-weight: inherit; overflow-wrap: break-word; white-space: break-spaces; font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; word-break: keep-all; padding: 0.125rem 0.25rem;">systemInit()</code> -> <code style="font-size: 90%; color: #18171d; word-wrap: break-word; background-color: #ececef; border-radius: .25rem; font-weight: inherit; overflow-wrap: break-word; white-space: break-spaces; font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; word-break: keep-all; padding: 0.125rem 0.25rem;">setupPLL()</code></p>
<p dir="auto" style="color: #3a383f; margin: 0 0 1rem;" align="initial">The SRAM critical</p>
<div class="gl-relative markdown-code-block js-markdown-code">
<pre class="code highlight js-syntax-highlight language-plaintext" v-pre="true" style="display: block; font-size: 90%; color: #3a383f; line-height: 1.6em; overflow-x: auto; border-radius: .25rem; position: relative; font-family: monospace; font-variant-ligatures: none; word-break: break-all; word-wrap: break-word; background-color: #ffffff; -premailer-cellpadding: 0; -premailer-cellspacing: 0; -premailer-width: 100%; margin: 0 0 1rem; padding: 12px; border: 1px solid #dcdcde;"><code style="font-size: inherit; color: inherit; word-wrap: normal; word-break: keep-all; background-color: inherit; border-radius: .25rem; white-space: pre; margin-top: 0; font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; overflow-wrap: normal; padding: unset;"><span id="LC1" class="line" lang="plaintext" style="margin-top: 0;">pbistRun(0x08300020U, /* ESRAM Single Port PBIST */</span>
<span id="LC2" class="line" lang="plaintext"> (uint32)PBIST_March13N_SP);`</span></code></pre>
<copy-code></copy-code><insert-code-snippet></insert-code-snippet>
</div>
<p dir="auto" style="color: #3a383f; margin: 0 0 1rem;" align="initial">is after this call chain in <code style="font-size: 90%; color: #18171d; word-wrap: break-word; background-color: #ececef; border-radius: .25rem; margin-top: 0; font-weight: inherit; overflow-wrap: break-word; white-space: break-spaces; font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; word-break: keep-all; padding: 0.125rem 0.25rem;">_c_int00</code> so even there it seem that risk of running setup before SRAM is fully tested latter is considered acceptable. If there is some real interest and ideally even funding or some other kind of cooperation from these, who want to build expensive (may it be multi-millions dollars one) missions then there is chance to do deeper analyze for RTEMS. I have colleagues who use Hercules based systems in certified products with certified tools and complete setups, but their code is systemless loop even without interrupts to ensure highest SIL levels. But such analysis is really time consumpting and I am already quite exhausted by my attempt to clean breakages there and support TMS570 and my attempts on OpenOCD side still fails on TMS570LC4357 and on TMS570LS3137 I have working only older version. But I have found more steps forward in newer one. But still not enough. It is frustrating that Ti has library sources and documentation so it would be so easy if there is interest to cooperate instead of hiding, rule and extract maximum short-term profit. I have sent some plea for help after years again so we will see.</p>
<p dir="auto" style="color: #3a383f; margin: 0;" align="initial">I have some followup questions for WAF, but I open new thread under this MR.</p>
</div>
<p style="color: #777777;">
<a href="https://gitlab.rtems.org/ppisa">Pavel Pisa</a>
started a new
<a href="https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1187#note_148170">discussion</a>:
</p>
<div class="md" style="border-bottom-width: 4px; border-bottom-color: #ededed; border-bottom-style: solid; padding-bottom: 1em; position: relative; z-index: 1; color: #3a383f; word-wrap: break-word;">
<p dir="auto" style="color: #3a383f; margin: 0 0 1rem;" align="initial">I have plea for help with WAF. For testing, it would be much better if the choice of inclusion or skipping full TMS570 system initialization and in depth self-testing (enabled by <code style="font-size: 90%; color: #18171d; word-wrap: break-word; background-color: #ececef; border-radius: .25rem; margin-top: 0; font-weight: inherit; overflow-wrap: break-word; white-space: break-spaces; font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; word-break: keep-all; padding: 0.125rem 0.25rem;">TMS570_USE_HWINIT_STARTUP</code>) can be chosen independently on BSP selection.</p>
<p dir="auto" style="color: #3a383f; margin: 0 0 1rem;" align="initial">There is the current list of BSPs and default behavior</p>
<ul dir="auto" style="text-align: initial; list-style-type: disc; margin: 0 0 1rem; padding: 0;">
<li style="margin-top: 0; line-height: 1.6em; margin-left: 25px; padding-left: 3px;">
<code style="font-size: 90%; color: #18171d; word-wrap: break-word; background-color: #ececef; border-radius: .25rem; margin-top: 0; font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; white-space: pre-wrap; overflow-wrap: break-word; word-break: keep-all; padding: 0.125rem 0.25rem;">tms570lc4357_hdk</code> - Flash application full HWINIT including SRAM</li>
<li style="line-height: 1.6em; margin-left: 25px; padding-left: 3px;"><code style="font-size: 90%; color: #18171d; word-wrap: break-word; background-color: #ececef; border-radius: .25rem; margin-top: 0; font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; white-space: pre-wrap; overflow-wrap: break-word; word-break: keep-all; padding: 0.125rem 0.25rem;">tms570lc4357_hdk_intram</code></li>
<li style="line-height: 1.6em; margin-left: 25px; padding-left: 3px;"><code style="font-size: 90%; color: #18171d; word-wrap: break-word; background-color: #ececef; border-radius: .25rem; margin-top: 0; font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; white-space: pre-wrap; overflow-wrap: break-word; word-break: keep-all; padding: 0.125rem 0.25rem;">tms570lc4357_hdk_sdram</code></li>
<li style="line-height: 1.6em; margin-left: 25px; padding-left: 3px;">
<code style="font-size: 90%; color: #18171d; word-wrap: break-word; background-color: #ececef; border-radius: .25rem; margin-top: 0; font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; white-space: pre-wrap; overflow-wrap: break-word; word-break: keep-all; padding: 0.125rem 0.25rem;">tms570ls3137_hdk</code> - Flash application full HWINIT including SRAM</li>
<li style="line-height: 1.6em; margin-left: 25px; padding-left: 3px;"><code style="font-size: 90%; color: #18171d; word-wrap: break-word; background-color: #ececef; border-radius: .25rem; margin-top: 0; font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; white-space: pre-wrap; overflow-wrap: break-word; word-break: keep-all; padding: 0.125rem 0.25rem;">tms570ls3137_hdk_intram</code></li>
<li style="line-height: 1.6em; margin-left: 25px; padding-left: 3px;"><code style="font-size: 90%; color: #18171d; word-wrap: break-word; background-color: #ececef; border-radius: .25rem; margin-top: 0; font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; white-space: pre-wrap; overflow-wrap: break-word; word-break: keep-all; padding: 0.125rem 0.25rem;">tms570ls3137_hdk_sdram</code></li>
</ul>
<p dir="auto" style="color: #3a383f; margin: 0 0 1rem;" align="initial"><code style="font-size: 90%; color: #18171d; word-wrap: break-word; background-color: #ececef; border-radius: .25rem; margin-top: 0; font-weight: inherit; overflow-wrap: break-word; white-space: break-spaces; font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; word-break: keep-all; padding: 0.125rem 0.25rem;">intram</code> variant can be run with HWINIT when SRAM init and full test are skipped. That is very useful variant for testing. But I do not want to introduce more BSP variants. I would prefer to use modification of the <code style="font-size: 90%; color: #18171d; word-wrap: break-word; background-color: #ececef; border-radius: .25rem; font-weight: inherit; overflow-wrap: break-word; white-space: break-spaces; font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; word-break: keep-all; padding: 0.125rem 0.25rem;">config.ini</code> generated by</p>
<div class="gl-relative markdown-code-block js-markdown-code">
<pre class="code highlight js-syntax-highlight language-plaintext" v-pre="true" style="display: block; font-size: 90%; color: #3a383f; line-height: 1.6em; overflow-x: auto; border-radius: .25rem; position: relative; font-family: monospace; font-variant-ligatures: none; word-break: break-all; word-wrap: break-word; background-color: #ffffff; -premailer-cellpadding: 0; -premailer-cellspacing: 0; -premailer-width: 100%; margin: 0 0 1rem; padding: 12px; border: 1px solid #dcdcde;"><code style="font-size: inherit; color: inherit; word-wrap: normal; word-break: keep-all; background-color: inherit; border-radius: .25rem; white-space: pre; margin-top: 0; font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; overflow-wrap: normal; padding: unset;"><span id="LC1" class="line" lang="plaintext" style="margin-top: 0;">$RTEMS_DIR/waf bspdefaults --rtems-bsps=arm/tms570ls3137_hdk_intram -t "$RTEMS_DIR" -o "$MY_DIR" --prefix "/opt/rtems/7" >config.ini</span></code></pre>
<copy-code></copy-code><insert-code-snippet></insert-code-snippet>
</div>
<p dir="auto" style="color: #3a383f; margin: 0 0 1rem;" align="initial">I have solved HWINIT enable for SRAM build by YAM files modification for now</p>
<div class="gl-relative markdown-code-block js-markdown-code">
<pre data-canonical-lang="diff" data-lang-params="--git a/spec/build/bsps/arm/tms570/objhwinitlc4357hdk.yml b/spec/build/bsps/arm/tms570/objhwinitlc4357hdk.yml" class="code highlight js-syntax-highlight language-diff" v-pre="true" style="display: block; font-size: 90%; color: #3a383f; line-height: 1.6em; overflow-x: auto; border-radius: .25rem; position: relative; font-family: monospace; font-variant-ligatures: none; word-break: break-all; word-wrap: break-word; background-color: #ffffff; -premailer-cellpadding: 0; -premailer-cellspacing: 0; -premailer-width: 100%; margin: 0 0 1rem; padding: 12px; border: 1px solid #dcdcde;"><code style="font-size: inherit; color: inherit; word-wrap: normal; word-break: keep-all; background-color: inherit; border-radius: .25rem; white-space: pre; margin-top: 0; font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; overflow-wrap: normal; padding: unset;"><span id="LC1" class="line" lang="diff" style="margin-top: 0;"><span class="gh" style="margin-top: 0; color: #800080; font-weight: 600;">index 1b91475804..71ca6b8b47 100644</span></span>
<span id="LC2" class="line" lang="diff"><span class="gd" style="margin-top: 0; color: #000000; background-color: #ffdddd;">--- a/spec/build/bsps/arm/tms570/objhwinitlc4357hdk.yml</span></span>
<span id="LC3" class="line" lang="diff"><span class="gi" style="margin-top: 0; color: #000000; background-color: #ddffdd;">+++ b/spec/build/bsps/arm/tms570/objhwinitlc4357hdk.yml</span></span>
<span id="LC4" class="line" lang="diff"><span class="p" style="margin-top: 0;">@@ -5,7 +5,9 @@</span> copyrights:</span>
<span id="LC5" class="line" lang="diff"> - Copyright (C) 2023 embedded brains GmbH & Co. KG</span>
<span id="LC6" class="line" lang="diff"> cppflags: []</span>
<span id="LC7" class="line" lang="diff"> cxxflags: []</span>
<span id="LC8" class="line" lang="diff"><span class="gd" style="margin-top: 0; color: #000000; background-color: #ffdddd;">-enabled-by: arm/tms570lc4357_hdk</span></span>
<span id="LC9" class="line" lang="diff"><span class="gi" style="margin-top: 0; color: #000000; background-color: #ddffdd;">+enabled-by:</span></span>
<span id="LC10" class="line" lang="diff"><span class="gi" style="margin-top: 0; color: #000000; background-color: #ddffdd;">+ - arm/tms570lc4357_hdk</span></span>
<span id="LC11" class="line" lang="diff"><span class="gi" style="margin-top: 0; color: #000000; background-color: #ddffdd;">+ - arm/tms570lc4357_hdk_intram</span></span>
<span id="LC12" class="line" lang="diff"> includes: []</span>
<span id="LC13" class="line" lang="diff"> install: []</span>
<span id="LC14" class="line" lang="diff"> links: []</span>
<span id="LC15" class="line" lang="diff"><span class="gh" style="margin-top: 0; color: #800080; font-weight: 600;">diff --git a/spec/build/bsps/arm/tms570/objhwinitls3137hdk.yml b/spec/build/bsps/arm/tms570/objhwinitls3137hdk.yml</span></span>
<span id="LC16" class="line" lang="diff"><span class="gh" style="margin-top: 0; color: #800080; font-weight: 600;">index 9971a0a86f..a7ebcc25f3 100644</span></span>
<span id="LC17" class="line" lang="diff"><span class="gd" style="margin-top: 0; color: #000000; background-color: #ffdddd;">--- a/spec/build/bsps/arm/tms570/objhwinitls3137hdk.yml</span></span>
<span id="LC18" class="line" lang="diff"><span class="gi" style="margin-top: 0; color: #000000; background-color: #ddffdd;">+++ b/spec/build/bsps/arm/tms570/objhwinitls3137hdk.yml</span></span>
<span id="LC19" class="line" lang="diff"><span class="p" style="margin-top: 0;">@@ -5,7 +5,9 @@</span> copyrights:</span>
<span id="LC20" class="line" lang="diff"> - Copyright (C) 2023 embedded brains GmbH & Co. KG</span>
<span id="LC21" class="line" lang="diff"> cppflags: []</span>
<span id="LC22" class="line" lang="diff"> cxxflags: []</span>
<span id="LC23" class="line" lang="diff"><span class="gd" style="margin-top: 0; color: #000000; background-color: #ffdddd;">-enabled-by: arm/tms570ls3137_hdk</span></span>
<span id="LC24" class="line" lang="diff"><span class="gi" style="margin-top: 0; color: #000000; background-color: #ddffdd;">+enabled-by:</span></span>
<span id="LC25" class="line" lang="diff"><span class="gi" style="margin-top: 0; color: #000000; background-color: #ddffdd;">+ - arm/tms570ls3137_hdk</span></span>
<span id="LC26" class="line" lang="diff"><span class="gi" style="margin-top: 0; color: #000000; background-color: #ddffdd;">+ - arm/tms570ls3137_hdk_intram</span></span>
<span id="LC27" class="line" lang="diff"> includes: []</span>
<span id="LC28" class="line" lang="diff"> install: []</span>
<span id="LC29" class="line" lang="diff"> links: []</span>
<span id="LC30" class="line" lang="diff"><span class="gh" style="margin-top: 0; color: #800080; font-weight: 600;">diff --git a/spec/build/bsps/arm/tms570/optlowinit.yml b/spec/build/bsps/arm/tms570/optlowinit.yml</span></span>
<span id="LC31" class="line" lang="diff"><span class="gh" style="margin-top: 0; color: #800080; font-weight: 600;">index 32b84aa08c..2f2cb3a99e 100644</span></span>
<span id="LC32" class="line" lang="diff"><span class="gd" style="margin-top: 0; color: #000000; background-color: #ffdddd;">--- a/spec/build/bsps/arm/tms570/optlowinit.yml</span></span>
<span id="LC33" class="line" lang="diff"><span class="gi" style="margin-top: 0; color: #000000; background-color: #ddffdd;">+++ b/spec/build/bsps/arm/tms570/optlowinit.yml</span></span>
<span id="LC34" class="line" lang="diff"><span class="p" style="margin-top: 0;">@@ -9,7 +9,9 @@</span> copyrights:</span>
<span id="LC35" class="line" lang="diff"> default:</span>
<span id="LC36" class="line" lang="diff"> - enabled-by:</span>
<span id="LC37" class="line" lang="diff"> - arm/tms570ls3137_hdk</span>
<span id="LC38" class="line" lang="diff"><span class="gi" style="margin-top: 0; color: #000000; background-color: #ddffdd;">+ - arm/tms570ls3137_hdk_intram</span></span>
<span id="LC39" class="line" lang="diff"> - arm/tms570lc4357_hdk</span>
<span id="LC40" class="line" lang="diff"><span class="gi" style="margin-top: 0; color: #000000; background-color: #ddffdd;">+ - arm/tms570lc4357_hdk_intram</span></span>
<span id="LC41" class="line" lang="diff"> value: true</span>
<span id="LC42" class="line" lang="diff"> - enabled-by: true</span>
<span id="LC43" class="line" lang="diff"> value: false</span></code></pre>
<copy-code></copy-code><insert-code-snippet></insert-code-snippet>
</div>
<p dir="auto" style="color: #3a383f; margin: 0 0 1rem;" align="initial">I have tried to find how to conventionalize it from <code style="font-size: 90%; color: #18171d; word-wrap: break-word; background-color: #ececef; border-radius: .25rem; margin-top: 0; font-weight: inherit; overflow-wrap: break-word; white-space: break-spaces; font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; word-break: keep-all; padding: 0.125rem 0.25rem;">TMS570_USE_HWINIT_STARTUP</code> in <code style="font-size: 90%; color: #18171d; word-wrap: break-word; background-color: #ececef; border-radius: .25rem; font-weight: inherit; overflow-wrap: break-word; white-space: break-spaces; font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; word-break: keep-all; padding: 0.125rem 0.25rem;">config.ini</code> but it did not work for me.</p>
<p dir="auto" style="color: #3a383f; margin: 0 0 1rem;" align="initial">I would like to use conditions even for</p>
<ul dir="auto" style="text-align: initial; list-style-type: disc; margin: 0 0 1rem; padding: 0;">
<li style="margin-top: 0; line-height: 1.6em; margin-left: 25px; padding-left: 3px;">bsps/arm/tms570/start/hwinit-lc4357-hdk.c</li>
<li style="line-height: 1.6em; margin-left: 25px; padding-left: 3px;">bsps/arm/tms570/start/hwinit-ls3137-hdk.c</li>
</ul>
<p dir="auto" style="color: #3a383f; margin: 0 0 1rem;" align="initial">and more other locations to be based on <code style="font-size: 90%; color: #18171d; word-wrap: break-word; background-color: #ececef; border-radius: .25rem; margin-top: 0; font-weight: inherit; overflow-wrap: break-word; white-space: break-spaces; font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; word-break: keep-all; padding: 0.125rem 0.25rem;">TMS570_VARIANT</code> and or <code style="font-size: 90%; color: #18171d; word-wrap: break-word; background-color: #ececef; border-radius: .25rem; font-weight: inherit; overflow-wrap: break-word; white-space: break-spaces; font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; word-break: keep-all; padding: 0.125rem 0.25rem;">TMS570_VARIANT_${TMS570_VARIANT}</code>. But again, I have not found how to use it properly.</p>
<p dir="auto" style="color: #3a383f; margin: 0 0 1rem;" align="initial">In the fact, I have removed one such attempt in previous YML set</p>
<div class="gl-relative markdown-code-block js-markdown-code">
<pre class="code highlight js-syntax-highlight language-plaintext" v-pre="true" style="display: block; font-size: 90%; color: #3a383f; line-height: 1.6em; overflow-x: auto; border-radius: .25rem; position: relative; font-family: monospace; font-variant-ligatures: none; word-break: break-all; word-wrap: break-word; background-color: #ffffff; -premailer-cellpadding: 0; -premailer-cellspacing: 0; -premailer-width: 100%; margin: 0 0 1rem; padding: 12px; border: 1px solid #dcdcde;"><code style="font-size: inherit; color: inherit; word-wrap: normal; word-break: keep-all; background-color: inherit; border-radius: .25rem; white-space: pre; margin-top: 0; font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; overflow-wrap: normal; padding: unset;"><span id="LC1" class="line" lang="plaintext" style="margin-top: 0;">diff --git a/spec/build/bsps/arm/tms570/optmemsramsize.yml b/spec/build/bsps/arm/tms570/optmemsramsize.yml</span>
<span id="LC2" class="line" lang="plaintext">index 515819daad..87536c9364 100644</span>
<span id="LC3" class="line" lang="plaintext">--- a/spec/build/bsps/arm/tms570/optmemsramsize.yml</span>
<span id="LC4" class="line" lang="plaintext">+++ b/spec/build/bsps/arm/tms570/optmemsramsize.yml</span>
<span id="LC5" class="line" lang="plaintext">@@ -7,7 +7,10 @@ build-type: option</span>
<span id="LC6" class="line" lang="plaintext"> copyrights:</span>
<span id="LC7" class="line" lang="plaintext"> - Copyright (C) 2023 embedded brains GmbH & Co. KG</span>
<span id="LC8" class="line" lang="plaintext"> default:</span>
<span id="LC9" class="line" lang="plaintext">-- enabled-by: TMS570_VARIANT_4357</span>
<span id="LC10" class="line" lang="plaintext">+- enabled-by:</span>
<span id="LC11" class="line" lang="plaintext">+ - arm/tms570lc4357_hdk</span>
<span id="LC12" class="line" lang="plaintext">+ - arm/tms570lc4357_hdk_intram</span>
<span id="LC13" class="line" lang="plaintext">+ - arm/tms570lc4357_hdk_sdram</span>
<span id="LC14" class="line" lang="plaintext"> value: 0x00080000</span>
<span id="LC15" class="line" lang="plaintext"> - enabled-by: true</span>
<span id="LC16" class="line" lang="plaintext"> value: 0x00040000</span></code></pre>
<copy-code></copy-code><insert-code-snippet></insert-code-snippet>
</div>
<p dir="auto" style="color: #3a383f; margin: 0;" align="initial">because it seems that use of define setup by <code style="font-size: 90%; color: #18171d; word-wrap: break-word; background-color: #ececef; border-radius: .25rem; margin-top: 0; font-weight: inherit; overflow-wrap: break-word; white-space: break-spaces; font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; word-break: keep-all; padding: 0.125rem 0.25rem;">TMS570_VARIANT</code> does not wok in build scripts and incorrect values has been chosen. There should/could be/work test for for <code style="font-size: 90%; color: #18171d; word-wrap: break-word; background-color: #ececef; border-radius: .25rem; font-weight: inherit; overflow-wrap: break-word; white-space: break-spaces; font-family: "GitLab Mono", "JetBrains Mono", "Menlo", "DejaVu Sans Mono", "Liberation Mono", "Consolas", "Ubuntu Mono", "Courier New", "andale mono", "lucida console", monospace; font-variant-ligatures: none; word-break: keep-all; padding: 0.125rem 0.25rem;">TMS570_VARIANT</code> equal to 4357, but I am not sure how to setup it in WAF.</p>
</div>
</div>
<div class="footer" style="margin-top: 10px;">
<p style="font-size: small; color: #626168;">
—
<br>
<a href="https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1187">View it on GitLab</a>.
<br>
You're receiving this email because of your account on <a target="_blank" rel="noopener noreferrer" href="https://gitlab.rtems.org">gitlab.rtems.org</a>. <a href="https://gitlab.rtems.org/-/sent_notifications/3-ewilgbu75n7b5kvoo0jplxlmc/unsubscribe" target="_blank" rel="noopener noreferrer">Unsubscribe</a> from this thread · <a href="https://gitlab.rtems.org/-/profile/notifications" target="_blank" rel="noopener noreferrer" class="mng-notif-link">Manage all notifications</a> · <a href="https://gitlab.rtems.org/help" target="_blank" rel="noopener noreferrer" class="help-link">Help</a>
<span style="color: transparent; font-size: 0; display: none; overflow: hidden; opacity: 0; width: 0; height: 0; max-width: 0; max-height: 0;">
Notification message regarding https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1187 at 1775550927
</span>
<script type="application/ld+json">{"@context":"http://schema.org","@type":"EmailMessage","action":{"@type":"ViewAction","name":"View Merge request","url":"https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1187"}}</script>
</p>
</div>
</body>
</html>