Re: [ECOS] hal_idle_thread_action on ARM

[Jonathan Larmour <jifl at eCosCentric dot com>]

Daniel Néri wrote:
>         Greetings,
>
> The ARM architecture HAL defines a dummy hal_idle_thread_action.
> Which is the best way to override this in a variant or platform HAL?

Since it's empty, but included all the time from the arch/hal_misc.c, it's 
 probably better just to remove it from hal_arch.h and hal/misc.c.

For overriding it, I'd probably recommend adding:

#ifdef CYGINT_HAL_ARM_PLF_ARCH_H
# include <cyg/hal/plf_arch.h>
#endif

and then adding CDL in the arch HAL to add a cdl_interface 
CYGINT_HAL_ARM_PLF_ARCH_H which platform CDL that supplies that header 
could then implement.

Just my opinion though.

Jifl
--
eCosCentric    http://www.eCosCentric.com/    The eCos and RedBoot experts
--[ "You can complain because roses have thorns, or you ]--
--[  can rejoice because thorns have roses." -Lincoln   ]-- Opinions==mine


--
Before posting, please read the FAQ: http://sources.redhat.com/fom/ecos
and search the list archive: http://sources.redhat.com/ml/ecos-discuss