[U-Boot] [PATCH] p1022ds: add video support

Detlev Zundel
2010-09-22T15:30:41Z

Hi Timur,

> Detlev Zundel wrote:
>> This "but there are other examples doing just the same!" arguing is
>> getting tiresome. Please accept that this is not an argument capable of
>> convincing people - the opposite is true.
>
> It's also very frustrating when developers look at the way U-Boot does things,
> then spend countless hours or days implementing something similar, only be told,
> "Well, we know this is how it's done, but it's wrong, so you need to rewrite
> your code."

It is a simple fact that we cannot remove every part of U-Boot code
which is not in alignment with the (current) design principles. The
design principles of _every_ software project will develop over its
existence and there will always be old code around which is not how it
would be done if starting today. This will happen for every moderately
large project and should be "common sense" for software engineers.

The only way I can see around this is if the project has a sufficiently
large number of "janitors" cleaning up such things. And you know as
well as I do that we simply do not have such people for U-Boot, although
the work would be much appreciated.

It is not the first and not the second time that you argument this way.
Who stops you from posting a small mail with the intent of your changes
to get early feedback from others if in doubt? Not only in this context
I would advise doing this whenever possible.

Cheers
Detlev

--
Another helpful hint for successful MIME processing:

application/msword; rm -f %s; description="MS Word Text";
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-40 Fax: (+49)-8142-66989-80 Email: dzu at denx.de