Trans.: Re: at91 watchdog on eb42

[davarn@free.fr]

I redo var_io and watchdog patches concidering all your remarks.
var_io was big because i did not used the latest version to make my code.

Selon Nick Garnett <nickg@ecoscentric.com>:

> davarn@free.fr writes:
>
> > Hello,
> >
> > The change is really the minimum, even if it is quite important.
> > Tis is due to the fact that on eb42 board, if Interrupt is used in init_hw,
> the
> > object is not already intialized so the attach function assert.
> > Moving it in the start function make the thing work, and does not change
> the
> > behavior at all.
> > To be closer to the original code, it is necessary to investigate object
> > intialization on eb42 board.
>
> I was also concerned about changes to var_io.h. Your patch seems to be
> deleting large numbers of definitions. I'm also not sure what the
> change to AT91_US_BAUD() is going to do.
>
>
> The problem with the interrupt object can be fixed by adding
> CYGBLD_ATTRIB_INIT_PRI() to the interrupt object definition:
>
> static Cyg_Interrupt wdint(
>     CYGNUM_HAL_INTERRUPT_WATCHDOG,
>     INT_PRIO,
>     0,
>     isr,
>     NULL
>   ) CYGBLD_ATTRIB_INIT_PRI( CYG_INIT_DRIVERS );
>
>
> This only seems to affect some targets, we haven't worked out why
> this is yet.
>
> --
> Nick Garnett                    eCos Kernel Architect
> http://www.ecoscentric.com/     The eCos and RedBoot experts
>
>

-------------- next part --------------
A non-text attachment was scrubbed...
Name: watchdog_at91.cxx.patch
Type: application/octet-stream
Size: 4100 bytes
Desc: not available
URL: <http://sourceware.org/pipermail/ecos-patches/attachments/20040812/9dba430a/attachment.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: var_io.h.patch
Type: application/octet-stream
Size: 1606 bytes
Desc: not available
URL: <http://sourceware.org/pipermail/ecos-patches/attachments/20040812/9dba430a/attachment-0001.obj>