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 20065C47258 for ; Thu, 25 Jan 2024 09:21:16 +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=TRZBpQFgbhy+FSAlAxuWX/dbquwxtTsN6j26+f1UwEk=; b=dc9CFZlNBgqB7c 1MsfvX6RDvhUA9aniAEYGjl//UzFGNlmdouTOtDLYxoaKbOpHcUCgt+5MGQ2FWHgBqWrwG5plSJ1N mCw8uyyjrZVt7DDxAgVLMouCP/4UtfgjGZhmSCddj12Kz7A787Q2AE42WGVOxd0sBFo/ihYebLS+H ODrqyzKeLzQZ5Map5Vmrk/HBMDQa7nSY06p5eHvfJOw5XutW39b5dK8vIFQMbDBHTA4awFykttDl3 lMKNqC0U+sGmPHixh0RWDNO9uKikwQezdsV0bkT9VPgAEwCaKQQ6jwy1nHKM3ohd2Mxn0Zo54JM0j /DT+Vd9AsjyvF1+RTcrg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1rSvv4-007WF2-1y; Thu, 25 Jan 2024 09:21:10 +0000 Received: from relay8-d.mail.gandi.net ([217.70.183.201]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1rSvv0-007WDn-0x; Thu, 25 Jan 2024 09:21:08 +0000 Received: by mail.gandi.net (Postfix) with ESMTPSA id DA3881BF219; Thu, 25 Jan 2024 09:20:59 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1706174461; 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=eKjAu14meoD0UWLGa1Hctnp45OsMD9+KH2IhUcbWrL4=; b=VQ86jc4jMi7kZpSX93HTuIEk4dHcoUQO2jYVvHJKrFbWKkXwZunjal+i7HO2/oynf2bGKQ CYDTSSZAWfKlxhC0kyB9aI8HFAj//uey3jcaKSEEz/1HaS2w6h2ETfSokmazYkVCoJN4pd 9QXJE49MHmPyiPllw9VhmA2BRY5LEvfRdNWiuXqt33gGxfFbwv7BZcjNxLofLpQbSeSc+6 tD9beY3Sn/veWmIUV2Ys8WpFFrOjXXy1++Q9ki3bL0dINJ1K0ewx2Ag+0RXhMRaZ875MXs +YHWh7+uEfOy3+IwWuICh8XKLw/5yed/TAzlzTOX+PxM1i1sqZwE8qGfLKzE+g== Date: Thu, 25 Jan 2024 10:20:58 +0100 From: Miquel Raynal To: William Zhang Cc: Florian Fainelli , David Regan , dregan@mail.com, richard@nod.at, vigneshr@ti.com, robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org, computersforpeace@gmail.com, kdasu.kdev@gmail.com, linux-mtd@lists.infradead.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, joel.peshkin@broadcom.com, tomer.yacoby@broadcom.com, dan.beygelman@broadcom.com, anand.gore@broadcom.com, kursad.oney@broadcom.com, rafal@milecki.pl, bcm-kernel-feedback-list@broadcom.com, andre.przywara@arm.com, baruch@tkos.co.il, linux-arm-kernel@lists.infradead.org, dan.carpenter@linaro.org Subject: Re: [PATCH v3 02/10] ARM: dts: broadcom: bcmbca: Add NAND controller node Message-ID: <20240125102058.5ae46d8a@xps-13> In-Reply-To: <722d6bdf-9a43-436d-a9e2-4c21e1deb8c3@broadcom.com> References: <20240124030458.98408-1-dregan@broadcom.com> <20240124030458.98408-3-dregan@broadcom.com> <20240124183008.04a1bcb0@xps-13> <2c10a764-f74f-45b2-8bba-77c40468f4b5@broadcom.com> <85c97d00-a973-46c6-974c-3dfa587ae873@broadcom.com> <722d6bdf-9a43-436d-a9e2-4c21e1deb8c3@broadcom.com> 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-20240125_012106_701873_8368A74A X-CRM114-Status: GOOD ( 25.16 ) 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 SGkgV2lsbGlhbSwKCj4gPj4+PiArwqDCoMKgwqDCoMKgwqAgbmFuZF9jb250cm9sbGVyOiBuYW5k LWNvbnRyb2xsZXJAMTgwMCB7Cj4gPj4+PiArwqDCoMKgwqDCoMKgwqDCoMKgwqDCoCAjYWRkcmVz cy1jZWxscyA9IDwxPjsKPiA+Pj4+ICvCoMKgwqDCoMKgwqDCoMKgwqDCoMKgICNzaXplLWNlbGxz ID0gPDA+Owo+ID4+Pj4gK8KgwqDCoMKgwqDCoMKgwqDCoMKgwqAgY29tcGF0aWJsZSA9ICJicmNt LG5hbmQtYmNtNjMxMzgiLCA+Pj4+ICJicmNtLGJyY21uYW5kLXY3LjEiLCAiYnJjbSxicmNtbmFu ZCI7Cj4gPj4+PiArwqDCoMKgwqDCoMKgwqDCoMKgwqDCoCByZWcgPSA8MHgxODAwIDB4NjAwPiwg PDB4MjAwMCAweDEwPjsKPiA+Pj4+ICvCoMKgwqDCoMKgwqDCoMKgwqDCoMKgIHJlZy1uYW1lcyA9 ICJuYW5kIiwgIm5hbmQtaW50LWJhc2UiOwo+ID4+Pj4gK8KgwqDCoMKgwqDCoMKgwqDCoMKgwqAg YnJjbSxuYW5kLXVzZS13cCA9IDwwPjsKPiA+Pj4+ICvCoMKgwqDCoMKgwqDCoMKgwqDCoMKgIHN0 YXR1cyA9ICJkaXNhYmxlZCI7Cj4gPj4+PiArCj4gPj4+PiArwqDCoMKgwqDCoMKgwqDCoMKgwqDC oCBuYW5kY3M6IG5hbmRAMCB7Cj4gPj4+PiArwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKg IGNvbXBhdGlibGUgPSAiYnJjbSxuYW5kY3MiOwo+ID4+Pj4gK8KgwqDCoMKgwqDCoMKgwqDCoMKg wqDCoMKgwqDCoCByZWcgPSA8MD47Cj4gPj4+PiArwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDC oMKgIG5hbmQtb24tZmxhc2gtYmJ0Owo+ID4+Pj4gK8KgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKg wqDCoCBicmNtLG5hbmQtZWNjLXVzZS1zdHJhcDsgIAo+ID4+Pgo+ID4+PiBEZXNjcmliaW5nIHRo ZSBOQU5EIGNoaXAgaW4gYSBTb0MgRFRTSSBkb2VzIG5vdCBsb29rIHJlbGV2YW50IHRvIG1lLgo+ ID4+PiBFdmVuIG1vcmUgaWYgeW91IGFkZCBzb21ldGhpbmcgbGlrZSB0aGlzIG5hbmQtZWNjLXVz ZS1zdHJhcCBzZXR0aW5nCj4gPj4+IHdoaWNoIGlzIHZlcnkgYm9hcmQgZGVwZW5kZW50Lgo+ID4+ PiAgCj4gPj4gSSBhbSBub3Qgc3VyZSBpZiBJIHVuZGVyc3RhbmQgeW91IGNvbW1lbnRzIGNvcnJl Y3RseSBidXQgYXJlIHlvdSA+PiBzdWdnZXN0aW5nIHRvIHB1dCB0aGlzIHdob2xlIG5hbmQgY29u dHJvbGxlciBub2RlIGludG8gZWFjaCBib2FyZCBkdHM/ID4+IFdlIGhhdmUgb3RoZXIgaXAgYmxv Y2sgbm9kZXMgbGlrZSBTUEksIHVhcnQgaW4gdGhpcyBzYW1lIHNvYyBkdHNpIGZpbGUgPj4gdG9v LsKgIEZvciBhbGwgdGhlIGJjbWJjYSBzb2MgZHRzaSBJIGFtIHVwZGF0aW5nIGhlcmUoYW5kIGl0 cyBib2FyZCA+PiBkZXNpZ24pLCB3ZSBhbHdheXMgdXNlIHRoZSBzdHJhcCB0byBmb3IgZWNjIHNl dHRpbmcuwqAgU28gSSB0aG91Z2h0IGl0ID4+IHNob3VsZCBiZSBva2F5IHRvIHB1dCBicmNtLG5h bmQtZWNjLXVzZS1zdHJhcCBpbiB0aGUgZGVmYXVsdCBkdHNpID4+IGZpbGUuIEZvciBhbnkgYm9h cmQgdGhhdCB1c2VzIHRoZSByYXcgbmFuZCBuYW5kLWVjYyBwcm9wZXJ0eSwgdGhlID4+IGJvYXJk IGR0cyBjYW4gZG8gc28gYW5kIG92ZXJyaWRlIHRoZSBicmNtLG5hbmQtZWNjLXVzZS1zdHJhcCBz ZXR0aW5nLiAgCj4gPiAKPiA+IEkgcmVhZCBNaXF1ZWwncyBjb21tZW50IGFzIG1lYW5pbmcgdGhh dCB0aGUgbmFuZGNzIGFrYSB0aGUgTkFORCA+IGNoaXAvZmxhc2ggcGFydCBkZXNjcmlwdGlvbiBz aG91bGQgYmUgaW4gdGhlIGJvYXJkIC5kdHMgZmlsZSwgd2hpbGUgdGhlID4gY29udHJvbGxlciBp dHNlbGYgY2FuIHJlbWFpbiBpbiB0aGUgLmR0c2kgZmlsZSB3aXRoIGl0cyBzdGF0dXMgPSA+ICJk aXNhYmxlZCIgcHJvcGVydHkuCj4gPiAKPiA+IEFyZSB0aGVyZSBjdXN0b21lciBib2FyZHMsIHRo YXQgaXMgbm9uIHJlZmVyZW5jZSBib2FyZHMgdGhhdCBtaWdodCBjaG9zZSA+IGEgZGlmZmVyZW50 IGNoaXAgc2VsZWN0IG51bWJlciBhbmQvb3Igbm90IHVzZSB0aGUgc3RyYXAgc2V0dGluZ3M/ICAK PiBJbiBCQ01CQ0EgU29DLCB0aGVyZSBpcyBvbmx5IG9uZSBjcyBhbmQgY3VzdG9tZXIgZGVzaWdu IGFsc28gaGF2ZSB0byB1c2Ugc3RyYXAgZm9yIHRoZSBib290cm9tIHRvIGJvb3QgdXAgcHJvcGVy bHkuICBUaGV5IGNhbiBvdmVycmlkZSBpdCB3aXRoIGR0cyBpbiBsaW51eCBidXQgSSBkb24ndCB0 aGluayBhbnkgY3VzdG9tZXIgd291bGQgZG8gdGhhdC4KPiAKPiBNYXliZSB0aGUgbmFuZC1vbi1m bGFzaC1iYnQgY291bGQgYmUgcG9zc2libGUgaXRlbSB0aGF0IGN1c3RvbWVyIG1heSBoYXZlIHRv IHNldCBpdCBkaWZmZXJlbnRseSBpZiB0aGV5IGRvbid0IGZvbGxvdyBvdXIgcmVmZXJlbmNlIHNv ZnR3YXJlIGRlc2lnbi4KPiAKPiBJIHdpbGwgbW92ZSB0aGUgbmFuZC1vbi1mbGFzaC1iYnQgdG8g dGhlIGJvYXJkIGR0cyBidXQgSSB3b3VsZCBsaWtlIHRvIGtlZXAgdGhlIG90aGVyIGRlZmF1bHQg bmFuZGNzIHNldHRpbmdzIGluIFNvQy5kc3RpIGlmIHRoYXQgaXMgbm90IHRvbyBvdXQgb2YgdGhl IGNvbnZlbnRpb25hbCBydWxlIGFuZCBNaXF1ZWwgaXMgb2theSB3aXRoIGl0LgoKSSB0aGluayB0 aGVyZSBpcyBhIGdsb2JhbCBtaXN1bmRlcnN0YW5kaW5nIHJlZ2FyZGluZyB0aGUgdXNlIG9mIHRo ZQpuYW5kLWVjYy0qIHByb3BlcnRpZXMuIFRoZXNlIGFyZSBub3QgdGhlIGRlZmF1bHQuIFRoZSBk ZWZhdWx0IGlzIHRoZSBPUwpjaG9pY2UgYW5kIGRlcGVuZHMgb24gdGhlIE5BTkQgY2FwYWJpbGl0 aWVzLiBUaGUgT1Mgd2lsbCBhbHdheXMgdHJ5IHRvCm1hdGNoIHRoZSBjbG9zZXN0IEVDQyBzZXR0 aW5ncyBvZmZlcmVkIGJ5IHRoZSBlbmdpbmUsIGJhc2VkIG9uIHRoZSBOQU5ECmNoaXAgcmVxdWly ZW1lbnRzIHdoaWNoIGFyZSBkaXNjb3ZlcmFibGUuIElmIHlvdSB3YW50IHRvIG1heGltaXplIHlv dXIKc3RyZW5ndGgsIGl0IGlzIGFsc28gcG9zc2libGUgdG8gdGVsbCB0aGUgT1Mgd2l0aCBhIGRl ZGljYXRlZCAoZ2VuZXJpYykKcHJvcGVydHkuIEFuZCBvbmx5IGlmIHlvdSB3YW50IHNvbWV0aGlu ZyBkaWZmZXJlbnQsIHlvdSBtYXkgdXNlIHRoZXNlCnByb3BlcnRpZXMsIGJ1dCB0aGV5IHNob3Vs ZCBiZSB0aGUgZXhjZXB0aW9uIHJhdGhlciB0aGFuIHRoZSBydWxlLgoKT3ZlcnJpZGluZyB0aGlz IHdpdGggYSBzdHJhcCBpcyBhIGJhZCBoYXJkd2FyZSBkZXNpZ24gb24gY29tbWVyY2lhbApwcm9k dWN0cyBJTU8uIEkgYW0gdG90YWxseSBmaW5lIHdpdGggdGhlIGlkZWEgb2YgYSBzdHJhcCB0byBj aG9vc2UKdGhlIEVDQyBjb25maWd1cmF0aW9uIGZvciBkZXZlbG9wbWVudCBib2FyZHMvZXZhbHVh dGlvbiBraXRzLCBidXQgb25jZQp5b3UndmUgZGVjaWRlZCB3aGljaCBzZXR0aW5nIHlvdSB3YW50 IHlvdSBjYW5ub3QgY2hhbmdlIGl0IGZvciB0aGUKbGlmZXRpbWUgb2YgeW91ciBwcm9qZWN0IChv ciB3aXRoIGEgbG90IG9mIGRpZmZpY3VsdGllcykgc28gSSBkb24ndCBzZWUKdGhlIHBvaW50IG9m IHN1Y2ggYSBzdHJhcC4gU28gcmVhbGx5LCBJIGRvbid0IGxpa2UgdGhlIGlkZWEgb2YgZGVmaW5p bmcKYnkgZGVmYXVsdCBhIHZhcmlhYmxlIHdoaWNoIGFza3MgZm9yIGFuIG92ZXJyaWRlIG9mIHRo ZSBkZWZhdWx0cywgZXZlbgp0aG91Z2ggbWFueSBvZiB5b3VyIGN1c3RvbWVycyBtaWdodCB3YW50 IHRvIHVzZSB0aGF0LgoKU28sIGFueXRoaW5nIHRoYXQgaXMgZGVzaWduIGRlcGVuZGVudCAodGhl IGNoaXAgQ1MsIEVDQwpjb25maWd1cmF0aW9uLCBldGMpIHNob3VsZCBnbyBpbnRvIHRoZSBib2Fy ZCBEVFMsIGFuZCB3aGF0IGlzIFNvQwpyZWxhdGVkIGhhcmR3YXJlIChsaWtlIHRoZSBkZWZpbml0 aW9uIG9mIHRoZSBOQU5EIGNvbnRyb2xsZXIpIHNob3VsZApzdGF5IGluIHRoZSBEVFNJLCBhcyBw cm9wZXJseSBjbGFyaWZpZWQgYnkgRmxvcmlhbi4KClRoYW5rcywKTWlxdcOobAoKX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCkxpbnV4IE1URCBk aXNjdXNzaW9uIG1haWxpbmcgbGlzdApodHRwOi8vbGlzdHMuaW5mcmFkZWFkLm9yZy9tYWlsbWFu L2xpc3RpbmZvL2xpbnV4LW10ZC8K 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 30758C47258 for ; Thu, 25 Jan 2024 09:21: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=ScD0f6prNdTVMfu9CX5a75Cei1gjZXWfG8KqNdFDAR8=; b=ngpG8+aacdcHzn vqEip1HVJ60Yh8yB8/rQ8WbObIqzQvD2nPczuVXmCfUyl6L2mTOKgvWV6xKXlN33sirGp+8Pdjyu/ WH0NDryhVpG1ebrgSBba0U4+hOm87qbpbuU942xWOVkV4npR7PHGM5tAwxZJQ+YMh0F3ZEY7ndFPV r03deAmvwRHRzycC+xEtIsZJwGMn0c5chxAhYHrsVCxNT/jXVMLBx/Tds8Jm3YXB2w1PVUuVCkySw S8G/GUVEoGoyjWWWijdADlKUmGZQGasEwBkw5hk3R8AXOQEahe3siMh79gMS2JvhbD+GWI8mltnMi prxC+HdxPXd0jl3YE+JQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1rSvv3-007WEy-33; Thu, 25 Jan 2024 09:21:09 +0000 Received: from relay8-d.mail.gandi.net ([217.70.183.201]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1rSvv0-007WDn-0x; Thu, 25 Jan 2024 09:21:08 +0000 Received: by mail.gandi.net (Postfix) with ESMTPSA id DA3881BF219; Thu, 25 Jan 2024 09:20:59 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1706174461; 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=eKjAu14meoD0UWLGa1Hctnp45OsMD9+KH2IhUcbWrL4=; b=VQ86jc4jMi7kZpSX93HTuIEk4dHcoUQO2jYVvHJKrFbWKkXwZunjal+i7HO2/oynf2bGKQ CYDTSSZAWfKlxhC0kyB9aI8HFAj//uey3jcaKSEEz/1HaS2w6h2ETfSokmazYkVCoJN4pd 9QXJE49MHmPyiPllw9VhmA2BRY5LEvfRdNWiuXqt33gGxfFbwv7BZcjNxLofLpQbSeSc+6 tD9beY3Sn/veWmIUV2Ys8WpFFrOjXXy1++Q9ki3bL0dINJ1K0ewx2Ag+0RXhMRaZ875MXs +YHWh7+uEfOy3+IwWuICh8XKLw/5yed/TAzlzTOX+PxM1i1sqZwE8qGfLKzE+g== Date: Thu, 25 Jan 2024 10:20:58 +0100 From: Miquel Raynal To: William Zhang Cc: Florian Fainelli , David Regan , dregan@mail.com, richard@nod.at, vigneshr@ti.com, robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org, computersforpeace@gmail.com, kdasu.kdev@gmail.com, linux-mtd@lists.infradead.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, joel.peshkin@broadcom.com, tomer.yacoby@broadcom.com, dan.beygelman@broadcom.com, anand.gore@broadcom.com, kursad.oney@broadcom.com, rafal@milecki.pl, bcm-kernel-feedback-list@broadcom.com, andre.przywara@arm.com, baruch@tkos.co.il, linux-arm-kernel@lists.infradead.org, dan.carpenter@linaro.org Subject: Re: [PATCH v3 02/10] ARM: dts: broadcom: bcmbca: Add NAND controller node Message-ID: <20240125102058.5ae46d8a@xps-13> In-Reply-To: <722d6bdf-9a43-436d-a9e2-4c21e1deb8c3@broadcom.com> References: <20240124030458.98408-1-dregan@broadcom.com> <20240124030458.98408-3-dregan@broadcom.com> <20240124183008.04a1bcb0@xps-13> <2c10a764-f74f-45b2-8bba-77c40468f4b5@broadcom.com> <85c97d00-a973-46c6-974c-3dfa587ae873@broadcom.com> <722d6bdf-9a43-436d-a9e2-4c21e1deb8c3@broadcom.com> 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-20240125_012106_701873_8368A74A X-CRM114-Status: GOOD ( 25.16 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org SGkgV2lsbGlhbSwKCj4gPj4+PiArwqDCoMKgwqDCoMKgwqAgbmFuZF9jb250cm9sbGVyOiBuYW5k LWNvbnRyb2xsZXJAMTgwMCB7Cj4gPj4+PiArwqDCoMKgwqDCoMKgwqDCoMKgwqDCoCAjYWRkcmVz cy1jZWxscyA9IDwxPjsKPiA+Pj4+ICvCoMKgwqDCoMKgwqDCoMKgwqDCoMKgICNzaXplLWNlbGxz ID0gPDA+Owo+ID4+Pj4gK8KgwqDCoMKgwqDCoMKgwqDCoMKgwqAgY29tcGF0aWJsZSA9ICJicmNt LG5hbmQtYmNtNjMxMzgiLCA+Pj4+ICJicmNtLGJyY21uYW5kLXY3LjEiLCAiYnJjbSxicmNtbmFu ZCI7Cj4gPj4+PiArwqDCoMKgwqDCoMKgwqDCoMKgwqDCoCByZWcgPSA8MHgxODAwIDB4NjAwPiwg PDB4MjAwMCAweDEwPjsKPiA+Pj4+ICvCoMKgwqDCoMKgwqDCoMKgwqDCoMKgIHJlZy1uYW1lcyA9 ICJuYW5kIiwgIm5hbmQtaW50LWJhc2UiOwo+ID4+Pj4gK8KgwqDCoMKgwqDCoMKgwqDCoMKgwqAg YnJjbSxuYW5kLXVzZS13cCA9IDwwPjsKPiA+Pj4+ICvCoMKgwqDCoMKgwqDCoMKgwqDCoMKgIHN0 YXR1cyA9ICJkaXNhYmxlZCI7Cj4gPj4+PiArCj4gPj4+PiArwqDCoMKgwqDCoMKgwqDCoMKgwqDC oCBuYW5kY3M6IG5hbmRAMCB7Cj4gPj4+PiArwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKg IGNvbXBhdGlibGUgPSAiYnJjbSxuYW5kY3MiOwo+ID4+Pj4gK8KgwqDCoMKgwqDCoMKgwqDCoMKg wqDCoMKgwqDCoCByZWcgPSA8MD47Cj4gPj4+PiArwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDC oMKgIG5hbmQtb24tZmxhc2gtYmJ0Owo+ID4+Pj4gK8KgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKg wqDCoCBicmNtLG5hbmQtZWNjLXVzZS1zdHJhcDsgIAo+ID4+Pgo+ID4+PiBEZXNjcmliaW5nIHRo ZSBOQU5EIGNoaXAgaW4gYSBTb0MgRFRTSSBkb2VzIG5vdCBsb29rIHJlbGV2YW50IHRvIG1lLgo+ ID4+PiBFdmVuIG1vcmUgaWYgeW91IGFkZCBzb21ldGhpbmcgbGlrZSB0aGlzIG5hbmQtZWNjLXVz ZS1zdHJhcCBzZXR0aW5nCj4gPj4+IHdoaWNoIGlzIHZlcnkgYm9hcmQgZGVwZW5kZW50Lgo+ID4+ PiAgCj4gPj4gSSBhbSBub3Qgc3VyZSBpZiBJIHVuZGVyc3RhbmQgeW91IGNvbW1lbnRzIGNvcnJl Y3RseSBidXQgYXJlIHlvdSA+PiBzdWdnZXN0aW5nIHRvIHB1dCB0aGlzIHdob2xlIG5hbmQgY29u dHJvbGxlciBub2RlIGludG8gZWFjaCBib2FyZCBkdHM/ID4+IFdlIGhhdmUgb3RoZXIgaXAgYmxv Y2sgbm9kZXMgbGlrZSBTUEksIHVhcnQgaW4gdGhpcyBzYW1lIHNvYyBkdHNpIGZpbGUgPj4gdG9v LsKgIEZvciBhbGwgdGhlIGJjbWJjYSBzb2MgZHRzaSBJIGFtIHVwZGF0aW5nIGhlcmUoYW5kIGl0 cyBib2FyZCA+PiBkZXNpZ24pLCB3ZSBhbHdheXMgdXNlIHRoZSBzdHJhcCB0byBmb3IgZWNjIHNl dHRpbmcuwqAgU28gSSB0aG91Z2h0IGl0ID4+IHNob3VsZCBiZSBva2F5IHRvIHB1dCBicmNtLG5h bmQtZWNjLXVzZS1zdHJhcCBpbiB0aGUgZGVmYXVsdCBkdHNpID4+IGZpbGUuIEZvciBhbnkgYm9h cmQgdGhhdCB1c2VzIHRoZSByYXcgbmFuZCBuYW5kLWVjYyBwcm9wZXJ0eSwgdGhlID4+IGJvYXJk IGR0cyBjYW4gZG8gc28gYW5kIG92ZXJyaWRlIHRoZSBicmNtLG5hbmQtZWNjLXVzZS1zdHJhcCBz ZXR0aW5nLiAgCj4gPiAKPiA+IEkgcmVhZCBNaXF1ZWwncyBjb21tZW50IGFzIG1lYW5pbmcgdGhh dCB0aGUgbmFuZGNzIGFrYSB0aGUgTkFORCA+IGNoaXAvZmxhc2ggcGFydCBkZXNjcmlwdGlvbiBz aG91bGQgYmUgaW4gdGhlIGJvYXJkIC5kdHMgZmlsZSwgd2hpbGUgdGhlID4gY29udHJvbGxlciBp dHNlbGYgY2FuIHJlbWFpbiBpbiB0aGUgLmR0c2kgZmlsZSB3aXRoIGl0cyBzdGF0dXMgPSA+ICJk aXNhYmxlZCIgcHJvcGVydHkuCj4gPiAKPiA+IEFyZSB0aGVyZSBjdXN0b21lciBib2FyZHMsIHRo YXQgaXMgbm9uIHJlZmVyZW5jZSBib2FyZHMgdGhhdCBtaWdodCBjaG9zZSA+IGEgZGlmZmVyZW50 IGNoaXAgc2VsZWN0IG51bWJlciBhbmQvb3Igbm90IHVzZSB0aGUgc3RyYXAgc2V0dGluZ3M/ICAK PiBJbiBCQ01CQ0EgU29DLCB0aGVyZSBpcyBvbmx5IG9uZSBjcyBhbmQgY3VzdG9tZXIgZGVzaWdu IGFsc28gaGF2ZSB0byB1c2Ugc3RyYXAgZm9yIHRoZSBib290cm9tIHRvIGJvb3QgdXAgcHJvcGVy bHkuICBUaGV5IGNhbiBvdmVycmlkZSBpdCB3aXRoIGR0cyBpbiBsaW51eCBidXQgSSBkb24ndCB0 aGluayBhbnkgY3VzdG9tZXIgd291bGQgZG8gdGhhdC4KPiAKPiBNYXliZSB0aGUgbmFuZC1vbi1m bGFzaC1iYnQgY291bGQgYmUgcG9zc2libGUgaXRlbSB0aGF0IGN1c3RvbWVyIG1heSBoYXZlIHRv IHNldCBpdCBkaWZmZXJlbnRseSBpZiB0aGV5IGRvbid0IGZvbGxvdyBvdXIgcmVmZXJlbmNlIHNv ZnR3YXJlIGRlc2lnbi4KPiAKPiBJIHdpbGwgbW92ZSB0aGUgbmFuZC1vbi1mbGFzaC1iYnQgdG8g dGhlIGJvYXJkIGR0cyBidXQgSSB3b3VsZCBsaWtlIHRvIGtlZXAgdGhlIG90aGVyIGRlZmF1bHQg bmFuZGNzIHNldHRpbmdzIGluIFNvQy5kc3RpIGlmIHRoYXQgaXMgbm90IHRvbyBvdXQgb2YgdGhl IGNvbnZlbnRpb25hbCBydWxlIGFuZCBNaXF1ZWwgaXMgb2theSB3aXRoIGl0LgoKSSB0aGluayB0 aGVyZSBpcyBhIGdsb2JhbCBtaXN1bmRlcnN0YW5kaW5nIHJlZ2FyZGluZyB0aGUgdXNlIG9mIHRo ZQpuYW5kLWVjYy0qIHByb3BlcnRpZXMuIFRoZXNlIGFyZSBub3QgdGhlIGRlZmF1bHQuIFRoZSBk ZWZhdWx0IGlzIHRoZSBPUwpjaG9pY2UgYW5kIGRlcGVuZHMgb24gdGhlIE5BTkQgY2FwYWJpbGl0 aWVzLiBUaGUgT1Mgd2lsbCBhbHdheXMgdHJ5IHRvCm1hdGNoIHRoZSBjbG9zZXN0IEVDQyBzZXR0 aW5ncyBvZmZlcmVkIGJ5IHRoZSBlbmdpbmUsIGJhc2VkIG9uIHRoZSBOQU5ECmNoaXAgcmVxdWly ZW1lbnRzIHdoaWNoIGFyZSBkaXNjb3ZlcmFibGUuIElmIHlvdSB3YW50IHRvIG1heGltaXplIHlv dXIKc3RyZW5ndGgsIGl0IGlzIGFsc28gcG9zc2libGUgdG8gdGVsbCB0aGUgT1Mgd2l0aCBhIGRl ZGljYXRlZCAoZ2VuZXJpYykKcHJvcGVydHkuIEFuZCBvbmx5IGlmIHlvdSB3YW50IHNvbWV0aGlu ZyBkaWZmZXJlbnQsIHlvdSBtYXkgdXNlIHRoZXNlCnByb3BlcnRpZXMsIGJ1dCB0aGV5IHNob3Vs ZCBiZSB0aGUgZXhjZXB0aW9uIHJhdGhlciB0aGFuIHRoZSBydWxlLgoKT3ZlcnJpZGluZyB0aGlz IHdpdGggYSBzdHJhcCBpcyBhIGJhZCBoYXJkd2FyZSBkZXNpZ24gb24gY29tbWVyY2lhbApwcm9k dWN0cyBJTU8uIEkgYW0gdG90YWxseSBmaW5lIHdpdGggdGhlIGlkZWEgb2YgYSBzdHJhcCB0byBj aG9vc2UKdGhlIEVDQyBjb25maWd1cmF0aW9uIGZvciBkZXZlbG9wbWVudCBib2FyZHMvZXZhbHVh dGlvbiBraXRzLCBidXQgb25jZQp5b3UndmUgZGVjaWRlZCB3aGljaCBzZXR0aW5nIHlvdSB3YW50 IHlvdSBjYW5ub3QgY2hhbmdlIGl0IGZvciB0aGUKbGlmZXRpbWUgb2YgeW91ciBwcm9qZWN0IChv ciB3aXRoIGEgbG90IG9mIGRpZmZpY3VsdGllcykgc28gSSBkb24ndCBzZWUKdGhlIHBvaW50IG9m IHN1Y2ggYSBzdHJhcC4gU28gcmVhbGx5LCBJIGRvbid0IGxpa2UgdGhlIGlkZWEgb2YgZGVmaW5p bmcKYnkgZGVmYXVsdCBhIHZhcmlhYmxlIHdoaWNoIGFza3MgZm9yIGFuIG92ZXJyaWRlIG9mIHRo ZSBkZWZhdWx0cywgZXZlbgp0aG91Z2ggbWFueSBvZiB5b3VyIGN1c3RvbWVycyBtaWdodCB3YW50 IHRvIHVzZSB0aGF0LgoKU28sIGFueXRoaW5nIHRoYXQgaXMgZGVzaWduIGRlcGVuZGVudCAodGhl IGNoaXAgQ1MsIEVDQwpjb25maWd1cmF0aW9uLCBldGMpIHNob3VsZCBnbyBpbnRvIHRoZSBib2Fy ZCBEVFMsIGFuZCB3aGF0IGlzIFNvQwpyZWxhdGVkIGhhcmR3YXJlIChsaWtlIHRoZSBkZWZpbml0 aW9uIG9mIHRoZSBOQU5EIGNvbnRyb2xsZXIpIHNob3VsZApzdGF5IGluIHRoZSBEVFNJLCBhcyBw cm9wZXJseSBjbGFyaWZpZWQgYnkgRmxvcmlhbi4KClRoYW5rcywKTWlxdcOobAoKX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KbGludXgtYXJtLWtlcm5lbCBt YWlsaW5nIGxpc3QKbGludXgtYXJtLWtlcm5lbEBsaXN0cy5pbmZyYWRlYWQub3JnCmh0dHA6Ly9s aXN0cy5pbmZyYWRlYWQub3JnL21haWxtYW4vbGlzdGluZm8vbGludXgtYXJtLWtlcm5lbAo= From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from relay8-d.mail.gandi.net (relay8-d.mail.gandi.net [217.70.183.201]) (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 193FB1BC35; Thu, 25 Jan 2024 09:21:03 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.70.183.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1706174467; cv=none; b=hTNO4ol+myBHH2CF0aqL2ibEE9uEHNWYoUA9h7uU9hQQgDDwJmqaw8/D1oygl0Qf0vxzB+H6KjnTBtIex1ofhwSj/bLqLlbAs+jV+GYMdkm6tn8nMCctqiDsWvcClN/kQjViIC5zE1Ef/PM06b9Xjb2IezJv5Q5Kj+Xv3mJDIBI= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1706174467; c=relaxed/simple; bh=vqNkEAabo6aICt8fsmv4NUb7Td0W6lltgyPnkptXJFQ=; h=Date:From:To:Cc:Subject:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=OBQ8Z7/xkCr3S+Ve14GoSqd4FmcKFeNLdM7EA9RvEuKJ3Ri6R6lPB/lb0GZDyD1a40hlt47elr7TzHNdsk/l3u3gz5FMB9RCpq4ZjyOJmj5MJpTXPTPHyHe5tAQKDbEXvipRSorRcAIQ1MAHpcqRZTMZyzQrXjOl7KlHb9aVe7s= 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=VQ86jc4j; arc=none smtp.client-ip=217.70.183.201 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="VQ86jc4j" Received: by mail.gandi.net (Postfix) with ESMTPSA id DA3881BF219; Thu, 25 Jan 2024 09:20:59 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1706174461; 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=eKjAu14meoD0UWLGa1Hctnp45OsMD9+KH2IhUcbWrL4=; b=VQ86jc4jMi7kZpSX93HTuIEk4dHcoUQO2jYVvHJKrFbWKkXwZunjal+i7HO2/oynf2bGKQ CYDTSSZAWfKlxhC0kyB9aI8HFAj//uey3jcaKSEEz/1HaS2w6h2ETfSokmazYkVCoJN4pd 9QXJE49MHmPyiPllw9VhmA2BRY5LEvfRdNWiuXqt33gGxfFbwv7BZcjNxLofLpQbSeSc+6 tD9beY3Sn/veWmIUV2Ys8WpFFrOjXXy1++Q9ki3bL0dINJ1K0ewx2Ag+0RXhMRaZ875MXs +YHWh7+uEfOy3+IwWuICh8XKLw/5yed/TAzlzTOX+PxM1i1sqZwE8qGfLKzE+g== Date: Thu, 25 Jan 2024 10:20:58 +0100 From: Miquel Raynal To: William Zhang Cc: Florian Fainelli , David Regan , dregan@mail.com, richard@nod.at, vigneshr@ti.com, robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org, computersforpeace@gmail.com, kdasu.kdev@gmail.com, linux-mtd@lists.infradead.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, joel.peshkin@broadcom.com, tomer.yacoby@broadcom.com, dan.beygelman@broadcom.com, anand.gore@broadcom.com, kursad.oney@broadcom.com, rafal@milecki.pl, bcm-kernel-feedback-list@broadcom.com, andre.przywara@arm.com, baruch@tkos.co.il, linux-arm-kernel@lists.infradead.org, dan.carpenter@linaro.org Subject: Re: [PATCH v3 02/10] ARM: dts: broadcom: bcmbca: Add NAND controller node Message-ID: <20240125102058.5ae46d8a@xps-13> In-Reply-To: <722d6bdf-9a43-436d-a9e2-4c21e1deb8c3@broadcom.com> References: <20240124030458.98408-1-dregan@broadcom.com> <20240124030458.98408-3-dregan@broadcom.com> <20240124183008.04a1bcb0@xps-13> <2c10a764-f74f-45b2-8bba-77c40468f4b5@broadcom.com> <85c97d00-a973-46c6-974c-3dfa587ae873@broadcom.com> <722d6bdf-9a43-436d-a9e2-4c21e1deb8c3@broadcom.com> Organization: Bootlin X-Mailer: Claws Mail 4.0.0 (GTK+ 3.24.33; 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 William, > >>>> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 nand_controller: nand-co= ntroller@1800 { > >>>> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 = #address-cells =3D <1>; > >>>> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 = #size-cells =3D <0>; > >>>> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 = compatible =3D "brcm,nand-bcm63138", >>>> "brcm,brcmnand-v7.1", "brcm,brcmn= and"; > >>>> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 = reg =3D <0x1800 0x600>, <0x2000 0x10>; > >>>> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 = reg-names =3D "nand", "nand-int-base"; > >>>> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 = brcm,nand-use-wp =3D <0>; > >>>> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 = status =3D "disabled"; > >>>> + > >>>> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 = nandcs: nand@0 { > >>>> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0=C2=A0 compatible =3D "brcm,nandcs"; > >>>> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0=C2=A0 reg =3D <0>; > >>>> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0=C2=A0 nand-on-flash-bbt; > >>>> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0=C2=A0 brcm,nand-ecc-use-strap; =20 > >>> > >>> Describing the NAND chip in a SoC DTSI does not look relevant to me. > >>> Even more if you add something like this nand-ecc-use-strap setting > >>> which is very board dependent. > >>> =20 > >> I am not sure if I understand you comments correctly but are you >> su= ggesting to put this whole nand controller node into each board dts? >> We = have other ip block nodes like SPI, uart in this same soc dtsi file >> too.= =C2=A0 For all the bcmbca soc dtsi I am updating here(and its board >> desi= gn), we always use the strap to for ecc setting.=C2=A0 So I thought it >> s= hould be okay to put brcm,nand-ecc-use-strap in the default dtsi >> file. F= or any board that uses the raw nand nand-ecc property, the >> board dts can= do so and override the brcm,nand-ecc-use-strap setting. =20 > >=20 > > I read Miquel's comment as meaning that the nandcs aka the NAND > chip/= flash part description should be in the board .dts file, while the > contro= ller itself can remain in the .dtsi file with its status =3D > "disabled" p= roperty. > >=20 > > Are there customer boards, that is non reference boards that might chos= e > a different chip select number and/or not use the strap settings? =20 > In BCMBCA SoC, there is only one cs and customer design also have to use = strap for the bootrom to boot up properly. They can override it with dts i= n linux but I don't think any customer would do that. >=20 > Maybe the nand-on-flash-bbt could be possible item that customer may have= to set it differently if they don't follow our reference software design. >=20 > I will move the nand-on-flash-bbt to the board dts but I would like to ke= ep the other default nandcs settings in SoC.dsti if that is not too out of = the conventional rule and Miquel is okay with it. I think there is a global misunderstanding regarding the use of the nand-ecc-* properties. These are not the default. The default is the OS choice and depends on the NAND capabilities. The OS will always try to match the closest ECC settings offered by the engine, based on the NAND chip requirements which are discoverable. If you want to maximize your strength, it is also possible to tell the OS with a dedicated (generic) property. And only if you want something different, you may use these properties, but they should be the exception rather than the rule. Overriding this with a strap is a bad hardware design on commercial products IMO. I am totally fine with the idea of a strap to choose the ECC configuration for development boards/evaluation kits, but once you've decided which setting you want you cannot change it for the lifetime of your project (or with a lot of difficulties) so I don't see the point of such a strap. So really, I don't like the idea of defining by default a variable which asks for an override of the defaults, even though many of your customers might want to use that. So, anything that is design dependent (the chip CS, ECC configuration, etc) should go into the board DTS, and what is SoC related hardware (like the definition of the NAND controller) should stay in the DTSI, as properly clarified by Florian. Thanks, Miqu=C3=A8l