[Bug 1000761] eCos support for MPC5xxx MCUs

[bugzilla-daemon at bugs dot ecos dot sourceware dot org]

Please do not reply to this email. Use the web interface provided at:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=1000761

--- Comment #61 from Ilija Kocho <ilijak@siva.com.mk> 2012-10-05 14:05:54 BST ---
(In reply to comment #58)

Hi Stefan

> Hi ecos Maintainers, Jonathan,
> any progress on the review of our contribution ? It has almost been two months
> and we heard nothing so far. I was hoping, that active contributions are more
> appreciated ?!

In comment #32 and comment #33 Jifl mentiond duplicate files, and they are
still present in recent attachment #1872 (aka /MPC5xxx HAL packages/ ).
There are a lot of definitions repeating across /include/ directories of all
platforms, in files such as: var_regs.h, mpc5*.h, plf_intr.h, etc. These can,
and should be consolidated in respective files placed in var/current/include
directory.

Also some of these files, notably mpc5*.h have copyright banners that are
incompatible with eCos licence.

This is my impression, but could the number of platforms be reduced, at least
for initial port? I have impression that the aim is to cover every chip (which
is not bad) but could some of these platforms be consolidated? It will make the
review and maintenance easier.

Regards

Ilija

-- 
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.