This is the mail archive of the
ecos-bugs@sourceware.org
mailing list for the eCos project.
[Bug 1001184] Cortex-m arch: hal_intr.h includes var_intr.h tooearly.
- From: bugzilla-daemon at bugs dot ecos dot sourceware dot org
- To: unassigned at bugs dot 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 tooearly.
- Auto-submitted: auto-generated
- References: <bug-1001184-777@http.bugs.ecos.sourceware.org/>
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.