[{"id":1774257,"web_url":"http://patchwork.ozlabs.org/comment/1774257/","msgid":"<61a675e1-a3ae-c8f4-32fd-cf548c4047df@denx.de>","list_archive_url":null,"date":"2017-09-24T16:50:01","subject":"Re: [U-Boot] [PATCH 0/5] Follow the Linux Kernel in building dtc as\n\tneeded","submitter":{"id":12009,"url":"http://patchwork.ozlabs.org/api/people/12009/","name":"Marek Vasut","email":"marex@denx.de"},"content":"On 09/24/2017 04:26 PM, Tom Rini wrote:\n> The following series has been applied. I am posting this for the record.\n> \n> For the record, I do not believe that using git submodules is a good approach\n> here. We have a small amount of code that we need here, and happily we can\n> leverage infrastructur e from the Linux Kernel.\n> \n> Speaking of, this is not the first time we have run into problems deviating\n> from the workflow of the kernel. The problems of having large number of\n> warnings, or not stemmed from not just leve raging all of the infrastructure\n> from the kernel. So related, yes, fixes for these warnings should come in,\n> and as always, if they're in the upstream kernel dts as well, they should be\n> fixed there.\n\nSo any comments regarding bundling external tools were ignored, even\nthough the discussion was not finished, great.\n\nFurthermore, there was zero time to review this series, it was just\napplied and posted afterward ? What sort of practice is this ?\n\n> Thanks all.\n>","headers":{"Return-Path":"","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=)","Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3y0Y9f6CC1z9sRq\n\tfor ;\n\tMon, 25 Sep 2017 02:50:13 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid E01CAC220E2; Sun, 24 Sep 2017 16:50:07 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id CABEDC22007;\n\tSun, 24 Sep 2017 16:50:04 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 3564EC22007; Sun, 24 Sep 2017 16:50:03 +0000 (UTC)","from mail-out.m-online.net (mail-out.m-online.net [212.18.0.10])\n\tby lists.denx.de (Postfix) with ESMTPS id E6EA7C21EF1\n\tfor ; Sun, 24 Sep 2017 16:50:02 +0000 (UTC)","from frontend01.mail.m-online.net (unknown [192.168.8.182])\n\tby mail-out.m-online.net (Postfix) with ESMTP id 3y0Y9Q4GGfz1qrfK;\n\tSun, 24 Sep 2017 18:50:02 +0200 (CEST)","from localhost (dynscan1.mnet-online.de [192.168.6.70])\n\tby mail.m-online.net (Postfix) with ESMTP id 3y0Y9Q2ckYz1qqkP;\n\tSun, 24 Sep 2017 18:50:02 +0200 (CEST)","from mail.mnet-online.de ([192.168.8.182])\n\tby localhost (dynscan1.mail.m-online.net [192.168.6.70]) (amavisd-new,\n\tport 10024)\n\twith ESMTP id cF6WXytQuPa6; Sun, 24 Sep 2017 18:50:01 +0200 (CEST)","from [IPv6:::1] (unknown [195.140.253.167])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby mail.mnet-online.de (Postfix) with ESMTPSA;\n\tSun, 24 Sep 2017 18:50:01 +0200 (CEST)"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=-0.7 required=5.0 tests=RCVD_IN_DNSWL_LOW,\n\tRCVD_IN_MSPIKE_H3,\n\tRCVD_IN_MSPIKE_WL autolearn=unavailable autolearn_force=no\n\tversion=3.4.0","X-Virus-Scanned":"amavisd-new at mnet-online.de","X-Auth-Info":"F/ddT7pWuST9z3K1glMwUzEe8HEVjxRUEBJAbGJfyG4=","To":"Tom Rini ","References":"<1506263222-7080-1-git-send-email-trini@konsulko.com>","From":"Marek Vasut ","Message-ID":"<61a675e1-a3ae-c8f4-32fd-cf548c4047df@denx.de>","Date":"Sun, 24 Sep 2017 18:50:01 +0200","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.3.0","MIME-Version":"1.0","In-Reply-To":"<1506263222-7080-1-git-send-email-trini@konsulko.com>","Content-Language":"en-US","Cc":"Vagrant Cascadian , u-boot@lists.denx.de","Subject":"Re: [U-Boot] [PATCH 0/5] Follow the Linux Kernel in building dtc as\n\tneeded","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" "}},{"id":1774266,"web_url":"http://patchwork.ozlabs.org/comment/1774266/","msgid":"<20170924172845.GV30205@bill-the-cat>","list_archive_url":null,"date":"2017-09-24T17:28:45","subject":"Re: [U-Boot] [PATCH 0/5] Follow the Linux Kernel in building dtc as\n\tneeded","submitter":{"id":65875,"url":"http://patchwork.ozlabs.org/api/people/65875/","name":"Tom Rini","email":"trini@konsulko.com"},"content":"On Sun, Sep 24, 2017 at 06:50:01PM +0200, Marek Vasut wrote:\n> On 09/24/2017 04:26 PM, Tom Rini wrote:\n> > The following series has been applied. I am posting this for the record.\n> > \n> > For the record, I do not believe that using git submodules is a good approach\n> > here. We have a small amount of code that we need here, and happily we can\n> > leverage infrastructur e from the Linux Kernel.\n> > \n> > Speaking of, this is not the first time we have run into problems deviating\n> > from the workflow of the kernel. The problems of having large number of\n> > warnings, or not stemmed from not just leve raging all of the infrastructure\n> > from the kernel. So related, yes, fixes for these warnings should come in,\n> > and as always, if they're in the upstream kernel dts as well, they should be\n> > fixed there.\n> \n> So any comments regarding bundling external tools were ignored, even\n> though the discussion was not finished, great.\n> \n> Furthermore, there was zero time to review this series, it was just\n> applied and posted afterward ? What sort of practice is this ?\n\nYes. I put on my slightly-less-than-benevolent dictator hat today, and\nexplained my reasoning. I do feel bad that you're rather unhappy with\nthe overall situation, but no, I believe this is the right answer.\n\nNow, there's some follow up further re-syncing with the kernel that\ncould be done (scripts/Makefile.lib/extrawarn can be further re-matched\nup with the kernel now, but that would have made a larger delta than\n\"just migrate to providing the tool\").","headers":{"Return-Path":"","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=)","ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=konsulko.com header.i=@konsulko.com\n\theader.b=\"j8ISRLLn\"; dkim-atps=neutral"],"Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3y0Z2M1kyWz9sRm\n\tfor ;\n\tMon, 25 Sep 2017 03:28:57 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 6DFB8C220E2; Sun, 24 Sep 2017 17:28:53 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 90EB3C22007;\n\tSun, 24 Sep 2017 17:28:51 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 959BCC22007; Sun, 24 Sep 2017 17:28:49 +0000 (UTC)","from mail-yw0-f179.google.com (mail-yw0-f179.google.com\n\t[209.85.161.179])\n\tby lists.denx.de (Postfix) with ESMTPS id 1BFE3C21EF1\n\tfor ; Sun, 24 Sep 2017 17:28:49 +0000 (UTC)","by mail-yw0-f179.google.com with SMTP id x131so3407048ywa.10\n\tfor ; Sun, 24 Sep 2017 10:28:49 -0700 (PDT)","from bill-the-cat ([2606:a000:1401:811b:9cbd:ca90:bff9:8e97])\n\tby smtp.gmail.com with ESMTPSA id\n\ti206sm1985321ywg.27.2017.09.24.10.28.45\n\t(version=TLS1_2 cipher=AES128-SHA bits=128/128);\n\tSun, 24 Sep 2017 10:28:45 -0700 (PDT)"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=-0.0 required=5.0 tests=RCVD_IN_DNSWL_NONE,\n\tRCVD_IN_MSPIKE_H3, RCVD_IN_MSPIKE_WL,\n\tT_DKIM_INVALID autolearn=unavailable\n\tautolearn_force=no version=3.4.0","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=konsulko.com;\n\ts=google; \n\th=date:from:to:cc:subject:message-id:references:mime-version\n\t:content-disposition:in-reply-to:user-agent;\n\tbh=xIMRbg7vcCKZ2W11UDr4YJvDiRRcou9wlZdPYvZ4vIs=;\n\tb=j8ISRLLnk5cbf8roLqZVMtr2OAxcoVPBgYCYLmvgMmTyS7WmWwBcQrerSQY7JgoTvZ\n\tL/+WS5YuPJKncFkkX7f8Zl17sErOmwC2N9Tatj5jl4Cpr03lgqgwEEBoxhir52OYAySH\n\tZ7LGIkiQx5Rea535YnHHrxg+v1V2By6RZ4hc0=","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:date:from:to:cc:subject:message-id:references\n\t:mime-version:content-disposition:in-reply-to:user-agent;\n\tbh=xIMRbg7vcCKZ2W11UDr4YJvDiRRcou9wlZdPYvZ4vIs=;\n\tb=kZNETwqFoW8FF8ZCJfKcdZUZzP88SyNvbgNm1HGSf3Z6OSvyCsAGjZlWLHoFhkTltt\n\tWB8ByxiXT2kY9/GjPuGLZhv59KcO4kmOv/XR9LYS1fpRwlPAGln7tYxj4djdMNYVnezS\n\t3fCAB3+uptedREvOPnaUmzdte4/8DZdvVjS+yolo5TYiO0iryVFEarmxkh8hQf0Tl9v8\n\tkseBhm7DdFpQNwVKL+LGhuTvZL0fzc68hBOnbVlouVr4GqwHRrQwXTPU1VKncOQp3QHH\n\t2cLb3/uq60N6JN+LV6OGxMygDSo0s/PgFOQeuuoD5LHhIJ1krszA6OFwecxMoHIgqjFS\n\tqe/w==","X-Gm-Message-State":"AHPjjUgmKCwZoyYnexLLe5pJ920g1w9+9K+rVlRCN19vB3QYfXcJi+hr\n\ttAAUtXLpipGY673t0Nwy8ZMM0Q==","X-Google-Smtp-Source":"AOwi7QCA2TUz9UBjnDnG/LD27aq/umcmHPC0TY7db1LIRwdtgSBKM4XGMOv1OCOADmxXV/09fsMTVA==","X-Received":"by 10.37.191.129 with SMTP id l1mr3176723ybk.158.1506274127560; \n\tSun, 24 Sep 2017 10:28:47 -0700 (PDT)","Date":"Sun, 24 Sep 2017 13:28:45 -0400","From":"Tom Rini ","To":"Marek Vasut ","Message-ID":"<20170924172845.GV30205@bill-the-cat>","References":"<1506263222-7080-1-git-send-email-trini@konsulko.com>\n\t<61a675e1-a3ae-c8f4-32fd-cf548c4047df@denx.de>","MIME-Version":"1.0","In-Reply-To":"<61a675e1-a3ae-c8f4-32fd-cf548c4047df@denx.de>","User-Agent":"Mutt/1.5.21 (2010-09-15)","Cc":"Vagrant Cascadian , u-boot@lists.denx.de","Subject":"Re: [U-Boot] [PATCH 0/5] Follow the Linux Kernel in building dtc as\n\tneeded","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Content-Type":"multipart/mixed;\n\tboundary=\"===============4751811661925221313==\"","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" "}},{"id":1774286,"web_url":"http://patchwork.ozlabs.org/comment/1774286/","msgid":"<7a2909a5-9d5e-3e7d-cdd0-a6e8468661d5@denx.de>","list_archive_url":null,"date":"2017-09-24T20:39:30","subject":"Re: [U-Boot] [PATCH 0/5] Follow the Linux Kernel in building dtc as\n\tneeded","submitter":{"id":70701,"url":"http://patchwork.ozlabs.org/api/people/70701/","name":"Lukasz Majewski","email":"lukma@denx.de"},"content":"Hi Tom,\n\n> On Sun, Sep 24, 2017 at 06:50:01PM +0200, Marek Vasut wrote:\n>> On 09/24/2017 04:26 PM, Tom Rini wrote:\n>>> The following series has been applied. I am posting this for the record.\n>>>\n>>> For the record, I do not believe that using git submodules is a good approach\n>>> here. We have a small amount of code that we need here, and happily we can\n>>> leverage infrastructur e from the Linux Kernel.\n>>>\n>>> Speaking of, this is not the first time we have run into problems deviating\n>>> from the workflow of the kernel. The problems of having large number of\n>>> warnings, or not stemmed from not just leve raging all of the infrastructure\n>>> from the kernel. So related, yes, fixes for these warnings should come in,\n>>> and as always, if they're in the upstream kernel dts as well, they should be\n>>> fixed there.\n>>\n>> So any comments regarding bundling external tools were ignored, even\n>> though the discussion was not finished, great.\n>>\n>> Furthermore, there was zero time to review this series, it was just\n>> applied and posted afterward ? What sort of practice is this ?\n> \n> Yes. I put on my slightly-less-than-benevolent dictator hat today, and\n\nLinus style? :D Management\n\n> explained my reasoning. I do feel bad that you're rather unhappy with\n> the overall situation, but no, I believe this is the right answer.\n> \n> Now, there's some follow up further re-syncing with the kernel that\n> could be done (scripts/Makefile.lib/extrawarn can be further re-matched\n> up with the kernel now, but that would have made a larger delta than\n> \"just migrate to providing the tool\").\n> \n\nBut seriously, do we have some kind of road map for this?\n\nI mean if we have copied the dtc tool into u-boot temporarily (until \ndistros catch up) or will it stay with us forever ?","headers":{"Return-Path":"","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=)","Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3y0fGS6KtVz9s3T\n\tfor ;\n\tMon, 25 Sep 2017 06:39:43 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 01992C220E5; Sun, 24 Sep 2017 20:39:38 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id A284EC220AF;\n\tSun, 24 Sep 2017 20:39:35 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 8F485C220AF; Sun, 24 Sep 2017 20:39:33 +0000 (UTC)","from mail-out.m-online.net (mail-out.m-online.net [212.18.0.9])\n\tby lists.denx.de (Postfix) with ESMTPS id 155AAC21C2B\n\tfor ; Sun, 24 Sep 2017 20:39:33 +0000 (UTC)","from frontend01.mail.m-online.net (unknown [192.168.8.182])\n\tby mail-out.m-online.net (Postfix) with ESMTP id 3y0fGD5fRNz1qqyP;\n\tSun, 24 Sep 2017 22:39:32 +0200 (CEST)","from localhost (dynscan1.mnet-online.de [192.168.6.70])\n\tby mail.m-online.net (Postfix) with ESMTP id 3y0fGD5129z1qqkP;\n\tSun, 24 Sep 2017 22:39:32 +0200 (CEST)","from mail.mnet-online.de ([192.168.8.182])\n\tby localhost (dynscan1.mail.m-online.net [192.168.6.70]) (amavisd-new,\n\tport 10024)\n\twith ESMTP id RZEvzt41m8cR; Sun, 24 Sep 2017 22:39:31 +0200 (CEST)","from [192.168.2.222] (89-64-27-66.dynamic.chello.pl [89.64.27.66])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby mail.mnet-online.de (Postfix) with ESMTPSA;\n\tSun, 24 Sep 2017 22:39:31 +0200 (CEST)"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=-0.7 required=5.0 tests=RCVD_IN_DNSWL_LOW,\n\tRCVD_IN_MSPIKE_H3,\n\tRCVD_IN_MSPIKE_WL autolearn=unavailable autolearn_force=no\n\tversion=3.4.0","X-Virus-Scanned":"amavisd-new at mnet-online.de","X-Auth-Info":"73fPxwXo2EOBwAeu0U5qinA/bNLr3Ul8UvZwUkNpGpA=","To":"Tom Rini , Marek Vasut ","References":"<1506263222-7080-1-git-send-email-trini@konsulko.com>\n\t<61a675e1-a3ae-c8f4-32fd-cf548c4047df@denx.de>\n\t<20170924172845.GV30205@bill-the-cat>","From":"=?utf-8?q?=C5=81ukasz_Majewski?= ","Organization":"DENX","Message-ID":"<7a2909a5-9d5e-3e7d-cdd0-a6e8468661d5@denx.de>","Date":"Sun, 24 Sep 2017 22:39:30 +0200","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.2.1","MIME-Version":"1.0","In-Reply-To":"<20170924172845.GV30205@bill-the-cat>","Content-Language":"en-US","Cc":"Vagrant Cascadian , u-boot@lists.denx.de","Subject":"Re: [U-Boot] [PATCH 0/5] Follow the Linux Kernel in building dtc as\n\tneeded","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Content-Transfer-Encoding":"base64","Content-Type":"text/plain; charset=\"utf-8\"; Format=\"flowed\"","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" "}},{"id":1774312,"web_url":"http://patchwork.ozlabs.org/comment/1774312/","msgid":"<20170924220006.GD29028@bill-the-cat>","list_archive_url":null,"date":"2017-09-24T22:00:06","subject":"Re: [U-Boot] [PATCH 0/5] Follow the Linux Kernel in building dtc as\n\tneeded","submitter":{"id":65875,"url":"http://patchwork.ozlabs.org/api/people/65875/","name":"Tom Rini","email":"trini@konsulko.com"},"content":"On Sun, Sep 24, 2017 at 10:39:30PM +0200, Łukasz Majewski wrote:\n> Hi Tom,\n> \n> >On Sun, Sep 24, 2017 at 06:50:01PM +0200, Marek Vasut wrote:\n> >>On 09/24/2017 04:26 PM, Tom Rini wrote:\n> >>>The following series has been applied. I am posting this for the record.\n> >>>\n> >>>For the record, I do not believe that using git submodules is a good approach\n> >>>here. We have a small amount of code that we need here, and happily we can\n> >>>leverage infrastructur e from the Linux Kernel.\n> >>>\n> >>>Speaking of, this is not the first time we have run into problems deviating\n> >>>from the workflow of the kernel. The problems of having large number of\n> >>>warnings, or not stemmed from not just leve raging all of the infrastructure\n> >>>from the kernel. So related, yes, fixes for these warnings should come in,\n> >>>and as always, if they're in the upstream kernel dts as well, they should be\n> >>>fixed there.\n> >>\n> >>So any comments regarding bundling external tools were ignored, even\n> >>though the discussion was not finished, great.\n> >>\n> >>Furthermore, there was zero time to review this series, it was just\n> >>applied and posted afterward ? What sort of practice is this ?\n> >\n> >Yes. I put on my slightly-less-than-benevolent dictator hat today, and\n> \n> Linus style? :D Management\n\nIt is not my general preference, but yes.\n\n> >explained my reasoning. I do feel bad that you're rather unhappy with\n> >the overall situation, but no, I believe this is the right answer.\n> >\n> >Now, there's some follow up further re-syncing with the kernel that\n> >could be done (scripts/Makefile.lib/extrawarn can be further re-matched\n> >up with the kernel now, but that would have made a larger delta than\n> >\"just migrate to providing the tool\").\n> \n> But seriously, do we have some kind of road map for this?\n> \n> I mean if we have copied the dtc tool into u-boot temporarily (until\n> distros catch up) or will it stay with us forever ?\n\nI think long term we'll match what the kernel does and keep synced with\nupstream, just what we need. I'm thinking I ought to see about pushing\na re-sync in the kernel for the next -rc1 there as I know it has been\nlamented that it hasn't happened in a bit.","headers":{"Return-Path":"","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=)","ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=konsulko.com header.i=@konsulko.com\n\theader.b=\"V3JgqSGW\"; dkim-atps=neutral"],"Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3y0h3R6ZsXz9t3t\n\tfor ;\n\tMon, 25 Sep 2017 08:00:19 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 8E135C2217A; Sun, 24 Sep 2017 22:00:12 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id A57F3C220AF;\n\tSun, 24 Sep 2017 22:00:09 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 21CFCC220AF; Sun, 24 Sep 2017 22:00:08 +0000 (UTC)","from mail-yw0-f177.google.com (mail-yw0-f177.google.com\n\t[209.85.161.177])\n\tby lists.denx.de (Postfix) with ESMTPS id 81712C21C4F\n\tfor ; Sun, 24 Sep 2017 22:00:07 +0000 (UTC)","by mail-yw0-f177.google.com with SMTP id w9so3605371ywi.11\n\tfor ; Sun, 24 Sep 2017 15:00:07 -0700 (PDT)","from bill-the-cat ([2606:a000:1401:811b:9cbd:ca90:bff9:8e97])\n\tby smtp.gmail.com with ESMTPSA id\n\tr29sm2235350ywa.46.2017.09.24.15.00.04\n\t(version=TLS1_2 cipher=AES128-SHA bits=128/128);\n\tSun, 24 Sep 2017 15:00:05 -0700 (PDT)"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=-0.0 required=5.0 tests=RCVD_IN_DNSWL_NONE,\n\tRCVD_IN_MSPIKE_H3, RCVD_IN_MSPIKE_WL,\n\tT_DKIM_INVALID autolearn=unavailable\n\tautolearn_force=no version=3.4.0","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=konsulko.com;\n\ts=google; \n\th=date:from:to:cc:subject:message-id:references:mime-version\n\t:content-disposition:in-reply-to:user-agent;\n\tbh=o9C8p3IgTQVcbRliIkzsJ+5WI9ful7KPyWlb8kW1W2Y=;\n\tb=V3JgqSGWHWRa8oX6TltNpEbS6RNmV5mVUaKJ9Sr++J9cQeci4zjs6Umj70icxBJmy7\n\txJHKRXTR7Z2HFOKOhc22czFeQIywMtXywufZqQSy+F7vbME1Acv4uGH1/frHWw9Qejpk\n\tlRgK/nI4n03RNmEQ3Xit3PL1x4ZWyslAiOAbc=","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:date:from:to:cc:subject:message-id:references\n\t:mime-version:content-disposition:in-reply-to:user-agent;\n\tbh=o9C8p3IgTQVcbRliIkzsJ+5WI9ful7KPyWlb8kW1W2Y=;\n\tb=t8VE5PPzEjTathnMIJWHxLVfPZXFqBcwhr9ERaBC3PdsUZZaV7Ang4fZoKkWYRhdbf\n\teyrj/KGaGpRAK8g4gyvidc6FtZ7RnYn0QA/YjMenfCr1kj4LvkzioS7/ahZS74xcRKLG\n\tNT4rK+ufT80b17kUNc6RHbkBPsYyihCIIsYl6AYteDRkgoU9URxBdAcURNwxaRuiNrcO\n\tg5SAbFxuMTDSxmfspO5jEMGULHtI2aOU+KbTXHVCUxWNmrFNMlV50kElyoHepMAExVV1\n\tYKkp30MgyCVTPpT78kglcfKkdnUe+mxqBmSqWixm3LzSLYijf8oSUJcQNL2y05kz9RTe\n\tKHKA==","X-Gm-Message-State":"AHPjjUhST5ySKmzUTA8gw+uATWJ3Z700DC2CqN7HreDNiPbiSQVhB8VS\n\t+JbbCC9js7P3yn8o0bAPb7yuvA==","X-Google-Smtp-Source":"AOwi7QCuijuBS6E537pYHiMRJdFGWIyNpgzBvuNNFyvurdVHt7L8D3jM4jGc2KIyvHq/ncJHpbOz1g==","X-Received":"by 10.37.75.65 with SMTP id y62mr3365355yba.347.1506290406309;\n\tSun, 24 Sep 2017 15:00:06 -0700 (PDT)","Date":"Sun, 24 Sep 2017 18:00:06 -0400","From":"Tom Rini ","To":"=?utf-8?q?=C5=81ukasz?= Majewski ","Message-ID":"<20170924220006.GD29028@bill-the-cat>","References":"<1506263222-7080-1-git-send-email-trini@konsulko.com>\n\t<61a675e1-a3ae-c8f4-32fd-cf548c4047df@denx.de>\n\t<20170924172845.GV30205@bill-the-cat>\n\t<7a2909a5-9d5e-3e7d-cdd0-a6e8468661d5@denx.de>","MIME-Version":"1.0","In-Reply-To":"<7a2909a5-9d5e-3e7d-cdd0-a6e8468661d5@denx.de>","User-Agent":"Mutt/1.5.21 (2010-09-15)","Cc":"Marek Vasut , Vagrant Cascadian ,\n\tu-boot@lists.denx.de","Subject":"Re: [U-Boot] [PATCH 0/5] Follow the Linux Kernel in building dtc as\n\tneeded","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Content-Type":"multipart/mixed;\n\tboundary=\"===============7106968981169714223==\"","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" "}},{"id":1774511,"web_url":"http://patchwork.ozlabs.org/comment/1774511/","msgid":"<20170925082828.C092312015D@gemini.denx.de>","list_archive_url":null,"date":"2017-09-25T08:28:28","subject":"Re: [U-Boot] [PATCH 0/5] Follow the Linux Kernel in building dtc as\n\tneeded","submitter":{"id":387,"url":"http://patchwork.ozlabs.org/api/people/387/","name":"Wolfgang Denk","email":"wd@denx.de"},"content":"Dear Tom,\n\nIn message <1506263222-7080-1-git-send-email-trini@konsulko.com> you wrote:\n>\n> The following series has been applied. I am posting this for the record.\n\nArgghhh... This is not the way things should be done.\n\nI am definitely unhappy here - not so much about the patches, but\nabout the procedure.\n\nEven as a maintainer you are supposed to post the patches first.\nAnd then _wait_ and give people a chance to review the patches.\nAnd then decide based on the review results.\n\nShortcutting an ongoing discussion with such a \"management decision\"\ndoes not match my understanding of how U-Boot development takes\nplace. Accepting unreviewed patches because of the powers of a\ncustodian/maintainer is IMO a strict no-go.\n\n\nWolfgang Denk","headers":{"Return-Path":"","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=)","Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3y0y0W0Jt5z9t5c\n\tfor ;\n\tMon, 25 Sep 2017 18:28:41 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 16DACC21D78; Mon, 25 Sep 2017 08:28:35 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 3EACEC21C4F;\n\tMon, 25 Sep 2017 08:28:33 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 4AE66C21C4F; Mon, 25 Sep 2017 08:28:32 +0000 (UTC)","from mail-out.m-online.net (mail-out.m-online.net [212.18.0.9])\n\tby lists.denx.de (Postfix) with ESMTPS id E75D6C21C26\n\tfor ; Mon, 25 Sep 2017 08:28:31 +0000 (UTC)","from frontend01.mail.m-online.net (unknown [192.168.8.182])\n\tby mail-out.m-online.net (Postfix) with ESMTP id 3y0y0H5Ctxz1qsDQ\n\tfor ; Mon, 25 Sep 2017 10:28:31 +0200 (CEST)","from localhost (dynscan1.mnet-online.de [192.168.6.70])\n\tby mail.m-online.net (Postfix) with ESMTP id 3y0y0H53Fxz1r0wd\n\tfor ; Mon, 25 Sep 2017 10:28:31 +0200 (CEST)","from mail.mnet-online.de ([192.168.8.182])\n\tby localhost (dynscan1.mail.m-online.net [192.168.6.70]) (amavisd-new,\n\tport 10024) with ESMTP id BT-fZWxdISy8 for ;\n\tMon, 25 Sep 2017 10:28:31 +0200 (CEST)","from janitor.denx.de (unknown [62.91.23.180])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby mail.mnet-online.de (Postfix) with ESMTPSA\n\tfor ; Mon, 25 Sep 2017 10:28:31 +0200 (CEST)","by janitor.denx.de (Postfix, from userid 119)\n\tid 88D7C47B; Mon, 25 Sep 2017 10:28:30 +0200 (CEST)","from gemini.denx.de (gemini.denx.de [10.4.0.2])\n\tby janitor.denx.de (Postfix) with ESMTPS id 1F9153E3;\n\tMon, 25 Sep 2017 10:28:29 +0200 (CEST)","from gemini.denx.de (localhost [IPv6:::1])\n\tby gemini.denx.de (Postfix) with ESMTP id C092312015D;\n\tMon, 25 Sep 2017 10:28:28 +0200 (CEST)"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=-0.7 required=5.0 tests=RCVD_IN_DNSWL_LOW,\n\tRCVD_IN_MSPIKE_H3,\n\tRCVD_IN_MSPIKE_WL autolearn=unavailable autolearn_force=no\n\tversion=3.4.0","X-Virus-Scanned":"amavisd-new at mnet-online.de","X-Auth-Info":"h5iF1vUNO69gcwqIh1zDJmKYHwJKGA6iY3Wm2Je1H7Q=","To":"Tom Rini ","From":"Wolfgang Denk ","MIME-Version":"1.0","In-reply-to":"<1506263222-7080-1-git-send-email-trini@konsulko.com>","References":"<1506263222-7080-1-git-send-email-trini@konsulko.com>","Comments":"In-reply-to Tom Rini \n\tmessage dated \"Sun, 24 Sep 2017 10:26:57 -0400.\"","Date":"Mon, 25 Sep 2017 10:28:28 +0200","Message-Id":"<20170925082828.C092312015D@gemini.denx.de>","Cc":"u-boot@lists.denx.de","Subject":"Re: [U-Boot] [PATCH 0/5] Follow the Linux Kernel in building dtc as\n\tneeded","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" "}},{"id":1774517,"web_url":"http://patchwork.ozlabs.org/comment/1774517/","msgid":"<5384e97f-0095-4a37-8b9f-bd5d82ebd4a3@denx.de>","list_archive_url":null,"date":"2017-09-25T08:32:49","subject":"Re: [U-Boot] [PATCH 0/5] Follow the Linux Kernel in building dtc as\n\tneeded","submitter":{"id":12009,"url":"http://patchwork.ozlabs.org/api/people/12009/","name":"Marek Vasut","email":"marex@denx.de"},"content":"On 09/24/2017 07:28 PM, Tom Rini wrote:\n> On Sun, Sep 24, 2017 at 06:50:01PM +0200, Marek Vasut wrote:\n>> On 09/24/2017 04:26 PM, Tom Rini wrote:\n>>> The following series has been applied. I am posting this for the record.\n>>>\n>>> For the record, I do not believe that using git submodules is a good approach\n>>> here. We have a small amount of code that we need here, and happily we can\n>>> leverage infrastructur e from the Linux Kernel.\n>>>\n>>> Speaking of, this is not the first time we have run into problems deviating\n>>> from the workflow of the kernel. The problems of having large number of\n>>> warnings, or not stemmed from not just leve raging all of the infrastructure\n>>> from the kernel. So related, yes, fixes for these warnings should come in,\n>>> and as always, if they're in the upstream kernel dts as well, they should be\n>>> fixed there.\n>>\n>> So any comments regarding bundling external tools were ignored, even\n>> though the discussion was not finished, great.\n>>\n>> Furthermore, there was zero time to review this series, it was just\n>> applied and posted afterward ? What sort of practice is this ?\n> \n> Yes. I put on my slightly-less-than-benevolent dictator hat today, and\n> explained my reasoning. I do feel bad that you're rather unhappy with\n> the overall situation, but no, I believe this is the right answer.\n\nI believe we have a well-established process of submitting patches,\ngetting a review and then applying them. If the head maintainer doesn't\nfollow the process, why should anyone else ?\n\nI believe you should be an example of doing it right, but what you did\nhere is the exact opposite.\n\nThe discussion how to solve this problem properly was also unfinished,\nyet a decision was taken without any community agreement. What is the\npoint of discussing anything on the ML if the decision gets ignored ?\n\n> Now, there's some follow up further re-syncing with the kernel that\n> could be done (scripts/Makefile.lib/extrawarn can be further re-matched\n> up with the kernel now, but that would have made a larger delta than\n> \"just migrate to providing the tool\").\n>","headers":{"Return-Path":"","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=)","Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3y0y6k52lYz9t62\n\tfor ;\n\tMon, 25 Sep 2017 18:34:06 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 5089BC21F95; Mon, 25 Sep 2017 08:33:38 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 1BD30C21F83;\n\tMon, 25 Sep 2017 08:33:33 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 59305C21E37; Mon, 25 Sep 2017 08:33:27 +0000 (UTC)","from mail-out.m-online.net (mail-out.m-online.net [212.18.0.10])\n\tby lists.denx.de (Postfix) with ESMTPS id 02862C21F85\n\tfor ; Mon, 25 Sep 2017 08:33:24 +0000 (UTC)","from frontend01.mail.m-online.net (unknown [192.168.8.182])\n\tby mail-out.m-online.net (Postfix) with ESMTP id 3y0y5v5lxKz1qshg;\n\tMon, 25 Sep 2017 10:33:23 +0200 (CEST)","from localhost (dynscan1.mnet-online.de [192.168.6.70])\n\tby mail.m-online.net (Postfix) with ESMTP id 3y0y5v54ssz1r0wX;\n\tMon, 25 Sep 2017 10:33:23 +0200 (CEST)","from mail.mnet-online.de ([192.168.8.182])\n\tby localhost (dynscan1.mail.m-online.net [192.168.6.70]) (amavisd-new,\n\tport 10024)\n\twith ESMTP id N_dNA2kodPBj; Mon, 25 Sep 2017 10:33:22 +0200 (CEST)","from [IPv6:::1] (unknown [195.140.253.167])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby mail.mnet-online.de (Postfix) with ESMTPSA;\n\tMon, 25 Sep 2017 10:33:22 +0200 (CEST)"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=-0.7 required=5.0 tests=RCVD_IN_DNSWL_LOW,\n\tRCVD_IN_MSPIKE_H3,\n\tRCVD_IN_MSPIKE_WL autolearn=unavailable autolearn_force=no\n\tversion=3.4.0","X-Virus-Scanned":"amavisd-new at mnet-online.de","X-Auth-Info":"zI8R8hcldWlQaxeCneXN54OoYK7mvoi0Esf0cEN9UsE=","To":"Tom Rini ","References":"<1506263222-7080-1-git-send-email-trini@konsulko.com>\n\t<61a675e1-a3ae-c8f4-32fd-cf548c4047df@denx.de>\n\t<20170924172845.GV30205@bill-the-cat>","From":"Marek Vasut ","Message-ID":"<5384e97f-0095-4a37-8b9f-bd5d82ebd4a3@denx.de>","Date":"Mon, 25 Sep 2017 10:32:49 +0200","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.3.0","MIME-Version":"1.0","In-Reply-To":"<20170924172845.GV30205@bill-the-cat>","Content-Language":"en-US","Cc":"Vagrant Cascadian , u-boot@lists.denx.de","Subject":"Re: [U-Boot] [PATCH 0/5] Follow the Linux Kernel in building dtc as\n\tneeded","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" "}}]