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 176E3ECAAD8 for ; Fri, 23 Sep 2022 15:49:01 +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=l/HYl6QBDoSnuyzk8+x9kfRh3ZY2bpW6wZbXdudabPY=; b=1VBC7FKRr/bUWW g8pDjydF4lw6DRhQNh0FdL25Gy8QyCxuBZoXsqOJv5qOiiCFELFxnFkePmj4VxceETM7CVvrfwR5M 43Ms98NlgZ1tgg1NNZnPA61FFhcloD5CgKQPTzCohBnYVy/UiDkhPLvMpBECBk7EiEvG3PajOH+Ix O5Nx1UEXtX9E3HDpJQ04fz/Vl1R8SHmsHzMk3aKmfBarJke9x4IvoaHfmjvr+QljCOuJni8cYPLFe nFqwO+HAioahcKPhiaWl3q7OkfULFoXL8494212vX6lkC84Td5u3KkvfdAtelSPRaqj8jLvpH/Sje +QBKBIYqgsGLUYo+a3Ow==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1obkub-004sre-Gn; Fri, 23 Sep 2022 15:48:21 +0000 Received: from relay9-d.mail.gandi.net ([2001:4b98:dc4:8::229]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1obkuO-004snH-GV; Fri, 23 Sep 2022 15:48:10 +0000 Received: (Authenticated sender: miquel.raynal@bootlin.com) by mail.gandi.net (Postfix) with ESMTPSA id 5E00BFF809; Fri, 23 Sep 2022 15:48:01 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1663948083; 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=lo2OfiFsituiAjWzRKD+8mam5gTPh7hGEapVViTm41I=; b=fsi/9hSaY/0DUzrW2fRSEnNXqB9wsVkDc8+Ln6V2dgFk/WsWpD8ZUPRF9l1bhZLlpHMooS UDcKQSzp/PN4D6sKnKmilsjeKp3JQSGnxpTHDes8cuHi+m/PO+RLvmALwZiwdf/6KUl4UD ZDslRsUikY86FVM167/dIxig6MmRRNlOf4FfrTrqo5uh42ZpuaJnLPGBe696TODaRiCRBy KkmORG0fN6VwaMO9W7rgdunJtTMk9nrQFTPYCT76qaqVL2YDNmc9rs2P9I/Us4UOwiudae eeyXGsdPbxc12KynT7BaS50Xt5hi9ehrLZdQQcfofIRLaD3gkDcCKydFYDShFA== Date: Fri, 23 Sep 2022 17:47:59 +0200 From: Miquel Raynal To: Michael Walle Cc: Richard Weinberger , Vignesh Raghavendra , Rob Herring , Krzysztof Kozlowski , Srinivas Kandagatla , Shawn Guo , Li Yang , =?UTF-8?B?UmFmYcWCIE1pxYJlY2tp?= , Frank Rowand , linux-mtd@lists.infradead.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Ahmad Fatoum , Philipp Zabel , Robert Marko , Thomas Petazzoni Subject: Re: [PATCH v2 00/20] nvmem: core: introduce NVMEM layouts Message-ID: <20220923174759.299e504e@xps-13> In-Reply-To: <20220901221857.2600340-1-michael@walle.cc> References: <20220901221857.2600340-1-michael@walle.cc> Organization: Bootlin X-Mailer: Claws Mail 4.0.0 (GTK+ 3.24.33; 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-20220923_084808_888552_6799CD6D X-CRM114-Status: GOOD ( 31.39 ) 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 SGkgTWljaGFlbCwKCkkgaGF2ZSBhIGZldyBhZGRpdGlvbmFsIHF1ZXN0aW9ucyByZWdhcmRpbmcg dGhlIGJpbmRpbmdzLgoKbWljaGFlbEB3YWxsZS5jYyB3cm90ZSBvbiBGcmksICAyIFNlcCAyMDIy IDAwOjE4OjM3ICswMjAwOgoKPiBUaGlzIGlzIG5vdyB0aGUgdGhpcmQgYXR0ZW1wdCB0byBmZXRj aCB0aGUgTUFDIGFkZHJlc3NlcyBmcm9tIHRoZSBWUEQKPiBmb3IgdGhlIEtvbnRyb24gc2wyOCBi b2FyZHMuIFByZXZpb3VzIGRpc2N1c3Npb25zIGNhbiBiZSBmb3VuZCBoZXJlOgo+IGh0dHBzOi8v bG9yZS5rZXJuZWwub3JnL2xrbWwvMjAyMTEyMjgxNDI1NDkuMTI3NTQxMi0xLW1pY2hhZWxAd2Fs bGUuY2MvCj4gCj4gCj4gTlZNRU0gY2VsbHMgYXJlIHR5cGljYWxseSBhZGRlZCBieSBib2FyZCBj b2RlIG9yIGJ5IHRoZSBkZXZpY2V0cmVlLiBCdXQKPiBhcyB0aGUgY2VsbHMgZ2V0IG1vcmUgY29t cGxleCwgdGhlcmUgaXMgKHZhbGlkKSBwdXNoIGJhY2sgZnJvbSB0aGUKPiBkZXZpY2V0cmVlIG1h aW50YWluZXJzIHRvIG5vdCBwdXQgdGhhdCBoYW5kbGluZyBpbiB0aGUgZGV2aWNldHJlZS4KPiAK PiBUaGVyZWZvcmUsIGludHJvZHVjZSBOVk1FTSBsYXlvdXRzLiBUaGV5IG9wZXJhdGUgb24gdGhl IE5WTUVNIGRldmljZSBhbmQKPiBjYW4gYWRkIGNlbGxzIGR1cmluZyBydW50aW1lLiBUaGF0IHdh eSBpdCBpcyBwb3NzaWJsZSB0byBhZGQgbW9yZSBjb21wbGV4Cj4gY2VsbHMgdGhhbiBpdCBpcyBw b3NzaWJsZSByaWdodCBub3cgd2l0aCB0aGUgb2Zmc2V0L2xlbmd0aC9iaXRzCj4gZGVzY3JpcHRp b24gaW4gdGhlIGRldmljZSB0cmVlLiBGb3IgZXhhbXBsZSwgeW91IGNhbiBoYXZlIHBvc3QgcHJv Y2Vzc2luZwo+IGZvciBpbmRpdmlkdWFsIGNlbGxzICh0aGluayBvZiBlbmRpYW4gc3dhcHBpbmcs IG9yIGV0aGVybmV0IG9mZnNldAo+IGhhbmRsaW5nKS4KPiAKPiBUaGUgaW14LW9jb3RwIGRyaXZl ciBpcyB0aGUgb25seSB1c2VyIG9mIHRoZSBnbG9iYWwgcG9zdCBwcm9jZXNzaW5nIGhvb2ssCj4g Y29udmVydCBpdCB0byBudm1lbSBsYXlvdXRzIGFuZCBkcm9wIHRoZSBnbG9iYWwgcG9zdCBwb2Nl c3NpbmcgaG9vay4gUGxlYXNlCj4gbm90ZSwgdGhhdCB0aGlzIGNoYW5nZSBpcyBvbmx5IGNvbXBp bGUtdGltZSB0ZXN0ZWQuCj4gCj4gWW91IGNhbiBhbHNvIGhhdmUgY2VsbHMgd2hpY2ggaGF2ZSBu byBzdGF0aWMgb2Zmc2V0LCBsaWtlIHRoZQo+IG9uZXMgaW4gYW4gdS1ib290IGVudmlyb25tZW50 LiBUaGUgbGFzdCBwYXRjaGVzIHdpbGwgY29udmVydCB0aGUgY3VycmVudAo+IHUtYm9vdCBlbnZp cm9ubWVudCBkcml2ZXIgdG8gYSBOVk1FTSBsYXlvdXQgYW5kIGxpZnRpbmcgdGhlIHJlc3RyaWN0 aW9uCj4gdGhhdCBpdCBvbmx5IHdvcmtzIHdpdGggbXRkIGRldmljZXMuIEJ1dCBhcyBpdCB3aWxs IGNoYW5nZSB0aGUgcmVxdWlyZWQKPiBjb21wYXRpYmxlIHN0cmluZ3MsIGl0IGlzIG1hcmtlZCBh cyBSRkMgZm9yIG5vdy4gSXQgYWxzbyBuZWVkcyB0byBoYXZlCj4gaXRzIGRldmljZSB0cmVlIHNj aGVtYSB1cGRhdGUgd2hpY2ggaXMgbGVmdCBvdXQgaGVyZS4gVGhlc2UgdHdvIHBhdGNoZXMKPiBh cmUgbm90IGV4cGVjdGVkIHRvIGJlIGFwcGxpZWQsIGJ1dCByYXRoZXIgdG8gc2hvdyBhbm90aGVy IGV4YW1wbGUgb2YKPiBob3cgdG8gdXNlIHRoZSBsYXlvdXRzLgo+IAo+IEZvciBub3csIHRoZSBs YXlvdXRzIGFyZSBzZWxlY3RlZCBieSBhIHNwZWNpZmljIGNvbXBhdGlibGUgc3RyaW5nIGluIGEK PiBkZXZpY2UgdHJlZS4gRS5nLiB0aGUgVlBEIG9uIHRoZSBrb250cm9uIHNsMjggZG8gKHdpdGhp biBhIFNQSSBmbGFzaCBub2RlKToKPiAgIGNvbXBhdGlibGUgPSAia29udHJvbixzbDI4LXZwZCIs ICJ1c2VyLW90cCI7Cj4gb3IgaWYgeW91J2QgdXNlIHRoZSB1LWJvb3QgZW52aXJvbm1lbnQgKHdp dGhpbiBhbiBNVEQgcGF0aXRpb24pOgo+ICAgY29tcGF0aWJsZSA9ICJ1LWJvb3QsZW52IiwgIm52 bWVtIjsKPiAKPiBUaGUgInVzZXItb3RwIiAob3IgIm52bWVtIikgd2lsbCBsZWFkIHRvIGEgTlZN RU0gZGV2aWNlLCB0aGUKPiAia29udHJvbixzbDI4LXZwZCIgKG9yICJ1LWJvb3QsZW52Iikgd2ls bCB0aGVuIGFwcGx5IHRoZSBzcGVjaWZpYyBsYXlvdXQKPiBvbiB0b3Agb2YgdGhlIE5WTUVNIGRl dmljZS4KClNvIGlmIEkgdW5kZXJzdGFuZCBjb3JyZWN0bHksIHRoZXJlIHNob3VsZCBiZToKLSBv bmUgRFQgbm9kZSBkZWZpbmluZyB0aGUgc3RvcmFnZSBtZWRpdW0gZWVwcm9tL210ZC93aGF0ZXZl ciwKLSBhbm90aGVyIERUIG5vZGUgZGVmaW5pbmcgdGhlIG52bWVtIGRldmljZSB3aXRoIHR3byBj b21wYXRpYmxlcywgdGhlCiAgIm52bWVtIiAob3IgInVzZXItb3RwIikgYW5kIHRoZSBsYXlvdXQu CklzIHRoaXMgY29ycmVjdD8gQWN0dWFsbHkgSSB3YXMgYSBiaXQgc3VycHJpc2VkIGJlY2F1c2Ug Z2VuZXJhbGx5CnNwZWFraW5nLCBEVCBtYWludGFpbmVycyAocmlnaHRmdWxseSkgZG8gbm90IHdh bnQgdG8gZGVzY3JpYmUgaG93IHdlCnVzZSBkZXZpY2VzLCB0aGUgbnZtZW0gYWJzdHJhY3Rpb24g bG9va3MgbGlrZSBhIExpbnV4IHRoaW5nIHdoZW4gb24gdG9wCm9mIG10ZCBkZXZpY2VzIGZvciBp bnN0YW5jZSwgc28gSSBqdXN0IHdhbnRlZCB0byBjb25maXJtIHRoaXMgcG9pbnQuCgpUaGVuLCBh cyB3ZSBoYXZlIGFuIG52bWVtIGRldmljZSBkZXNjcmliZWQgaW4gdGhlIERULCB3aHkgbm90IGp1 c3QKcG9pbnRpbmcgYXQgdGhlIG52bWVtIGRldmljZSBmcm9tIHRoZSBjZWxsIGNvbnN1bWVyLCBy YXRoZXIgdGhhbiBzdGlsbApoYXZpbmcgdGhlIG5lZWQgdG8gZGVmaW5lIGFsbCB0aGUgY2VsbHMg dGhhdCB0aGUgbnZtZW0gZGV2aWNlIHdpbGwKcHJvZHVjZSBpbiB0aGUgRFQ/CgpNYXliZSBhbiBl eGFtcGxlIHRvIHNob3cgd2hhdCBJIG1lYW4uIEhlcmUgaXMgdGhlIGN1cnJlbnQgd2F5OgoKbnZt ZW1fcHJvdmlkZXI6IG52bWVtLXByb3ZpZGVyIHsKCXByb3BlcnRpZXM7CgoJbXljZWxsOiBteV9j ZWxsIHsKCQlbcHJvcGVydGllcztdCgl9Owp9OwoKQW5kIHdlIHBvaW50IHRvIGEgY2VsbCB3aXRo OgoKCW52bWVtLWNlbGxzID0gPCZteWNlbGw+OwoKQnV0LCBhcyBmb3IgdGhlIHRsdiB0YWJsZXMs IHRoZXJlIGFyZSBtYW55IGNlbGxzIHRoYXQgd2lsbCBiZSBwcm9kdWNlZCwKYW5kIHRoZSBkcml2 ZXIgbWF5IGFueXdheSBqdXN0IGFzayBmb3IgdGhlIGNlbGwgbmFtZSAoZWcuIHBlcmZvcm1pbmcg YQpsb29rdXAgb2YgdGhlICJtYWMtYWRkcmVzcyIgY2VsbCBuYW1lKSwgc28gd2h5IGJvdGhlcmlu ZyB0byBkZXNjcmliZSBhbGwKdGhlIGNlbGxzIGluIHRoZSBEVCwgbGlrZToKCgludm1lbS1jZWxs cy1wcm92aWRlcnMgPSA8Jm52bWVtX3Byb3ZpZGVyPjsKCldoYXQgZG8geW91IHRoaW5rPwoKTWF5 YmUgZm9yIHRoZSBtYWMgYWRkcmVzc2VzIHRoaXMgaXMgYSBiaXQgbGltaXRpbmcgYXMsIGluIHBy YWN0aWNlLCB3ZQpvZnRlbiBoYXZlIGJhc2UgbWFjIGFkZHJlc3NlcyBhdmFpbGFibGUgYW5kIHVz aW5nOgoKCW52bWVtLWNlbGxzID0gPCZteWNlbGwgSU5ERVg+OwoKYWxsb3dzIHRvIGR5bmFtaWNh bGx5IGNyZWF0ZSBtYW55IGRpZmZlcmVudCBtYWMgYWRkcmVzc2VzLCBidXQgSSB3b25kZXIKaWYg dGhlIGFwcHJvYWNoIHdvdWxkIGJlIGludGVyZXN0aW5nIGZvciBvdGhlciBjZWxsIHR5cGVzLiBK dXN0IGFuIG9wZW4KcXVlc3Rpb24uCgpUaGFua3MsCk1pcXXDqGwKCl9fX19fX19fX19fX19fX19f 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 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 1CF93C6FA82 for ; Fri, 23 Sep 2022 15:49:22 +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=OvoLqwZzQcz+Jds/Kx8kBPj9FTvXwMy+bX3W1Q374f0=; b=iG0IyplngmVDK2 4G4a+ESWtCJXIkWas0fLhfoUcWauipDJIhAmpBNO7eO9GwDWCDsP2FNdbMFy/kigP6d5ePvT8nrdt DcrQkn4+WNGkwrB/2EuWHPwtsDLq/al0ckKdDf9+4Pg0sRIB+8V6MdzALfQQ2UqjaFqII3mPVMTep EUxMMM9/9TOEiOetIjMROpz5BCvrmbR0xNO8gwLGlOW6cTuV2sMpObMp961jnzKvbg0Fbl92DUY7Q Qr1rHCvuc4rp2eILU5a0OD+OlDVZiPXItliTTbFJPi+GLoE/vvLlFzpgfeBkWaqiMTf1FvfaPFTYB jWkE4QfRu5EITMBA4eKw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1obkuT-004spI-0c; Fri, 23 Sep 2022 15:48:13 +0000 Received: from relay9-d.mail.gandi.net ([2001:4b98:dc4:8::229]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1obkuO-004snH-GV; Fri, 23 Sep 2022 15:48:10 +0000 Received: (Authenticated sender: miquel.raynal@bootlin.com) by mail.gandi.net (Postfix) with ESMTPSA id 5E00BFF809; Fri, 23 Sep 2022 15:48:01 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1663948083; 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=lo2OfiFsituiAjWzRKD+8mam5gTPh7hGEapVViTm41I=; b=fsi/9hSaY/0DUzrW2fRSEnNXqB9wsVkDc8+Ln6V2dgFk/WsWpD8ZUPRF9l1bhZLlpHMooS UDcKQSzp/PN4D6sKnKmilsjeKp3JQSGnxpTHDes8cuHi+m/PO+RLvmALwZiwdf/6KUl4UD ZDslRsUikY86FVM167/dIxig6MmRRNlOf4FfrTrqo5uh42ZpuaJnLPGBe696TODaRiCRBy KkmORG0fN6VwaMO9W7rgdunJtTMk9nrQFTPYCT76qaqVL2YDNmc9rs2P9I/Us4UOwiudae eeyXGsdPbxc12KynT7BaS50Xt5hi9ehrLZdQQcfofIRLaD3gkDcCKydFYDShFA== Date: Fri, 23 Sep 2022 17:47:59 +0200 From: Miquel Raynal To: Michael Walle Cc: Richard Weinberger , Vignesh Raghavendra , Rob Herring , Krzysztof Kozlowski , Srinivas Kandagatla , Shawn Guo , Li Yang , =?UTF-8?B?UmFmYcWCIE1pxYJlY2tp?= , Frank Rowand , linux-mtd@lists.infradead.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Ahmad Fatoum , Philipp Zabel , Robert Marko , Thomas Petazzoni Subject: Re: [PATCH v2 00/20] nvmem: core: introduce NVMEM layouts Message-ID: <20220923174759.299e504e@xps-13> In-Reply-To: <20220901221857.2600340-1-michael@walle.cc> References: <20220901221857.2600340-1-michael@walle.cc> Organization: Bootlin X-Mailer: Claws Mail 4.0.0 (GTK+ 3.24.33; 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-20220923_084808_888552_6799CD6D X-CRM114-Status: GOOD ( 31.39 ) 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 SGkgTWljaGFlbCwKCkkgaGF2ZSBhIGZldyBhZGRpdGlvbmFsIHF1ZXN0aW9ucyByZWdhcmRpbmcg dGhlIGJpbmRpbmdzLgoKbWljaGFlbEB3YWxsZS5jYyB3cm90ZSBvbiBGcmksICAyIFNlcCAyMDIy IDAwOjE4OjM3ICswMjAwOgoKPiBUaGlzIGlzIG5vdyB0aGUgdGhpcmQgYXR0ZW1wdCB0byBmZXRj aCB0aGUgTUFDIGFkZHJlc3NlcyBmcm9tIHRoZSBWUEQKPiBmb3IgdGhlIEtvbnRyb24gc2wyOCBi b2FyZHMuIFByZXZpb3VzIGRpc2N1c3Npb25zIGNhbiBiZSBmb3VuZCBoZXJlOgo+IGh0dHBzOi8v bG9yZS5rZXJuZWwub3JnL2xrbWwvMjAyMTEyMjgxNDI1NDkuMTI3NTQxMi0xLW1pY2hhZWxAd2Fs bGUuY2MvCj4gCj4gCj4gTlZNRU0gY2VsbHMgYXJlIHR5cGljYWxseSBhZGRlZCBieSBib2FyZCBj b2RlIG9yIGJ5IHRoZSBkZXZpY2V0cmVlLiBCdXQKPiBhcyB0aGUgY2VsbHMgZ2V0IG1vcmUgY29t cGxleCwgdGhlcmUgaXMgKHZhbGlkKSBwdXNoIGJhY2sgZnJvbSB0aGUKPiBkZXZpY2V0cmVlIG1h aW50YWluZXJzIHRvIG5vdCBwdXQgdGhhdCBoYW5kbGluZyBpbiB0aGUgZGV2aWNldHJlZS4KPiAK PiBUaGVyZWZvcmUsIGludHJvZHVjZSBOVk1FTSBsYXlvdXRzLiBUaGV5IG9wZXJhdGUgb24gdGhl IE5WTUVNIGRldmljZSBhbmQKPiBjYW4gYWRkIGNlbGxzIGR1cmluZyBydW50aW1lLiBUaGF0IHdh eSBpdCBpcyBwb3NzaWJsZSB0byBhZGQgbW9yZSBjb21wbGV4Cj4gY2VsbHMgdGhhbiBpdCBpcyBw b3NzaWJsZSByaWdodCBub3cgd2l0aCB0aGUgb2Zmc2V0L2xlbmd0aC9iaXRzCj4gZGVzY3JpcHRp b24gaW4gdGhlIGRldmljZSB0cmVlLiBGb3IgZXhhbXBsZSwgeW91IGNhbiBoYXZlIHBvc3QgcHJv Y2Vzc2luZwo+IGZvciBpbmRpdmlkdWFsIGNlbGxzICh0aGluayBvZiBlbmRpYW4gc3dhcHBpbmcs IG9yIGV0aGVybmV0IG9mZnNldAo+IGhhbmRsaW5nKS4KPiAKPiBUaGUgaW14LW9jb3RwIGRyaXZl ciBpcyB0aGUgb25seSB1c2VyIG9mIHRoZSBnbG9iYWwgcG9zdCBwcm9jZXNzaW5nIGhvb2ssCj4g Y29udmVydCBpdCB0byBudm1lbSBsYXlvdXRzIGFuZCBkcm9wIHRoZSBnbG9iYWwgcG9zdCBwb2Nl c3NpbmcgaG9vay4gUGxlYXNlCj4gbm90ZSwgdGhhdCB0aGlzIGNoYW5nZSBpcyBvbmx5IGNvbXBp bGUtdGltZSB0ZXN0ZWQuCj4gCj4gWW91IGNhbiBhbHNvIGhhdmUgY2VsbHMgd2hpY2ggaGF2ZSBu byBzdGF0aWMgb2Zmc2V0LCBsaWtlIHRoZQo+IG9uZXMgaW4gYW4gdS1ib290IGVudmlyb25tZW50 LiBUaGUgbGFzdCBwYXRjaGVzIHdpbGwgY29udmVydCB0aGUgY3VycmVudAo+IHUtYm9vdCBlbnZp cm9ubWVudCBkcml2ZXIgdG8gYSBOVk1FTSBsYXlvdXQgYW5kIGxpZnRpbmcgdGhlIHJlc3RyaWN0 aW9uCj4gdGhhdCBpdCBvbmx5IHdvcmtzIHdpdGggbXRkIGRldmljZXMuIEJ1dCBhcyBpdCB3aWxs IGNoYW5nZSB0aGUgcmVxdWlyZWQKPiBjb21wYXRpYmxlIHN0cmluZ3MsIGl0IGlzIG1hcmtlZCBh cyBSRkMgZm9yIG5vdy4gSXQgYWxzbyBuZWVkcyB0byBoYXZlCj4gaXRzIGRldmljZSB0cmVlIHNj aGVtYSB1cGRhdGUgd2hpY2ggaXMgbGVmdCBvdXQgaGVyZS4gVGhlc2UgdHdvIHBhdGNoZXMKPiBh cmUgbm90IGV4cGVjdGVkIHRvIGJlIGFwcGxpZWQsIGJ1dCByYXRoZXIgdG8gc2hvdyBhbm90aGVy IGV4YW1wbGUgb2YKPiBob3cgdG8gdXNlIHRoZSBsYXlvdXRzLgo+IAo+IEZvciBub3csIHRoZSBs YXlvdXRzIGFyZSBzZWxlY3RlZCBieSBhIHNwZWNpZmljIGNvbXBhdGlibGUgc3RyaW5nIGluIGEK PiBkZXZpY2UgdHJlZS4gRS5nLiB0aGUgVlBEIG9uIHRoZSBrb250cm9uIHNsMjggZG8gKHdpdGhp biBhIFNQSSBmbGFzaCBub2RlKToKPiAgIGNvbXBhdGlibGUgPSAia29udHJvbixzbDI4LXZwZCIs ICJ1c2VyLW90cCI7Cj4gb3IgaWYgeW91J2QgdXNlIHRoZSB1LWJvb3QgZW52aXJvbm1lbnQgKHdp dGhpbiBhbiBNVEQgcGF0aXRpb24pOgo+ICAgY29tcGF0aWJsZSA9ICJ1LWJvb3QsZW52IiwgIm52 bWVtIjsKPiAKPiBUaGUgInVzZXItb3RwIiAob3IgIm52bWVtIikgd2lsbCBsZWFkIHRvIGEgTlZN RU0gZGV2aWNlLCB0aGUKPiAia29udHJvbixzbDI4LXZwZCIgKG9yICJ1LWJvb3QsZW52Iikgd2ls bCB0aGVuIGFwcGx5IHRoZSBzcGVjaWZpYyBsYXlvdXQKPiBvbiB0b3Agb2YgdGhlIE5WTUVNIGRl dmljZS4KClNvIGlmIEkgdW5kZXJzdGFuZCBjb3JyZWN0bHksIHRoZXJlIHNob3VsZCBiZToKLSBv bmUgRFQgbm9kZSBkZWZpbmluZyB0aGUgc3RvcmFnZSBtZWRpdW0gZWVwcm9tL210ZC93aGF0ZXZl ciwKLSBhbm90aGVyIERUIG5vZGUgZGVmaW5pbmcgdGhlIG52bWVtIGRldmljZSB3aXRoIHR3byBj b21wYXRpYmxlcywgdGhlCiAgIm52bWVtIiAob3IgInVzZXItb3RwIikgYW5kIHRoZSBsYXlvdXQu CklzIHRoaXMgY29ycmVjdD8gQWN0dWFsbHkgSSB3YXMgYSBiaXQgc3VycHJpc2VkIGJlY2F1c2Ug Z2VuZXJhbGx5CnNwZWFraW5nLCBEVCBtYWludGFpbmVycyAocmlnaHRmdWxseSkgZG8gbm90IHdh bnQgdG8gZGVzY3JpYmUgaG93IHdlCnVzZSBkZXZpY2VzLCB0aGUgbnZtZW0gYWJzdHJhY3Rpb24g bG9va3MgbGlrZSBhIExpbnV4IHRoaW5nIHdoZW4gb24gdG9wCm9mIG10ZCBkZXZpY2VzIGZvciBp bnN0YW5jZSwgc28gSSBqdXN0IHdhbnRlZCB0byBjb25maXJtIHRoaXMgcG9pbnQuCgpUaGVuLCBh cyB3ZSBoYXZlIGFuIG52bWVtIGRldmljZSBkZXNjcmliZWQgaW4gdGhlIERULCB3aHkgbm90IGp1 c3QKcG9pbnRpbmcgYXQgdGhlIG52bWVtIGRldmljZSBmcm9tIHRoZSBjZWxsIGNvbnN1bWVyLCBy YXRoZXIgdGhhbiBzdGlsbApoYXZpbmcgdGhlIG5lZWQgdG8gZGVmaW5lIGFsbCB0aGUgY2VsbHMg dGhhdCB0aGUgbnZtZW0gZGV2aWNlIHdpbGwKcHJvZHVjZSBpbiB0aGUgRFQ/CgpNYXliZSBhbiBl eGFtcGxlIHRvIHNob3cgd2hhdCBJIG1lYW4uIEhlcmUgaXMgdGhlIGN1cnJlbnQgd2F5OgoKbnZt ZW1fcHJvdmlkZXI6IG52bWVtLXByb3ZpZGVyIHsKCXByb3BlcnRpZXM7CgoJbXljZWxsOiBteV9j ZWxsIHsKCQlbcHJvcGVydGllcztdCgl9Owp9OwoKQW5kIHdlIHBvaW50IHRvIGEgY2VsbCB3aXRo OgoKCW52bWVtLWNlbGxzID0gPCZteWNlbGw+OwoKQnV0LCBhcyBmb3IgdGhlIHRsdiB0YWJsZXMs IHRoZXJlIGFyZSBtYW55IGNlbGxzIHRoYXQgd2lsbCBiZSBwcm9kdWNlZCwKYW5kIHRoZSBkcml2 ZXIgbWF5IGFueXdheSBqdXN0IGFzayBmb3IgdGhlIGNlbGwgbmFtZSAoZWcuIHBlcmZvcm1pbmcg YQpsb29rdXAgb2YgdGhlICJtYWMtYWRkcmVzcyIgY2VsbCBuYW1lKSwgc28gd2h5IGJvdGhlcmlu ZyB0byBkZXNjcmliZSBhbGwKdGhlIGNlbGxzIGluIHRoZSBEVCwgbGlrZToKCgludm1lbS1jZWxs cy1wcm92aWRlcnMgPSA8Jm52bWVtX3Byb3ZpZGVyPjsKCldoYXQgZG8geW91IHRoaW5rPwoKTWF5 YmUgZm9yIHRoZSBtYWMgYWRkcmVzc2VzIHRoaXMgaXMgYSBiaXQgbGltaXRpbmcgYXMsIGluIHBy YWN0aWNlLCB3ZQpvZnRlbiBoYXZlIGJhc2UgbWFjIGFkZHJlc3NlcyBhdmFpbGFibGUgYW5kIHVz aW5nOgoKCW52bWVtLWNlbGxzID0gPCZteWNlbGwgSU5ERVg+OwoKYWxsb3dzIHRvIGR5bmFtaWNh bGx5IGNyZWF0ZSBtYW55IGRpZmZlcmVudCBtYWMgYWRkcmVzc2VzLCBidXQgSSB3b25kZXIKaWYg dGhlIGFwcHJvYWNoIHdvdWxkIGJlIGludGVyZXN0aW5nIGZvciBvdGhlciBjZWxsIHR5cGVzLiBK dXN0IGFuIG9wZW4KcXVlc3Rpb24uCgpUaGFua3MsCk1pcXXDqGwKCl9fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCmxpbnV4LWFybS1rZXJuZWwgbWFpbGluZyBs aXN0CmxpbnV4LWFybS1rZXJuZWxAbGlzdHMuaW5mcmFkZWFkLm9yZwpodHRwOi8vbGlzdHMuaW5m cmFkZWFkLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2xpbnV4LWFybS1rZXJuZWwK 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 06770C6FA82 for ; Fri, 23 Sep 2022 15:50:02 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232797AbiIWPt7 (ORCPT ); Fri, 23 Sep 2022 11:49:59 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:53044 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232762AbiIWPtU (ORCPT ); Fri, 23 Sep 2022 11:49:20 -0400 Received: from relay9-d.mail.gandi.net (relay9-d.mail.gandi.net [IPv6:2001:4b98:dc4:8::229]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 1F569146F84; Fri, 23 Sep 2022 08:48:05 -0700 (PDT) Received: (Authenticated sender: miquel.raynal@bootlin.com) by mail.gandi.net (Postfix) with ESMTPSA id 5E00BFF809; Fri, 23 Sep 2022 15:48:01 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1663948083; 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=lo2OfiFsituiAjWzRKD+8mam5gTPh7hGEapVViTm41I=; b=fsi/9hSaY/0DUzrW2fRSEnNXqB9wsVkDc8+Ln6V2dgFk/WsWpD8ZUPRF9l1bhZLlpHMooS UDcKQSzp/PN4D6sKnKmilsjeKp3JQSGnxpTHDes8cuHi+m/PO+RLvmALwZiwdf/6KUl4UD ZDslRsUikY86FVM167/dIxig6MmRRNlOf4FfrTrqo5uh42ZpuaJnLPGBe696TODaRiCRBy KkmORG0fN6VwaMO9W7rgdunJtTMk9nrQFTPYCT76qaqVL2YDNmc9rs2P9I/Us4UOwiudae eeyXGsdPbxc12KynT7BaS50Xt5hi9ehrLZdQQcfofIRLaD3gkDcCKydFYDShFA== Date: Fri, 23 Sep 2022 17:47:59 +0200 From: Miquel Raynal To: Michael Walle Cc: Richard Weinberger , Vignesh Raghavendra , Rob Herring , Krzysztof Kozlowski , Srinivas Kandagatla , Shawn Guo , Li Yang , =?UTF-8?B?UmFmYcWCIE1pxYJlY2tp?= , Frank Rowand , linux-mtd@lists.infradead.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Ahmad Fatoum , Philipp Zabel , Robert Marko , Thomas Petazzoni Subject: Re: [PATCH v2 00/20] nvmem: core: introduce NVMEM layouts Message-ID: <20220923174759.299e504e@xps-13> In-Reply-To: <20220901221857.2600340-1-michael@walle.cc> References: <20220901221857.2600340-1-michael@walle.cc> 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 Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Michael, I have a few additional questions regarding the bindings. michael@walle.cc wrote on Fri, 2 Sep 2022 00:18:37 +0200: > This is now the third attempt to fetch the MAC addresses from the VPD > for the Kontron sl28 boards. Previous discussions can be found here: > https://lore.kernel.org/lkml/20211228142549.1275412-1-michael@walle.cc/ >=20 >=20 > NVMEM cells are typically added by board code or by the devicetree. But > as the cells get more complex, there is (valid) push back from the > devicetree maintainers to not put that handling in the devicetree. >=20 > Therefore, introduce NVMEM layouts. They operate on the NVMEM device and > can add cells during runtime. That way it is possible to add more complex > cells than it is possible right now with the offset/length/bits > description in the device tree. For example, you can have post processing > for individual cells (think of endian swapping, or ethernet offset > handling). >=20 > The imx-ocotp driver is the only user of the global post processing hook, > convert it to nvmem layouts and drop the global post pocessing hook. Plea= se > note, that this change is only compile-time tested. >=20 > You can also have cells which have no static offset, like the > ones in an u-boot environment. The last patches will convert the current > u-boot environment driver to a NVMEM layout and lifting the restriction > that it only works with mtd devices. But as it will change the required > compatible strings, it is marked as RFC for now. It also needs to have > its device tree schema update which is left out here. These two patches > are not expected to be applied, but rather to show another example of > how to use the layouts. >=20 > For now, the layouts are selected by a specific compatible string in a > device tree. E.g. the VPD on the kontron sl28 do (within a SPI flash node= ): > compatible =3D "kontron,sl28-vpd", "user-otp"; > or if you'd use the u-boot environment (within an MTD patition): > compatible =3D "u-boot,env", "nvmem"; >=20 > The "user-otp" (or "nvmem") will lead to a NVMEM device, the > "kontron,sl28-vpd" (or "u-boot,env") will then apply the specific layout > on top of the NVMEM device. So if I understand correctly, there should be: - one DT node defining the storage medium eeprom/mtd/whatever, - another DT node defining the nvmem device with two compatibles, the "nvmem" (or "user-otp") and the layout. Is this correct? Actually I was a bit surprised because generally speaking, DT maintainers (rightfully) do not want to describe how we use devices, the nvmem abstraction looks like a Linux thing when on top of mtd devices for instance, so I just wanted to confirm this point. Then, as we have an nvmem device described in the DT, why not just pointing at the nvmem device from the cell consumer, rather than still having the need to define all the cells that the nvmem device will produce in the DT? Maybe an example to show what I mean. Here is the current way: nvmem_provider: nvmem-provider { properties; mycell: my_cell { [properties;] }; }; And we point to a cell with: nvmem-cells =3D <&mycell>; But, as for the tlv tables, there are many cells that will be produced, and the driver may anyway just ask for the cell name (eg. performing a lookup of the "mac-address" cell name), so why bothering to describe all the cells in the DT, like: nvmem-cells-providers =3D <&nvmem_provider>; What do you think? Maybe for the mac addresses this is a bit limiting as, in practice, we often have base mac addresses available and using: nvmem-cells =3D <&mycell INDEX>; allows to dynamically create many different mac addresses, but I wonder if the approach would be interesting for other cell types. Just an open question. Thanks, Miqu=C3=A8l