<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Oct 4, 2019 at 12:02 AM Sebastian Huber <<a href="mailto:sebastian.huber@embedded-brains.de">sebastian.huber@embedded-brains.de</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 04/10/2019 00:08, Joel Sherrill wrote:<br>
> This adds back the capability for the BSP to configure an<br>
> initial extension that is specific to itself. The parameter<br>
> BSP_INITIAL_EXTENSION was taken over by having a standard<br>
> fatal extension installed using the same name.<br>
> ---<br>
>   cpukit/include/rtems/confdefs.h | 3 +++<br>
>   1 file changed, 3 insertions(+)<br>
> <br>
> diff --git a/cpukit/include/rtems/confdefs.h b/cpukit/include/rtems/confdefs.h<br>
> index 5eb5425..e1a255a 100644<br>
> --- a/cpukit/include/rtems/confdefs.h<br>
> +++ b/cpukit/include/rtems/confdefs.h<br>
> @@ -2136,6 +2136,9 @@ extern rtems_initialization_tasks_table Initialization_tasks[];<br>
>       #if defined(CONFIGURE_INITIAL_EXTENSIONS)<br>
>         CONFIGURE_INITIAL_EXTENSIONS,<br>
>       #endif<br>
> +    #if defined(BSP_INITIAL2_EXTENSIONS)<br>
> +      BSP_INITIAL2_EXTENSIONS,<br>
> +    #endif<br>
>       #if defined(BSP_INITIAL_EXTENSION)<br>
>         BSP_INITIAL_EXTENSION<br>
>       #endif<br>
> <br>
<br>
I don't think this patch is necessary. A BSP is free to provide its own <br>
initial extension. Just don't add the<br>
<br>
#include <bsp/default-initial-extension.h><br>
<br>
to the bsp.h.<br></blockquote><div><br></div><div>And why would I want to lose the default fatal extension? The point of BSP_INITIAL_EXTENSION</div><div>was for a BSP to add it's own extensions. I like the default extensions but the BSP should still have a</div><div>hook. </div><div><br></div><div>I would prefer to rename this to BSP_FATAL_EXTENSION and leave the BSP_INITIAL_EXTENSION</div><div>available.</div><div><br></div><div>I think I could fix this by for this BSP by something like this:</div><div><br></div><div>#include <bsp/default-initial-extension.h><br></div><div><br></div><div>#undef BSP_INITIAL_EXTENSION</div><div>#define BSP_INITIAL_EXTENSION </div><div>   {</div><div>     ... switch extension ...,</div><div>    ... default fatal extension </div><div>  }</div><div><br></div><div>Is that what you are proposing a user should do if they want to another extension but still</div><div>use the default fatal extension?</div><div><br></div><div>FWIW other BSPs uses this default extension and just provide their own bsp_fatal_extension() </div><div>implementation. That seems OK and slightly different from my use case.</div><div><br></div><div>When BSP_INITIAL_EXTENSION was re-purposed for the default extension, it took it away as</div><div>an unused configuration point available to a BSP developer.</div><div><br></div><div>--joel</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
-- <br>
Sebastian Huber, embedded brains GmbH<br>
<br>
Address : Dornierstr. 4, D-82178 Puchheim, Germany<br>
Phone   : +49 89 189 47 41-16<br>
Fax     : +49 89 189 47 41-09<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>
</blockquote></div></div>