All of lore.kernel.org
 help / color / mirror / Atom feed
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 11:23:45 -0400	[thread overview]
Message-ID: <20170927152345.GV3112@bill-the-cat> (raw)
In-Reply-To: <20170927145627.57E011201AF@gemini.denx.de>

On Wed, Sep 27, 2017 at 04:56:27PM +0200, Wolfgang Denk wrote:
> Dear Tom,
> 
> In message <20170927143112.GT3112@bill-the-cat> you wrote:
> > 
> > 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:
> 
> These are all technical arguments. You could have raised these in the
> ongoing discussion.  If they are good, convincing arguments, you
> should have no problems to have them accepted.

To which I did for some of them, and others I was going to follow up
with, but left aside rather than further the thread.

> What I'm complainiung about is not WHAT was commited, but HOW it was
> committed.
> 
> Marek wrote:
> 
> | I believe we have a well-established process of submitting patches,
> | getting a review and then applying them. If the head maintainer doesn't
> | follow the process, why should anyone else ?
> 
> You have a role model function. Please stick to the rules!

Right.  And part of the job of the custodian is to, on occasion, put
their foot down, on technical grounds, and say when we must do X.  This
happens from time to time anyways.  It rarely happens between
custodians, and I think this is occurrence number 2 in just over 5
years.  And given the related issue of "everyone" being broken unless
they have a new DTC installed, I didn't want to wait nearly so long try
and convince Marek as I did with Albert and the unaligned access thing
last time.  Nor did I want to go with reverting the FDT changes so we
can continue cycling around on what amounts to correcting the previous
mistake of not just including dtc.

-- 
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/2d51349b/attachment.sig>

      reply	other threads:[~2017-09-27 15:23 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   ` [U-Boot] [PATCH 0/5] REVERT Follow the Linux Kernel in building dtc as needed Tom Rini
2017-09-27 14:56     ` Wolfgang Denk
2017-09-27 15:23       ` Tom Rini [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=20170927152345.GV3112@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.