public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Detlev Zundel <dzu@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] Add support to Colibri PXA-270 board
Date: Tue, 07 Apr 2009 20:41:18 +0200	[thread overview]
Message-ID: <m2ocv82to1.fsf@ohwell.denx.de> (raw)
In-Reply-To: <9e0cf0bf0904071126u4f0cf332x35fb378dcc5c1979@mail.gmail.com> (Alon Bar-Lev's message of "Tue, 7 Apr 2009 21:26:18 +0300")

Hi Alon,

> I don't understand why you are so angry...

Maybe because you misunderstood me.  Let me assure you, that I'm not
angry.  I only wanted to make a clear statement that there are people on
this list who spend really lots of time in the "unsexy job" of cleaning
up code.

This usually takes way longer than writing new code and of course this
is also completely invisible to the casual reader of the sorce code.
One also never gets any return from it, because one "does not add any
new shiny features".

Because of this, not many people attend to this type of work, so what I
try to do is to ensure that we *need as few cleanup as possible* right
upfront.

> There is no simple implementation of code reuse within the boards.

Are you sure?  You did not look very hard then.  In a strict sense it
may be true that "board directories" do not share much code (although
even that is done by some), but this is missing the point.

If something proves to be board-independent, it usually means it is only
architecture specific, so it can go there and be reused by *all* boards
of that architecture.

> You know the code much better than I do.
> Please free to drop this implementation.

Oh please, we really value all contributions, but if we were to accept
everything unconditionally, the code would explode in no time.

Please reconsider and try to reuse the code instead of copying it.
Maybe put it into pxa270 specific places.  Maybe Stefano has some good
ideas?

Cheers
  Detlev

-- 
Markov does it in chains.
--
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

      reply	other threads:[~2009-04-07 18:41 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-04-07  8:56 [U-Boot] [PATCH] Add support to Colibri PXA-270 board Alon Bar-Lev
2009-04-07 11:55 ` Detlev Zundel
2009-04-07 17:16   ` Alon Bar-Lev
2009-04-07 18:22     ` Detlev Zundel
2009-04-07 18:26       ` Alon Bar-Lev
2009-04-07 18:41         ` Detlev Zundel [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=m2ocv82to1.fsf@ohwell.denx.de \
    --to=dzu@denx.de \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox