hal_idle_thread_action on ARM

[Daniel Néri]

Jonathan Larmour <jifl@eCosCentric.com> wrote:

> 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.

As I didn't quite share Jonathan's opinion, I finally came up with my
own solution (attached below). I'll make a separate post with a
HAL_IDLE_THREAD_ACTION implementation for the new AT91 variant HAL.


Regards,
  --Daniel


-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: text/x-patch
Size: 4506 bytes
Desc: Configurable HAL_IDLE_THREAD_ACTION for ARM
URL: <http://sourceware.org/pipermail/ecos-patches/attachments/20030516/5ba98035/attachment.bin>