<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_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>
<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">><br>
> Also, I should mention that patching the appropriate bsp.h to use<br>
> CONFIGURE_INTERRUPT_ STACK_SIZE works too, but assumed that the BSP_<br>
> prefix was preferred in this context...</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>
<p dir="ltr">> T<br>
> _______________________________________________<br>
> users mailing list<br>
> <a href="mailto:users@rtems.org">users@rtems.org</a><br>
> <a href="http://lists.rtems.org/mailman/listinfo/users">http://lists.rtems.org/mailman/listinfo/users</a><br></p>