From: Tom Rini <trini@konsulko.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 0/5] REVERT Follow the Linux Kernel in building dtc as needed
Date: Wed, 27 Sep 2017 10:31:12 -0400 [thread overview]
Message-ID: <20170927143112.GT3112@bill-the-cat> (raw)
In-Reply-To: <20170927141524.5920-1-wd@denx.de>
On Wed, Sep 27, 2017 at 04:15:19PM +0200, Wolfgang Denk wrote:
> This patch series reverts some patches that should never have been
> merged into mainline at all:
>
> - They were applied in violation of the well established rule that
> ALL patches must be posted on the mailing list before, and given
> enough time for review by the community.
> - Obviously, these patches have not been Acked-by: by anybody.
> - They implement a solution that probably does not represent the
> majority of the developers who discussed this issue. They were
> applied cutting short the ongoing discussion, without giving a
> good, comprehensible reason. Alternative solutions were
> presented before, but this patch series ignores them.
>
> U-Boot is a community project with an established and documented
> development process. No single person, not even a custodian or
> maintainer, shall have the power to push through any changes without
> or against the consent of the community. If we want to maintain
> these standards, we cannot accept this. We therefore have only one
> choice: revert these patches now, and let them undergo the necessary
> review process on the mailing list, allowing for the usual grace
> period of two weeks. Sorry.
Well, NAK. When you passed the "benevolent dictator" hat over, you
passed it over. I value your input, and I don't lightly over-rule
feedback. So, lets summarize things:
- We need a modern device tree compiler.
- Everyone needs a modern device tree compiler, because everyone should
be running 'make tests'.
- This is not the first, or second, or possibly even third time we've
hit problems of people using different versions of dtc and seeing
different results / problems.
- The Linux Kernel has avoided this problem for forever by just bundling
the minimum required parts of dtc in order to build things.
So, I've imported the infrastructure to keep things in sync, and
imported upstream dtc.
Now, the best argument I've heard against this is "bundling is bad". I
don't disagree, and the best way to fix this problem would be for
distributions to start shipping a new enough version here that the
kernel can drop it out, just like they did for linux-firmware recently.
A whole lot more people care about the kernel than they do about U-Boot,
and that's a more viable path forward. It would even make some kernel
folks happy.
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170927/6031580e/attachment.sig>
next prev parent reply other threads:[~2017-09-27 14:31 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-09-24 14:26 [U-Boot] [PATCH 0/5] Follow the Linux Kernel in building dtc as needed Tom Rini
2017-09-24 14:26 ` [U-Boot] [PATCH 1/5] scripts/dtc: Update to upstream version v1.4.3 Tom Rini
2017-10-03 18:24 ` Rob Herring
2017-10-03 18:28 ` Tom Rini
2017-10-03 18:30 ` Rob Herring
2017-10-03 18:44 ` Tom Rini
2017-09-24 14:26 ` [U-Boot] [PATCH 2/5] scripts/dtc: Update to upstream version v1.4.4 Tom Rini
2017-09-24 14:27 ` [U-Boot] [PATCH 3/5] scripts/dtc: Update to upstream version v1.4.4-50-gfe50bd1ecc1d Tom Rini
2017-09-24 14:27 ` [U-Boot] [PATCH 4/5] dtc: Switch to building and using our own dtc unless provided Tom Rini
2017-09-24 14:27 ` [U-Boot] [PATCH 5/5] tools/mkimage: Make the path to the dtc binary that mkimage calls configurable Tom Rini
2017-09-24 16:50 ` [U-Boot] [PATCH 0/5] Follow the Linux Kernel in building dtc as needed Marek Vasut
2017-09-24 17:28 ` Tom Rini
2017-09-24 20:39 ` Łukasz Majewski
2017-09-24 22:00 ` Tom Rini
2017-09-25 8:32 ` Marek Vasut
2017-09-25 8:28 ` Wolfgang Denk
2017-09-27 14:15 ` [U-Boot] [PATCH 0/5] REVERT " Wolfgang Denk
2017-09-27 14:15 ` [U-Boot] [PATCH 1/5] Revert "tools/mkimage: Make the path to the dtc binary that mkimage calls configurable" Wolfgang Denk
2017-09-27 14:15 ` [U-Boot] [PATCH 2/5] Revert "dtc: Switch to building and using our own dtc unless provided" Wolfgang Denk
2017-09-27 14:15 ` [U-Boot] [PATCH 3/5] Revert "scripts/dtc: Update to upstream version v1.4.4-50-gfe50bd1ecc1d" Wolfgang Denk
2017-09-27 14:15 ` [U-Boot] [PATCH 4/5] Revert "scripts/dtc: Update to upstream version v1.4.4" Wolfgang Denk
2017-09-27 14:15 ` [U-Boot] [PATCH 5/5] Revert "scripts/dtc: Update to upstream version v1.4.3" Wolfgang Denk
2017-09-27 14:31 ` Tom Rini [this message]
2017-09-27 14:56 ` [U-Boot] [PATCH 0/5] REVERT Follow the Linux Kernel in building dtc as needed Wolfgang Denk
2017-09-27 15:23 ` Tom Rini
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=20170927143112.GT3112@bill-the-cat \
--to=trini@konsulko.com \
--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.