This is the mail archive of the
ecos-bugs@sourceware.org
mailing list for the eCos project.
[Bug 1000761] eCos support for MPC5xxx MCUs
- From: bugzilla-daemon at ecoscentric dot com
- To: ecos-bugs at ecos dot sourceware dot org
- Date: Thu, 6 Aug 2009 15:30:25 +0100
- Subject: [Bug 1000761] eCos support for MPC5xxx MCUs
- References: <bug-1000761-13@http.bugzilla.ecoscentric.com/>
http://bugzilla.ecoscentric.com/show_bug.cgi?id=1000761
--- Comment #19 from Stefan Singer <Stefan.Singer@freescale.com> 2009-08-06 15:30:23 ---
Hi Nick,
I am currently on vacation, so I can not test it on a target, but it looks good
when I compile it. I will test on real HW next week. Now I have only two
outstanding problems:
problem I:
You write:
>The intended use of the BOOK_E option is that the HAL for the appropriate
>variant should have a "requires" statement for it. Otherwise we could end up
>with twisty mazes of conditions in the architecture HAL.
I have done that now, but that means, everytime you select the template, you
will get a conflict window. I am probably not familiar enough with the whole
options of cdl files, but can this not be avoided ? I know, that for templates
options can be forced, but can I do this also for the cdl file ?
problem II:
you did not comment on the proposed changes in hal_misc.c regarding the Macro
for unified cache, where you had used
#ifndef HAL_UCACHE_ENABLE
which is used as a Macro function call in all other architectures and where I
had proposed to change that to "HAL_CACHE_IS_UCACHE". I need the macro function
call for devices with our e200z6 core, so if you are not agreeing to change
that other macro, than I would need to change the name of this, which I think
would be unclean, since this is used in all other architectures.
Can you please look at that ?
--
Configure bugmail: http://bugzilla.ecoscentric.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.