This is the mail archive of the ecos-patches@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 too early.


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

--- Comment #8 from Sergei Gavrikov <sergei.gavrikov@gmail.com> 2011-04-04 14:29:12 BST ---
(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?

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


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