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:22:16 +0200	[thread overview]
Message-ID: <m2ws9w2ujr.fsf@ohwell.denx.de> (raw)
In-Reply-To: <9e0cf0bf0904071016j2fc2cdddt93a9f1155e211193@mail.gmail.com> (Alon Bar-Lev's message of "Tue, 7 Apr 2009 20:16:16 +0300")

Hi Alon,

> It was not my intention to add publicity my name, but provide
> a solution for users, as it took a lot of work to make it
> work without many changes from other implementation.

Adding someones name to code that one has written is primarily not for
publicity reasons but for somple tracking reasons.  Who do you think
people will turn to if they see problems in the code?  What would you
say if someone mailed you about a problem in "your" code which you have
never seen before?

> Any way to do it using a reuse would be super, although
> this is not the only duplication in u-boot board tree...

Yes, and maybe you realize how *damn* (sorry) some of us work to
eliminate this.  I never tire of stating that the cost of fixing a
problem in software is exponential to the time of its discovery.

Translated into U-Boot context this means, that we do *not* allow simple
code duplication.  If such a case slips through the review process, this
is nowhere near a reason to allow it the next time.

So please, reconsider adding to the mess instead of decreasing it or
even keeping the mess-level constant ;)

Cheers
  Detlev

-- 
The 82558 B-step and later generation devices do not maintain a link
in D3 if PME is disabled or if the device does not have power.
                                        -- Intel documentation
--
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:22 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 [this message]
2009-04-07 18:26       ` Alon Bar-Lev
2009-04-07 18:41         ` Detlev Zundel

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