This is the mail archive of the ecos-bugs@sourceware.org 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]

[Bug 1001184] Cortex-m arch: hal_intr.h includes var_intr.h tooearly.


Please do not reply to this email. Use the web interface provided at:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001184

--- Comment #9 from Ilija Kocho <ilijak@siva.com.mk> 2011-04-04 21:50:03 BST ---
(In reply to comment #8)
> (In reply to comment #7)
> > (In reply to comment #6)
> > > (In reply to comment #5)
> > > > Sergei, Sorry but I do not quite understand what to do? What is the
> > > > acronym CL?
> > > 
> > > Ah, Ilija, I'm sorry CL = ChangeLog. I meaned and to fix your ChangeLog's
> > > record.
> > > 
> > > Sergei
> > 
> > OK - Could it be:
> >  ... after definition of
> > - CYGNUM_HAL_INTERRUPT_EXTERNAL
> > + Cortex-M architecture common exception vectors
> 
> Ilija, I kept in mind something likes the below
> 
> * include/hal_intr.h: Place CYGNUM_HAL_INTERRUPT_RTC
> cond. definition properly. Remove duplicated inclusion of
> var_intr.h header.
> 
> Why do we need to talk about the exception vectors? As far as I could
> understand CYGNUM_HAL_INTERRUPT_RTC definition is misplaced and there
> is dup. header inclusion in hal_intr.h. So, you fix it and should add
> record *about* it. Did I miss something?

Maybe I tried to be too descriptive. If we opt for short record I think the
main point is placing #include var_intr.h at right place so I would restate:

 * include/hal_intr.h: Remove double inclusion of var_intr.h header and include
   it at proper place.

-- 
Configure bugmail: http://bugs.ecos.sourceware.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]