<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Dec 8, 2015 at 9:09 AM, Sebastian Huber <span dir="ltr"><<a href="mailto:sebastian.huber@embedded-brains.de" target="_blank">sebastian.huber@embedded-brains.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><span class="">On 08/12/15 16:03, Joel Sherrill wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
What BSPs/architectures have you tested?<br>
</blockquote>
<br></span>
I temporarily moved the splinkersets01 test to the samples/ticker and tested that all BSPs build and link this test.<br>
<br>
I executed the splinkersets01 test on sis, psim and arm_realview_pbx_a9_qemu.<span class=""><br>
<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<br>
Is this something that breaks on a per BSP basis? or per architecture basis?<br>
I am assuming that since it is linker based, each BSP could have broken linkcmds.<br>
Is that right?<br>
</blockquote>
<br></span>
It breaks on a per linker command file basis. Since all the maintained BSPs use a linkercmds.base, which shouldn't be a big issue.<br>
<br></blockquote><div><br></div><div>That means a LOT of the BSPs are broken. You have defined maintained in your own way.</div><div>There are only a handful of architectures with linkcmds.base in them:</div><div><br></div><div><div>./or1k/shared/startup/linkcmds.base</div><div>./arm/shared/startup/linkcmds.base</div><div>./m68k/shared/startup/linkcmds.base</div><div>./powerpc/tqm8xx/startup/linkcmds.base</div><div>./powerpc/gen5200/startup/linkcmds.base</div><div>./powerpc/shared/startup/linkcmds.base</div><div>./sparc/shared/startup/linkcmds.base</div></div><div><br></div><div>I am not sure how many BSPs within arm, m68k, or powerpc actually use the linkcmds.base.</div><div><br></div><div>By my count, 13 of 94 BSP families have linkcmds.base. </div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
For requirements on the linker command file, see new chapter in user manual. However BSPs should not deal with this in copy and paste linker command files and instead use a linkercmds.base file.<div class=""><div class="h5"><br></div></div></blockquote><div><br></div><div>So 85% of the BSP families don't use linkcmds.base and by the above statement,</div><div>they must immediately be migrated to linkcmds.base. </div><div><br></div><div>Unless you have a plan to address this problem, I am on the side of rejecting the</div><div>part of this patch that changes the initialization.  And the issue must be addressed</div><div>before this can be merged.</div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class=""><div class="h5">
<br>
-- <br>
Sebastian Huber, embedded brains GmbH<br>
<br>
Address : Dornierstr. 4, D-82178 Puchheim, Germany<br>
Phone   : <a href="tel:%2B49%2089%20189%2047%2041-16" value="+4989189474116" target="_blank">+49 89 189 47 41-16</a><br>
Fax     : <a href="tel:%2B49%2089%20189%2047%2041-09" value="+4989189474109" target="_blank">+49 89 189 47 41-09</a><br>
E-Mail  : <a href="mailto:sebastian.huber@embedded-brains.de" target="_blank">sebastian.huber@embedded-brains.de</a><br>
PGP     : Public key available on request.<br>
<br>
Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG.<br>
<br>
</div></div></blockquote></div><br></div></div>