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 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 8A3CFC433EF for ; Wed, 24 Nov 2021 13:55:34 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=Hyz1UmZn0UEa+ziB43lMlt0Hw+Q3FGtNDAoci2GzD6c=; b=SFHBYli5izHH63 uKHQBT7TOxH6wsLYwmTjhUKPPhEiEamSUY6CO/k9aoo5+u/dG77H4fpV5o0KouG7SC/1O/dfJ5Inz 15lSVxKqHLJHwA7YRgP/RQQ045XYazUNrTW63nd6RN6dJi8h6cdkswBZHdqKwOBi4ao1zFcArIdC1 XNCIyR/5JEY0x5+lEShNFkt9Vx6PF89OoG9dy1+rw5S7Wyad6G9t9dzuHmP2nvrEa+qpaY9FE96ho GLrQEA5e5iNNUm+ZlyRRb4QBAuQyAdsjoxohSavnpxLditqar8Nm2VyCotgqNclWgrv6l1ftC/9/I j5uGpzynBvhsE4keLGXg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1mpsjV-004wZs-EI; Wed, 24 Nov 2021 13:54:45 +0000 Received: from relay10.mail.gandi.net ([217.70.178.230]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1mpsjR-004wYi-Ov for linux-mtd@lists.infradead.org; Wed, 24 Nov 2021 13:54:44 +0000 Received: (Authenticated sender: miquel.raynal@bootlin.com) by relay10.mail.gandi.net (Postfix) with ESMTPSA id 22E4A24000A; Wed, 24 Nov 2021 13:54:36 +0000 (UTC) Date: Wed, 24 Nov 2021 14:54:35 +0100 From: Miquel Raynal To: Rob Herring Cc: Tudor Ambarus , linux-mtd@lists.infradead.org, Julien Su , linux-spi@vger.kernel.org, Jaime Liao , Thomas Petazzoni , Richard Weinberger , Vignesh Raghavendra , Boris Brezillon , Mark Brown , devicetree@vger.kernel.org, Xiangsheng Hou , linux-kernel@vger.kernel.org Subject: Re: [PATCH 04/18] dt-bindings: mtd: spi-nand: Convert spi-nand description file to yaml Message-ID: <20211124145435.4bd01001@xps13> In-Reply-To: References: <20211020142809.349347-1-miquel.raynal@bootlin.com> <20211020142809.349347-5-miquel.raynal@bootlin.com> <1634764487.038982.2962586.nullmailer@robh.at.kernel.org> <20211021160932.4169f66f@xps13> Organization: Bootlin X-Mailer: Claws Mail 3.17.7 (GTK+ 2.24.32; x86_64-pc-linux-gnu) MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20211124_055442_108357_A3FFE1F3 X-CRM114-Status: GOOD ( 41.70 ) X-BeenThere: linux-mtd@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Sender: "linux-mtd" Errors-To: linux-mtd-bounces+linux-mtd=archiver.kernel.org@lists.infradead.org SGkgUm9iLAoKcm9iaEBrZXJuZWwub3JnIHdyb3RlIG9uIEZyaSwgMjIgT2N0IDIwMjEgMTc6MzY6 MTggLTA1MDA6Cgo+IE9uIFRodSwgT2N0IDIxLCAyMDIxIGF0IDA0OjA5OjMyUE0gKzAyMDAsIE1p cXVlbCBSYXluYWwgd3JvdGU6Cj4gPiBIaSBSb2IsCj4gPiAKPiA+IHJvYmhAa2VybmVsLm9yZyB3 cm90ZSBvbiBXZWQsIDIwIE9jdCAyMDIxIDE2OjE0OjQ3IC0wNTAwOgo+ID4gICAKPiA+ID4gT24g V2VkLCAyMCBPY3QgMjAyMSAxNjoyNzo1NSArMDIwMCwgTWlxdWVsIFJheW5hbCB3cm90ZTogIAo+ ID4gPiA+IExldCdzIGdldCByaWQgb2Ygc3BpLW5hbmQudHh0IGJ5IGNvbnZlcnRpbmcgaXQgdG8g eWFtbCBzY2hlbWEuIFdoaWxlIGF0Cj4gPiA+ID4gY29udmVydGluZyB0aGlzIGZpbGUsIGxldCdz IGFjdHVhbGx5IHB1bGwgYWxsIHRoZSBnZW5lcmljIHByb3BlcnRpZXMKPiA+ID4gPiBmcm9tIG5h bmQtY2hpcC55YW1sIHdoaWNoIG1pZ2h0IGFwcGx5IHRvIGEgU1BJLU5BTkQgY2hpcC4KPiA+ID4g PiAKPiA+ID4gPiBTaWduZWQtb2ZmLWJ5OiBNaXF1ZWwgUmF5bmFsIDxtaXF1ZWwucmF5bmFsQGJv b3RsaW4uY29tPgo+ID4gPiA+IC0tLQo+ID4gPiA+ICAuLi4vZGV2aWNldHJlZS9iaW5kaW5ncy9t dGQvc3BpLW5hbmQudHh0ICAgICAgfCAgNSAtLS0tCj4gPiA+ID4gIC4uLi9kZXZpY2V0cmVlL2Jp bmRpbmdzL210ZC9zcGktbmFuZC55YW1sICAgICB8IDI3ICsrKysrKysrKysrKysrKysrKysKPiA+ ID4gPiAgMiBmaWxlcyBjaGFuZ2VkLCAyNyBpbnNlcnRpb25zKCspLCA1IGRlbGV0aW9ucygtKQo+ ID4gPiA+ICBkZWxldGUgbW9kZSAxMDA2NDQgRG9jdW1lbnRhdGlvbi9kZXZpY2V0cmVlL2JpbmRp bmdzL210ZC9zcGktbmFuZC50eHQKPiA+ID4gPiAgY3JlYXRlIG1vZGUgMTAwNjQ0IERvY3VtZW50 YXRpb24vZGV2aWNldHJlZS9iaW5kaW5ncy9tdGQvc3BpLW5hbmQueWFtbAo+ID4gPiA+ICAgICAK PiA+ID4gCj4gPiA+IE15IGJvdCBmb3VuZCBlcnJvcnMgcnVubmluZyAnbWFrZSBEVF9DSEVDS0VS X0ZMQUdTPS1tIGR0X2JpbmRpbmdfY2hlY2snCj4gPiA+IG9uIHlvdXIgcGF0Y2ggKERUX0NIRUNL RVJfRkxBR1MgaXMgbmV3IGluIHY1LjEzKToKPiA+ID4gCj4gPiA+IHlhbWxsaW50IHdhcm5pbmdz L2Vycm9yczoKPiA+ID4gCj4gPiA+IGR0c2NoZW1hL2R0YyB3YXJuaW5ncy9lcnJvcnM6Cj4gPiA+ IFVua25vd24gZmlsZSByZWZlcmVuY2VkOiBbRXJybm8gMl0gTm8gc3VjaCBmaWxlIG9yIGRpcmVj dG9yeTogJy91c3IvbG9jYWwvbGliL3B5dGhvbjMuOC9kaXN0LXBhY2thZ2VzL2R0c2NoZW1hL3Nj aGVtYXMvbXRkL25hbmQtY2hpcC55YW1sJwo+ID4gPiB4YXJnczogZHQtZG9jLXZhbGlkYXRlOiBl eGl0ZWQgd2l0aCBzdGF0dXMgMjU1OyBhYm9ydGluZwo+ID4gPiBtYWtlWzFdOiAqKiogRGVsZXRp bmcgZmlsZSAnRG9jdW1lbnRhdGlvbi9kZXZpY2V0cmVlL2JpbmRpbmdzL3NwaS9zbnBzLGR3LWFw Yi1zc2kuZXhhbXBsZS5kdC55YW1sJwo+ID4gPiBVbmtub3duIGZpbGUgcmVmZXJlbmNlZDogW0Vy cm5vIDJdIE5vIHN1Y2ggZmlsZSBvciBkaXJlY3Rvcnk6ICcvdXNyL2xvY2FsL2xpYi9weXRob24z LjgvZGlzdC1wYWNrYWdlcy9kdHNjaGVtYS9zY2hlbWFzL210ZC9uYW5kLWNoaXAueWFtbCcKPiA+ ID4gbWFrZVsxXTogKioqIFtzY3JpcHRzL01ha2VmaWxlLmxpYjozODU6IERvY3VtZW50YXRpb24v ZGV2aWNldHJlZS9iaW5kaW5ncy9zcGkvc25wcyxkdy1hcGItc3NpLmV4YW1wbGUuZHQueWFtbF0g RXJyb3IgMjU1Cj4gPiA+IG1ha2VbMV06ICoqKiBXYWl0aW5nIGZvciB1bmZpbmlzaGVkIGpvYnMu Li4uCj4gPiA+IG1ha2U6ICoqKiBbTWFrZWZpbGU6MTQ0MTogZHRfYmluZGluZ19jaGVja10gRXJy b3IgMiAgCj4gPiAKPiA+IEkgYW0gbm90IGFibGUgdG8gcmVwcm9kdWNlIHRoaXMgZXJyb3IgYW5k IGluIGdlbmVyYWwgSSBkb24ndCB1bmRlcnN0YW5kCj4gPiBpdC4gVGhlcmUgaXMgbm8gcmVsYXRp b25zaGlwIGJldHdlZW4gdGhpcyBjaGFuZ2UgYW5kCj4gPiBzbnBzLGR3LWFwYi1zc2kueWFtbC4g QWxzbyB0aGUgZmFjdCB0aGF0IG5hbmQtY2hpcC15YW1sIGRvIG5vdCBleGlzdCwKPiA+IGl0IHdh cyBqdXN0IGNyZWF0ZWQgaW4gdGhlIHBhdGNoIGJlZm9yZSBzbyBJIHdvbmRlciBob3cgbXVjaCBJ IHNob3VsZAo+ID4gdHJ1c3QgdGhpcyBlcnJvci4gIAo+IAo+IEkgdGhpbmsgeW91IGNhbiBpZ25v cmUgdGhpcy4gVGhlIHByaW9yIHBhdGNoIHNob3VsZCBoYXZlIGJlZW4gYXBwbGllZCwgCj4gYnV0 IGxvb2tzIGxpa2UgaXQgd2Fzbid0LiBNeSBzY3JpcHQncyBwYXRjaCBhcHBseWluZyBsb2dpYyBp cyBub3Qgd2hhdCAKPiBJJ2QgY2FsbCByb2J1c3QuCgpPaywgSSB1bmRlcnN0YW5kIHdoYXQncyBo YXBwZW5pbmcgYmVoaW5kIHRoZSBzY2VuZXMsIG5vIHByb2JsZW0gd2l0aAp0aGlzLgoKPiAKPiA+ IEFsc28sIG1heWJlIEkgYW0gbm90IHVzaW5nIHRoZSB0b29scyBwcm9wZXJseSwgYnV0IGl0IGlz IHZlcnkgaGFyZCB0bwo+ID4gc2VuZCBjb3JyZWN0IGJpbmRpbmdzIGF0IHRoZSBmaXJzdCB0cnku IFJ1bm5pbmcgbWFrZSBkdF9iaW5kaW5nX2NoZWNrCj4gPiB0YWtlcyBhZ2VzLCBhbnkgY2hhbmdl IGluIG9uZSB5YW1sIGZpbGUgd2lsbCByZWNoZWNrIHRoZSBlbnRpcmUgZGF0YQo+ID4gYmFzZSBh bmQgZmlsdGVyaW5nIG91dCBvbiBhIHNpbmdsZSB5YW1sIGZpbGUgaXMgZ2VuZXJhbGx5IHRvbwo+ ID4gcmVzdHJpY3RpdmUgYW5kIHN0aWxsIHByaW50cyB1bnJlbGF0ZWQgZXJyb3JzIG9mIHN5bnRh eCBvbiBvdGhlciBmaWxlcy4gIAo+IAo+IERvIHlvdSBzZXQgJ0RUX1NDSEVNQV9GSUxFUyc/IFRo YXQgd2lsbCBjaGVjayBqdXN0IHRoZSBzY2hlbWEgeW91IHNldCAKPiBpdCB0by4gWW91IHN0aWxs IG5lZWQgdG8gbm90IHNldCBpdCBhdCB0aGUgZW5kIGJlY2F1c2UgYW55IHNjaGVtYSBjb3VsZCAK PiBhcHBseSB0byBhbnkgZXhhbXBsZSwgc28gd2UgaGF2ZSB0byBjaGVjayBldmVyeXRoaW5nLgoK QWN0dWFsbHkgSSB0aGluayB0aGlzIGlzIHdoYXQgY291bGQgYmUgZW5oYW5jZWQ6IHdoZW4gSSB1 c2UKRFRfU0NIRU1BX0ZJTEVTIHRoZSBvdXRwdXQgaXMgYWx3YXlzIHBvbGx1dGVkIHdpdGggZXJy b3JzIHdpdGggbm90aGluZwooYXQgbGVhc3QgZnJvbSBteSB1c2VyIHBvaW50IG9mIHZpZXcpIHJl bGF0ZWQgdG8gdGhlIGZpbGVzIEkgYW0Kd29ya2luZyBvbi4gU2VlIFsxXSBmb3IgYW4gZXhhbXBs ZSBvZiBvdXRwdXQgdGhhdCBJIGZvdW5kIGhhcmQgdG8gcGFyc2UKZm9yIGVycm9ycyByZWxhdGVk IHRvIG15IGNoYW5nZXM6IEkgYW0gbG9va2luZyBmb3IgaXNzdWVzIGluCm5hbmQtY2hpcC55YW1s IGFuZCAoNTAgc2Vjb25kcyBsYXRlcikgSSBnZXQgd2FybmluZ3MgZm9yOgotIGlsaXRlayxpbGk5 MzQxLnlhbWwKLSBzbnBzLGR3Y21zaGMtc2RoY2kueWFtbAotIHRpLHNuNjVkc2k4My55YW1sCi0g bnhwLGR3bWFjLWlteC55YW1sCi0gZnNsLGlteDZxLXBjaWUueWFtbAoKRG8geW91IHRoaW5rIHRo ZSByZXBvcnRpbmcgY2FuIGJlIGltcHJvdmVkPwoKPiBBbHNvIHVzaW5nIERUX1NDSEVNQV9GSUxF UyBzaG91bGQgYmUgYSBiaXQgZmFzdGVyIHdpdGggd2hhdCdzIHF1ZXVlZCBmb3IgCj4gNS4xNi4K CkdyZWF0IQoKPiAKPiA+IEkgZG9uJ3Qga25vdyBob3cgbXVjaCBvZiB0aGlzIGlzIGFjdHVhbGx5 IGV4cGVjdGVkIGFuZC9vciBpZiBzb21lb25lIGlzCj4gPiB3b3JraW5nIG9uIGl0LiAgCj4gCj4g RHVlIHRvIHB5dGhvbiBzdGFydHVwIHRpbWVzIGJlaW5nIHNsb3csIGl0IHR1cm5zIG91dCB0byBn ZW5lcmFsbHkgYmUgCj4gZmFzdGVyIHRvIG5vdCBoYXZlIG1ha2UgdHJhY2sgY2hhbmdlcyBhbmQg ZG8gdGhpbmdzIGluY3JlbWVudGFsbHkuIAo+IFRoYXQncyB3aHkgYWxsIHRoZSBzY2hlbWEgYXJl IGNoZWNrZWQgYXQgb25jZSAodGhvdWdoIHNoYXJkZWQgd2l0aCAKPiB4YXJncykuIFNvIEknbSBu b3QgcmVhbGx5IHN1cmUgdGhlcmUncyBtdWNoIHdlIGNhbiBkby4gSSd2ZSBjZXJ0YWlubHkgCj4g aW52ZXN0aWdhdGVkIGl0LgoKSSB1bmRlcnN0YW5kIGl0J3Mgbm90IHRoYXQgZWFzeSBhbmQgdGFr ZXMgYSBsb3Qgb2YgdGltZSwgdGhhbmtzIGFueXdheQpmb3IgYWxsIHlvdXIgd29yayBvbiB0aGlz IHRvcGljLgoKVGhhbmtzLApNaXF1w6hsCgpbMV0KJCB0aW1lIG1ha2UgZHRfYmluZGluZ19jaGVj ayBEVF9TQ0hFTUFfRklMRVM9RG9jdW1lbnRhdGlvbi9kZXZpY2V0cmVlL2JpbmRpbmdzL210ZC9u YW5kLWNoaXAueWFtbAogIExJTlQgICAgRG9jdW1lbnRhdGlvbi9kZXZpY2V0cmVlL2JpbmRpbmdz Ci4vRG9jdW1lbnRhdGlvbi9kZXZpY2V0cmVlL2JpbmRpbmdzL2Rpc3BsYXkvcGFuZWwvaWxpdGVr LGlsaTkzNDEueWFtbDoyNTo5OiBbd2FybmluZ10gd3JvbmcgaW5kZW50YXRpb246IGV4cGVjdGVk IDEwIGJ1dCBmb3VuZCA4IChpbmRlbnRhdGlvbikKICBDSEtEVCAgIERvY3VtZW50YXRpb24vZGV2 aWNldHJlZS9iaW5kaW5ncy9wcm9jZXNzZWQtc2NoZW1hLWV4YW1wbGVzLmpzb24KL2hvbWUvbXJh eW5hbC9tYWNyb25peC9saW51eC9Eb2N1bWVudGF0aW9uL2RldmljZXRyZWUvYmluZGluZ3MvcGNp LzogcHJvcGVydGllczppbnRlcnJ1cHQtbmFtZXM6ICdvbmVPZicgY29uZGl0aW9uYWwgZmFpbGVk LCBvbmUgbXVzdCBiZSBmaXhlZDoKCVt7J2NvbnN0JzogJ21zaSd9XSBpcyB0b28gc2hvcnQKCUZh bHNlIHNjaGVtYSBkb2VzIG5vdCBhbGxvdyAxCgloaW50OiAibWluSXRlbXMiIGlzIG9ubHkgbmVl ZGVkIGlmIGxlc3MgdGhhbiB0aGUgIml0ZW1zIiBsaXN0IGxlbmd0aAoJZnJvbSBzY2hlbWEgJGlk OiBodHRwOi8vZGV2aWNldHJlZS5vcmcvbWV0YS1zY2hlbWFzL2l0ZW1zLnlhbWwjCi9ob21lL21y YXluYWwvbWFjcm9uaXgvbGludXgvRG9jdW1lbnRhdGlvbi9kZXZpY2V0cmVlL2JpbmRpbmdzL21t Yy9zbnBzLGR3Y21zaGMtc2RoY2kueWFtbDogcHJvcGVydGllczpyZWc6ICdvbmVPZicgY29uZGl0 aW9uYWwgZmFpbGVkLCBvbmUgbXVzdCBiZSBmaXhlZDoKCVt7J2Rlc2NyaXB0aW9uJzogJ09mZnNl dCBhbmQgbGVuZ3RoIG9mIHRoZSByZWdpc3RlciBzZXQgZm9yIHRoZSBkZXZpY2UnfV0gaXMgdG9v IHNob3J0CglGYWxzZSBzY2hlbWEgZG9lcyBub3QgYWxsb3cgMQoJaGludDogIm1pbkl0ZW1zIiBp cyBvbmx5IG5lZWRlZCBpZiBsZXNzIHRoYW4gdGhlICJpdGVtcyIgbGlzdCBsZW5ndGgKCWZyb20g c2NoZW1hICRpZDogaHR0cDovL2RldmljZXRyZWUub3JnL21ldGEtc2NoZW1hcy9pdGVtcy55YW1s IwovaG9tZS9tcmF5bmFsL21hY3Jvbml4L2xpbnV4L0RvY3VtZW50YXRpb24vZGV2aWNldHJlZS9i aW5kaW5ncy9kaXNwbGF5L2JyaWRnZS86IHByb3BlcnRpZXM6cG9ydHM6cHJvcGVydGllczpwb3J0 QDA6cHJvcGVydGllczplbmRwb2ludDpwcm9wZXJ0aWVzOmRhdGEtbGFuZXM6IHsncmVxdWlyZWQn OiBbJ21heEl0ZW1zJ119IGlzIG5vdCBhbGxvd2VkIGZvciB7J2Rlc2NyaXB0aW9uJzogJ2FycmF5 IG9mIHBoeXNpY2FsIERTSSBkYXRhIGxhbmUgaW5kZXhlcy4nLCAnbWluSXRlbXMnOiAxLCAnbWF4 SXRlbXMnOiA0LCAnaXRlbXMnOiBbeydjb25zdCc6IDF9LCB7J2NvbnN0JzogMn0sIHsnY29uc3Qn OiAzfSwgeydjb25zdCc6IDR9XX0KCWhpbnQ6ICJtYXhJdGVtcyIgaXMgbm90IG5lZWRlZCB3aXRo IGFuICJpdGVtcyIgbGlzdAoJZnJvbSBzY2hlbWEgJGlkOiBodHRwOi8vZGV2aWNldHJlZS5vcmcv bWV0YS1zY2hlbWFzL2l0ZW1zLnlhbWwjCi9ob21lL21yYXluYWwvbWFjcm9uaXgvbGludXgvRG9j dW1lbnRhdGlvbi9kZXZpY2V0cmVlL2JpbmRpbmdzL2Rpc3BsYXkvYnJpZGdlL3RpLHNuNjVkc2k4 My55YW1sOiBwcm9wZXJ0aWVzOnBvcnRzOnByb3BlcnRpZXM6cG9ydEAxOnByb3BlcnRpZXM6ZW5k cG9pbnQ6cHJvcGVydGllczpkYXRhLWxhbmVzOiB7J3JlcXVpcmVkJzogWydtYXhJdGVtcyddfSBp cyBub3QgYWxsb3dlZCBmb3IgeydkZXNjcmlwdGlvbic6ICdhcnJheSBvZiBwaHlzaWNhbCBEU0kg ZGF0YSBsYW5lIGluZGV4ZXMuJywgJ21pbkl0ZW1zJzogMSwgJ21heEl0ZW1zJzogNCwgJ2l0ZW1z JzogW3snY29uc3QnOiAxfSwgeydjb25zdCc6IDJ9LCB7J2NvbnN0JzogM30sIHsnY29uc3QnOiA0 fV19CgloaW50OiAibWF4SXRlbXMiIGlzIG5vdCBuZWVkZWQgd2l0aCBhbiAiaXRlbXMiIGxpc3QK CWZyb20gc2NoZW1hICRpZDogaHR0cDovL2RldmljZXRyZWUub3JnL21ldGEtc2NoZW1hcy9pdGVt cy55YW1sIwovaG9tZS9tcmF5bmFsL21hY3Jvbml4L2xpbnV4L0RvY3VtZW50YXRpb24vZGV2aWNl dHJlZS9iaW5kaW5ncy9uZXQvbnhwLGR3bWFjLWlteC55YW1sOiBwcm9wZXJ0aWVzOmNsb2Nrczog eydyZXF1aXJlZCc6IFsnbWF4SXRlbXMnXX0gaXMgbm90IGFsbG93ZWQgZm9yIHsnbWluSXRlbXMn OiAzLCAnbWF4SXRlbXMnOiA1LCAnaXRlbXMnOiBbeydkZXNjcmlwdGlvbic6ICdNQUMgaG9zdCBj bG9jayd9LCB7J2Rlc2NyaXB0aW9uJzogJ01BQyBhcGIgY2xvY2snfSwgeydkZXNjcmlwdGlvbic6 ICdNQUMgdGltZXIgY2xvY2snfSwgeydkZXNjcmlwdGlvbic6ICdNQUMgUkdNSUkgVFggY2xvY2sn fSwgeydkZXNjcmlwdGlvbic6ICdFUU9TIE1FTSBjbG9jayd9XX0KCWhpbnQ6ICJtYXhJdGVtcyIg aXMgbm90IG5lZWRlZCB3aXRoIGFuICJpdGVtcyIgbGlzdAoJZnJvbSBzY2hlbWEgJGlkOiBodHRw Oi8vZGV2aWNldHJlZS5vcmcvbWV0YS1zY2hlbWFzL2l0ZW1zLnlhbWwjCiAgU0NIRU1BICBEb2N1 bWVudGF0aW9uL2RldmljZXRyZWUvYmluZGluZ3MvcHJvY2Vzc2VkLXNjaGVtYS1leGFtcGxlcy5q c29uCi9ob21lL21yYXluYWwvbWFjcm9uaXgvbGludXgvRG9jdW1lbnRhdGlvbi9kZXZpY2V0cmVl L2JpbmRpbmdzL3BjaS9mc2wsaW14NnEtcGNpZS55YW1sOiBpZ25vcmluZywgZXJyb3IgaW4gc2No ZW1hOiBwcm9wZXJ0aWVzOiBpbnRlcnJ1cHQtbmFtZXMKd2FybmluZzogbm8gc2NoZW1hIGZvdW5k IGluIGZpbGU6IC4vRG9jdW1lbnRhdGlvbi9kZXZpY2V0cmVlL2JpbmRpbmdzL3BjaS9mc2wsaW14 NnEtcGNpZS55YW1sCi9ob21lL21yYXluYWwvbWFjcm9uaXgvbGludXgvRG9jdW1lbnRhdGlvbi9k ZXZpY2V0cmVlL2JpbmRpbmdzL21tYy9zbnBzLGR3Y21zaGMtc2RoY2kueWFtbDogaWdub3Jpbmcs IGVycm9yIGluIHNjaGVtYTogcHJvcGVydGllczogcmVnCndhcm5pbmc6IG5vIHNjaGVtYSBmb3Vu ZCBpbiBmaWxlOiAuL0RvY3VtZW50YXRpb24vZGV2aWNldHJlZS9iaW5kaW5ncy9tbWMvc25wcyxk d2Ntc2hjLXNkaGNpLnlhbWwKL2hvbWUvbXJheW5hbC9tYWNyb25peC9saW51eC9Eb2N1bWVudGF0 aW9uL2RldmljZXRyZWUvYmluZGluZ3MvZGlzcGxheS9icmlkZ2UvdGksc242NWRzaTgzLnlhbWw6 IGlnbm9yaW5nLCBlcnJvciBpbiBzY2hlbWE6IHByb3BlcnRpZXM6IHBvcnRzOiBwcm9wZXJ0aWVz OiBwb3J0QDA6IHByb3BlcnRpZXM6IGVuZHBvaW50OiBwcm9wZXJ0aWVzOiBkYXRhLWxhbmVzCndh cm5pbmc6IG5vIHNjaGVtYSBmb3VuZCBpbiBmaWxlOiAuL0RvY3VtZW50YXRpb24vZGV2aWNldHJl ZS9iaW5kaW5ncy9kaXNwbGF5L2JyaWRnZS90aSxzbjY1ZHNpODMueWFtbAovaG9tZS9tcmF5bmFs L21hY3Jvbml4L2xpbnV4L0RvY3VtZW50YXRpb24vZGV2aWNldHJlZS9iaW5kaW5ncy9uZXQvbnhw LGR3bWFjLWlteC55YW1sOiBpZ25vcmluZywgZXJyb3IgaW4gc2NoZW1hOiBwcm9wZXJ0aWVzOiBj bG9ja3MKd2FybmluZzogbm8gc2NoZW1hIGZvdW5kIGluIGZpbGU6IC4vRG9jdW1lbnRhdGlvbi9k ZXZpY2V0cmVlL2JpbmRpbmdzL25ldC9ueHAsZHdtYWMtaW14LnlhbWwKICBEVEVYICAgIERvY3Vt ZW50YXRpb24vZGV2aWNldHJlZS9iaW5kaW5ncy9tdGQvbmFuZC1jaGlwLmV4YW1wbGUuZHRzCiAg RFRDICAgICBEb2N1bWVudGF0aW9uL2RldmljZXRyZWUvYmluZGluZ3MvbXRkL25hbmQtY2hpcC5l eGFtcGxlLmR0LnlhbWwKICBDSEVDSyAgIERvY3VtZW50YXRpb24vZGV2aWNldHJlZS9iaW5kaW5n cy9tdGQvbmFuZC1jaGlwLmV4YW1wbGUuZHQueWFtbAoKcmVhbAkwbTM4LDk1OHMKdXNlcgkwbTUx LDYwNHMKc3lzCTBtMCw2MzJzCgpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX18KTGludXggTVREIGRpc2N1c3Npb24gbWFpbGluZyBsaXN0Cmh0dHA6 Ly9saXN0cy5pbmZyYWRlYWQub3JnL21haWxtYW4vbGlzdGluZm8vbGludXgtbXRkLwo= 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 Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id B3E8DC433FE for ; Wed, 24 Nov 2021 13:56:44 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1353286AbhKXN7w convert rfc822-to-8bit (ORCPT ); Wed, 24 Nov 2021 08:59:52 -0500 Received: from relay10.mail.gandi.net ([217.70.178.230]:37749 "EHLO relay10.mail.gandi.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1352984AbhKXN5v (ORCPT ); Wed, 24 Nov 2021 08:57:51 -0500 Received: (Authenticated sender: miquel.raynal@bootlin.com) by relay10.mail.gandi.net (Postfix) with ESMTPSA id 22E4A24000A; Wed, 24 Nov 2021 13:54:36 +0000 (UTC) Date: Wed, 24 Nov 2021 14:54:35 +0100 From: Miquel Raynal To: Rob Herring Cc: Tudor Ambarus , linux-mtd@lists.infradead.org, Julien Su , linux-spi@vger.kernel.org, Jaime Liao , Thomas Petazzoni , Richard Weinberger , Vignesh Raghavendra , Boris Brezillon , Mark Brown , devicetree@vger.kernel.org, Xiangsheng Hou , linux-kernel@vger.kernel.org Subject: Re: [PATCH 04/18] dt-bindings: mtd: spi-nand: Convert spi-nand description file to yaml Message-ID: <20211124145435.4bd01001@xps13> In-Reply-To: References: <20211020142809.349347-1-miquel.raynal@bootlin.com> <20211020142809.349347-5-miquel.raynal@bootlin.com> <1634764487.038982.2962586.nullmailer@robh.at.kernel.org> <20211021160932.4169f66f@xps13> Organization: Bootlin X-Mailer: Claws Mail 3.17.7 (GTK+ 2.24.32; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8BIT Precedence: bulk List-ID: X-Mailing-List: linux-spi@vger.kernel.org Hi Rob, robh@kernel.org wrote on Fri, 22 Oct 2021 17:36:18 -0500: > On Thu, Oct 21, 2021 at 04:09:32PM +0200, Miquel Raynal wrote: > > Hi Rob, > > > > robh@kernel.org wrote on Wed, 20 Oct 2021 16:14:47 -0500: > > > > > On Wed, 20 Oct 2021 16:27:55 +0200, Miquel Raynal wrote: > > > > Let's get rid of spi-nand.txt by converting it to yaml schema. While at > > > > converting this file, let's actually pull all the generic properties > > > > from nand-chip.yaml which might apply to a SPI-NAND chip. > > > > > > > > Signed-off-by: Miquel Raynal > > > > --- > > > > .../devicetree/bindings/mtd/spi-nand.txt | 5 ---- > > > > .../devicetree/bindings/mtd/spi-nand.yaml | 27 +++++++++++++++++++ > > > > 2 files changed, 27 insertions(+), 5 deletions(-) > > > > delete mode 100644 Documentation/devicetree/bindings/mtd/spi-nand.txt > > > > create mode 100644 Documentation/devicetree/bindings/mtd/spi-nand.yaml > > > > > > > > > > My bot found errors running 'make DT_CHECKER_FLAGS=-m dt_binding_check' > > > on your patch (DT_CHECKER_FLAGS is new in v5.13): > > > > > > yamllint warnings/errors: > > > > > > dtschema/dtc warnings/errors: > > > Unknown file referenced: [Errno 2] No such file or directory: '/usr/local/lib/python3.8/dist-packages/dtschema/schemas/mtd/nand-chip.yaml' > > > xargs: dt-doc-validate: exited with status 255; aborting > > > make[1]: *** Deleting file 'Documentation/devicetree/bindings/spi/snps,dw-apb-ssi.example.dt.yaml' > > > Unknown file referenced: [Errno 2] No such file or directory: '/usr/local/lib/python3.8/dist-packages/dtschema/schemas/mtd/nand-chip.yaml' > > > make[1]: *** [scripts/Makefile.lib:385: Documentation/devicetree/bindings/spi/snps,dw-apb-ssi.example.dt.yaml] Error 255 > > > make[1]: *** Waiting for unfinished jobs.... > > > make: *** [Makefile:1441: dt_binding_check] Error 2 > > > > I am not able to reproduce this error and in general I don't understand > > it. There is no relationship between this change and > > snps,dw-apb-ssi.yaml. Also the fact that nand-chip-yaml do not exist, > > it was just created in the patch before so I wonder how much I should > > trust this error. > > I think you can ignore this. The prior patch should have been applied, > but looks like it wasn't. My script's patch applying logic is not what > I'd call robust. Ok, I understand what's happening behind the scenes, no problem with this. > > > Also, maybe I am not using the tools properly, but it is very hard to > > send correct bindings at the first try. Running make dt_binding_check > > takes ages, any change in one yaml file will recheck the entire data > > base and filtering out on a single yaml file is generally too > > restrictive and still prints unrelated errors of syntax on other files. > > Do you set 'DT_SCHEMA_FILES'? That will check just the schema you set > it to. You still need to not set it at the end because any schema could > apply to any example, so we have to check everything. Actually I think this is what could be enhanced: when I use DT_SCHEMA_FILES the output is always polluted with errors with nothing (at least from my user point of view) related to the files I am working on. See [1] for an example of output that I found hard to parse for errors related to my changes: I am looking for issues in nand-chip.yaml and (50 seconds later) I get warnings for: - ilitek,ili9341.yaml - snps,dwcmshc-sdhci.yaml - ti,sn65dsi83.yaml - nxp,dwmac-imx.yaml - fsl,imx6q-pcie.yaml Do you think the reporting can be improved? > Also using DT_SCHEMA_FILES should be a bit faster with what's queued for > 5.16. Great! > > > I don't know how much of this is actually expected and/or if someone is > > working on it. > > Due to python startup times being slow, it turns out to generally be > faster to not have make track changes and do things incrementally. > That's why all the schema are checked at once (though sharded with > xargs). So I'm not really sure there's much we can do. I've certainly > investigated it. I understand it's not that easy and takes a lot of time, thanks anyway for all your work on this topic. Thanks, Miquèl [1] $ time make dt_binding_check DT_SCHEMA_FILES=Documentation/devicetree/bindings/mtd/nand-chip.yaml LINT Documentation/devicetree/bindings ./Documentation/devicetree/bindings/display/panel/ilitek,ili9341.yaml:25:9: [warning] wrong indentation: expected 10 but found 8 (indentation) CHKDT Documentation/devicetree/bindings/processed-schema-examples.json /home/mraynal/macronix/linux/Documentation/devicetree/bindings/pci/: properties:interrupt-names: 'oneOf' conditional failed, one must be fixed: [{'const': 'msi'}] is too short False schema does not allow 1 hint: "minItems" is only needed if less than the "items" list length from schema $id: http://devicetree.org/meta-schemas/items.yaml# /home/mraynal/macronix/linux/Documentation/devicetree/bindings/mmc/snps,dwcmshc-sdhci.yaml: properties:reg: 'oneOf' conditional failed, one must be fixed: [{'description': 'Offset and length of the register set for the device'}] is too short False schema does not allow 1 hint: "minItems" is only needed if less than the "items" list length from schema $id: http://devicetree.org/meta-schemas/items.yaml# /home/mraynal/macronix/linux/Documentation/devicetree/bindings/display/bridge/: properties:ports:properties:port@0:properties:endpoint:properties:data-lanes: {'required': ['maxItems']} is not allowed for {'description': 'array of physical DSI data lane indexes.', 'minItems': 1, 'maxItems': 4, 'items': [{'const': 1}, {'const': 2}, {'const': 3}, {'const': 4}]} hint: "maxItems" is not needed with an "items" list from schema $id: http://devicetree.org/meta-schemas/items.yaml# /home/mraynal/macronix/linux/Documentation/devicetree/bindings/display/bridge/ti,sn65dsi83.yaml: properties:ports:properties:port@1:properties:endpoint:properties:data-lanes: {'required': ['maxItems']} is not allowed for {'description': 'array of physical DSI data lane indexes.', 'minItems': 1, 'maxItems': 4, 'items': [{'const': 1}, {'const': 2}, {'const': 3}, {'const': 4}]} hint: "maxItems" is not needed with an "items" list from schema $id: http://devicetree.org/meta-schemas/items.yaml# /home/mraynal/macronix/linux/Documentation/devicetree/bindings/net/nxp,dwmac-imx.yaml: properties:clocks: {'required': ['maxItems']} is not allowed for {'minItems': 3, 'maxItems': 5, 'items': [{'description': 'MAC host clock'}, {'description': 'MAC apb clock'}, {'description': 'MAC timer clock'}, {'description': 'MAC RGMII TX clock'}, {'description': 'EQOS MEM clock'}]} hint: "maxItems" is not needed with an "items" list from schema $id: http://devicetree.org/meta-schemas/items.yaml# SCHEMA Documentation/devicetree/bindings/processed-schema-examples.json /home/mraynal/macronix/linux/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.yaml: ignoring, error in schema: properties: interrupt-names warning: no schema found in file: ./Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.yaml /home/mraynal/macronix/linux/Documentation/devicetree/bindings/mmc/snps,dwcmshc-sdhci.yaml: ignoring, error in schema: properties: reg warning: no schema found in file: ./Documentation/devicetree/bindings/mmc/snps,dwcmshc-sdhci.yaml /home/mraynal/macronix/linux/Documentation/devicetree/bindings/display/bridge/ti,sn65dsi83.yaml: ignoring, error in schema: properties: ports: properties: port@0: properties: endpoint: properties: data-lanes warning: no schema found in file: ./Documentation/devicetree/bindings/display/bridge/ti,sn65dsi83.yaml /home/mraynal/macronix/linux/Documentation/devicetree/bindings/net/nxp,dwmac-imx.yaml: ignoring, error in schema: properties: clocks warning: no schema found in file: ./Documentation/devicetree/bindings/net/nxp,dwmac-imx.yaml DTEX Documentation/devicetree/bindings/mtd/nand-chip.example.dts DTC Documentation/devicetree/bindings/mtd/nand-chip.example.dt.yaml CHECK Documentation/devicetree/bindings/mtd/nand-chip.example.dt.yaml real 0m38,958s user 0m51,604s sys 0m0,632s