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 47782C369D5 for ; Wed, 25 Sep 2024 11:58:33 +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=JyjHnwNwDrzXQTUwxhqXsHSsbjesbLHrb4gOQs8xIHE=; b=VMNcemyEc6nK1k crZ455hB1wmG5Vffglpco0woAfQMdM+hX80zGRQUh1P/eAQyfcZSuKuCyC7kgh7HqkrrJ+fUB6cSg Rg3NlFKiVsyPFf3Xh5g6Yg1dja7/TxKwDnv75zOD12o3fO1PkrUKYBqaqlfGMVbJnVmDzXUkmz4vq MNfSQ3wXoUuBlOYZJMoxETyLCWi5uRmennxH43H+B8XjlLUpDI9f+wy2/pWdUzLLJNeZ2M0VPdcir QA1N20DZsSUXnpV9///Ou7PboSDbBQ50L7vtwT8F4I7PjW8+XcRuaYb55MGPaauF++XyG1ckzvL7N ypUgyQpWH3o8B8EJ+yJw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1stQf8-00000004z8e-1hTQ; Wed, 25 Sep 2024 11:58:30 +0000 Received: from relay9-d.mail.gandi.net ([2001:4b98:dc4:8::229]) by bombadil.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1stQZp-00000004xfZ-2nq8 for linux-mtd@lists.infradead.org; Wed, 25 Sep 2024 11:53:03 +0000 Received: by mail.gandi.net (Postfix) with ESMTPSA id 4A1B4FF803; Wed, 25 Sep 2024 11:52:57 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1727265178; 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=rEB2l7uMtdKl3fE8fywDslPAGKGpR8hMKHGK5fGclE0=; b=dgh4/71kLNp+tfE+mazcuCvgP5lMlVVsW2znVEw6E3ALQZLnLgpfBO+yAJ4aJhTmbi1Atb 8m/3NxWtLwLSYMbN+j+SFfZ7VBcDMEkdz6OqtWlJZQVeWEcii+PX7mh1LzCR93Ch0VZ1KE 6B6PgdNSSVECwXBW1RPOUDRf+j/iCEm+6qZSMlja6AMGQiOjXbmtnMa7r48TpPhnnGmn7V C/efCLt2z+aFw8lOAuHZlA7DII8kyQ9CYHTQM4P+QDfVTW0DHzIxCg6vGLk+A++rTtpGtR pk6Mb0iGSL3pGZxDHjThVX9Meu5aefbTpp1jtX2UekudfYcNym8G3oR/dLGPqg== Date: Wed, 25 Sep 2024 13:52:56 +0200 From: Miquel Raynal To: Christian Marangi Cc: Richard Weinberger , Vignesh Raghavendra , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Saravana Kannan , Florian Fainelli , Thomas Bogendoerfer , Wolfram Sang , linux-mtd@lists.infradead.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Lorenzo Bianconi , upstream@airoha.com Subject: Re: [PATCH 2/3] dt-bindings: mtd: Add Documentation for Airoha fixed-partitions Message-ID: <20240925135256.32d3a0f7@xps-13> In-Reply-To: <66f3f58e.5d0a0220.5d655.b48a@mx.google.com> References: <20240925101422.8373-1-ansuelsmth@gmail.com> <20240925101422.8373-3-ansuelsmth@gmail.com> <20240925133003.619c40c4@xps-13> <66f3f58e.5d0a0220.5d655.b48a@mx.google.com> Organization: Bootlin X-Mailer: Claws Mail 4.2.0 (GTK 3.24.41; 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-20240925_045302_167339_80EC4211 X-CRM114-Status: GOOD ( 34.59 ) 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 SGkgQ2hyaXN0aWFuLAoKYW5zdWVsc210aEBnbWFpbC5jb20gd3JvdGUgb24gV2VkLCAyNSBTZXAg MjAyNCAxMzozNTozOCArMDIwMDoKCj4gT24gV2VkLCBTZXAgMjUsIDIwMjQgYXQgMDE6MzA6MDNQ TSArMDIwMCwgTWlxdWVsIFJheW5hbCB3cm90ZToKPiA+IEhpIENocmlzdGlhbiwKPiA+IAo+ID4g YW5zdWVsc210aEBnbWFpbC5jb20gd3JvdGUgb24gV2VkLCAyNSBTZXAgMjAyNCAxMjoxMzo1OCAr MDIwMDoKPiA+ICAgCj4gPiA+IEFkZCBEb2N1bWVudGF0aW9uIGZvciBBaXJvaGEgZml4ZWQtcGFy dGl0aW9ucyBjb21wYXRpYmxlcy4KPiA+ID4gCj4gPiA+IEFpcm9oYSBiYXNlZCBTb0MgZGVjbGFy ZSBhIGRlZGljYXRlZCBwYXJ0aXRpb24gYXQgdGhlIGVuZCBvZiB0aGUgZmxhc2ggdG8KPiA+ID4g c3RvcmUgY2FsaWJyYXRpb24gYW5kIGRldmljZSBzcGVjaWZpYyBkYXRhLCBpbiBhZGRpdGlvbiB0 byBmaXhlZAo+ID4gPiBwYXJ0aXRpb25zLgo+ID4gPiAKPiA+ID4gVGhlIG9mZnNldCBvZiB0aGlz IHNwZWNpYWwgcGFydGl0aW9uIGlzIG5vdCB3ZWxsIGRlZmluZWQgYXMgYSBjdXN0b20gYmFkCj4g PiA+IGJsb2NrIG1hbmFnZW1lbnQgZHJpdmVyIGlzIHVzZWQgdGhhdCByZXNlcnZlIHNwYWNlIGF0 IHRoZSBlbmQgb2YgdGhlIGZsYXNoLgo+ID4gPiAKPiA+ID4gVGhpcyBiaW5kaW5nIGFsbG93cyBk ZWZpbmluZyBhbGwgZml4ZWQgcGFydGl0aW9ucyBhbmQgbWFya2luZyB0aGUgbGFzdCBvbmUKPiA+ ID4gdG8gZGV0ZWN0IHRoZSBjb3JyZWN0IG9mZnNldC4KPiA+ID4gCj4gPiA+IFNpZ25lZC1vZmYt Ynk6IENocmlzdGlhbiBNYXJhbmdpIDxhbnN1ZWxzbXRoQGdtYWlsLmNvbT4KPiA+ID4gLS0tCj4g PiA+ICAuLi4vcGFydGl0aW9ucy9haXJvaGEsZml4ZWQtcGFydGl0aW9ucy55YW1sICAgfCA4MCAr KysrKysrKysrKysrKysrKysrCj4gPiA+ICAuLi4vYmluZGluZ3MvbXRkL3BhcnRpdGlvbnMvcGFy dGl0aW9ucy55YW1sICAgfCAgMSArCj4gPiA+ICAyIGZpbGVzIGNoYW5nZWQsIDgxIGluc2VydGlv bnMoKykKPiA+ID4gIGNyZWF0ZSBtb2RlIDEwMDY0NCBEb2N1bWVudGF0aW9uL2RldmljZXRyZWUv YmluZGluZ3MvbXRkL3BhcnRpdGlvbnMvYWlyb2hhLGZpeGVkLXBhcnRpdGlvbnMueWFtbAo+ID4g PiAKPiA+ID4gZGlmZiAtLWdpdCBhL0RvY3VtZW50YXRpb24vZGV2aWNldHJlZS9iaW5kaW5ncy9t dGQvcGFydGl0aW9ucy9haXJvaGEsZml4ZWQtcGFydGl0aW9ucy55YW1sIGIvRG9jdW1lbnRhdGlv bi9kZXZpY2V0cmVlL2JpbmRpbmdzL210ZC9wYXJ0aXRpb25zL2Fpcm9oYSxmaXhlZC1wYXJ0aXRp b25zLnlhbWwKPiA+ID4gbmV3IGZpbGUgbW9kZSAxMDA2NDQKPiA+ID4gaW5kZXggMDAwMDAwMDAw MDAwLi5hNDVkZjUxMDY1YWYKPiA+ID4gLS0tIC9kZXYvbnVsbAo+ID4gPiArKysgYi9Eb2N1bWVu dGF0aW9uL2RldmljZXRyZWUvYmluZGluZ3MvbXRkL3BhcnRpdGlvbnMvYWlyb2hhLGZpeGVkLXBh cnRpdGlvbnMueWFtbAo+ID4gPiBAQCAtMCwwICsxLDgwIEBACj4gPiA+ICsjIFNQRFgtTGljZW5z ZS1JZGVudGlmaWVyOiBHUEwtMi4wLW9ubHkgT1IgQlNELTItQ2xhdXNlCj4gPiA+ICslWUFNTCAx LjIKPiA+ID4gKy0tLQo+ID4gPiArJGlkOiBodHRwOi8vZGV2aWNldHJlZS5vcmcvc2NoZW1hcy9t dGQvcGFydGl0aW9ucy9haXJvaGEsZml4ZWQtcGFydGl0aW9ucy55YW1sIwo+ID4gPiArJHNjaGVt YTogaHR0cDovL2RldmljZXRyZWUub3JnL21ldGEtc2NoZW1hcy9jb3JlLnlhbWwjCj4gPiA+ICsK PiA+ID4gK3RpdGxlOiBBaXJvaGEgU29DIHBhcnRpdGlvbmluZwo+ID4gPiArCj4gPiA+ICtkZXNj cmlwdGlvbjogfAo+ID4gPiArICBBaXJvaGEgYmFzZWQgU29DIGRlY2xhcmUgYSBkZWRpY2F0ZWQg cGFydGl0aW9uIGF0IHRoZSBlbmQgb2YgdGhlIGZsYXNoIHRvCj4gPiA+ICsgIHN0b3JlIGNhbGli cmF0aW9uIGFuZCBkZXZpY2Ugc3BlY2lmaWMgZGF0YSwgaW4gYWRkaXRpb24gdG8gZml4ZWQgcGFy dGl0aW9ucy4KPiA+ID4gKwo+ID4gPiArICBUaGUgb2Zmc2V0IG9mIHRoaXMgc3BlY2lhbCBwYXJ0 aXRpb24gaXMgbm90IHdlbGwgZGVmaW5lZCBhcyBhIGN1c3RvbSBiYWQgYmxvY2sKPiA+ID4gKyAg bWFuYWdlbWVudCBkcml2ZXIgaXMgdXNlZCB0aGF0IHJlc2VydmUgc3BhY2UgYXQgdGhlIGVuZCBv ZiB0aGUgZmxhc2guCj4gPiA+ICsKPiA+ID4gKyAgVGhpcyBiaW5kaW5nIGFsbG93cyBkZWZpbmlu ZyBhbGwgZml4ZWQgcGFydGl0aW9ucyBhbmQgbWFya2luZyB0aGUgbGFzdCBvbmUgdG8KPiA+ID4g KyAgZGV0ZWN0IHRoZSBjb3JyZWN0IG9mZnNldCBmcm9tIHRoZSBuZXcgZW5kIG9mIHRoZSBmbGFz aC4KPiA+ID4gKwo+ID4gPiArbWFpbnRhaW5lcnM6Cj4gPiA+ICsgIC0gQ2hyaXN0aWFuIE1hcmFu Z2kgPGFuc3VlbHNtdGhAZ21haWwuY29tPgo+ID4gPiArCj4gPiA+ICtzZWxlY3Q6IGZhbHNlCj4g PiA+ICsKPiA+ID4gK3Byb3BlcnRpZXM6Cj4gPiA+ICsgIGNvbXBhdGlibGU6Cj4gPiA+ICsgICAg Y29uc3Q6IGFpcm9oYSxmaXhlZC1wYXJ0aXRpb25zCj4gPiA+ICsKPiA+ID4gKyAgIiNhZGRyZXNz LWNlbGxzIjoKPiA+ID4gKyAgICBlbnVtOiBbIDEsIDIgXQo+ID4gPiArCj4gPiA+ICsgICIjc2l6 ZS1jZWxscyI6Cj4gPiA+ICsgICAgZW51bTogWyAxLCAyIF0KPiA+ID4gKwo+ID4gPiArcGF0dGVy blByb3BlcnRpZXM6Cj4gPiA+ICsgICJecGFydGl0aW9uQFswLTlhLWZdKyQiOgo+ID4gPiArICAg ICRyZWY6IHBhcnRpdGlvbi55YW1sIwo+ID4gPiArICAgIHByb3BlcnRpZXM6Cj4gPiA+ICsgICAg ICBjb21wYXRpYmxlOgo+ID4gPiArICAgICAgICBjb25zdDogYWlyb2hhLGR5bmFtaWMtYXJ0Cj4g PiA+ICsgICAgdW5ldmFsdWF0ZWRQcm9wZXJ0aWVzOiBmYWxzZQo+ID4gPiArCj4gPiA+ICtyZXF1 aXJlZDoKPiA+ID4gKyAgLSAiI2FkZHJlc3MtY2VsbHMiCj4gPiA+ICsgIC0gIiNzaXplLWNlbGxz Igo+ID4gPiArCj4gPiA+ICthZGRpdGlvbmFsUHJvcGVydGllczogZmFsc2UKPiA+ID4gKwo+ID4g PiArZXhhbXBsZXM6Cj4gPiA+ICsgIC0gfAo+ID4gPiArICAgIHBhcnRpdGlvbnMgewo+ID4gPiAr ICAgICAgICBjb21wYXRpYmxlID0gImFpcm9oYSxmaXhlZC1wYXJ0aXRpb25zIjsKPiA+ID4gKyAg ICAgICAgI2FkZHJlc3MtY2VsbHMgPSA8MT47Cj4gPiA+ICsgICAgICAgICNzaXplLWNlbGxzID0g PDE+Owo+ID4gPiArCj4gPiA+ICsgICAgICAgIHBhcnRpdGlvbkAwIHsKPiA+ID4gKyAgICAgICAg ICBsYWJlbCA9ICJib290bG9hZGVyIjsKPiA+ID4gKyAgICAgICAgICByZWcgPSA8MHgwMDAwMDAw MCAweDAwMDgwMDAwPjsKPiA+ID4gKyAgICAgICAgfTsKPiA+ID4gKwo+ID4gPiArICAgICAgICBw YXJ0aXRpb25AODAwMDAgewo+ID4gPiArICAgICAgICAgIGxhYmVsID0gInRjbGludXgiOwo+ID4g PiArICAgICAgICAgIHJlZyA9IDwweDAwMDgwMDAwIDB4MDI4MDAwMDA+Owo+ID4gPiArICAgICAg ICB9Owo+ID4gPiArCj4gPiA+ICsgICAgICAgIHBhcnRpdGlvbkAyODgwMDAwIHsKPiA+ID4gKyAg ICAgICAgICBsYWJlbCA9ICJ0Y2xpbnV4X3NsYXZlIjsKPiA+ID4gKyAgICAgICAgICByZWcgPSA8 MHgwMjg4MDAwMCAweDAyODAwMDAwPjsKPiA+ID4gKyAgICAgICAgfTsKPiA+ID4gKwo+ID4gPiAr ICAgICAgICBwYXJ0aXRpb25ANTA4MDAwMCB7Cj4gPiA+ICsgICAgICAgICAgbGFiZWwgPSAicm9v dGZzX2RhdGEiOwo+ID4gPiArICAgICAgICAgIHJlZyA9IDwweDUwODAwMDAgMHgwMDgwMDAwMD47 Cj4gPiA+ICsgICAgICAgIH07Cj4gPiA+ICsKPiA+ID4gKyAgICAgICAgcGFydGl0aW9uQGZmZmZm ZmZmIHsKPiA+ID4gKyAgICAgICAgICBjb21wYXRpYmxlID0gImFpcm9oYSxkeW5hbWljLWFydCI7 Cj4gPiA+ICsgICAgICAgICAgbGFiZWwgPSAiYXJ0IjsKPiA+ID4gKyAgICAgICAgICByZWcgPSA8 MHhmZmZmZmZmZiAweDAwMzAwMDAwPjsgIAo+ID4gCj4gPiBJJ20gYSBsaXR0bGUgYml0IHB1enps ZWQgYnkgdGhpcyBraW5kIG9mIGluZm9ybWF0aW9uIHdoaWNoIGlzIGtub3duIHRvCj4gPiBiZSB3 cm9uZy4gQXMgdGhlIHBhcnRpdGlvbiBvZmZzZXQgYW5kIHNpemUgbXVzdCBiZSBkeW5hbWljYWxs eQo+ID4gY2FsY3VsYXRlZCwgdGhpcyByZWcgcHJvcGVydHkgKGFzIHdlbGwgYXMgdGhlIHNpemUg cGFyYW1ldGVyIG9mIHRoZQo+ID4gcHJldmlvdXMgb25lKSBhcmUgbm90YWJseSB3cm9uZy4gSSBn dWVzcyB3ZSBhcmUgbm90IGZ1bGx5IGNvbnN0cmFpbmVkCj4gPiBieSB0aGUgZml4ZWQtcGFydGl0 aW9ucyBzY2hlbWEgaGVyZSwgc28gY291bGQgd2UgYXZvaWQgdGhlIHJlZyBwcm9wZXJ0eQo+ID4g aW4gdGhlIGFpcm9oYSxkeW5hbWljLWFydCBwYXJ0aXRpb24/IE1heWJlIHdlIGFsc28gbmVlZCBh ICNkZWZpbmUgZm9yIGEKPiA+IHNwZWNpZmljIHBsYWNlaG9sZGVyIGluIHRoZSBwZW51bHRpbWF0 ZSByZWcgcHJvcGVydHkgdG9vIChmb3IgdGhlIHNpemUpLgo+ID4gIAo+IAo+IE1heWJlIGluc3Rl YWQgb2YgcmVnIHdlIGNhbiB1c2UgYSBwcm9wZXJ0eSBsaWtlIHNpemU/Cj4gCj4gQ2FuIHlvdSBi ZXR0ZXIgZWxhYm9yYXRlIHRoZSBzdWdnZXN0aW9uIGFib3V0IHRoZSAjZGVmaW5lPwo+IAo+IERv IHlvdSBtZWFuIGZvciBjYXNlIHdoZXJlIHRoZSBsYXN0IHBhcnRpdGlvbiBtaWdodCBvdmVybGFw Cj4gd2l0aCB0aGUgcGVudWx0aW1hdGU/IEhvbmVzdGx5IGluIHN1Y2ggY2FzZSBJIHdvdWxkIGVy cm9yIGhhcmQsIHRoYXQKPiBjYXNlIGhhcHBlbiB3aGVuIHRvbyBtdWNoIHNwYWNlIGlzIHJlc2Vy dmVkIGFuZCB0aGF0IGlzIGEKPiBtaXNjb25maWd1cmF0aW9uIG9mIHRoZSBzeXN0ZW0gKGRldmVs b3BlciBlcnJvcikKClRoYXQncyBub3Qgd2hhdCBJIG1lYW4uCgpJbiB0aGUgYWJvdmUgY2FzZSB5 b3Ugc2F5IHBhcnRpdGlvbiAicGFydGl0aW9uQDUwODAwMDAiIGlzIDB4ODAwMDAwCmJ5dGVzIGxv bmcuIFRoaXMgaXMgb2J2aW91c2x5IHdyb25nIG90aGVyd2lzZSB5b3Ugd291bGQga25vdyB3aGVy ZSB0aGUKYXJ0IHBhcnRpdGlvbiBzdGFydHMuIEFuZCByaWdodCBhZnRlciB5b3UncmUgc2F5aW5n IHBhcnRpdGlvbgoicGFydGl0aW9uQGZmZmZmZmZmIiBzdGFydHMgYXQgMHhmZmZmZmZmZiBhbmQg aXMgMHgzMDAwMDAgYnl0ZXMgbG9uZy4KVGhpcyBpcyBhbHNvIHdyb25nIGJlY2F1c2UgMHhmZmZm ZmZmZiBpcyBub3QgYSB2YWxpZCBzdGFydCBhZGRyZXNzIGFuZApJSVVDIDB4MzAwMDAwIGlzIGFs c28gdW5rbm93biBhbmQgZHluYW1pY2FsbHkgZGVyaXZlZC4KClNvIGZvciB0aGUgYXJ0IHBhcnRp dGlvbiBteSBhZHZpc2UgaWYgeW91IGtub3cgbm90aGluZyBhYm91dCB0aGUKc3RhcnQvbGVuZ3Ro IGlzIHRvIGp1c3Qgc2tpcCB0aGUgcmVnIHByb3BlcnR5LiBGb3IgdGhlIHByZXZpb3VzCnBhcnRp dGlvbiBJJ2QgbWF5YmUgdXNlIGEgZGVmaW5pdGlvbiAod2hvc2UgbmFtZSBpcyB0byBkaXNjdXNz KSBpbnN0ZWFkCm9mIHRoZSB3cm9uZyBzaXplIGFyZ3VtZW50ICh0aGUgc3RhcnQgb2Zmc2V0IGJl aW5nIGNvcnJlY3Qgb24gaGlzIHNpZGUpLgoKVGhhbmtzLCBNaXF1w6hsCgpfX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KTGludXggTVREIGRpc2N1 c3Npb24gbWFpbGluZyBsaXN0Cmh0dHA6Ly9saXN0cy5pbmZyYWRlYWQub3JnL21haWxtYW4vbGlz dGluZm8vbGludXgtbXRkLwo= From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from relay9-d.mail.gandi.net (relay9-d.mail.gandi.net [217.70.183.199]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id AE8E31C688B; Wed, 25 Sep 2024 11:53:05 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.70.183.199 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1727265188; cv=none; b=UYMBNTbtOtIDfRjpXhkSKDKfJnaPS6WyO09xkBXQF9+c7voYernbOoAwZrHR0laB7jw0sKI6JAViseEIyHzJ7RKW1Xdr4wBMwHG8gR1F5mn5Zi9TATBuu7rqq4w+ZFftwg7OQFHxvSc2Z4/VDTC5GHVTD29Ofg1Lyi4qF2DzA3c= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1727265188; c=relaxed/simple; bh=+R7DW9dKSPV/DqDxWXrxZWtidjyjzc2zeQUtweCGWk8=; h=Date:From:To:Cc:Subject:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=fgWxr/eS3sS0rKI82iUdzecXzLJyU5PH8VGliRwlB//coDdrOQazlBrDNyh1PRAMjpT4pxJNd/JEhdolXauTL/hckaGO+e0I6bs0kOcKEp/ABgcwIOBcDkE/gQfqhY81jBvYwGQL8iAsmEH4WrJvtiSlZc9JynTgwZQJ7stcIHY= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=bootlin.com; spf=pass smtp.mailfrom=bootlin.com; dkim=pass (2048-bit key) header.d=bootlin.com header.i=@bootlin.com header.b=dgh4/71k; arc=none smtp.client-ip=217.70.183.199 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=bootlin.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=bootlin.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=bootlin.com header.i=@bootlin.com header.b="dgh4/71k" Received: by mail.gandi.net (Postfix) with ESMTPSA id 4A1B4FF803; Wed, 25 Sep 2024 11:52:57 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1727265178; 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=rEB2l7uMtdKl3fE8fywDslPAGKGpR8hMKHGK5fGclE0=; b=dgh4/71kLNp+tfE+mazcuCvgP5lMlVVsW2znVEw6E3ALQZLnLgpfBO+yAJ4aJhTmbi1Atb 8m/3NxWtLwLSYMbN+j+SFfZ7VBcDMEkdz6OqtWlJZQVeWEcii+PX7mh1LzCR93Ch0VZ1KE 6B6PgdNSSVECwXBW1RPOUDRf+j/iCEm+6qZSMlja6AMGQiOjXbmtnMa7r48TpPhnnGmn7V C/efCLt2z+aFw8lOAuHZlA7DII8kyQ9CYHTQM4P+QDfVTW0DHzIxCg6vGLk+A++rTtpGtR pk6Mb0iGSL3pGZxDHjThVX9Meu5aefbTpp1jtX2UekudfYcNym8G3oR/dLGPqg== Date: Wed, 25 Sep 2024 13:52:56 +0200 From: Miquel Raynal To: Christian Marangi Cc: Richard Weinberger , Vignesh Raghavendra , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Saravana Kannan , Florian Fainelli , Thomas Bogendoerfer , Wolfram Sang , linux-mtd@lists.infradead.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Lorenzo Bianconi , upstream@airoha.com Subject: Re: [PATCH 2/3] dt-bindings: mtd: Add Documentation for Airoha fixed-partitions Message-ID: <20240925135256.32d3a0f7@xps-13> In-Reply-To: <66f3f58e.5d0a0220.5d655.b48a@mx.google.com> References: <20240925101422.8373-1-ansuelsmth@gmail.com> <20240925101422.8373-3-ansuelsmth@gmail.com> <20240925133003.619c40c4@xps-13> <66f3f58e.5d0a0220.5d655.b48a@mx.google.com> Organization: Bootlin X-Mailer: Claws Mail 4.2.0 (GTK 3.24.41; x86_64-pc-linux-gnu) Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable X-GND-Sasl: miquel.raynal@bootlin.com Hi Christian, ansuelsmth@gmail.com wrote on Wed, 25 Sep 2024 13:35:38 +0200: > On Wed, Sep 25, 2024 at 01:30:03PM +0200, Miquel Raynal wrote: > > Hi Christian, > >=20 > > ansuelsmth@gmail.com wrote on Wed, 25 Sep 2024 12:13:58 +0200: > > =20 > > > Add Documentation for Airoha fixed-partitions compatibles. > > >=20 > > > Airoha based SoC declare a dedicated partition at the end of the flas= h to > > > store calibration and device specific data, in addition to fixed > > > partitions. > > >=20 > > > The offset of this special partition is not well defined as a custom = bad > > > block management driver is used that reserve space at the end of the = flash. > > >=20 > > > This binding allows defining all fixed partitions and marking the las= t one > > > to detect the correct offset. > > >=20 > > > Signed-off-by: Christian Marangi > > > --- > > > .../partitions/airoha,fixed-partitions.yaml | 80 +++++++++++++++++= ++ > > > .../bindings/mtd/partitions/partitions.yaml | 1 + > > > 2 files changed, 81 insertions(+) > > > create mode 100644 Documentation/devicetree/bindings/mtd/partitions/= airoha,fixed-partitions.yaml > > >=20 > > > diff --git a/Documentation/devicetree/bindings/mtd/partitions/airoha,= fixed-partitions.yaml b/Documentation/devicetree/bindings/mtd/partitions/ai= roha,fixed-partitions.yaml > > > new file mode 100644 > > > index 000000000000..a45df51065af > > > --- /dev/null > > > +++ b/Documentation/devicetree/bindings/mtd/partitions/airoha,fixed-p= artitions.yaml > > > @@ -0,0 +1,80 @@ > > > +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause > > > +%YAML 1.2 > > > +--- > > > +$id: http://devicetree.org/schemas/mtd/partitions/airoha,fixed-parti= tions.yaml# > > > +$schema: http://devicetree.org/meta-schemas/core.yaml# > > > + > > > +title: Airoha SoC partitioning > > > + > > > +description: | > > > + Airoha based SoC declare a dedicated partition at the end of the f= lash to > > > + store calibration and device specific data, in addition to fixed p= artitions. > > > + > > > + The offset of this special partition is not well defined as a cust= om bad block > > > + management driver is used that reserve space at the end of the fla= sh. > > > + > > > + This binding allows defining all fixed partitions and marking the = last one to > > > + detect the correct offset from the new end of the flash. > > > + > > > +maintainers: > > > + - Christian Marangi > > > + > > > +select: false > > > + > > > +properties: > > > + compatible: > > > + const: airoha,fixed-partitions > > > + > > > + "#address-cells": > > > + enum: [ 1, 2 ] > > > + > > > + "#size-cells": > > > + enum: [ 1, 2 ] > > > + > > > +patternProperties: > > > + "^partition@[0-9a-f]+$": > > > + $ref: partition.yaml# > > > + properties: > > > + compatible: > > > + const: airoha,dynamic-art > > > + unevaluatedProperties: false > > > + > > > +required: > > > + - "#address-cells" > > > + - "#size-cells" > > > + > > > +additionalProperties: false > > > + > > > +examples: > > > + - | > > > + partitions { > > > + compatible =3D "airoha,fixed-partitions"; > > > + #address-cells =3D <1>; > > > + #size-cells =3D <1>; > > > + > > > + partition@0 { > > > + label =3D "bootloader"; > > > + reg =3D <0x00000000 0x00080000>; > > > + }; > > > + > > > + partition@80000 { > > > + label =3D "tclinux"; > > > + reg =3D <0x00080000 0x02800000>; > > > + }; > > > + > > > + partition@2880000 { > > > + label =3D "tclinux_slave"; > > > + reg =3D <0x02880000 0x02800000>; > > > + }; > > > + > > > + partition@5080000 { > > > + label =3D "rootfs_data"; > > > + reg =3D <0x5080000 0x00800000>; > > > + }; > > > + > > > + partition@ffffffff { > > > + compatible =3D "airoha,dynamic-art"; > > > + label =3D "art"; > > > + reg =3D <0xffffffff 0x00300000>; =20 > >=20 > > I'm a little bit puzzled by this kind of information which is known to > > be wrong. As the partition offset and size must be dynamically > > calculated, this reg property (as well as the size parameter of the > > previous one) are notably wrong. I guess we are not fully constrained > > by the fixed-partitions schema here, so could we avoid the reg property > > in the airoha,dynamic-art partition? Maybe we also need a #define for a > > specific placeholder in the penultimate reg property too (for the size). > > =20 >=20 > Maybe instead of reg we can use a property like size? >=20 > Can you better elaborate the suggestion about the #define? >=20 > Do you mean for case where the last partition might overlap > with the penultimate? Honestly in such case I would error hard, that > case happen when too much space is reserved and that is a > misconfiguration of the system (developer error) That's not what I mean. In the above case you say partition "partition@5080000" is 0x800000 bytes long. This is obviously wrong otherwise you would know where the art partition starts. And right after you're saying partition "partition@ffffffff" starts at 0xffffffff and is 0x300000 bytes long. This is also wrong because 0xffffffff is not a valid start address and IIUC 0x300000 is also unknown and dynamically derived. So for the art partition my advise if you know nothing about the start/length is to just skip the reg property. For the previous partition I'd maybe use a definition (whose name is to discuss) instead of the wrong size argument (the start offset being correct on his side). Thanks, Miqu=C3=A8l