This is the mail archive of the ecos-maintainers@sources.redhat.com mailing list for the eCos project.
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |
Other format: | [Raw text] |
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 > >
Attachment:
watchdog_at91.cxx.patch
Description: Binary data
Attachment:
var_io.h.patch
Description: Binary data
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |