From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Sun, 24 Sep 2017 13:28:45 -0400 Subject: [U-Boot] [PATCH 0/5] Follow the Linux Kernel in building dtc as needed In-Reply-To: <61a675e1-a3ae-c8f4-32fd-cf548c4047df@denx.de> References: <1506263222-7080-1-git-send-email-trini@konsulko.com> <61a675e1-a3ae-c8f4-32fd-cf548c4047df@denx.de> Message-ID: <20170924172845.GV30205@bill-the-cat> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Sun, Sep 24, 2017 at 06:50:01PM +0200, Marek Vasut wrote: > On 09/24/2017 04:26 PM, Tom Rini wrote: > > The following series has been applied. I am posting this for the record. > > > > For the record, I do not believe that using git submodules is a good approach > > here. We have a small amount of code that we need here, and happily we can > > leverage infrastructur e from the Linux Kernel. > > > > Speaking of, this is not the first time we have run into problems deviating > > from the workflow of the kernel. The problems of having large number of > > warnings, or not stemmed from not just leve raging all of the infrastructure > > from the kernel. So related, yes, fixes for these warnings should come in, > > and as always, if they're in the upstream kernel dts as well, they should be > > fixed there. > > So any comments regarding bundling external tools were ignored, even > though the discussion was not finished, great. > > Furthermore, there was zero time to review this series, it was just > applied and posted afterward ? What sort of practice is this ? Yes. I put on my slightly-less-than-benevolent dictator hat today, and explained my reasoning. I do feel bad that you're rather unhappy with the overall situation, but no, I believe this is the right answer. Now, there's some follow up further re-syncing with the kernel that could be done (scripts/Makefile.lib/extrawarn can be further re-matched up with the kernel now, but that would have made a larger delta than "just migrate to providing the tool"). -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: Digital signature URL: