<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Mar 13, 2019 at 2:38 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">What about this patch? The placement of rtems_board_support_package() <br>
into <rtems/version.h> is a bit odd, but I think it fits into the scope <br>
of these functions. You normally include <rtems/version.h> if you want <br>
to print some system information. Another place would be <rtems.h> right <br>
after rtems_get_version_string().<br></blockquote><div><br></div><div>I think it is OK although I agree with you it is odd.</div><div><br></div><div>If these are intended to be public APIs, I do not see any documentation for</div><div>them in rtems-docs.   </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
On 01/02/2019 08:05, Sebastian Huber wrote:<br>
> ---<br>
>   bsps/shared/rtems-version.c           | 6 ++++++<br>
>   cpukit/include/rtems/version.h        | 7 +++++++<br>
>   testsuites/sptests/spversion01/init.c | 1 +<br>
>   3 files changed, 14 insertions(+)<br>
><br>
> diff --git a/bsps/shared/rtems-version.c b/bsps/shared/rtems-version.c<br>
> index 16f74070bc..99b98966e6 100644<br>
> --- a/bsps/shared/rtems-version.c<br>
> +++ b/bsps/shared/rtems-version.c<br>
> @@ -7,6 +7,7 @@<br>
>    *  <a href="http://www.rtems.org/license/LICENSE" rel="noreferrer" target="_blank">http://www.rtems.org/license/LICENSE</a>.<br>
>    */<br>
>   <br>
> +#include <rtems/version.h><br>
>   #include <rtems/system.h><br>
>   <br>
>   #include <bspopts.h><br>
> @@ -27,3 +28,8 @@<br>
>   const char _RTEMS_version[] =<br>
>     "rtems-" RTEMS_VERSION " (" CPU_NAME "/" CPU_MODEL_NAME "/"<br>
>     RTEMS_XSTRING( RTEMS_BSP ) ")";<br>
> +<br>
> +const char *rtems_board_support_package( void )<br>
> +{<br>
> +  return RTEMS_XSTRING( RTEMS_BSP );<br>
> +}<br>
> diff --git a/cpukit/include/rtems/version.h b/cpukit/include/rtems/version.h<br>
> index b806cb8c2f..9bee2e8d10 100644<br>
> --- a/cpukit/include/rtems/version.h<br>
> +++ b/cpukit/include/rtems/version.h<br>
> @@ -67,6 +67,13 @@ int rtems_version_revision( void );<br>
>    */<br>
>   const char *rtems_version_control_key( void );<br>
>   <br>
> +/**<br>
> + * @brief Returns the board support package name.<br>
> + *<br>
> + * @return The board support package name.<br>
> + */<br>
> +const char *rtems_board_support_package( void );<br>
> +<br>
>   /** @} */<br>
>   <br>
>   #ifdef __cplusplus<br>
> diff --git a/testsuites/sptests/spversion01/init.c b/testsuites/sptests/spversion01/init.c<br>
> index f4edcb2cdc..fc38577691 100644<br>
> --- a/testsuites/sptests/spversion01/init.c<br>
> +++ b/testsuites/sptests/spversion01/init.c<br>
> @@ -29,6 +29,7 @@ static rtems_task Init(<br>
>     printf("Minor    : %d\n", rtems_version_minor());<br>
>     printf("Revision : %d\n", rtems_version_revision());<br>
>     printf("VC Key   : %s\n", rtems_version_control_key());<br>
> +  printf("BSP      : %s\n", rtems_board_support_package());<br>
>   <br>
>     TEST_END();<br>
>   <br>
<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>
<br>
_______________________________________________<br>
devel mailing list<br>
<a href="mailto:devel@rtems.org" target="_blank">devel@rtems.org</a><br>
<a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a></blockquote></div></div>