This is the mail archive of the
ecos-bugs@sourceware.org
mailing list for the eCos project.
[Bug 1001184] New: Cortex-m arch: hal_intr.h includes var_intr.h tooearly.
- From: bugzilla-daemon at bugs dot ecos dot sourceware dot org
- To: ecos-bugs at ecos dot sourceware dot org
- Date: Fri, 1 Apr 2011 15:09:56 +0100
- Subject: [Bug 1001184] New: Cortex-m arch: hal_intr.h includes var_intr.h tooearly.
- Auto-submitted: auto-generated
Please do not reply to this email. Use the web interface provided at:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001184
Summary: Cortex-m arch: hal_intr.h includes var_intr.h too
early.
Product: eCos
Version: 3.0
Platform: All
OS/Version: Cortex-M
Status: UNCONFIRMED
Severity: normal
Priority: low
Component: HAL
AssignedTo: unassigned@bugs.ecos.sourceware.org
ReportedBy: ilijak@siva.com.mk
CC: ecos-bugs@ecos.sourceware.org
Class: Advice Request
Created an attachment (id=1195)
--> (http://bugs.ecos.sourceware.org/attachment.cgi?id=1195)
Include var_intr.h after definition of hal_intr.h macros but allows overriding
of CYGNUM_HAL_INTERRUPT_RTC.
In Cortex-M architecture, the header hal_intr.h includes var_intr.h twice.
Second one is of course with no effect, but the first one is too early.
Namely some macros that are defined in hal_intr.h, such as
CYGNUM_HAL_INTERRUPT_EXTERNAL, are used in hal-intr.h. This doesn't seem to
show ill effect so far since usage is by other macros.
Note: The early inclusion of var_intr.h has been introduced by Bug 1001090
Attachment 1092 in order to enable overriding of CYGNUM_HAL_INTERRUPT_RTC by
variant/platform.
The proposed patch includes var_intr.h after definition of hal_intr.h macros
but also allows overriding of CYGNUM_HAL_INTERRUPT_RTC.
--
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.