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 966A8C25B6B for ; Wed, 25 Oct 2023 08:11:44 +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=n7PFLkurfsDiWX1FtUpo3eJdZyRIkgDhYWQeq/DXo0k=; b=ds5/7dr8u4RL4N 0cgPyNIc0ZbwFn+wz27DJr4rYzy1SeERnGQgf1xy/oO658+hJJ87q2lq63AaLJi320+qRNdJ2w6r7 y3jgkvRWfGghlFnouh/32j8l2h6Avm6DMbWHutZPzKxW9KrlQHEH8yAMkE+jhLHsKugaWkvKa6BoA rSDUeOpvagNLG5gqjHdeWIQQM9mdwBYfl3/S53YJ07sw6qx8VuIANEIMnY9+MdkgMsj1AItuN5tu+ wrQVuwJNF776OE56hsuGXODb4bqXDuiozUX/f/b+xUWL1fP6iHoZJFLx0senmnBvilYAgAdOYF5ey KzyHV/gogzrxsuhFEM9w==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qvYzI-00BfXV-2e; Wed, 25 Oct 2023 08:11:36 +0000 Received: from relay4-d.mail.gandi.net ([217.70.183.196]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qvYzF-00BfVa-0J for linux-mtd@lists.infradead.org; Wed, 25 Oct 2023 08:11:35 +0000 Received: by mail.gandi.net (Postfix) with ESMTPSA id B0E7FE000C; Wed, 25 Oct 2023 08:11:27 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1698221488; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=kMsuCLZa5HO4/dTq8bW3lK1ng6FHaXYOgfnq2P4eOQg=; b=WpaHPssdmaUYYCP6ILGX0jXGmmE/YE/hq7pPHyHUfKycPqwyvpeThDlzSTmxf5Ou/U8LPK 6xbQM3cFgvQAUxwbh2R7OUMPXF0KMmrFbJgjg/8LuhWiKG5TTjx4S8caNs0H2LU64Easwg fJ2/fvW0VucZ+04ktbrfh7kF0CrEni45GoGImZyTX/RsZrcAshVZntfXs08QoYAil3WXyv uSjUYnShdesomb97EqP4C2SacSYMGjg2BpmOPGqwlcIgY8pZUaOJ71hemiQihaRoWPscmr SK1xm9WNW35kipLA05wVJSf+d6y3gMGJudeEhF6oLcu9FPgRwn0NXJuDG9o9VQ== Date: Wed, 25 Oct 2023 10:11:26 +0200 From: Miquel Raynal To: Simon Glass Cc: Rob Herring , devicetree@vger.kernel.org, linux-mtd@lists.infradead.org, Michael Walle , U-Boot Mailing List , Tom Rini , Conor Dooley , Krzysztof Kozlowski , Richard Weinberger , Vignesh Raghavendra , linux-kernel@vger.kernel.org Subject: Re: [PATCH v4 2/3] dt-bindings: mtd: binman-partition: Add binman compatibles Message-ID: <20231025101126.2234a74c@xps-13> In-Reply-To: References: <20231009220436.2164245-1-sjg@chromium.org> <20231009220436.2164245-2-sjg@chromium.org> <20231024161644.GB3707756-robh@kernel.org> Organization: Bootlin X-Mailer: Claws Mail 4.0.0 (GTK+ 3.24.33; x86_64-pc-linux-gnu) MIME-Version: 1.0 X-GND-Sasl: miquel.raynal@bootlin.com X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20231025_011133_466086_70738054 X-CRM114-Status: GOOD ( 45.84 ) 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 SGkgU2ltb24sCgpzamdAY2hyb21pdW0ub3JnIHdyb3RlIG9uIFR1ZSwgMjQgT2N0IDIwMjMgMTQ6 NDA6NTQgLTA3MDA6Cgo+IEhpIFJvYiwKPiAKPiBPbiBUdWUsIDI0IE9jdCAyMDIzIGF0IDA5OjE2 LCBSb2IgSGVycmluZyA8cm9iaEBrZXJuZWwub3JnPiB3cm90ZToKPiA+Cj4gPiBPbiBNb24sIE9j dCAwOSwgMjAyMyBhdCAwNDowNDoxNFBNIC0wNjAwLCBTaW1vbiBHbGFzcyB3cm90ZTogIAo+ID4g PiBBZGQgdHdvIGNvbXBhdGlibGUgZm9yIGJpbm1hbiBlbnRyaWVzLCBhcyBhIHN0YXJ0aW5nIHBv aW50IGZvciB0aGUKPiA+ID4gc2NoZW1hLgo+ID4gPgo+ID4gPiBOb3RlIHRoYXQsIGFmdGVyIGRp c2N1c3Npb24gb24gdjIsIHdlIGRlY2lkZWQgdG8ga2VlcCB0aGUgZXhpc3RpbmcKPiA+ID4gbWVh bmluZyBvZiBsYWJlbCBzbyBhcyBub3QgdG8gcmVxdWlyZSBjaGFuZ2VzIHRvIGV4aXN0aW5nIHVz ZXJzcGFjZQo+ID4gPiBzb2Z0d2FyZSB3aGVuIG1vdmluZyB0byB1c2UgYmlubWFuIG5vZGVzIHRv IHNwZWNpZnkgdGhlIGZpcm13YXJlCj4gPiA+IGxheW91dC4KPiA+ID4KPiA+ID4gU2lnbmVkLW9m Zi1ieTogU2ltb24gR2xhc3MgPHNqZ0BjaHJvbWl1bS5vcmc+Cj4gPiA+IC0tLQo+ID4gPgo+ID4g PiBDaGFuZ2VzIGluIHY0Ogo+ID4gPiAtIENvcnJlY3Qgc2VsZWN0aW9uIG9mIG11bHRpcGxlIGNv bXBhdGlibGUgc3RyaW5ncwo+ID4gPgo+ID4gPiBDaGFuZ2VzIGluIHYzOgo+ID4gPiAtIERyb3Ag Zml4ZWQtcGFydGl0aW9ucyBmcm9tIHRoZSBleGFtcGxlCj4gPiA+IC0gVXNlIGNvbXBhdGlibGUg aW5zdGVhZCBvZiBsYWJlbAo+ID4gPgo+ID4gPiBDaGFuZ2VzIGluIHYyOgo+ID4gPiAtIFVzZSBw bGFpbiBwYXJ0aXRpb25AeHh4IGZvciB0aGUgbm9kZSBuYW1lCj4gPiA+Cj4gPiA+ICAuLi4vbXRk L3BhcnRpdGlvbnMvYmlubWFuLXBhcnRpdGlvbi55YW1sICAgICAgfCA0OSArKysrKysrKysrKysr KysrKysrCj4gPiA+ICAxIGZpbGUgY2hhbmdlZCwgNDkgaW5zZXJ0aW9ucygrKQo+ID4gPiAgY3Jl YXRlIG1vZGUgMTAwNjQ0IERvY3VtZW50YXRpb24vZGV2aWNldHJlZS9iaW5kaW5ncy9tdGQvcGFy dGl0aW9ucy9iaW5tYW4tcGFydGl0aW9uLnlhbWwKPiA+ID4KPiA+ID4gZGlmZiAtLWdpdCBhL0Rv Y3VtZW50YXRpb24vZGV2aWNldHJlZS9iaW5kaW5ncy9tdGQvcGFydGl0aW9ucy9iaW5tYW4tcGFy dGl0aW9uLnlhbWwgYi9Eb2N1bWVudGF0aW9uL2RldmljZXRyZWUvYmluZGluZ3MvbXRkL3BhcnRp dGlvbnMvYmlubWFuLXBhcnRpdGlvbi55YW1sCj4gPiA+IG5ldyBmaWxlIG1vZGUgMTAwNjQ0Cj4g PiA+IGluZGV4IDAwMDAwMDAwMDAwMC4uMzVhMzIwMzU5ZWMxCj4gPiA+IC0tLSAvZGV2L251bGwK PiA+ID4gKysrIGIvRG9jdW1lbnRhdGlvbi9kZXZpY2V0cmVlL2JpbmRpbmdzL210ZC9wYXJ0aXRp b25zL2Jpbm1hbi1wYXJ0aXRpb24ueWFtbAo+ID4gPiBAQCAtMCwwICsxLDQ5IEBACj4gPiA+ICsj IFNQRFgtTGljZW5zZS1JZGVudGlmaWVyOiAoR1BMLTIuMCBPUiBCU0QtMi1DbGF1c2UpCj4gPiA+ ICsjIENvcHlyaWdodCAyMDIzIEdvb2dsZSBMTEMKPiA+ID4gKwo+ID4gPiArJVlBTUwgMS4yCj4g PiA+ICstLS0KPiA+ID4gKyRpZDogaHR0cDovL2RldmljZXRyZWUub3JnL3NjaGVtYXMvbXRkL3Bh cnRpdGlvbnMvYmlubWFuLXBhcnRpdGlvbi55YW1sIwo+ID4gPiArJHNjaGVtYTogaHR0cDovL2Rl dmljZXRyZWUub3JnL21ldGEtc2NoZW1hcy9jb3JlLnlhbWwjCj4gPiA+ICsKPiA+ID4gK3RpdGxl OiBCaW5tYW4gcGFydGl0aW9uCj4gPiA+ICsKPiA+ID4gK21haW50YWluZXJzOgo+ID4gPiArICAt IFNpbW9uIEdsYXNzIDxzamdAY2hyb21pdW0ub3JnPgo+ID4gPiArCj4gPiA+ICtzZWxlY3Q6IGZh bHNlICAKPiA+Cj4gPiBTbyB0aGlzIHNjaGVtYSBpcyBuZXZlciB1c2VkLiAnc2VsZWN0OiBmYWxz ZScgaXMgb25seSB1c2VmdWwgaWYKPiA+IHNvbWV0aGluZyBlbHNlIGlmIHJlZmVyZW5jaW5nIHRo ZSBzY2hlbWEuICAKPiAKPiBPSy4gSXMgdGhlcmUgYSB1c2VyIGd1aWRlIHRvIHRoaXMgc29tZXdo ZXJlPyBJIHJlYWxseSBkb24ndCB1bmRlcnN0YW5kCj4gaXQgdmVyeSB3ZWxsLgoKVGhlIGV4YW1w bGUtc2NoZW1hLnlhbWwgYXQgdGhlIHJvb3Qgb2YgdGhlIGR0LWJpbmRpbmdzIGRpcmVjdG9yeSBp cwp3ZWxsIGNvbW1lbnRlZC4KCj4gPiA+ICtkZXNjcmlwdGlvbjogfAo+ID4gPiArICBUaGlzIGNv cnJlc3BvbmRzIHRvIGEgYmlubWFuICdlbnRyeScuIEl0IGlzIGEgc2luZ2xlIHBhcnRpdGlvbiB3 aGljaCBob2xkcwo+ID4gPiArICBkYXRhIG9mIGEgZGVmaW5lZCB0eXBlLgo+ID4gPiArCj4gPiA+ ICthbGxPZjoKPiA+ID4gKyAgLSAkcmVmOiAvc2NoZW1hcy9tdGQvcGFydGl0aW9ucy9wYXJ0aXRp b24ueWFtbCMKPiA+ID4gKwo+ID4gPiArcHJvcGVydGllczoKPiA+ID4gKyAgY29tcGF0aWJsZToK PiA+ID4gKyAgICBvbmVPZjoKPiA+ID4gKyAgICAgIC0gY29uc3Q6IGJpbm1hbixlbnRyeSAgICAg IyBnZW5lcmljIGJpbm1hbiBlbnRyeSAgCj4gPgo+ID4gJ2Jpbm1hbicgaXMgbm90IGEgdmVuZG9y LiBZb3UgY291bGQgYWRkIGl0IGlmIHlvdSB0aGluayB0aGF0J3MgdXNlZnVsLgo+ID4gUHJvYmFi bHkgbm90IHdpdGggb25seSAxIGNhc2UuLi4gIAo+IAo+IEkgdGhpbmsgaXQgaXMgYmVzdCB0byB1 c2UgdGhpcyBmb3IgZ2VuZXJpYyB0aGluZ3MgaW1wbGVtZW50ZWQgYnkKPiBiaW5tYW4sIHJhdGhl ciB0aGFuIHNvbWUgb3RoZXIgcHJvamVjdC4gRm9yIGV4YW1wbGUsIGJpbm1hbiBzdXBwb3J0cyBh Cj4gJ2ZpbGwnIHJlZ2lvbi4gSXQgYWxzbyBzdXBwb3J0cyBzZWN0aW9ucyB3aGljaCBhcmUgZ3Jv dXBzIG9mCj4gc3ViLWVudHJpZXMuIFNvIHdlIHdpbGwgbGlrZWx5IHN0YXJ0IHdpdGggaGFsZiBh IGRvemVuIG9mIHRoZXNlIGFuZCBpdAo+IHdpbGwgbGlrZWx5IGdyb3c6IGJpbm1hbixmaWxsLCBi aW5tYW4sc2VjdGlvbiwgYmlubWFuLGZpbGVzCj4gCj4gSWYgd2UgZG9uJ3QgdXNlICdiaW5tYW4n LCB3aGF0IGRvIHlvdSBzdWdnZXN0Pwo+IAo+ID4gIAo+ID4gPiArICAgICAgLSBpdGVtczoKPiA+ ID4gKyAgICAgICAgICAtIGNvbnN0OiB1LWJvb3QgICAgICAgIyB1LWJvb3QuYmluIGZyb20gVS1C b290IHByb2plY3QKPiA+ID4gKyAgICAgICAgICAtIGNvbnN0OiBhdGYtYmwzMSAgICAgIyBibDMx LmJpbiBvciBibDMxLmVsZiBmcm9tIFRGLUEgcHJvamVjdCAgCj4gPgo+ID4gUHJvYmFibHkgc2hv dWxkIHVzZSB0aGUgbmV3ICd0ZmEnIHJhdGhlciB0aGFuIG9sZCAnYXRmJy4gSXMgdGhpcyB0aGUK PiA+IG9ubHkgYmluYXJ5IGZvciBURkE/IFRoZSBuYW1pbmcgc2VlbXMgaW5jb25zaXN0ZW50IGlu IHRoYXQgZXZlcnkgaW1hZ2UKPiA+IGdvZXMgaW4gKG9yIGNhbiBnbyBpbikgYSBibD8/IHNlY3Rp b24uIFdoeSBkb2VzIFRGQSBoYXZlIGl0IGJ1dCB1LWJvb3QKPiA+IGRvZXNuJ3Q/IFBlcmhhcHMg Qkw/PyBpcyBvcnRob2dvbmFsIHRvIGRlZmluaW5nIHdoYXQgaXMgaW4gZWFjaAo+ID4gcGFydGl0 aW9uLiBQZXJoYXBzIHNvbWVvbmUgbW9yZSBmYW1pbGFyIHdpdGggYWxsIHRoaXMgdGhhbiBJIGFt IGNhbgo+ID4gY29tbWVudC4gIAo+IAo+IEZyb20gd2hhdCBJIGNhbiB0ZWxsIFRGLUEgY2FuIHBy b2R1Y2UgYWxsIHNvcnRzIG9mIGJpbmFyaWVzLCBvZiB3aGljaAo+IGJsMzEgaXMgb25lLiBVLUJv b3QgY2FuIGFsc28gcHJvZHVjZSBsb3RzIG9mIGJpbmFyaWVzLCBidXQgaXRzIG5hbWluZwo+IGlz IGRpZmZlcmVudCAodS1ib290LCB1LWJvb3Qtc3BsLCBldGMuKS4gQmVhciBpbiBtaW5kIHRoYXQg VS1Cb290IGlzCj4gdXNlZCBvbiBBUk0sIHdoZXJlIHRoaXMgdGVybWlub2xvZ3kgaXMgZGVmaW5l ZCwgYW5kIG9uIHg4NiAoZm9yCj4gZXhhbXBsZSksIHdoZXJlIGl0IGlzIG5vdC4KPiAKPiA+Cj4g PiBPbmNlIHlvdSBhY3R1YWxseSB0ZXN0IHRoaXMsIHlvdSdsbCBmaW5kIHlvdSBhcmUgc3BlY2lm eWluZzoKPiA+Cj4gPiBjb21wYXRpYmxlID0gInUtYm9vdCIsICJhdGYtYmwzMSI7ICAKPiAKPiBJ IGRvbid0IHVuZGVyc3RhbmQgdGhhdCwgc29ycnkuIEknbGwgc2VuZCBhIHY1IGFuZCBzZWUgaWYg dGhlIHByb2JsZW0gZ29lcyBhd2F5LgoKRm9yIG1lIHRoaXMgbWVhbnMgdGhlIHBhcnRpdGlvbiBj b250YWlucyBVLUJvb3QgYW5kIFRGLUEsIHdoaWNoIGlzCnByb2JhYmx5IG5vdCB3aGF0IHlvdSB3 YW50LiBJIGJlbGlldmUgUm9iIGlzIHNheWluZyB0aGF0IGhvdyB5b3UgZGVmaW5lCnRoZSBjb21w YXRpYmxlIHByb3BlcnR5IGFib3ZlIGRvZXMgbm90IG1hdGNoIHRoZSBleGFtcGxlcyBiZWxvdy4g RGlkCnlvdSBydW4gbWFrZSBkdF9iaW5kaW5nX2NoZWNrPwoKQWxzbywgZG8geW91IHJlYWxseSBu ZWVkIHRvIHNheSB3aGljaCBzb2Z0d2FyZSBwcm9qZWN0IHByb3ZpZGVzIGEKY29tcG9uZW50PyBX b3VsZCB1c2luZyAiYmwzMSIsICJibDMzIiwgZXRjIGJlIGVub3VnaD8gT3IgbWF5YmUgeW91CmNv dWxkIGhhdmUgZWcuICJibDMxLXRmLWEiIGFuZCAiYmwzMS11LWJvb3Qtc3BsIiAoaW4gdGhpcyBv cmRlcikgZm9yCmNsYXJpdHk/IFRoaXMgd2F5IG9uZSBrbm93cyB3aGljaCBzdGFnZSBhIHBhcnRp dGlvbiBjb250YWlucyBhbmQgYWxzbwp0aGUgc29mdHdhcmUgcHJvamVjdCB3aGljaCBwcm92aWRl ZCBpdC4KClRvIGJlIGhvbmVzdCBJIHN0aWxsIGRvbid0IGZ1bGx5IGdldCB3aGVyZSB5b3Ugd2Fu dCB0byBnbyBhbmQgSSBiZWxpZXZlCmEgbW9yZSBjb21wbGV0ZSBzY2hlbWEgd291bGQgcHJvYmFi bHkgaGVscCwgd2l0aCBkaWZmZXJlbnQgZXhhbXBsZXMsIHRvCmNhdGNoIHdoYXQgeW91IG5lZWQg YW5kIHdoeS4KCj4gPiA+ICthZGRpdGlvbmFsUHJvcGVydGllczogZmFsc2UKPiA+ID4gKwo+ID4g PiArZXhhbXBsZXM6Cj4gPiA+ICsgIC0gfAo+ID4gPiArICAgIHBhcnRpdGlvbnMgewo+ID4gPiAr ICAgICAgICBjb21wYXRpYmxlID0gImJpbm1hbiI7Cj4gPiA+ICsgICAgICAgICNhZGRyZXNzLWNl bGxzID0gPDE+Owo+ID4gPiArICAgICAgICAjc2l6ZS1jZWxscyA9IDwxPjsKPiA+ID4gKwo+ID4g PiArICAgICAgICBwYXJ0aXRpb25AMTAwMDAwIHsKPiA+ID4gKyAgICAgICAgICAgIGNvbXBhdGli bGUgPSAidS1ib290IjsKPiA+ID4gKyAgICAgICAgICAgIHJlZyA9IDwweDEwMDAwMCAweGYwMDAw MD47Cj4gPiA+ICsgICAgICAgIH07Cj4gPiA+ICsKPiA+ID4gKyAgICAgICAgcGFydGl0aW9uQDIw MDAwMCB7Cj4gPiA+ICsgICAgICAgICAgICBjb21wYXRpYmxlID0gImF0Zi1ibDMxIjsKPiA+ID4g KyAgICAgICAgICAgIHJlZyA9IDwweDIwMDAwMCAweDEwMDAwMD47Cj4gPiA+ICsgICAgICAgIH07 Cj4gPiA+ICsgICAgfTsKPiA+ID4gLS0KPiA+ID4gMi40Mi4wLjYwOS5nYmI3NmY0NjYwNi1nb29n Cj4gPiA+ICAKPiAKPiBSZWdhcmRzLAo+IFNpbW9uCgoKVGhhbmtzLApNaXF1w6hsCgpfX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KTGludXggTVRE IGRpc2N1c3Npb24gbWFpbGluZyBsaXN0Cmh0dHA6Ly9saXN0cy5pbmZyYWRlYWQub3JnL21haWxt YW4vbGlzdGluZm8vbGludXgtbXRkLwo= 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 phobos.denx.de (phobos.denx.de [85.214.62.61]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 75B63C25B47 for ; Wed, 25 Oct 2023 08:11:35 +0000 (UTC) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id E4D4E87442; Wed, 25 Oct 2023 10:11:33 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=reject dis=none) header.from=bootlin.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Authentication-Results: phobos.denx.de; dkim=pass (2048-bit key; unprotected) header.d=bootlin.com header.i=@bootlin.com header.b="WpaHPssd"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id 25DB88743F; Wed, 25 Oct 2023 10:11:33 +0200 (CEST) Received: from relay4-d.mail.gandi.net (relay4-d.mail.gandi.net [217.70.183.196]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by phobos.denx.de (Postfix) with ESMTPS id 26FCA871E7 for ; Wed, 25 Oct 2023 10:11:29 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=reject dis=none) header.from=bootlin.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=miquel.raynal@bootlin.com Received: by mail.gandi.net (Postfix) with ESMTPSA id B0E7FE000C; Wed, 25 Oct 2023 08:11:27 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1698221488; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=kMsuCLZa5HO4/dTq8bW3lK1ng6FHaXYOgfnq2P4eOQg=; b=WpaHPssdmaUYYCP6ILGX0jXGmmE/YE/hq7pPHyHUfKycPqwyvpeThDlzSTmxf5Ou/U8LPK 6xbQM3cFgvQAUxwbh2R7OUMPXF0KMmrFbJgjg/8LuhWiKG5TTjx4S8caNs0H2LU64Easwg fJ2/fvW0VucZ+04ktbrfh7kF0CrEni45GoGImZyTX/RsZrcAshVZntfXs08QoYAil3WXyv uSjUYnShdesomb97EqP4C2SacSYMGjg2BpmOPGqwlcIgY8pZUaOJ71hemiQihaRoWPscmr SK1xm9WNW35kipLA05wVJSf+d6y3gMGJudeEhF6oLcu9FPgRwn0NXJuDG9o9VQ== Date: Wed, 25 Oct 2023 10:11:26 +0200 From: Miquel Raynal To: Simon Glass Cc: Rob Herring , devicetree@vger.kernel.org, linux-mtd@lists.infradead.org, Michael Walle , U-Boot Mailing List , Tom Rini , Conor Dooley , Krzysztof Kozlowski , Richard Weinberger , Vignesh Raghavendra , linux-kernel@vger.kernel.org Subject: Re: [PATCH v4 2/3] dt-bindings: mtd: binman-partition: Add binman compatibles Message-ID: <20231025101126.2234a74c@xps-13> In-Reply-To: References: <20231009220436.2164245-1-sjg@chromium.org> <20231009220436.2164245-2-sjg@chromium.org> <20231024161644.GB3707756-robh@kernel.org> Organization: Bootlin X-Mailer: Claws Mail 4.0.0 (GTK+ 3.24.33; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable X-GND-Sasl: miquel.raynal@bootlin.com X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.39 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" X-Virus-Scanned: clamav-milter 0.103.8 at phobos.denx.de X-Virus-Status: Clean Hi Simon, sjg@chromium.org wrote on Tue, 24 Oct 2023 14:40:54 -0700: > Hi Rob, >=20 > On Tue, 24 Oct 2023 at 09:16, Rob Herring wrote: > > > > On Mon, Oct 09, 2023 at 04:04:14PM -0600, Simon Glass wrote: =20 > > > Add two compatible for binman entries, as a starting point for the > > > schema. > > > > > > Note that, after discussion on v2, we decided to keep the existing > > > meaning of label so as not to require changes to existing userspace > > > software when moving to use binman nodes to specify the firmware > > > layout. > > > > > > Signed-off-by: Simon Glass > > > --- > > > > > > Changes in v4: > > > - Correct selection of multiple compatible strings > > > > > > Changes in v3: > > > - Drop fixed-partitions from the example > > > - Use compatible instead of label > > > > > > Changes in v2: > > > - Use plain partition@xxx for the node name > > > > > > .../mtd/partitions/binman-partition.yaml | 49 +++++++++++++++++= ++ > > > 1 file changed, 49 insertions(+) > > > create mode 100644 Documentation/devicetree/bindings/mtd/partitions/= binman-partition.yaml > > > > > > diff --git a/Documentation/devicetree/bindings/mtd/partitions/binman-= partition.yaml b/Documentation/devicetree/bindings/mtd/partitions/binman-pa= rtition.yaml > > > new file mode 100644 > > > index 000000000000..35a320359ec1 > > > --- /dev/null > > > +++ b/Documentation/devicetree/bindings/mtd/partitions/binman-partiti= on.yaml > > > @@ -0,0 +1,49 @@ > > > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) > > > +# Copyright 2023 Google LLC > > > + > > > +%YAML 1.2 > > > +--- > > > +$id: http://devicetree.org/schemas/mtd/partitions/binman-partition.y= aml# > > > +$schema: http://devicetree.org/meta-schemas/core.yaml# > > > + > > > +title: Binman partition > > > + > > > +maintainers: > > > + - Simon Glass > > > + > > > +select: false =20 > > > > So this schema is never used. 'select: false' is only useful if > > something else if referencing the schema. =20 >=20 > OK. Is there a user guide to this somewhere? I really don't understand > it very well. The example-schema.yaml at the root of the dt-bindings directory is well commented. > > > +description: | > > > + This corresponds to a binman 'entry'. It is a single partition whi= ch holds > > > + data of a defined type. > > > + > > > +allOf: > > > + - $ref: /schemas/mtd/partitions/partition.yaml# > > > + > > > +properties: > > > + compatible: > > > + oneOf: > > > + - const: binman,entry # generic binman entry =20 > > > > 'binman' is not a vendor. You could add it if you think that's useful. > > Probably not with only 1 case... =20 >=20 > I think it is best to use this for generic things implemented by > binman, rather than some other project. For example, binman supports a > 'fill' region. It also supports sections which are groups of > sub-entries. So we will likely start with half a dozen of these and it > will likely grow: binman,fill, binman,section, binman,files >=20 > If we don't use 'binman', what do you suggest? >=20 > > =20 > > > + - items: > > > + - const: u-boot # u-boot.bin from U-Boot project > > > + - const: atf-bl31 # bl31.bin or bl31.elf from TF-A pro= ject =20 > > > > Probably should use the new 'tfa' rather than old 'atf'. Is this the > > only binary for TFA? The naming seems inconsistent in that every image > > goes in (or can go in) a bl?? section. Why does TFA have it but u-boot > > doesn't? Perhaps BL?? is orthogonal to defining what is in each > > partition. Perhaps someone more familar with all this than I am can > > comment. =20 >=20 > From what I can tell TF-A can produce all sorts of binaries, of which > bl31 is one. U-Boot can also produce lots of binaries, but its naming > is different (u-boot, u-boot-spl, etc.). Bear in mind that U-Boot is > used on ARM, where this terminology is defined, and on x86 (for > example), where it is not. >=20 > > > > Once you actually test this, you'll find you are specifying: > > > > compatible =3D "u-boot", "atf-bl31"; =20 >=20 > I don't understand that, sorry. I'll send a v5 and see if the problem goe= s away. For me this means the partition contains U-Boot and TF-A, which is probably not what you want. I believe Rob is saying that how you define the compatible property above does not match the examples below. Did you run make dt_binding_check? Also, do you really need to say which software project provides a component? Would using "bl31", "bl33", etc be enough? Or maybe you could have eg. "bl31-tf-a" and "bl31-u-boot-spl" (in this order) for clarity? This way one knows which stage a partition contains and also the software project which provided it. To be honest I still don't fully get where you want to go and I believe a more complete schema would probably help, with different examples, to catch what you need and why. > > > +additionalProperties: false > > > + > > > +examples: > > > + - | > > > + partitions { > > > + compatible =3D "binman"; > > > + #address-cells =3D <1>; > > > + #size-cells =3D <1>; > > > + > > > + partition@100000 { > > > + compatible =3D "u-boot"; > > > + reg =3D <0x100000 0xf00000>; > > > + }; > > > + > > > + partition@200000 { > > > + compatible =3D "atf-bl31"; > > > + reg =3D <0x200000 0x100000>; > > > + }; > > > + }; > > > -- > > > 2.42.0.609.gbb76f46606-goog > > > =20 >=20 > Regards, > Simon Thanks, Miqu=C3=A8l