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 39F01C433EF for ; Tue, 7 Dec 2021 08:54:27 +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=/2hYB0r+gzskP7PgFDYpTxw24m79e1N5i63pHlffwFE=; b=kk4kWDL6I5EKZX 3fcWDEX1f3pZqq8JOG4MhZco/R9YTAs6nDRWMqorwwh8us9Bj/ePYGNJkaKh6AUed5Qv3pD2u2snw BDjTIkQxRfuPzInv4GFIPMdprjKvAdgqb8i6OAnzdeHICD8Bs73uvlzjR2OJlYh7H7frAs6a0qn1A DnxknnQdeQ8awQLyadc4Naz8/1sYcsp4gNDPymRfiRW9SYV+YN7WBOeAMLJIEV1dLHE2QfYq0ODpa XXZoB6zJmN+afGoDOhfzGYVIqZHoFceszwBc2hrKnNAmOn9xtIPswj7tEyAOGz9j5s4r9ul6/jlD/ K48/MI1uXU0lxyEfjjmA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1muWEM-007dZm-Vx; Tue, 07 Dec 2021 08:53:47 +0000 Received: from relay4-d.mail.gandi.net ([217.70.183.196]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1muVyf-007YmB-1s for linux-mtd@lists.infradead.org; Tue, 07 Dec 2021 08:37:35 +0000 Received: (Authenticated sender: miquel.raynal@bootlin.com) by relay4-d.mail.gandi.net (Postfix) with ESMTPSA id 533BDE0014; Tue, 7 Dec 2021 08:37:29 +0000 (UTC) Date: Tue, 7 Dec 2021 09:37:27 +0100 From: Miquel Raynal To: Pratyush Yadav Cc: , , , , , , , , , , Subject: Re: [PATCH v3 2/3] spi: dt-bindings: Describe stacked/parallel memories modes Message-ID: <20211207093727.5261fe66@xps13> In-Reply-To: <20211207075721.ghwbruqcgi36fk72@ti.com> References: <20211206095921.33302-1-miquel.raynal@bootlin.com> <20211206095921.33302-3-miquel.raynal@bootlin.com> <20211207071406.c2ajc3shqybevvjj@ti.com> <0d97a420-685e-5120-3c09-d433382c02aa@microchip.com> <20211207075721.ghwbruqcgi36fk72@ti.com> 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-20211207_003733_438229_68E057B5 X-CRM114-Status: GOOD ( 39.23 ) 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 SGkgUHJhdHl1c2ggJiBUdWRvciwKCnAueWFkYXZAdGkuY29tIHdyb3RlIG9uIFR1ZSwgNyBEZWMg MjAyMSAxMzoyNzoyMyArMDUzMDoKCj4gT24gMDcvMTIvMjEgMDc6MzVBTSwgVHVkb3IuQW1iYXJ1 c0BtaWNyb2NoaXAuY29tIHdyb3RlOgo+ID4gT24gMTIvNy8yMSA5OjE0IEFNLCBQcmF0eXVzaCBZ YWRhdiB3cm90ZTogIAo+ID4gPiBFWFRFUk5BTCBFTUFJTDogRG8gbm90IGNsaWNrIGxpbmtzIG9y IG9wZW4gYXR0YWNobWVudHMgdW5sZXNzIHlvdSBrbm93IHRoZSBjb250ZW50IGlzIHNhZmUKPiA+ ID4gCj4gPiA+IE9uIDA2LzEyLzIxIDEwOjU5QU0sIE1pcXVlbCBSYXluYWwgd3JvdGU6ICAKPiA+ ID4+IERlc2NyaWJlIHR3byBuZXcgbWVtb3JpZXMgbW9kZXM6Cj4gPiA+PiAtIEEgc3RhY2tlZCBt b2RlIHdoZW4gdGhlIGJ1cyBpcyBjb21tb24gYnV0IHRoZSBhZGRyZXNzIHNwYWNlIGV4dGVuZGVk Cj4gPiA+PiAgIHdpdGggYW4gYWRkaXRpbmFscyB3aXJlcy4KPiA+ID4+IC0gQSBwYXJhbGxlbCBt b2RlIHdpdGggcGFyYWxsZWwgYnVzc2VzIGFjY2Vzc2luZyBwYXJhbGxlbCBmbGFzaGVzIHdoZXJl Cj4gPiA+PiAgIHRoZSBkYXRhIGlzIHNwcmVhZC4KPiA+ID4+Cj4gPiA+PiBTaWduZWQtb2ZmLWJ5 OiBNaXF1ZWwgUmF5bmFsIDxtaXF1ZWwucmF5bmFsQGJvb3RsaW4uY29tPgo+ID4gPj4gLS0tCj4g PiA+PiAgLi4uL2JpbmRpbmdzL3NwaS9zcGktcGVyaXBoZXJhbC1wcm9wcy55YW1sICAgIHwgMjEg KysrKysrKysrKysrKysrKysrKwo+ID4gPj4gIDEgZmlsZSBjaGFuZ2VkLCAyMSBpbnNlcnRpb25z KCspCj4gPiA+Pgo+ID4gPj4gZGlmZiAtLWdpdCBhL0RvY3VtZW50YXRpb24vZGV2aWNldHJlZS9i aW5kaW5ncy9zcGkvc3BpLXBlcmlwaGVyYWwtcHJvcHMueWFtbCBiL0RvY3VtZW50YXRpb24vZGV2 aWNldHJlZS9iaW5kaW5ncy9zcGkvc3BpLXBlcmlwaGVyYWwtcHJvcHMueWFtbAo+ID4gPj4gaW5k ZXggNWRkMjA5MjA2ZTg4Li4xM2FhNmEyMzc0YzkgMTAwNjQ0Cj4gPiA+PiAtLS0gYS9Eb2N1bWVu dGF0aW9uL2RldmljZXRyZWUvYmluZGluZ3Mvc3BpL3NwaS1wZXJpcGhlcmFsLXByb3BzLnlhbWwK PiA+ID4+ICsrKyBiL0RvY3VtZW50YXRpb24vZGV2aWNldHJlZS9iaW5kaW5ncy9zcGkvc3BpLXBl cmlwaGVyYWwtcHJvcHMueWFtbAo+ID4gPj4gQEAgLTgyLDYgKzgyLDI3IEBAIHByb3BlcnRpZXM6 Cj4gPiA+PiAgICAgIGRlc2NyaXB0aW9uOgo+ID4gPj4gICAgICAgIERlbGF5LCBpbiBtaWNyb3Nl Y29uZHMsIGFmdGVyIGEgd3JpdGUgdHJhbnNmZXIuCj4gPiA+Pgo+ID4gPj4gKyAgc3RhY2tlZC1t ZW1vcmllczoKPiA+ID4+ICsgICAgdHlwZTogYm9vbGVhbiAgCj4gPiA+IAo+ID4gPiBJIGRvbid0 IHRoaW5rIGEgYm9vbGVhbiBpcyBlbm91Z2ggdG8gY29tcGxldGVseSBkZXNjcmliZSB0aGUgbWVt b3J5Lgo+ID4gPiBTdXJlLCB5b3Ugc2F5IHRoZSBtZW1vcmllcyBhcmUgc3RhY2tlZCwgYnV0IHdo ZXJlIGRvIHlvdSBzcGVjaWZ5IHdoZW4gdG8KPiA+ID4gc3dpdGNoIHRoZSBDUz8gVGhleSBjb3Vs ZCBiZSB0d28gNTEyIE1pQiBtZW1vcmllcywgdHdvIDEgR2lCIG1lbW9yaWVzLAo+ID4gPiBvciBv bmUgNTEyIE1pQiBhbmQgb25lIDI1NiBNaUIuICAKPiA+IAo+ID4gSWYgdGhlIG11bHRpLWRpZSBm bGFzaCBjb250YWlucyBpZGVudGljYWwgZGllcyB0aGVuIHRoZSBkaWUgYm91bmRhcnkgY2FuIGJl Cj4gPiBkZXRlcm1pbmVkIHdpdGggZmxhc2hfc2l6ZSAvIG51bWJlcl9vZl9jcy4gQXJlIHRoZXJl IGFueSBtdWx0aSBkaWUgZmxhc2hlcwo+ID4gd2l0aCBkaWZmZXJlbnQgdHlwZXMgb2YgZGllcz8g IAo+IAo+IFRoZSB3YXkgSSBzZWUgaXQsIGEgbXVsdGktZGllIGZsYXNoIGlzIG5vdCBtdWNoIGRp ZmZlcmVudCBmcm9tIDIgCj4gaW5kZXBlbmRlbnQgZmxhc2hlcyBhdHRhY2hlZCB0byB0aGUgc2Ft ZSBTUEkgYnVzLiBTbyBpZiB3ZSBhcmUgZ29pbmcgdG8gCj4gaW1wbGVtZW50IHRoaXMgZmVhdHVy ZSwgSSB3YW50IGl0IHRvIGJlIGdlbmVyaWMgZW5vdWdoIHRvIGFsbG93IAo+IHN1cHBvcnRpbmcg dGhpcyB0eXBlIG9mIGhhcmR3YXJlIHNldHVwIGFzIHdlbGwuCj4gCj4gSSBhbSBub3QgYXdhcmUg b2YgYW55IGZsYXNoZXMgd2l0aCBhIGRpZmZlcmVudCBDUyBmb3IgZWFjaCBkaWUgKHRoYXQgCj4g aXNuJ3QgaGFuZGxlZCBieSB0aGUgZmxhc2ggaW50ZXJuYWxseSksIGxldCBhbG9uZSB3aXRoIGRp ZmZlcmVudCB0eXBlcyAKPiBvZiBkaWVzLiBJSVJDIGZyb20gb3VyIElSQyBjb252ZXJzYXRpb24s IE1pcXVlbCdzIHVzZSBjYXNlIHdhcyB1c2luZyAyIAo+IHNtYWxsZXIgaWRlbnRpY2FsIGZsYXNo ZXMgY29ubmVjdGVkIHRvIHRoZSBzYW1lIFNQSSBidXMgd2l0aCAxIENTIGVhY2guIAo+IERvIEkg cmVtZW1iZXIgdGhpcyByaWdodCBNaXF1ZWw/CgpJIG1hZGUgdGhlIGFzc3VtcHRpb24gdGhhdCBk aWVzIHdvdWxkIGJlIGlkZW50aWNhbCBpbiBvcmRlciB0byB1c2UgdGhpcwptb2RlLiBIb3dldmVy LCBpZiB5b3UgdGhpbmsgdGhpcyBpcyB0b28gcmlza3kgSSBzZWUgdHdvIGFsdGVybmF0aXZlczoK KiBLZWVwIHRoZSBiaW5kaW5ncyBhcyBJIHByb3Bvc2VkIGFuZCBpZiB3ZSBldmVyIGhhdmUgdGhl IGNhc2UsIGFkZAogIGFub3RoZXIgcHJvcGVydHksIHNvbWV0aGluZyBsaWtlOgoJc3RhY2tlZC1t ZW1vcmllczsKCXN0YWNrZWQtc2l6ZXMgPSA8eD4sIDx5PjsKKiBNZXJnZSB0aGVzZSB0d28gcHJv cGVydGllcyBpbnRvIG9uZToKCXN0YWNrZWQtbWVtb3JpZXMgPSA8eD4sIDx5PjsKCkJ1dCBUQkgg SSBwcmVmZXIgdGhlIGZvcm1lciBzb2x1dGlvbiBmb3IgdGhlc2UgdHdvIHJlYXNvbnM6CjEvIFlv dSBuZWVkIHRvIGtub3cgdGhlIGRldmljZXMgZXhhY3QgZ2VvbWV0cnkgd2hlbiB3cml0aW5nIHRo ZQogICBiaW5kaW5ncyB3aGlsZSB0aGlzIGlzIHNvbWV0aGluZyB0aGF0IGlzIHVzdWFsbHkgbGV0 IHRvIHRoZSBjb3JlIGFuZAogICB0aGUgaGFyZHdhcmUgZGVzaWduZXJzLgoyLyBJIGFtIG5vdCBz dXJlIHRoaXMgaXMgcmVhbGx5IGEgdmFsaWQgdXNlIGNhc2UuIElmIHdlIGV2ZXIgbmVlZCB0bwog ICBjb25jYXRlbmF0ZSB0d28gZGV2aWNlcywgaW4gcGFydGljdWxhciBpZiB0aGV5IGFyZSBkaWZm ZXJlbnQsIEkKICAgd291bGQgcHJlZmVyIHJldml2aW5nIHRoZSBtdGQtY29uY2F0IHNlcmllcyB3 aGljaCwgYmVzaWRlcyBsYWNraW5nIGEKICAgZHluYW1pYyBkaXNjb3ZlcnkgZmVhdHVyZSwgaXMg YWxtb3N0IHJlYWR5IHRvIGJlIHVzZWQuIFBsdXMsIGFkZGluZwogICB0b28gbXVjaCBjb21wbGV4 aXR5IHRvIHRoZSBjb3JlIGxvZ2ljIChzdWNoIGFzIGhhbmRsaW5nIGRpZmZlcmVudAogICBkaWUg c2l6ZXMpIG1pZ2h0IGltcGFjdCBuZWdhdGl2ZWx5IHRoZSBvdmVyYWxsIHBlcmZvcm1hbmNlcyBl dmVuIGZvcgogICBzaW1wbGVyIGRldmljZXMuCgo+ID4gPj4gKyAgICBkZXNjcmlwdGlvbjogU2V2 ZXJhbCBTUEkgbWVtb3JpZXMgY2FuIGJlIHdpcmVkIGluIHN0YWNrZWQgbW9kZS4KPiA+ID4+ICsg ICAgICBUaGlzIGJhc2ljYWxseSBtZWFucyB0aGF0IGVpdGhlciBhIGRldmljZSBmZWF0dXJlcyBz ZXZlcmFsIGNoaXAKPiA+ID4+ICsgICAgICBzZWxlY3RzLCBvciB0aGF0IGRpZmZlcmVudCBkZXZp Y2VzIG11c3QgYmUgc2VlbiBhcyBhIHNpbmdsZQo+ID4gPj4gKyAgICAgIGJpZ2dlciBjaGlwLiBU aGlzIGJhc2ljYWxseSBkb3VibGVzIChvciBtb3JlKSB0aGUgdG90YWwgYWRkcmVzcwo+ID4gPj4g KyAgICAgIHNwYWNlIHdpdGggb25seSBhIHNpbmdsZSBhZGRpdGlvbmFsIHdpcmUsIHdoaWxlIHN0 aWxsIG5lZWRpbmcKPiA+ID4+ICsgICAgICB0byByZXBlYXQgdGhlIGNvbW1hbmRzIHdoZW4gY3Jv c3NpbmcgYSBjaGlwIGJvdW5kYXJ5LiBYSVAgaXMKPiA+ID4+ICsgICAgICB1c3VhbGx5IG5vdCBz dXBwb3J0ZWQgaW4gdGhpcyBtb2RlLgo+ID4gPj4gKwo+ID4gPj4gKyAgcGFyYWxsZWwtbWVtb3Jp ZXM6Cj4gPiA+PiArICAgIHR5cGU6IGJvb2xlYW4gIAo+ID4gPiAKPiA+ID4gV2l0aCB0aGlzIEkg YXNzdW1lIGJvdGggbWVtb3JpZXMgaGF2ZSB0byBiZSB0aGUgc2FtZSBzaXplPyAgCj4gPiAKPiA+ IEl0IGxvb2tzIGxpa2UgdGhlIGFzc3VtcHRpb24gZm9yIGJvdGggY2FzZXMgaXMgdGhhdCB0aGUg ZGllcyBhcmUgaWRlbnRpY2FsLiAgCj4gCj4gSSB3b3VsZCBsaWtlIHRvIF9ub3RfIGFzc3VtZSB0 aGF0IGZvciBzdGFja2VkLW1lbW9yaWVzLCB1bmxlc3MgCj4gaW1wbGVtZW50aW5nIHRoYXQgYmVj b21lcyB0b28gY29tcGxpY2F0ZWQuCj4gCj4gPiAgIAo+ID4gPiAgIAo+ID4gPj4gKyAgICBkZXNj cmlwdGlvbjogU2V2ZXJhbCBTUEkgbWVtb3JpZXMgY2FuIGJlIHdpcmVkIGluIHBhcmFsbGVsIG1v ZGUuCj4gPiA+PiArICAgICAgVGhlIGRldmljZXMgYXJlIHBoeXNpY2FsbHkgb24gYSBkaWZmZXJl bnQgYnVzZXMgYnV0IHdpbGwgYWx3YXlzCj4gPiA+PiArICAgICAgYWN0IHN5bmNocm9ub3VzbHkg YXMgZWFjaCBkYXRhIHdvcmQgaXMgc3ByZWFkIGFjcm9zcyB0aGUKPiA+ID4+ICsgICAgICBkaWZm ZXJlbnQgbWVtb3JpZXMgKGVnLiBldmVuIGJpdHMgYXJlIHN0b3JlZCBpbiBvbmUgbWVtb3J5LCBv ZGQKPiA+ID4+ICsgICAgICBiaXRzIGluIHRoZSBvdGhlcikuIFRoaXMgYmFzaWNhbGx5IGRvdWJs ZXMgdGhlIGFkZHJlc3Mgc3BhY2UgYW5kCj4gPiA+PiArICAgICAgdGhlIHRocm91Z2hwdXQgd2hp bGUgZ3JlYXRseSBjb21wbGV4aWZ5aW5nIHRoZSB3aXJpbmcgYmVjYXVzZSBhcwo+ID4gPj4gKyAg ICAgIG1hbnkgYnVzc2VzIGFzIGRldmljZXMgbXVzdCBiZSB3aXJlZC4gWElQIGlzIHVzdWFsbHkg bm90Cj4gPiA+PiArICAgICAgc3VwcG9ydGVkIGluIHRoaXMgbW9kZS4KPiA+ID4+ICsKPiA+ID4+ ICAjIFRoZSBjb250cm9sbGVyIHNwZWNpZmljIHByb3BlcnRpZXMgZ28gaGVyZS4KPiA+ID4+ICBh bGxPZjoKPiA+ID4+ICAgIC0gJHJlZjogY2Rucyxxc3BpLW5vci1wZXJpcGhlcmFsLXByb3BzLnlh bWwjCj4gPiA+PiAtLQo+ID4gPj4gMi4yNy4wCj4gPiA+PiAgCj4gPiA+IAo+ID4gPiAtLQo+ID4g PiBSZWdhcmRzLAo+ID4gPiBQcmF0eXVzaCBZYWRhdgo+ID4gPiBUZXhhcyBJbnN0cnVtZW50cyBJ bmMuCj4gPiA+ICAgCj4gPiAgIAo+IAoKClRoYW5rcywKTWlxdcOobAoKX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCkxpbnV4IE1URCBkaXNjdXNz aW9uIG1haWxpbmcgbGlzdApodHRwOi8vbGlzdHMuaW5mcmFkZWFkLm9yZy9tYWlsbWFuL2xpc3Rp bmZvL2xpbnV4LW10ZC8K 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 29DBDC433EF for ; Tue, 7 Dec 2021 08:37:33 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229545AbhLGIlC convert rfc822-to-8bit (ORCPT ); Tue, 7 Dec 2021 03:41:02 -0500 Received: from relay4-d.mail.gandi.net ([217.70.183.196]:59543 "EHLO relay4-d.mail.gandi.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233093AbhLGIlC (ORCPT ); Tue, 7 Dec 2021 03:41:02 -0500 Received: (Authenticated sender: miquel.raynal@bootlin.com) by relay4-d.mail.gandi.net (Postfix) with ESMTPSA id 533BDE0014; Tue, 7 Dec 2021 08:37:29 +0000 (UTC) Date: Tue, 7 Dec 2021 09:37:27 +0100 From: Miquel Raynal To: Pratyush Yadav Cc: , , , , , , , , , , Subject: Re: [PATCH v3 2/3] spi: dt-bindings: Describe stacked/parallel memories modes Message-ID: <20211207093727.5261fe66@xps13> In-Reply-To: <20211207075721.ghwbruqcgi36fk72@ti.com> References: <20211206095921.33302-1-miquel.raynal@bootlin.com> <20211206095921.33302-3-miquel.raynal@bootlin.com> <20211207071406.c2ajc3shqybevvjj@ti.com> <0d97a420-685e-5120-3c09-d433382c02aa@microchip.com> <20211207075721.ghwbruqcgi36fk72@ti.com> 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 Pratyush & Tudor, p.yadav@ti.com wrote on Tue, 7 Dec 2021 13:27:23 +0530: > On 07/12/21 07:35AM, Tudor.Ambarus@microchip.com wrote: > > On 12/7/21 9:14 AM, Pratyush Yadav wrote: > > > EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe > > > > > > On 06/12/21 10:59AM, Miquel Raynal wrote: > > >> Describe two new memories modes: > > >> - A stacked mode when the bus is common but the address space extended > > >> with an additinals wires. > > >> - A parallel mode with parallel busses accessing parallel flashes where > > >> the data is spread. > > >> > > >> Signed-off-by: Miquel Raynal > > >> --- > > >> .../bindings/spi/spi-peripheral-props.yaml | 21 +++++++++++++++++++ > > >> 1 file changed, 21 insertions(+) > > >> > > >> diff --git a/Documentation/devicetree/bindings/spi/spi-peripheral-props.yaml b/Documentation/devicetree/bindings/spi/spi-peripheral-props.yaml > > >> index 5dd209206e88..13aa6a2374c9 100644 > > >> --- a/Documentation/devicetree/bindings/spi/spi-peripheral-props.yaml > > >> +++ b/Documentation/devicetree/bindings/spi/spi-peripheral-props.yaml > > >> @@ -82,6 +82,27 @@ properties: > > >> description: > > >> Delay, in microseconds, after a write transfer. > > >> > > >> + stacked-memories: > > >> + type: boolean > > > > > > I don't think a boolean is enough to completely describe the memory. > > > Sure, you say the memories are stacked, but where do you specify when to > > > switch the CS? They could be two 512 MiB memories, two 1 GiB memories, > > > or one 512 MiB and one 256 MiB. > > > > If the multi-die flash contains identical dies then the die boundary can be > > determined with flash_size / number_of_cs. Are there any multi die flashes > > with different types of dies? > > The way I see it, a multi-die flash is not much different from 2 > independent flashes attached to the same SPI bus. So if we are going to > implement this feature, I want it to be generic enough to allow > supporting this type of hardware setup as well. > > I am not aware of any flashes with a different CS for each die (that > isn't handled by the flash internally), let alone with different types > of dies. IIRC from our IRC conversation, Miquel's use case was using 2 > smaller identical flashes connected to the same SPI bus with 1 CS each. > Do I remember this right Miquel? I made the assumption that dies would be identical in order to use this mode. However, if you think this is too risky I see two alternatives: * Keep the bindings as I proposed and if we ever have the case, add another property, something like: stacked-memories; stacked-sizes = , ; * Merge these two properties into one: stacked-memories = , ; But TBH I prefer the former solution for these two reasons: 1/ You need to know the devices exact geometry when writing the bindings while this is something that is usually let to the core and the hardware designers. 2/ I am not sure this is really a valid use case. If we ever need to concatenate two devices, in particular if they are different, I would prefer reviving the mtd-concat series which, besides lacking a dynamic discovery feature, is almost ready to be used. Plus, adding too much complexity to the core logic (such as handling different die sizes) might impact negatively the overall performances even for simpler devices. > > >> + description: Several SPI memories can be wired in stacked mode. > > >> + This basically means that either a device features several chip > > >> + selects, or that different devices must be seen as a single > > >> + bigger chip. This basically doubles (or more) the total address > > >> + space with only a single additional wire, while still needing > > >> + to repeat the commands when crossing a chip boundary. XIP is > > >> + usually not supported in this mode. > > >> + > > >> + parallel-memories: > > >> + type: boolean > > > > > > With this I assume both memories have to be the same size? > > > > It looks like the assumption for both cases is that the dies are identical. > > I would like to _not_ assume that for stacked-memories, unless > implementing that becomes too complicated. > > > > > > > > >> + description: Several SPI memories can be wired in parallel mode. > > >> + The devices are physically on a different buses but will always > > >> + act synchronously as each data word is spread across the > > >> + different memories (eg. even bits are stored in one memory, odd > > >> + bits in the other). This basically doubles the address space and > > >> + the throughput while greatly complexifying the wiring because as > > >> + many busses as devices must be wired. XIP is usually not > > >> + supported in this mode. > > >> + > > >> # The controller specific properties go here. > > >> allOf: > > >> - $ref: cdns,qspi-nor-peripheral-props.yaml# > > >> -- > > >> 2.27.0 > > >> > > > > > > -- > > > Regards, > > > Pratyush Yadav > > > Texas Instruments Inc. > > > > > > Thanks, Miquèl