[rtems commit] bsp/mpc55xx: Add e200z7 specific idle task

[Joel Sherrill]

On 9/28/2012 9:47 AM, Sebastian Huber wrote:
> Module:    rtems
> Branch:    master
> Commit:    b762488ef5cd1fe18b620dd960dcbab9587aefe9
> Changeset: http://git.rtems.org/rtems/commit/?id=b762488ef5cd1fe18b620dd960dcbab9587aefe9
>
> Author:    Sebastian Huber <sebastian.huber at embedded-brains.de>
> Date:      Fri Sep 28 16:02:28 2012 +0200
>
> bsp/mpc55xx: Add e200z7 specific idle task
>
> ---
>
>   c/src/lib/libbsp/powerpc/mpc55xxevb/include/bsp.h |   13 +++++++++++++
>   1 files changed, 13 insertions(+), 0 deletions(-)
>
> diff --git a/c/src/lib/libbsp/powerpc/mpc55xxevb/include/bsp.h b/c/src/lib/libbsp/powerpc/mpc55xxevb/include/bsp.h
> index b486820..e1b1405 100644
> --- a/c/src/lib/libbsp/powerpc/mpc55xxevb/include/bsp.h
> +++ b/c/src/lib/libbsp/powerpc/mpc55xxevb/include/bsp.h
> @@ -74,6 +74,19 @@ rtems_status_code bsp_register_i2c(void);
>   
>   void bsp_restart(void *addr);
>   
> +#if MPC55XX_CHIP_TYPE / 10 == 567

Could you put parentheses around the "MPC55XX_CHIP_TYPE / 10"
portion?

And could you add a comment block explaining this?  Or even better what
does 567 mean? Should that be a constant?

> +  static void *mpc55xx_idle_task(uintptr_t arg)
> +  {
> +    while (true) {
> +      __asm__ volatile ("wait");
> +    }
> +
> +    return NULL;
> +  }
> +
> +  #define BSP_IDLE_TASK_BODY mpc55xx_idle_task
> +#endif
> +
>   #endif /* ASM */
>   
>   #ifdef __cplusplus
>
> _______________________________________________
> rtems-vc mailing list
> rtems-vc at rtems.org
> http://www.rtems.org/mailman/listinfo/rtems-vc


-- 
Joel Sherrill, Ph.D.             Director of Research&  Development
joel.sherrill at OARcorp.com        On-Line Applications Research
Ask me about RTEMS: a free RTOS  Huntsville AL 35805
     Support Available             (256) 722-9985