From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-4.4 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 178E8C433ED for ; Fri, 7 May 2021 10:53:43 +0000 (UTC) Received: from desiato.infradead.org (desiato.infradead.org [90.155.92.199]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 8C9B460FD7 for ; Fri, 7 May 2021 10:53:42 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 8C9B460FD7 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=linaro.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=desiato.20200630; h=Sender:Content-Transfer-Encoding :Content-Type:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:Cc:To:Subject:Message-ID:Date:From:In-Reply-To: References:MIME-Version:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=Hu3JPMdYv7/NL/QHuKCf5N+i1/3NwF4oN1hD4ypLvQg=; b=b4HSCAXvsWbDpzin2XQ2QCweC UDbD81eWb3WJzSZ05MXmm+Wzj2UmC4sxzL4i2tRKfsn8w+gbOxEDq7r/hMelH03G7WahvzTQaSzCW V0aDYYefJwhEBb3wuScFQXeyPp9CUvDoqiOq8qWaS9JBz2uf8yL5DqzC5gtmLQXjh2JgmZpLYRhXZ lUV0tsG8ibn/6VGj0j4BFnTzNMAmCcee3KnqocEk9O7hO2itkrC+yK/zJsdYJRCfxN9BdnTQHPPD9 1YNrK+p8etCM0gka2/mhaZjspoTH7Hfq7aGeqdzhPHgs7xwF5QsAbIP+8gXdN/Ha0gUM5sVvLqEyz hgWr17o9g==; Received: from localhost ([::1] helo=desiato.infradead.org) by desiato.infradead.org with esmtp (Exim 4.94 #2 (Red Hat Linux)) id 1ley5Q-006sdN-RP; Fri, 07 May 2021 10:52:01 +0000 Received: from bombadil.infradead.org ([2607:7c80:54:e::133]) by desiato.infradead.org with esmtps (Exim 4.94 #2 (Red Hat Linux)) id 1ley5N-006sdI-92 for linux-arm-kernel@desiato.infradead.org; Fri, 07 May 2021 10:51:57 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=bombadil.20210309; h=Content-Type:Cc:To:Subject:Message-ID :Date:From:In-Reply-To:References:MIME-Version:Sender:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description; bh=F33BGz4iH8dPjpEmF0N7tnuxayq0qldcPL97YnVagyw=; b=NPEXyK0iXs/sIwI4DHoHcs4Y0U QPi1iCluiAUrdM8SKqtem8iu2lSwZdpSWrfrlZszP3J6GlmBz9yXsYLhBPlS/CXQHbzdM8cx4Qiu6 f4YMh3awTILhH6GeJQlyxOT4ytMHqKZiOLZJdO3VONiAuMfhTT2PmYevQ+bqDiTpaUw5zi/GTIC8B 6uWEmCCjLDGvr3B39k8MFZwB3d3Ws21WYkZiyn/pGMNT9OmOGWT1kr9NIt77Gaz3mZYTZmuzLSP80 uLadyrAIk4IpZUUZ4d/Ao7SbL0X9rdCrNVq1O7AhJJhzobLHbrdQ98dAU5DPM0kJ2lDEjePcG8JeM i+JI02EA==; Received: from mail-lf1-x12e.google.com ([2a00:1450:4864:20::12e]) by bombadil.infradead.org with esmtps (Exim 4.94 #2 (Red Hat Linux)) id 1ley5I-006nMc-PR for linux-arm-kernel@lists.infradead.org; Fri, 07 May 2021 10:51:54 +0000 Received: by mail-lf1-x12e.google.com with SMTP id c11so12135126lfi.9 for ; Fri, 07 May 2021 03:51:51 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=F33BGz4iH8dPjpEmF0N7tnuxayq0qldcPL97YnVagyw=; b=RW/8+dN/lpgFSZ0df5MNc6Ki5DdxPUebMDu4kK08MUuv4ZBMgn5O8CE/YrnA80N/8s 45V4fPucN7eRvI+L50Th5xeDl+tyFXdyOnolupEH5Dm3j6KN2E0T/YbhCbFmf3ggqnJc uFkGqxpYaDIbIvmOmaUzIDHY/QVK0BX/qEbFIr3KtWFSt4JTzpdyOKSRFozudL1LIxhv wDULJGZVK9BEkc3y5wVQawO6Dl1IN1sxHz8xk8YjrplLAkzhkUig4p2M79yY8OZNz54X OrSffwbh4RRupTQnpWIAc65q2iK2wIxfXIV45cV7/OW/VgvqrZ0GxYp+tJ+srqSg6UDA tq8g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=F33BGz4iH8dPjpEmF0N7tnuxayq0qldcPL97YnVagyw=; b=EtLoGbl83b7/0pvMj0tmZrNCNOpMUkPiiOKNFQ3wE66kzXV8EcmK+IodbheOnfpYai 2XyrMyj4nsww6hCkLyau55yiR9GhK4zukim8BdBrK+dFyU/jUyN0iBg7mjilCmipOVp+ RC0rjR0Xm6xfn8JnuonixkHybBn5CjwE64ikcZ9QKTWpg5suvuOUVUal74nafbfegkED PkNIuFeXAKU1ExmWS9Tt45rKWXczzkrI/4O8jyI16EuoUG6HygQ2mqywBoCOt9s8CxiM BOnCont+Gl/7S+ZsXTUN7gtOSSqH8ggmyL83IfWmdVySpDw98bB8gvCVZTH7TmW+i43r 3BAA== X-Gm-Message-State: AOAM531CUQTQpqoD8e+bu0NaXAgvuhIm3AuQtpeIdpRSlQL4iYSVjW5b Xzs/pTdT/u0uIs1kf5M8sylaFzQwm/Oo7GEsqGCw3g== X-Google-Smtp-Source: ABdhPJzx7ls/vpMj3eUwiEfOSMkqTyuxko9P8MpdbiNR+6cpzIryOVSHbQTMGPND9R4/vV0DqWX1U8fhYfBfJ4XpTqg= X-Received: by 2002:a19:a418:: with SMTP id q24mr5846648lfc.649.1620384709817; Fri, 07 May 2021 03:51:49 -0700 (PDT) MIME-Version: 1.0 References: <20210503185228.1518131-1-clabbe@baylibre.com> <20210506203435.GA1432800@bjorn-Precision-5520> In-Reply-To: <20210506203435.GA1432800@bjorn-Precision-5520> From: Linus Walleij Date: Fri, 7 May 2021 12:51:39 +0200 Message-ID: Subject: Re: [PATCH 1/2] dt-bindings: pci: convert faraday,ftpci100 to yaml To: Bjorn Helgaas Cc: Corentin Labbe , Bjorn Helgaas , Rob Herring , Hans Ulli Kroll , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , Linux ARM , linux-kernel , linux-pci X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210507_035152_880610_52F7E8D0 X-CRM114-Status: GOOD ( 18.66 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Thu, May 6, 2021 at 10:34 PM Bjorn Helgaas wrote: > I think it's nicer when content changes are in a separate patch from > format conversion patches. Otherwise it's really hard to see the > content changes in the patch. > > Maybe a preliminary patch could fix whatever is actually broken? > > Rob suggested a bunch of things that could be dropped. Maybe those > could be removed in a second preliminary patch before the conversion? > Or maybe the removals are only possible *because* of the conversion? > I'm not a yaml expert. A bit of taste is involved. The old .txt bindings are for processing by human brain power. Those lack regular syntax and strictness because brains are designed for evolved natural languages. The YAML on the other hand is a chomsky type-3 strict regular language and the .yaml file (and includes) defines this strict regular grammar and as such admits less mistakes. The upside is that it enforces some order. In the process of moving to YAML we often discover a slew of mistakes and the initiative often comes with the ambition to add or modernize something. In this case I wouldn't care with stepwise fixing because the platform is modernized by a handful of people who all know what is going on, so there is noone to confuse other than the subsystem maintainer and the result will end up in the same kernel release anyway. Yours, Linus Walleij _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel