[PATCH 7/8] bsp/motorola_powerpc: Add MPCI support for Qemu

[Joel Sherrill]

Are there any instructions on how you run mptests on this BSP?

On Thu, Mar 24, 2016 at 9:57 AM, Sebastian Huber <
sebastian.huber at embedded-brains.de> wrote:

> ---
>  .../libbsp/powerpc/motorola_powerpc/Makefile.am    |   2 +
>  .../libbsp/powerpc/motorola_powerpc/include/bsp.h  |   8 +
>  .../libbsp/powerpc/motorola_powerpc/shmsupp/mpci.c | 375
> +++++++++++++++++++++
>  c/src/lib/libbsp/powerpc/shared/startup/bspstart.c |  10 +-
>  4 files changed, 393 insertions(+), 2 deletions(-)
>  create mode 100644
> c/src/lib/libbsp/powerpc/motorola_powerpc/shmsupp/mpci.c
>
> diff --git a/c/src/lib/libbsp/powerpc/motorola_powerpc/Makefile.am
> b/c/src/lib/libbsp/powerpc/motorola_powerpc/Makefile.am
> index e09820608..3576b75 100644
> --- a/c/src/lib/libbsp/powerpc/motorola_powerpc/Makefile.am
> +++ b/c/src/lib/libbsp/powerpc/motorola_powerpc/Makefile.am
> @@ -136,6 +136,8 @@ noinst_PROGRAMS         += ne2000.rel
>  ne2000_rel_SOURCES       = ../../i386/pc386/ne2000/ne2000.c
>  ne2000_rel_CPPFLAGS      = $(AM_CPPFLAGS) $(ne2000_CPPFLAGS)
>  ne2000_rel_LDFLAGS       = $(RTEMS_RELLDFLAGS)
> +
> +libbsp_a_SOURCES += shmsupp/mpci.c
>  endif
>

Is this guarded by the proper conditional? It looks like it is inside a
networking enabled one.


>  endif
>
> diff --git a/c/src/lib/libbsp/powerpc/motorola_powerpc/include/bsp.h
> b/c/src/lib/libbsp/powerpc/motorola_powerpc/include/bsp.h
> index 0439e87..b955d72 100644
> --- a/c/src/lib/libbsp/powerpc/motorola_powerpc/include/bsp.h
> +++ b/c/src/lib/libbsp/powerpc/motorola_powerpc/include/bsp.h
> @@ -242,6 +242,14 @@ void zero_bss(void);
>   */
>  void VIA_isa_bridge_interrupts_setup(void);
>
> +#ifdef RTEMS_MULTIPROCESSING
> +
> +extern rtems_mpci_table net_mpci_table;
> +
> +#define CONFIGURE_MP_MPCI_TABLE_POINTER &net_mpci_table
> +
> +#endif /* RTEMS_MULTIPROCESSING */
> +
>  #endif
>
>

Doesn't this imply that networking and MPCI must both be available? How is
this guaranteed?


>
> diff --git a/c/src/lib/libbsp/powerpc/shared/startup/bspstart.c
> b/c/src/lib/libbsp/powerpc/shared/startup/bspstart.c
> index 0f450fc..ca687ab 100644
> --- a/c/src/lib/libbsp/powerpc/shared/startup/bspstart.c
> +++ b/c/src/lib/libbsp/powerpc/shared/startup/bspstart.c
> @@ -87,13 +87,19 @@ unsigned int BSP_time_base_divisor;
>  void BSP_panic(char *s)
>  {
>    printk("%s PANIC %s\n",_RTEMS_version, s);
> -  __asm__ __volatile ("sc");
> +
> +  while (true) {
> +    /* Do nothing */
> +  }
>  }
>
>  void _BSP_Fatal_error(unsigned int v)
>  {
>    printk("%s PANIC ERROR %x\n",_RTEMS_version, v);
> -  __asm__ __volatile ("sc");
> +
> +  while (true) {
> +    /* Do nothing */
> +  }
>  }
>
>
What actually changed here?


>  /*
> --
> 1.8.4.5
>
> _______________________________________________
> devel mailing list
> devel at rtems.org
> http://lists.rtems.org/mailman/listinfo/devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rtems.org/pipermail/devel/attachments/20160324/2cc8893a/attachment-0002.html>