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
next prev parent 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.