This is the mail archive of the
mailing list for the eCos project.
[Bug 1001184] Cortex-m arch: hal_intr.h includes var_intr.h too early.
- From: bugzilla-daemon at bugs dot ecos dot sourceware dot org
- To: ecos-patches at ecos dot sourceware dot org
- Date: Mon, 4 Apr 2011 21:50:06 +0100
- Subject: [Bug 1001184] Cortex-m arch: hal_intr.h includes var_intr.h too early.
- Auto-submitted: auto-generated
- References: <email@example.com/>
Please do not reply to this email. Use the web interface provided at:
--- Comment #9 from Ilija Kocho <firstname.lastname@example.org> 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 on the CC list for the bug.