[En-Nut-Discussion] Porting Nut/OS to atmel's SAM3U (cortex-m3)

Harald Kipp harald.kipp at egnite.de
Thu Apr 8 17:49:06 CEST 2010


Hi all,

On 01.02.2010 10:29, Nikolay Zamotaev wrote:

> I posted patch done against svn revision r2881
> https://sourceforge.net/tracker/?func=detail&atid=410689&aid=2943586&group_id=34079

This patch had been posted about 2 months ago. As far as I can see,
nobody took care.

Nikolay probably spent a lot of time on this, it is quite large. Not
everyone, who is interested in running Nut/OS on Cortex-M3 may be able
to deal with Subversion and patches. My recommendation is, to add it to
the trunk now, so users can start playing with it.

Cons:

* The patch doesn't always follow our coding guidelines.
http://www.ethernut.de/en/documents/programming-style-guide.html
* I assume, that it is not very well tested by anyone except the author
himself.
* Even Nikolay himself admits, that it may be partly buggy.
* In the past I often applied patches without further checking and later
it took months for discovering and removing the bugs (e.g. register
usage of SAM7 context switching).
* I'm working towards a next stable release and don't like to have big
changes right now.

Pros:

* The patch contains very few changes on existing files.
* We may declare Cortex support as beta, even in a stable release.

What's your opinion?

What worries me most is, that more changes on the trunk may make the
patch unusable. Because of its size, it may later require a lot of
effort to fix this.

Last not least: Respect, Nikolay!

Harald




More information about the En-Nut-Discussion mailing list