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 22BAEC433EF for ; Mon, 16 May 2022 18:45:35 +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:In-Reply-To:MIME-Version:References: 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=lYfsbrNpgqh7kajRIV6P46rqT6263D8+dKa+U+4oC1g=; b=cMI2GLOqb9p0dp b34SnUYeduK2CPLj0QU1Od30s+sYhff/EKEOnD6LNvGcquX8pzHpprstpK03dvoPehMEe9CyFKSrw SBtEMXmc3T+F15E141MZNvEmapOjCzolsGYSV+fcRmdAtGI/eRmt0GesD0rxID6uzJrEf1KDdcA9Q 3OTuI+7wgaeruGvwQYKqrT+VcFrwBcEdMOJFouW5W05nleDCk6X7ukufbPtqGRDYVpJp0fRWveMof 70zHeiBhk5APIo2PaCQCUFsvAyzJxME9ril2yTkv8bgQAquuJTWs0z0Ftucn+LOPaaDTQIbguQ8Fp OY91/oNdp7RpOP3JIPtA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nqfiK-009dHN-Dd; Mon, 16 May 2022 18:45:04 +0000 Received: from mail-oi1-f173.google.com ([209.85.167.173]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1nqfiH-009dFi-2m for linux-mtd@lists.infradead.org; Mon, 16 May 2022 18:45:02 +0000 Received: by mail-oi1-f173.google.com with SMTP id e189so19691814oia.8 for ; Mon, 16 May 2022 11:44:59 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:content-transfer-encoding :in-reply-to; bh=UrqxUsCa56QU8+WchoN4MQg/QDcbGW/ieWtcKgIAWuY=; b=vVdCsk9hhhoSLC6Glqx2XOlF+hbkBQnW7kv/5ip/t4JZY//m4VfQRqK8+XFqzocGtL QDvY4ShiCyCfzBWJ4vsN7WYEXXeCtf78dqDOnPT6eMkTRK9hNRh143X2CmqsR2bsozaG qwy+3WHLYY1MSCy1g9ft7oDSq1PZhnqhqOKzrSdzrY9R3MtEhz9sC+nDZseOWrfcSW96 3ZvfSg5z6Z3ix8iwZ3H2Ihut9ku0A8Idh9gZJmxBXYZTYR16UoMXbnSkkGRMVwc1WbWP 22Bn5P30VbrfZTBgemLlER5elxbF8vTEfHImBRNKmghduKqQ+++MVLX3GvpgfLecFjF4 2XPQ== X-Gm-Message-State: AOAM5329tKt6epo3zkgPXXrzHESFqDaDgeFyxF+zbB0ZKKCq2f3JN/Jc TQROhL74Lye8g9GWZUhHWdvU2Sk0Ag== X-Google-Smtp-Source: ABdhPJzB1IgiGUY4GU8rBg46ZKFXuYfGday76Ep1taIi6V8o57el7+uGbXzh4JULTTeVbYvl9KmlwA== X-Received: by 2002:a05:6808:aa4:b0:325:91ff:43cc with SMTP id r4-20020a0568080aa400b0032591ff43ccmr8631919oij.166.1652726698693; Mon, 16 May 2022 11:44:58 -0700 (PDT) Received: from robh.at.kernel.org (66-90-144-107.dyn.grandenetworks.net. [66.90.144.107]) by smtp.gmail.com with ESMTPSA id y22-20020a4a6256000000b0035eb4e5a6cfsm4370484oog.37.2022.05.16.11.44.57 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 16 May 2022 11:44:57 -0700 (PDT) Received: (nullmailer pid 3062095 invoked by uid 1000); Mon, 16 May 2022 18:44:57 -0000 Date: Mon, 16 May 2022 13:44:57 -0500 From: Rob Herring To: =?utf-8?B?UmFmYcWCIE1pxYJlY2tp?= Cc: Ansuel Smith , Miquel Raynal , Richard Weinberger , Vignesh Raghavendra , Krzysztof Kozlowski , Manivannan Sadhasivam , linux-mtd@lists.infradead.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [RESEND PATCH v3 1/2] dt-bindings: mtd: partitions: Document new partition-dynamic nodes Message-ID: <20220516184457.GA3052710-robh@kernel.org> References: <20220429124825.21477-1-ansuelsmth@gmail.com> <20220429124825.21477-2-ansuelsmth@gmail.com> <6272e7a7.1c69fb81.dae8f.70aa@mx.google.com> <373bbbb1-a24f-32c6-1020-49c41ceda832@gmail.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <373bbbb1-a24f-32c6-1020-49c41ceda832@gmail.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220516_114501_168728_BA292060 X-CRM114-Status: GOOD ( 50.25 ) 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 T24gV2VkLCBNYXkgMDQsIDIwMjIgYXQgMTA6NTk6NDRQTSArMDIwMCwgUmFmYcWCIE1pxYJlY2tp IHdyb3RlOgo+IE9uIDQuMDUuMjAyMiAyMjo1MiwgQW5zdWVsIFNtaXRoIHdyb3RlOgo+ID4gT24g V2VkLCBNYXkgMDQsIDIwMjIgYXQgMTA6Mzk6MTRQTSArMDIwMCwgUmFmYcWCIE1pxYJlY2tpIHdy b3RlOgo+ID4gPiBPbiAyOS4wNC4yMDIyIDE0OjQ4LCBBbnN1ZWwgU21pdGggd3JvdGU6Cj4gPiA+ ID4gRG9jdW1lbnQgbmV3IHBhcnRpdGlvbi1keW5hbWljIG5vZGVzIHVzZWQgdG8gcHJvdmlkZSBh biBPRiBub2RlIGZvcgo+ID4gPiA+IHBhcnRpdGlvbiByZWdpc3RyZWQgYXQgcnVudGltZSBieSBw YXJzZXJzLiBUaGlzIGlzIHJlcXVpcmVkIGZvciBudm1lbQo+ID4gPiA+IHN5c3RlbSB0byBkZWNs YXJlIGFuZCBkZXRlY3QgbnZtZW0tY2VsbHMuCj4gPiA+ID4gCj4gPiA+ID4gV2l0aCB0aGVzZSBz cGVjaWFsIHBhcnRpdGlvbnMsIHRoZSByZWcgLyBvZmZzZXQgaXMgbm90IHJlcXVpcmVkLgo+ID4g PiA+IFRoZSBub2RlIG5hbWUgbXVzdCBiZSBpbiB0aGUgZm9ybSBvZiAicGFydGl0aW9uIG5hbWUi LWR5bmFtaWMuCj4gPiA+ID4gSWYgdGhlIHBhcnRpdGlvbiBjYW4ndCBiZSBkaXNwbGF5ZWQgdXNp bmcgdGhlIG5vZGUgbmFtZSwgaXQncyBwb3NzaWJsZQo+ID4gPiA+IHRvIHVzZSB0aGUgbGFiZWwg YmluZGluZyB0aGF0IHdpbGwgYmUgdXNlZCBpbnN0ZWFkIG9mIHRoZSBub2RlIG5hbWUuCj4gPiA+ ID4gVGhlIG5vZGUgbmFtZSBvciB0aGUgbGFiZWwgYmluZGluZyBpcyB1c2VkIHRvIG1hdGNoIHRo ZSBwYXJ0aXRpb24KPiA+ID4gPiBhbGxvY2F0ZWQgYnkgdGhlIHBhcnNlciBhdCBydW50aW1lIGFu ZCB0aGUgcGFyc2VyIHdpbGwgcHJvdmlkZSByZWcKPiA+ID4gPiBhbmQgb2Zmc2V0IG9mIHRoZSBt dGQuCj4gPiA+ID4gCj4gPiA+ID4gTlZNRU0gd2lsbCB1c2UgdGhlIGRhdGEgZnJvbSB0aGUgcGFy c2VyIGFuZCBwcm92aWRlIHRoZSBOVk1FTSBjZWxscwo+ID4gPiA+IGRlY2xhcmVkIGluIHRoZSBE VFMsICJjb25uZWN0aW5nIiB0aGUgZHluYW1pYyBwYXJ0aXRpb24gd2l0aCBhCj4gPiA+ID4gc3Rh dGljIGRlY2xhcmF0aW9uIG9mIGNlbGxzIGluIHRoZW0uCj4gPiA+ID4gCj4gPiA+ID4gU2lnbmVk LW9mZi1ieTogQW5zdWVsIFNtaXRoIDxhbnN1ZWxzbXRoQGdtYWlsLmNvbT4KPiA+ID4gPiAtLS0K PiA+ID4gPiAgICAuLi4vbXRkL3BhcnRpdGlvbnMvcGFydGl0aW9uLWR5bmFtaWMueWFtbCAgICAg fCA1NiArKysrKysrKysrKysrKysrKysrCj4gPiA+ID4gICAgLi4uL210ZC9wYXJ0aXRpb25zL3Fj b20sc21lbS1wYXJ0LnlhbWwgICAgICAgIHwgIDQgKysKPiA+ID4gPiAgICAyIGZpbGVzIGNoYW5n ZWQsIDYwIGluc2VydGlvbnMoKykKPiA+ID4gPiAgICBjcmVhdGUgbW9kZSAxMDA2NDQgRG9jdW1l bnRhdGlvbi9kZXZpY2V0cmVlL2JpbmRpbmdzL210ZC9wYXJ0aXRpb25zL3BhcnRpdGlvbi1keW5h bWljLnlhbWwKPiA+ID4gPiAKPiA+ID4gPiBkaWZmIC0tZ2l0IGEvRG9jdW1lbnRhdGlvbi9kZXZp Y2V0cmVlL2JpbmRpbmdzL210ZC9wYXJ0aXRpb25zL3BhcnRpdGlvbi1keW5hbWljLnlhbWwgYi9E b2N1bWVudGF0aW9uL2RldmljZXRyZWUvYmluZGluZ3MvbXRkL3BhcnRpdGlvbnMvcGFydGl0aW9u LWR5bmFtaWMueWFtbAo+ID4gPiA+IG5ldyBmaWxlIG1vZGUgMTAwNjQ0Cj4gPiA+ID4gaW5kZXgg MDAwMDAwMDAwMDAwLi5lMGVmYTU4ZTRmYWMKPiA+ID4gPiAtLS0gL2Rldi9udWxsCj4gPiA+ID4g KysrIGIvRG9jdW1lbnRhdGlvbi9kZXZpY2V0cmVlL2JpbmRpbmdzL210ZC9wYXJ0aXRpb25zL3Bh cnRpdGlvbi1keW5hbWljLnlhbWwKPiA+ID4gPiBAQCAtMCwwICsxLDU2IEBACj4gPiA+ID4gKyMg U1BEWC1MaWNlbnNlLUlkZW50aWZpZXI6IEdQTC0yLjAtb25seSBPUiBCU0QtMi1DbGF1c2UKPiA+ ID4gPiArJVlBTUwgMS4yCj4gPiA+ID4gKy0tLQo+ID4gPiA+ICskaWQ6IGh0dHA6Ly9kZXZpY2V0 cmVlLm9yZy9zY2hlbWFzL210ZC9wYXJ0aXRpb25zL3BhcnRpdGlvbi1keW5hbWljLnlhbWwjCj4g PiA+ID4gKyRzY2hlbWE6IGh0dHA6Ly9kZXZpY2V0cmVlLm9yZy9tZXRhLXNjaGVtYXMvY29yZS55 YW1sIwo+ID4gPiA+ICsKPiA+ID4gPiArdGl0bGU6IER5bmFtaWMgUGFydGl0aW9uCj4gPiA+IAo+ ID4gPiBJJ20gbm90IG5hdGl2ZSBidXQgdGhhdCAiRHluYW1pYyBQYXJ0aXRpb24iIHNvdW5kcyBw cmV0dHkgbmF0dXJhbCBhbmQKPiA+ID4gSSdtIHdvbmRlcmluZyBpZiB5b3Ugc2hvdWxkbid0IG1h a2UgdGhhdCBiaW5kaW5nIGR5bmFtaWMtcGFydGl0aW9uLnlhbWwKPiA+ID4gCj4gPiA+IEFueSBu YXRpdmVzIHRvIGNvbW1lbnQgb24gdGhpcz8gOikKPiA+ID4gCj4gPiA+IAo+ID4gCj4gPiBUaGUg bmFtaW5nIGZvciB0aGUgZmlsZSBpcyB1c2VkIHRvIGtlZXAgdGhlIHN0YW5kYXJkIG9mCj4gPiBb cGFyc2VyXS1wYXJ0aXRpb24ueWFtbC4gQWdyZWUgdGhhdCB3ZSBzaG91bGQgZmluZCBhIGJldHRl ciBuYW1pbmcgZm9yCj4gPiBhbGwgb2YgdGhpcy4KPiAKPiBTb21lb25lIGhhcyB0byBicmVhayB0 aGUgcGF0dGVybiBhbmQgc3RhcnQgbmFtaW5nIHRoaW5ncyBzYW5lIDspCj4gCj4gCj4gPiA+ID4g K2Rlc2NyaXB0aW9uOiB8Cj4gPiA+ID4gKyAgVGhpcyBiaW5kaW5nIGRlc2NyaWJlcyBhIHNpbmds ZSBmbGFzaCBwYXJ0aXRpb24gdGhhdCBpcyBkeW5hbWljYWxseSBhbGxvY2F0ZWQKPiA+ID4gPiAr ICBieSBhIGRlZGljYXRlZCBwYXJzZXIgdGhhdCBpcyBub3QgYSBmaXhlZC1wYXJ0aXRpb24gcGFy c2VyLgo+ID4gPiA+ICsKPiA+ID4gPiArICBBIGR5bmFtaWMgcGFydGl0aW9uIHJlcXVpcmUgdGhl IG5vZGUgZW5kaW5nIHdpdGggdGhlICItZHluYW1pYyIgdGFnIGFuZCBpZiB0aGUKPiA+ID4gPiAr ICBkeW5hbWljIHBhcnRpdGlvbiBuYW1lIGNhbid0IGJlIGRpc3BsYXllZCB1c2luZyB0aGUgbm9k ZSBuYW1lLCB0aGUgbGFiZWwKPiA+ID4gPiArICBwcm9wZXJ0aWVzIGNhbiBiZSB1c2VkLiBUaGUg bm9kZSBuYW1lIG9yIHRoZSBsYWJlbCBoYXZlIHRvIG1hdGNoIHRoZSBkeW5hbWljCj4gPiA+ID4g KyAgcGFydGl0aW9uIGFsbG9jYXRlZCBieSB0aGUgcGFyc2VyLgo+ID4gPiA+ICsKPiA+ID4gPiAr ICBUaGVzZSBzcGVjaWFsIHBhcnRpdGlvbiBkZWZpbml0aW9uIGNhbiBiZSB1c2VkIHRvIGdpdmUg YSBkeW5hbWljIHBhcnRpdGlvbgo+ID4gPiA+ICsgIGFuIE9GIG5vZGUgdG8gZGVjbGFyZSBOVk1F TSBjZWxscy4gQW4gZXhhbXBsZSBpcyBkZWNsYXJpbmcgdGhlIHBhcnRpdGlvbgo+ID4gPiA+ICsg IGxhYmVsIGFuZCBhbGwgdGhlIE5WTUVNIGNlbGxzIGluIGl0LiBUaGUgcGFyc2VyIHdpbGwgZGV0 ZWN0IHRoZSBjb3JyZWN0IHJlZwo+ID4gPiA+ICsgIGFuZCBvZmZzZXQgYW5kIHRoZSBOVk1FTSB3 aWxsIHJlZ2lzdGVyIHRoZSBjZWxscyBpbiBpdCBiYXNlZCBvbiB0aGUgZGF0YQo+ID4gPiA+ICsg IGV4dHJhY3RlZCBieSB0aGUgcGFyc2VyLgo+ID4gPiA+ICsKPiA+ID4gPiArbWFpbnRhaW5lcnM6 Cj4gPiA+ID4gKyAgLSBBbnN1ZWwgU21pdGggPGFuc3VlbHNtdGhAZ21haWwuY29tPgo+ID4gPiA+ ICsKPiA+ID4gPiArcHJvcGVydGllczoKPiA+ID4gPiArICBsYWJlbDoKPiA+ID4gPiArICAgIGRl c2NyaXB0aW9uOiBUaGUgbGFiZWwgLyBuYW1lIGZvciB0aGUgcGFydGl0aW9uIGFzc2lnbmVkIGJ5 IHRoZSBwYXJzZXIgYXQKPiA+ID4gPiArICAgICAgcnVudGltZS4gVGhpcyBpcyBuZWVkZWQgZm9y IHN5YnN5c3RlbSBsaWtlIE5WTUVNIHRvIGRlZmluZSBjZWxscyBhbmQKPiA+ID4gPiArICAgICAg cmVnaXN0ZXIgd2l0aCB0aGlzIE9GIG5vZGUuCj4gPiA+ID4gKwo+ID4gPiA+ICthZGRpdGlvbmFs UHJvcGVydGllczogdHJ1ZQo+ID4gPiA+ICsKPiA+ID4gPiArZXhhbXBsZXM6Cj4gPiA+ID4gKyAg LSB8Cj4gPiA+ID4gKyAgICBmbGFzaCB7Cj4gPiA+ID4gKyAgICAgIHBhcnRpdGlvbnMgewo+ID4g PiA+ICsgICAgICAgIGNvbXBhdGlibGUgPSAicWNvbSxzbWVtLXBhcnQiOwo+ID4gPiA+ICsKPiA+ ID4gPiArICAgICAgICBhcnQtZHluYW1pYyB7Cj4gPiA+ID4gKyAgICAgICAgICBjb21wYXRpYmxl ID0gIm52bWVtLWNlbGxzIjsKPiA+ID4gPiArICAgICAgICAgICNhZGRyZXNzLWNlbGxzID0gPDE+ Owo+ID4gPiA+ICsgICAgICAgICAgI3NpemUtY2VsbHMgPSA8MT47Cj4gPiA+ID4gKyAgICAgICAg ICBsYWJlbCA9ICIwOmFydCI7Cj4gPiA+ID4gKwo+ID4gPiA+ICsgICAgICAgICAgbWFjYWRkcl9h cnRfMDogbWFjYWRkckAwIHsKPiA+ID4gPiArICAgICAgICAgICAgcmVnID0gPDB4MCAweDY+Owo+ ID4gPiA+ICsgICAgICAgICAgfTsKPiA+ID4gPiArCj4gPiA+ID4gKyAgICAgICAgICBtYWNhZGRy X2FydF82OiBtYWNhZGRyQDYgewo+ID4gPiA+ICsgICAgICAgICAgICByZWcgPSA8MHg2IDB4Nj47 Cj4gPiA+ID4gKyAgICAgICAgICB9Owo+ID4gPiA+ICsgICAgICAgIH07Cj4gPiA+ID4gKyAgICAg IH07Cj4gPiA+ID4gKyAgICB9Owo+ID4gPiAKPiA+ID4gSSBzZWUgdGhhdCB3ZSBuZWVkIGEgcHJv cGVydHkgKGxpa2UgImxhYmVsIikgZm9yIHN0b3JpbmcgcGFydGl0aW9uIG5hbWUKPiA+ID4gYXMg aXQgbWF5IGNvbnRhaW4gY2hhcmFjdGVycyBub3QgYWxsb3dlZCBpbiAkbm9kZW5hbWUuCj4gPiA+ IAo+ID4gPiBJcyB0aGVyZSBhIHJlYXNvbiB0byBwbGF5IHdpdGggYWxsIHRoYXQgZm9vLWR5bmFt aWMgJG5vZGVuYW1lIHRoZW4/IFdpdGgKPiA+ID4gZmFsbGJhY2sgZnJvbSAibGFiZWwiIHRvIGV4 dHJhY3RpbmcgZm9vIGZyb20gKi1keW5hbWljIHBhdHRlcm4/Cj4gPiA+IAo+ID4gCj4gPiBIb25l c3RseSB0aGUgIi1keW5hbWljIiB0aGluZyBpcyB0byBjb3JyZWN0bHkgaGFuZGxlIHRoaXMgIiJz dHJhbmdlIiIKPiA+IERvY3VtZW50YXRpb24uIEF0IHRpbWVzIHVzaW5nIHRoZSBwYXR0ZXJuIGNh dXNlZCB0b25zIG9mIHByb2JsZW1zIHdpdGgKPiA+IHBhdHRlcm4gc28gSSBoYWQgdGhpcyBicmln aHQgaWRlYSBvZiB1c2luZyB0aGUgc3VmZml4ICItZHluYW1pYyIgdG8KPiA+IGNsZWFyeSBkaWZm ZXJlbnRpYXRlIHRoZXNlIHNwZWNpYWwgcGFydGl0aW9uIGZyb20gZml4ZWQgb25lLgo+ID4gCj4g PiA+IENvdWxkIHdlIGp1c3QgYmUgbGF6eSwga2VlcCB0aGluZ3Mgc2ltcGxlIGFuZCByZXF1aXJl ICJsYWJlbCIgcHJvcGVydHk/Cj4gPiA+IAo+ID4gCj4gPiBUaGlzIGlzIHByb2JsZW1hdGljIHRv IGNvcnJlY3RseSBhc3NpZ24gYSBwYXR0ZXJuUHJvcGVydGllcyB0byBhbnkgdXNlcgo+ID4gb3Ig dGhpcyBwYXJzZXIuCj4gPiAKPiA+ID4gVGhlbiB3ZSBjb3VsZCBlLmcuIHJlcXVpcmUgJG5vZGVu YW1lIHRvIGJlIHBhdHRlcm4gXnBhcnRpdGlvbi1bMC05YS1mXSskCj4gPiA+IEl0J3Mgd2hhdCBs ZWRzLWdwaW8ueWFtbCBkb2VzIGZvciByZWZlcmVuY2UuCj4gPiA+IAo+ID4gCj4gPiBNaGhoIG9r IEkgY2FuIHRvdGFsbHkgbWFrZSB0aGlzIGNoYW5nZS4gTXkgY29uY2VybiBpcyB0aGF0IHNvbWVv bmUgd291bGQKPiA+IGdldCBjb25mdXNlZCB0aGlua2luZyB0aGV5IGFyZSBmaXhlZCBwYXJ0aXRp b24gZGVjbGFyZWQgb24gdG9wIG9mIHRoZQo+ID4gcGFyc2VyLiBCdXQgeWhlYSB0aGlzIGNhbiBh bHNvIHdvcmsuLi4gSXQncyByZWFsbHkgYSBzaW1pbGFyCj4gPiBpbXBsZW1lbnRhdGlvbiBvZiB3 aGF0IEkgYWxyZWFkeSB0byB3aXRoIGR5bmFtaWMuIElmIHlvdSB3YW50IEkgY2FuIGRvCj4gPiB0 aGlzIGNoYW5nZSBhbmQgc2VuZCBhIHY0Lgo+IAo+IFdlIGNhbiB3YWl0IGZvciBtb3JlIHJldmll d3MsIGluIGNhc2Ugb3RoZXIgZGV2ZWxvcGVycyB0aGluayBteSBpZGVhCj4gbWF5IGJlIGNvbmZ1 c2luZy4gRm9yIG1lIHNlZWluZyBwYXJ0aXRpb24gd2l0aG91dCBhZGRyZXNzIGlzIGVhc3kgdG8K PiBpbnRlcnByZXQgYXMgZHluYW1pYyBwYXJ0aXRpb24gYnV0IGxldCdzIHNlZSB3aGF0IG90aGVy cyB0aGluay4KClRoYXQgc291bmRzIGdvb2QgdG8gbWUuCgpSb2IKCl9fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpMaW51eCBNVEQgZGlzY3Vzc2lv biBtYWlsaW5nIGxpc3QKaHR0cDovL2xpc3RzLmluZnJhZGVhZC5vcmcvbWFpbG1hbi9saXN0aW5m by9saW51eC1tdGQvCg== 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 BAE8EC433EF for ; Mon, 16 May 2022 18:45:01 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232738AbiEPSpB (ORCPT ); Mon, 16 May 2022 14:45:01 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:58808 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231972AbiEPSpA (ORCPT ); Mon, 16 May 2022 14:45:00 -0400 Received: from mail-oi1-f175.google.com (mail-oi1-f175.google.com [209.85.167.175]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 817BE3EA87; Mon, 16 May 2022 11:44:59 -0700 (PDT) Received: by mail-oi1-f175.google.com with SMTP id i66so19662503oia.11; Mon, 16 May 2022 11:44:59 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:content-transfer-encoding :in-reply-to; bh=UrqxUsCa56QU8+WchoN4MQg/QDcbGW/ieWtcKgIAWuY=; b=rWIu5JFJSRPHR3yb/4HfTwd90R52nRStsUvz+PougGkTG04AaV81O7MBWy4hInXmk7 +hBAC3wGOqorI3wxW4KSydp1yiM9GJsKe8N6Pg3rPEmOagU/qkP4XWImxT8AWr3F0SVM se7R0CHRV0AUW61YNR80VkrA+A8Wop/1J4pumBk+j+22tMLUqQId0yIMqbQIeD68lbcE RuDSp5ywHK7G6MvA7JJBslHKsYlDWon2JLzKjhxlHI03LU97HXQ5B3aVj9tkwtkuB4UR 3SClq9SPzowhfNEXrb2mrZo6qRrshdu4Oc/BemSytIsWslf4IvSULKU2Sh9rXkXWH06r Mpmg== X-Gm-Message-State: AOAM5331qesk17uDuSZt9ERuG1AhP60GMi5vOBzdduh70SH6alpKrZSk nxa2X3jGT7W8Z5sg0fqgjA== X-Google-Smtp-Source: ABdhPJzB1IgiGUY4GU8rBg46ZKFXuYfGday76Ep1taIi6V8o57el7+uGbXzh4JULTTeVbYvl9KmlwA== X-Received: by 2002:a05:6808:aa4:b0:325:91ff:43cc with SMTP id r4-20020a0568080aa400b0032591ff43ccmr8631919oij.166.1652726698693; Mon, 16 May 2022 11:44:58 -0700 (PDT) Received: from robh.at.kernel.org (66-90-144-107.dyn.grandenetworks.net. [66.90.144.107]) by smtp.gmail.com with ESMTPSA id y22-20020a4a6256000000b0035eb4e5a6cfsm4370484oog.37.2022.05.16.11.44.57 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 16 May 2022 11:44:57 -0700 (PDT) Received: (nullmailer pid 3062095 invoked by uid 1000); Mon, 16 May 2022 18:44:57 -0000 Date: Mon, 16 May 2022 13:44:57 -0500 From: Rob Herring To: =?utf-8?B?UmFmYcWCIE1pxYJlY2tp?= Cc: Ansuel Smith , Miquel Raynal , Richard Weinberger , Vignesh Raghavendra , Krzysztof Kozlowski , Manivannan Sadhasivam , linux-mtd@lists.infradead.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [RESEND PATCH v3 1/2] dt-bindings: mtd: partitions: Document new partition-dynamic nodes Message-ID: <20220516184457.GA3052710-robh@kernel.org> References: <20220429124825.21477-1-ansuelsmth@gmail.com> <20220429124825.21477-2-ansuelsmth@gmail.com> <6272e7a7.1c69fb81.dae8f.70aa@mx.google.com> <373bbbb1-a24f-32c6-1020-49c41ceda832@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <373bbbb1-a24f-32c6-1020-49c41ceda832@gmail.com> Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org On Wed, May 04, 2022 at 10:59:44PM +0200, Rafał Miłecki wrote: > On 4.05.2022 22:52, Ansuel Smith wrote: > > On Wed, May 04, 2022 at 10:39:14PM +0200, Rafał Miłecki wrote: > > > On 29.04.2022 14:48, Ansuel Smith wrote: > > > > Document new partition-dynamic nodes used to provide an OF node for > > > > partition registred at runtime by parsers. This is required for nvmem > > > > system to declare and detect nvmem-cells. > > > > > > > > With these special partitions, the reg / offset is not required. > > > > The node name must be in the form of "partition name"-dynamic. > > > > If the partition can't be displayed using the node name, it's possible > > > > to use the label binding that will be used instead of the node name. > > > > The node name or the label binding is used to match the partition > > > > allocated by the parser at runtime and the parser will provide reg > > > > and offset of the mtd. > > > > > > > > NVMEM will use the data from the parser and provide the NVMEM cells > > > > declared in the DTS, "connecting" the dynamic partition with a > > > > static declaration of cells in them. > > > > > > > > Signed-off-by: Ansuel Smith > > > > --- > > > > .../mtd/partitions/partition-dynamic.yaml | 56 +++++++++++++++++++ > > > > .../mtd/partitions/qcom,smem-part.yaml | 4 ++ > > > > 2 files changed, 60 insertions(+) > > > > create mode 100644 Documentation/devicetree/bindings/mtd/partitions/partition-dynamic.yaml > > > > > > > > diff --git a/Documentation/devicetree/bindings/mtd/partitions/partition-dynamic.yaml b/Documentation/devicetree/bindings/mtd/partitions/partition-dynamic.yaml > > > > new file mode 100644 > > > > index 000000000000..e0efa58e4fac > > > > --- /dev/null > > > > +++ b/Documentation/devicetree/bindings/mtd/partitions/partition-dynamic.yaml > > > > @@ -0,0 +1,56 @@ > > > > +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause > > > > +%YAML 1.2 > > > > +--- > > > > +$id: http://devicetree.org/schemas/mtd/partitions/partition-dynamic.yaml# > > > > +$schema: http://devicetree.org/meta-schemas/core.yaml# > > > > + > > > > +title: Dynamic Partition > > > > > > I'm not native but that "Dynamic Partition" sounds pretty natural and > > > I'm wondering if you shouldn't make that binding dynamic-partition.yaml > > > > > > Any natives to comment on this? :) > > > > > > > > > > The naming for the file is used to keep the standard of > > [parser]-partition.yaml. Agree that we should find a better naming for > > all of this. > > Someone has to break the pattern and start naming things sane ;) > > > > > > +description: | > > > > + This binding describes a single flash partition that is dynamically allocated > > > > + by a dedicated parser that is not a fixed-partition parser. > > > > + > > > > + A dynamic partition require the node ending with the "-dynamic" tag and if the > > > > + dynamic partition name can't be displayed using the node name, the label > > > > + properties can be used. The node name or the label have to match the dynamic > > > > + partition allocated by the parser. > > > > + > > > > + These special partition definition can be used to give a dynamic partition > > > > + an OF node to declare NVMEM cells. An example is declaring the partition > > > > + label and all the NVMEM cells in it. The parser will detect the correct reg > > > > + and offset and the NVMEM will register the cells in it based on the data > > > > + extracted by the parser. > > > > + > > > > +maintainers: > > > > + - Ansuel Smith > > > > + > > > > +properties: > > > > + label: > > > > + description: The label / name for the partition assigned by the parser at > > > > + runtime. This is needed for sybsystem like NVMEM to define cells and > > > > + register with this OF node. > > > > + > > > > +additionalProperties: true > > > > + > > > > +examples: > > > > + - | > > > > + flash { > > > > + partitions { > > > > + compatible = "qcom,smem-part"; > > > > + > > > > + art-dynamic { > > > > + compatible = "nvmem-cells"; > > > > + #address-cells = <1>; > > > > + #size-cells = <1>; > > > > + label = "0:art"; > > > > + > > > > + macaddr_art_0: macaddr@0 { > > > > + reg = <0x0 0x6>; > > > > + }; > > > > + > > > > + macaddr_art_6: macaddr@6 { > > > > + reg = <0x6 0x6>; > > > > + }; > > > > + }; > > > > + }; > > > > + }; > > > > > > I see that we need a property (like "label") for storing partition name > > > as it may contain characters not allowed in $nodename. > > > > > > Is there a reason to play with all that foo-dynamic $nodename then? With > > > fallback from "label" to extracting foo from *-dynamic pattern? > > > > > > > Honestly the "-dynamic" thing is to correctly handle this ""strange"" > > Documentation. At times using the pattern caused tons of problems with > > pattern so I had this bright idea of using the suffix "-dynamic" to > > cleary differentiate these special partition from fixed one. > > > > > Could we just be lazy, keep things simple and require "label" property? > > > > > > > This is problematic to correctly assign a patternProperties to any user > > or this parser. > > > > > Then we could e.g. require $nodename to be pattern ^partition-[0-9a-f]+$ > > > It's what leds-gpio.yaml does for reference. > > > > > > > Mhhh ok I can totally make this change. My concern is that someone would > > get confused thinking they are fixed partition declared on top of the > > parser. But yhea this can also work... It's really a similar > > implementation of what I already to with dynamic. If you want I can do > > this change and send a v4. > > We can wait for more reviews, in case other developers think my idea > may be confusing. For me seeing partition without address is easy to > interpret as dynamic partition but let's see what others think. That sounds good to me. Rob