[Bug 1000761] eCos support for MPC5xxx MCUs

[bugzilla-daemon at ecoscentric dot 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.