<div dir="ltr">Thanks. The ticket should now be closed with the patch applied<div>to master and the 4.11 branch.¬†</div><div><br></div><div>Since it was a plain diff, I had to turn it into a git patch. Check</div><div>out the link from the ticket. The git commit message also¬†</div><div>contained the annotation to close the ticket.</div><div><br></div><div>Don't worry. With git, we are all still learning. :)</div><div><br></div><div>Thanks again.</div><div><br></div><div><br></div><div><br></div><div>--joel</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Nov 3, 2016 at 7:52 AM, Joel Sherrill <span dir="ltr"><<a href="mailto:joel@rtems.org" target="_blank">joel@rtems.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><p dir="ltr"></p>
<p dir="ltr">On Nov 3, 2016 5:49 AM, "Tim Cussins" <timcussins@eml.cc> wrote:<br>
><br>
> On Thu, Nov 3, 2016, at 10:22 AM, Tim Cussins wrote:<br>
> > Currently 2 headers,<br>
> ><br>
> > virtex4/include/bsp.h<br>
> > virtex5/include/bsp.h<br>
> ><br>
> > refer to CONFIGURE_INTERRUPT_STACK_<wbr>MEMORY, which confdefs.h isn't too<br>
> > happy about.<br>
> ><br>
> > Can I assume those headers should be setting BSP_INTERRUPT_STACK_SIZE<br>
> > instead? FWIW it resolves the issue :)</p>
</span><p dir="ltr">That's the intention of the macros starting with BSP_. They are values where a BSP may desire to override the default. </p>
<p dir="ltr"><span class="">><br>
> Also, I should mention that patching the appropriate bsp.h to use<br></span>
> CONFIGURE_INTERRUPT_ STACK_SIZE works too, but assumed that the BSP_<span class=""><br>
> prefix was preferred in this context...</span></p>
<p dir="ltr">Since this is a release branch, can you file a ticket and provide a patch? Preferably to fix both. I am happy to apply it to the branch and master, if applicable.</p>
<p dir="ltr">I probably should so grep for CONFIGURE_ in all the BSPs. </p>
<p dir="ltr">Thanks.</p><div class="HOEnZb"><div class="h5">
<p dir="ltr">> T<br>
> ______________________________<wbr>_________________<br>
> users mailing list<br>
> <a href="mailto:users@rtems.org" target="_blank">users@rtems.org</a><br>
> <a href="http://lists.rtems.org/mailman/listinfo/users" target="_blank">http://lists.rtems.org/<wbr>mailman/listinfo/users</a><br></p>
</div></div></blockquote></div><br></div>