[U-Boot-Users] [DNX#2006062042000013] u-boot patch MicroSys MPX8349

[U-Boot patch tracking system]

Hallo Herr Loeffl, 

in message <1150816202.97887.1634914156.338.1 at castor.denx.de> you wrote:
>
> Your e-mail was registered at our system under the ticket
> number [DNX#2006062042000013].

Die Regeln im README unter "Submitting Patches" sind  immer  noch  in
Kraft, insbesondere:

    Patches shall be sent to the u-boot-users mailing list.

So schön unser Ticket-System ist, bisher kann darauf außer  uns  nie-
mand  zugreifen,  Ihr  Patch  ist  also  in einem Schwarzen Loch ver-
schwunden. So ist das nicht gedacht.


Was den Patch selbst betrifft: den möchte ich so  nicht  akzeptieren.
Hauptgrund  ist,  daß  Sie  den  ganzen  ECC-Init-  und  Testcode vom
MPC8349EMDS Board 1:1 in Ihren Code  kopiert  haben.  Eine  derartige
Duplikation  von  so umfangreichen Code-Teilen (weit über 300 Zeilen)
kann ich nicht zulassen, da diese nicht wartbart ist. Wenn Sie  diese
Funktionalität  benutzen  wollen,  muß  diese in separate, von beiden
Boards gemeinsame Files ausgegliedert werden.

Außerdem gibt es ein paar kleinere Verstöße gegen die Coding Style
Vorschriften (vor allem Trailing White Space).




Your Ticket-Team

-- 
Wolfgang Denk --
DENX Software Engineering:  Embedded and Realtime Systems, Embedded Linux
Phone: (+49)-8142-66989-0 Fax: (+49)-8142-66989-80 Email: support at denx.de
Powered by OTRS (http://otrs.org/)