All of lore.kernel.org
 help / color / mirror / Atom feed
From: Wolfgang Denk <wd@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] AT91 NAND om AT91SAM9260EK
Date: Mon, 12 Feb 2007 00:45:20 +0100	[thread overview]
Message-ID: <20070211234520.9F54135265F@atlas.denx.de> (raw)
In-Reply-To: Your message of "Sun, 11 Feb 2007 22:39:07 +0100." <00f201c74e25$71fdbf80$01c4af0a@Glamdring>

Dear Ulf,

in message <00f201c74e25$71fdbf80$01c4af0a@Glamdring> you wrote:
>
> A contributor has no business changing  a board file
> if he cannot reasonably verify that this will not break a board.

You are right in principle, but live is not always black and white.

> > I cannot help this, and we will definitely not wait to see an ACK for
> > each and every board that is listed in the U-Boot tree.
> 
> No, but as mentioned above, different parts of U-Boot should require
> the submitter to focus on different subsets of boards.

It ain't that easy. Take for example this stupid ". = .;" addition to
the linker scripts that was needed  to  make  some  versions  of  the
toolchain  happy  -  of course I add this to ALL boards without being
able to test even half of them.

And as mentioned before: if I have a global change  to  make,  and  I
cannot   get  response  from  board  /  architecture  maintainers  in
reasonable time, that change will be made globally, even if it breaks
their boards. It's IMHO better to have a board  clearly  broken  than
having all boards in N different patch states.

> The proposed dataflash patches in our other heated discussion
> for instance should only be accepted once proven that it actually
> works with dataflash memory chips.
> The proposer seemed to be happy if the stuff compiled...
> Since that mainly affects Atmel boards, he should make
> sure to get Atmel boards or sign up testers for it,
> and have it tested before even thinking about submit patches.

I disagree with you. You cannot expect any maintainer or custodian or
other person volunteering to contribute to U-Boot code (or any  other
free  software  project)  to "get Atmel boards or sign up testers for
it". There are board maintainers,  and  it  is  their  task  to  help
testing  things. And yes, this includes that they will sometimes have
to help sorting out problems created by other people. That's  how  it
has  been,  and  will remain, even if Atmel was shipping test systems
for free.

Or do you intend to get a cmc_pu2 board or sign up testers for it  to
check your patches? No, it will be me who has to run the tests, right?

> It really does not make sense to test his patches only
> on boards which does not use the dataflash.

If that's all he has, then he has no other choice.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, HRB 165235 Munich, CEO: Wolfgang Denk
Office:  Kirchenstr. 5,       D-82194 Groebenzell,            Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
"If the code and  the  comments  disagree,  then  both  are  probably
wrong."                                                - Norm Schryer

  reply	other threads:[~2007-02-11 23:45 UTC|newest]

Thread overview: 42+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <mailman.12584.1171099992.16820.u-boot-users@lists.sourceforge.net>
2007-02-10 20:31 ` [U-Boot-Users] AT91 NAND om AT91SAM9260EK Ivan Kuten
2007-02-11 16:43   ` Ulf Samuelsson
2007-02-11 18:04     ` Haavard Skinnemoen
2007-02-11 19:42       ` Ulf Samuelsson
2007-02-11 20:23         ` Haavard Skinnemoen
2007-02-11 20:29           ` Ulf Samuelsson
2007-02-11 20:54             ` Haavard Skinnemoen
2007-02-11 21:10           ` Wolfgang Denk
2007-02-11 21:39             ` Ulf Samuelsson
2007-02-11 23:45               ` Wolfgang Denk [this message]
2007-02-12  0:26                 ` Ulf Samuelsson
2007-02-12 15:18                   ` Haavard Skinnemoen
2007-02-12 18:40                     ` Ulf Samuelsson
2007-02-12 19:36                       ` Stefan Roese
2007-02-12 19:37                       ` Haavard Skinnemoen
2007-02-12 20:05                         ` Ulf Samuelsson
2007-02-11 21:54       ` Ulf Samuelsson
2007-02-11 11:49 Michel Benoit
2007-02-11 16:20 ` Ulf Samuelsson
     [not found] <mailman.5069.1170973304.16820.u-boot-users@lists.sourceforge.net>
2007-02-08 22:57 ` Ivan Kuten
     [not found] <c88e466f0702050752t16c18251gcddf40a7ec95469@mail.gmail.com>
2007-02-07 23:17 ` Ulf Samuelsson
2007-02-08  6:06   ` Stefan Roese
2007-02-08  8:34     ` Michel Benoit
2007-02-08 19:25       ` Ulf Samuelsson
2007-02-08 20:58         ` Haavard Skinnemoen
2007-02-08 22:20           ` Ulf Samuelsson
2007-02-09 15:45             ` Haavard Skinnemoen
2007-02-09 19:11               ` Ulf Samuelsson
2007-02-09 19:54                 ` Haavard Skinnemoen
2007-02-09 19:39               ` Wolfgang Denk
2007-02-09 22:18                 ` Ulf Samuelsson
2007-02-09 22:58                   ` Haavard Skinnemoen
2007-02-09 23:20                     ` Ulf Samuelsson
2007-02-09 23:42                       ` Haavard Skinnemoen
2007-02-10  0:10                         ` Ulf Samuelsson
2007-02-10  1:18                       ` Wolfgang Denk
2007-02-10  9:05                         ` Ulf Samuelsson
2007-02-10  7:23                     ` Stefan Roese
2007-02-10  1:15                   ` Wolfgang Denk
2007-02-10  7:32                     ` Stefan Roese
2007-02-10  9:29                     ` Ulf Samuelsson
2007-02-10  1:53       ` Ken Watson

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=20070211234520.9F54135265F@atlas.denx.de \
    --to=wd@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.