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 7497DC433FE for ; Mon, 28 Nov 2022 08:31:23 +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=ipWwKV9abQaH4PyN3Fp9dp9u6NTv8K5VZwhd0H3GLts=; b=trZOjY3XAs2mC7 CwFiwnWz1KBZ0+MmXFcoaI0oYN9EgK/YfU5E5R9YITjgXg8YU18K8jc2agHRoZugMrrBpplV5CApD vQF1jztd03jRvmy9a/t3hMX7bS0BPzlU2YVkyOlNps7j9fdAybDoLrZ7ufXHPfQpp4LHNJi5kXQ6L 5A341vF8F4dTtkwjGFa9UJu+c5JjOAzJt/l74nniF6NvaGsrDD6BWPua2pTXknCCpgShEHgpaWG33 5KE88MDgCijipQSH7Gpj7nMsgTjS2wAQ5mC17EBVotu552ugUa3SQqP4ZmPa8hCsq7ay043PG0f1R hdbFuK4qoiomjqMCUVaQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1ozZXk-000FD6-Lx; Mon, 28 Nov 2022 08:31:12 +0000 Received: from relay3-d.mail.gandi.net ([2001:4b98:dc4:8::223]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1ozZXL-000F0P-7z; Mon, 28 Nov 2022 08:30:49 +0000 Received: (Authenticated sender: miquel.raynal@bootlin.com) by mail.gandi.net (Postfix) with ESMTPSA id 6046760008; Mon, 28 Nov 2022 08:30:38 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1669624242; 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=V24Aaq988HpFuk8FMBp0+VYNbQHp6mco9JU3lqfOz1Q=; b=NnFQLqfWAY0F9/IpVrR3n0Tuv2AAexr6Q/CLUzGAXiupV52Xhw4ApUSmF7reojh1MykfhM o19kH/AoVRM3S3prp+q474utljNy1XVKFg7HJiLlJakBFZYzmU1+4mK7EbFJs+qsKOj1a3 DCj8u+zkhtu4wMAyXg+lagZx3Qswfhnlq9uGOAG6S7l2BF7paVOXBIxqMCXBR9VICXMt9T 5PvClBssOBYWZJi9Wr+hxF86l7vkC+Amv2qXIUGMa2PjFUaypSBwQN5ga7YUHBd2QgdziA oHLYHTPFmsIudsC3qABWWeLTVCwjw+v3G90S0ACBvX1dgkejEBZz4QfKNyih1A== Date: Mon, 28 Nov 2022 09:30:34 +0100 From: Miquel Raynal To: Michael Walle Cc: =?UTF-8?B?UmFmYcWCIE1pxYJlY2tp?= , Srinivas Kandagatla , Richard Weinberger , Vignesh Raghavendra , Rob Herring , Krzysztof Kozlowski , Shawn Guo , linux-mtd@lists.infradead.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, u-boot@lists.denx.de, =?UTF-8?B?UmFmYcWC?= =?UTF-8?B?IE1pxYJlY2tp?= Subject: Re: [PATCH V2 1/2] nvmem: core: refactor .cell_post_process() CB arguments Message-ID: <20221128093034.6646f191@xps-13> In-Reply-To: References: <20221128065923.1180-1-zajec5@gmail.com> 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-20221128_003047_793320_53E85C2B X-CRM114-Status: GOOD ( 20.82 ) 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 SGkgUmFmYWwsCgptaWNoYWVsQHdhbGxlLmNjIHdyb3RlIG9uIE1vbiwgMjggTm92IDIwMjIgMDg6 MzU6MjQgKzAxMDA6Cgo+IEFtIDIwMjItMTEtMjggMDc6NTksIHNjaHJpZWIgUmFmYcWCIE1pxYJl Y2tpOgo+ID4gRnJvbTogUmFmYcWCIE1pxYJlY2tpIDxyYWZhbEBtaWxlY2tpLnBsPgo+ID4gCj4g PiBQYXNzIHdob2xlIE5WTUVNIGNlbGwgc3RydWN0IGFuZCBsZW5ndGggcG9pbnRlciBhcyBhcmd1 bWVudHMgdG8gPiBjYWxsYmFjawo+ID4gZnVuY3Rpb25zLgo+ID4gCj4gPiBUaGlzIGFsbG93czoK PiA+IAo+ID4gMS4gQ2VsbHMgY29udGVudCB0byBiZSBtb2RpZmllZCBiYXNlZCBvbiBtb3JlIGlu Zm8KPiA+ICAgIFNvbWUgY2VsbHMgKGlkZW50aWZpZWQgYnkgdGhlaXIgbmFtZXMpIGNvbnRhaW4g c3BlY2lmaWMgZGF0YSB0aGF0Cj4gPiAgICBuZWVkcyBmdXJ0aGVyIHByb2Nlc3NpbmcuIFRoaXMg Y2FuIGJlIGUuZy4gTUFDIGFkZHJlc3Mgc3RvcmVkIGluIGFuCj4gPiAgICBBU0NJSSBmb3JtYXQu IE5WTUVNIGNvbnN1bWVycyBleHBlY3QgTUFDIHRvIGJlIHJlYWQgaW4gYSBiaW5hcnkgPiBmb3Jt Lgo+ID4gICAgTW9yZSBjb21wbGV4IGNlbGxzIG1heSBiZSBhZGRpdGlvbmFsbHkgZGVzY3JpYmVk IGluIERULiBUaGlzIGNoYW5nZQo+ID4gICAgYWxsb3dzIGFsc28gYWNjZXNzaW5nIHJlbGV2YW50 IERUIG5vZGVzIGFuZCByZWFkaW5nIGV4dHJhIGluZm8uCj4gPiAKPiA+IDIuIEFkanVzdGluZyBk YXRhIGxlbmd0aAo+ID4gICAgSWYgY2VsbCBwcm9jZXNzaW5nIHJlc3VsdHMgaW4gcmVmb3JtYXR0 aW5nIGl0LCBpdCdzIHJlcXVpcmVkIHRvCj4gPiAgICBhZGp1c3QgbGVuZ3RoLiBUaGlzIGFnYWlu IGFwcGxpZXMgZS5nLiB0byB0aGUgTUFDIGZvcm1hdCBjaGFuZ2UgZnJvbQo+ID4gICAgQVNDSUkg dG8gdGhlIGJ5dGUtYmFzZWQuCgpNaWNoYWVsJ3Mgc2VyaWVzIGJyaW5ncyByZWFkX3Bvc3RfcHJv Y2VzcywgaXNuJ3Qgd2hhdCB5b3UgbmVlZCBoZXJlPwoKPiA+IAo+ID4gTGF0ZXIgb24gd2UgbWF5 IGNvbnNpZGVyIG1vcmUgY2xlYW51cHMgJiBmZWF0dXJlcyBsaWtlOgo+ID4gMS4gRHJvcHBpbmcg ImNvbnN0IGNoYXIgKmlkIiBhbmQganVzdCB1c2luZyBOVk1FTSBjZWxsIG5hbWUKPiA+IDIuIEFk ZGluZyBleHRyYSBhcmd1bWVudCBmb3IgY2VsbHMgcHJvdmlkaW5nIG11bHRpcGxlIHZhbHVlcwo+ ID4gCj4gPiBTaWduZWQtb2ZmLWJ5OiBSYWZhxYIgTWnFgmVja2kgPHJhZmFsQG1pbGVja2kucGw+ Cj4gPiAtLS0KPiA+IFRoaXMgc29sdXRpb24gY29uZmxpY3RzIHdpdGggMSBwYXJ0IG9mIE1pY2hh ZWwncyB3b3JrOgo+ID4gW1BBVENIIHYyIDAwLzIwXSBudm1lbTogY29yZTogaW50cm9kdWNlIE5W TUVNIGxheW91dHMKPiA+IGh0dHBzOi8vbG9yZS5rZXJuZWwub3JnL2xpbnV4LWFybS1rZXJuZWwv MjAyMjA5MDEyMjE4NTcuMjYwMDM0MC0xLW1pY2hhZWxAd2FsbGUuY2MvCj4gPiAKPiA+IEluc3Rl YWQgb2Y6Cj4gPiAxLiBBZGRpbmcgTlZNRU0gY2VsbC1sZXZlbCBwb3N0X3Byb2Nlc3MgY2FsbGJh Y2sKPiA+IDIuIEFkZGluZyBjYWxsYmFjayAoLmZpeHVwX2NlbGxfaW5mbygpKSBmb3Igc2V0dGlu ZyBjYWxsYmFja3MKPiA+IDMuIERyb3BwaW5nIE5WTUVNIGRldmljZS1sZXZlbCBwb3N0X3Byb2Nl c3MgY2FsbGJhY2sKPiA+IEkgZGVjaWRlZCB0byByZWZhY3RvciBleGlzdGluZyBjYWxsYmFjay4K PiA+IAo+ID4gTWljaGFlbCdzIHdvcmsgb24gYWRkaW5nICNudm1lbS1jZWxsLWNlbGxzIHNob3Vs ZCBiZSBwb3NzaWJsZSB0byBlYXNpbHkKPiA+IHJlYmFzZSBvbiB0b3Agb2YgdGhvc2UgY2hhbmdl cy4gIAoKWWVhaCwgSSBndWVzcyBzaW5jZSBNaWNoYWVsJ3Mgc2VyaWVzIGhhcyBiZWVuIG91dCBm b3IgMiB5ZWFycyBhbmQgd2UKZmluYWxseSBhZ3JlZWQgb24gdGhlIGJpbmRpbmdzIHBsdXMgc29t ZSBpbXBsZW1lbnRhdGlvbiBwb2ludHMsIEkgd291bGQKZXhwZWN0IGl0IHRvIGJlIG1lcmdlZCB2 ZXJ5IHNvb24gKEkgZG9uJ3Qga25vdyBpZiBTcmluaXZhcyBzdGlsbCBwbGFucwp0byB0YWtlIGl0 IGZvciB0aGlzIHJlbGVhc2Ugb3IgZm9yIHRoZSBuZXh0PykgdW5sZXNzIHNvbWVvbmUgc3BlYWtz IHVwCmFnYWluc3QgaXQuCgo+IEFzIHlvdXJzIHNob3VsZCBiZSBlYXNpbHkgYWRkZWQgb24gdG9w IG9mIG15IHNlcmllcy4gSSd2ZSBzaG93ZWQgdGhhdAo+IHByb3ZpZGluZyBhIGdsb2JhbCBwb3N0 IHByb2Nlc3MgaG9vayBpcyBiYWQgYmVjYXVzZSB0aGF0IHdheSB5b3UgbmVlZAo+IHRvIGhhdmUg KmFsbCogY2VsbHMgb2YgeW91ciBkZXZpY2UgcmVhZC1vbmx5Lgo+IAo+IC1taWNoYWVsCgpUaGFu a3MsCk1pcXXDqGwKCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fXwpMaW51eCBNVEQgZGlzY3Vzc2lvbiBtYWlsaW5nIGxpc3QKaHR0cDovL2xpc3Rz LmluZnJhZGVhZC5vcmcvbWFpbG1hbi9saXN0aW5mby9saW51eC1tdGQvCg== 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 82CDBC4332F for ; Mon, 28 Nov 2022 08:31:53 +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=xDGrSbMA3ewY7iZXzXeKqyq8EJzurligYnblXHnc+f4=; b=t8qqN4zXE0V5CN WdLmJG6FeNebBpzUJwT+RhBNPhiEzTFoMtahyv0aCPfvhOUMMlWHucvSff90MO9MvOxS0tkUx/HKM ohBdNir01YzQ9MoqG6KiZuFKb2nn15L7pbjmlE0aiw5GFM4wMKCvfRMeBO3FT8K9NHUWC8Sv3Wcpf c4kMgc8mw9ci6OnbaBdLuEbyWBFJPrZYC5M2q+hboGR4YQL5AGA1YaWzTD/XA069Q1Mu5whTna0Hf V+hAl+ikHgWdkhwRJpyC0bL7YJpMc+T7+TfmDCx9ZWV8ARDBTa9oShv3opbvXBHfmmSqPlEqOJgAi kJEzTKRz+MeMv0h+laVg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1ozZXQ-000F60-Fy; Mon, 28 Nov 2022 08:30:52 +0000 Received: from relay3-d.mail.gandi.net ([2001:4b98:dc4:8::223]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1ozZXL-000F0P-7z; Mon, 28 Nov 2022 08:30:49 +0000 Received: (Authenticated sender: miquel.raynal@bootlin.com) by mail.gandi.net (Postfix) with ESMTPSA id 6046760008; Mon, 28 Nov 2022 08:30:38 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1669624242; 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=V24Aaq988HpFuk8FMBp0+VYNbQHp6mco9JU3lqfOz1Q=; b=NnFQLqfWAY0F9/IpVrR3n0Tuv2AAexr6Q/CLUzGAXiupV52Xhw4ApUSmF7reojh1MykfhM o19kH/AoVRM3S3prp+q474utljNy1XVKFg7HJiLlJakBFZYzmU1+4mK7EbFJs+qsKOj1a3 DCj8u+zkhtu4wMAyXg+lagZx3Qswfhnlq9uGOAG6S7l2BF7paVOXBIxqMCXBR9VICXMt9T 5PvClBssOBYWZJi9Wr+hxF86l7vkC+Amv2qXIUGMa2PjFUaypSBwQN5ga7YUHBd2QgdziA oHLYHTPFmsIudsC3qABWWeLTVCwjw+v3G90S0ACBvX1dgkejEBZz4QfKNyih1A== Date: Mon, 28 Nov 2022 09:30:34 +0100 From: Miquel Raynal To: Michael Walle Cc: =?UTF-8?B?UmFmYcWCIE1pxYJlY2tp?= , Srinivas Kandagatla , Richard Weinberger , Vignesh Raghavendra , Rob Herring , Krzysztof Kozlowski , Shawn Guo , linux-mtd@lists.infradead.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, u-boot@lists.denx.de, =?UTF-8?B?UmFmYcWC?= =?UTF-8?B?IE1pxYJlY2tp?= Subject: Re: [PATCH V2 1/2] nvmem: core: refactor .cell_post_process() CB arguments Message-ID: <20221128093034.6646f191@xps-13> In-Reply-To: References: <20221128065923.1180-1-zajec5@gmail.com> 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-20221128_003047_793320_53E85C2B X-CRM114-Status: GOOD ( 20.82 ) 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 SGkgUmFmYWwsCgptaWNoYWVsQHdhbGxlLmNjIHdyb3RlIG9uIE1vbiwgMjggTm92IDIwMjIgMDg6 MzU6MjQgKzAxMDA6Cgo+IEFtIDIwMjItMTEtMjggMDc6NTksIHNjaHJpZWIgUmFmYcWCIE1pxYJl Y2tpOgo+ID4gRnJvbTogUmFmYcWCIE1pxYJlY2tpIDxyYWZhbEBtaWxlY2tpLnBsPgo+ID4gCj4g PiBQYXNzIHdob2xlIE5WTUVNIGNlbGwgc3RydWN0IGFuZCBsZW5ndGggcG9pbnRlciBhcyBhcmd1 bWVudHMgdG8gPiBjYWxsYmFjawo+ID4gZnVuY3Rpb25zLgo+ID4gCj4gPiBUaGlzIGFsbG93czoK PiA+IAo+ID4gMS4gQ2VsbHMgY29udGVudCB0byBiZSBtb2RpZmllZCBiYXNlZCBvbiBtb3JlIGlu Zm8KPiA+ICAgIFNvbWUgY2VsbHMgKGlkZW50aWZpZWQgYnkgdGhlaXIgbmFtZXMpIGNvbnRhaW4g c3BlY2lmaWMgZGF0YSB0aGF0Cj4gPiAgICBuZWVkcyBmdXJ0aGVyIHByb2Nlc3NpbmcuIFRoaXMg Y2FuIGJlIGUuZy4gTUFDIGFkZHJlc3Mgc3RvcmVkIGluIGFuCj4gPiAgICBBU0NJSSBmb3JtYXQu IE5WTUVNIGNvbnN1bWVycyBleHBlY3QgTUFDIHRvIGJlIHJlYWQgaW4gYSBiaW5hcnkgPiBmb3Jt Lgo+ID4gICAgTW9yZSBjb21wbGV4IGNlbGxzIG1heSBiZSBhZGRpdGlvbmFsbHkgZGVzY3JpYmVk IGluIERULiBUaGlzIGNoYW5nZQo+ID4gICAgYWxsb3dzIGFsc28gYWNjZXNzaW5nIHJlbGV2YW50 IERUIG5vZGVzIGFuZCByZWFkaW5nIGV4dHJhIGluZm8uCj4gPiAKPiA+IDIuIEFkanVzdGluZyBk YXRhIGxlbmd0aAo+ID4gICAgSWYgY2VsbCBwcm9jZXNzaW5nIHJlc3VsdHMgaW4gcmVmb3JtYXR0 aW5nIGl0LCBpdCdzIHJlcXVpcmVkIHRvCj4gPiAgICBhZGp1c3QgbGVuZ3RoLiBUaGlzIGFnYWlu IGFwcGxpZXMgZS5nLiB0byB0aGUgTUFDIGZvcm1hdCBjaGFuZ2UgZnJvbQo+ID4gICAgQVNDSUkg dG8gdGhlIGJ5dGUtYmFzZWQuCgpNaWNoYWVsJ3Mgc2VyaWVzIGJyaW5ncyByZWFkX3Bvc3RfcHJv Y2VzcywgaXNuJ3Qgd2hhdCB5b3UgbmVlZCBoZXJlPwoKPiA+IAo+ID4gTGF0ZXIgb24gd2UgbWF5 IGNvbnNpZGVyIG1vcmUgY2xlYW51cHMgJiBmZWF0dXJlcyBsaWtlOgo+ID4gMS4gRHJvcHBpbmcg ImNvbnN0IGNoYXIgKmlkIiBhbmQganVzdCB1c2luZyBOVk1FTSBjZWxsIG5hbWUKPiA+IDIuIEFk ZGluZyBleHRyYSBhcmd1bWVudCBmb3IgY2VsbHMgcHJvdmlkaW5nIG11bHRpcGxlIHZhbHVlcwo+ ID4gCj4gPiBTaWduZWQtb2ZmLWJ5OiBSYWZhxYIgTWnFgmVja2kgPHJhZmFsQG1pbGVja2kucGw+ Cj4gPiAtLS0KPiA+IFRoaXMgc29sdXRpb24gY29uZmxpY3RzIHdpdGggMSBwYXJ0IG9mIE1pY2hh ZWwncyB3b3JrOgo+ID4gW1BBVENIIHYyIDAwLzIwXSBudm1lbTogY29yZTogaW50cm9kdWNlIE5W TUVNIGxheW91dHMKPiA+IGh0dHBzOi8vbG9yZS5rZXJuZWwub3JnL2xpbnV4LWFybS1rZXJuZWwv MjAyMjA5MDEyMjE4NTcuMjYwMDM0MC0xLW1pY2hhZWxAd2FsbGUuY2MvCj4gPiAKPiA+IEluc3Rl YWQgb2Y6Cj4gPiAxLiBBZGRpbmcgTlZNRU0gY2VsbC1sZXZlbCBwb3N0X3Byb2Nlc3MgY2FsbGJh Y2sKPiA+IDIuIEFkZGluZyBjYWxsYmFjayAoLmZpeHVwX2NlbGxfaW5mbygpKSBmb3Igc2V0dGlu ZyBjYWxsYmFja3MKPiA+IDMuIERyb3BwaW5nIE5WTUVNIGRldmljZS1sZXZlbCBwb3N0X3Byb2Nl c3MgY2FsbGJhY2sKPiA+IEkgZGVjaWRlZCB0byByZWZhY3RvciBleGlzdGluZyBjYWxsYmFjay4K PiA+IAo+ID4gTWljaGFlbCdzIHdvcmsgb24gYWRkaW5nICNudm1lbS1jZWxsLWNlbGxzIHNob3Vs ZCBiZSBwb3NzaWJsZSB0byBlYXNpbHkKPiA+IHJlYmFzZSBvbiB0b3Agb2YgdGhvc2UgY2hhbmdl cy4gIAoKWWVhaCwgSSBndWVzcyBzaW5jZSBNaWNoYWVsJ3Mgc2VyaWVzIGhhcyBiZWVuIG91dCBm b3IgMiB5ZWFycyBhbmQgd2UKZmluYWxseSBhZ3JlZWQgb24gdGhlIGJpbmRpbmdzIHBsdXMgc29t ZSBpbXBsZW1lbnRhdGlvbiBwb2ludHMsIEkgd291bGQKZXhwZWN0IGl0IHRvIGJlIG1lcmdlZCB2 ZXJ5IHNvb24gKEkgZG9uJ3Qga25vdyBpZiBTcmluaXZhcyBzdGlsbCBwbGFucwp0byB0YWtlIGl0 IGZvciB0aGlzIHJlbGVhc2Ugb3IgZm9yIHRoZSBuZXh0PykgdW5sZXNzIHNvbWVvbmUgc3BlYWtz IHVwCmFnYWluc3QgaXQuCgo+IEFzIHlvdXJzIHNob3VsZCBiZSBlYXNpbHkgYWRkZWQgb24gdG9w IG9mIG15IHNlcmllcy4gSSd2ZSBzaG93ZWQgdGhhdAo+IHByb3ZpZGluZyBhIGdsb2JhbCBwb3N0 IHByb2Nlc3MgaG9vayBpcyBiYWQgYmVjYXVzZSB0aGF0IHdheSB5b3UgbmVlZAo+IHRvIGhhdmUg KmFsbCogY2VsbHMgb2YgeW91ciBkZXZpY2UgcmVhZC1vbmx5Lgo+IAo+IC1taWNoYWVsCgpUaGFu a3MsCk1pcXXDqGwKCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fCmxpbnV4LWFybS1rZXJuZWwgbWFpbGluZyBsaXN0CmxpbnV4LWFybS1rZXJuZWxAbGlzdHMu aW5mcmFkZWFkLm9yZwpodHRwOi8vbGlzdHMuaW5mcmFkZWFkLm9yZy9tYWlsbWFuL2xpc3RpbmZv L2xpbnV4LWFybS1rZXJuZWwK 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 157E9C433FE for ; Mon, 28 Nov 2022 08:30:48 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229696AbiK1Iaq (ORCPT ); Mon, 28 Nov 2022 03:30:46 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:42918 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229674AbiK1Iap (ORCPT ); Mon, 28 Nov 2022 03:30:45 -0500 Received: from relay3-d.mail.gandi.net (relay3-d.mail.gandi.net [IPv6:2001:4b98:dc4:8::223]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 9376313D5E; Mon, 28 Nov 2022 00:30:43 -0800 (PST) Received: (Authenticated sender: miquel.raynal@bootlin.com) by mail.gandi.net (Postfix) with ESMTPSA id 6046760008; Mon, 28 Nov 2022 08:30:38 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1669624242; 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=V24Aaq988HpFuk8FMBp0+VYNbQHp6mco9JU3lqfOz1Q=; b=NnFQLqfWAY0F9/IpVrR3n0Tuv2AAexr6Q/CLUzGAXiupV52Xhw4ApUSmF7reojh1MykfhM o19kH/AoVRM3S3prp+q474utljNy1XVKFg7HJiLlJakBFZYzmU1+4mK7EbFJs+qsKOj1a3 DCj8u+zkhtu4wMAyXg+lagZx3Qswfhnlq9uGOAG6S7l2BF7paVOXBIxqMCXBR9VICXMt9T 5PvClBssOBYWZJi9Wr+hxF86l7vkC+Amv2qXIUGMa2PjFUaypSBwQN5ga7YUHBd2QgdziA oHLYHTPFmsIudsC3qABWWeLTVCwjw+v3G90S0ACBvX1dgkejEBZz4QfKNyih1A== Date: Mon, 28 Nov 2022 09:30:34 +0100 From: Miquel Raynal To: Michael Walle Cc: =?UTF-8?B?UmFmYcWCIE1pxYJlY2tp?= , Srinivas Kandagatla , Richard Weinberger , Vignesh Raghavendra , Rob Herring , Krzysztof Kozlowski , Shawn Guo , linux-mtd@lists.infradead.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, u-boot@lists.denx.de, =?UTF-8?B?UmFmYcWC?= =?UTF-8?B?IE1pxYJlY2tp?= Subject: Re: [PATCH V2 1/2] nvmem: core: refactor .cell_post_process() CB arguments Message-ID: <20221128093034.6646f191@xps-13> In-Reply-To: References: <20221128065923.1180-1-zajec5@gmail.com> 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: devicetree@vger.kernel.org Hi Rafal, michael@walle.cc wrote on Mon, 28 Nov 2022 08:35:24 +0100: > Am 2022-11-28 07:59, schrieb Rafa=C5=82 Mi=C5=82ecki: > > From: Rafa=C5=82 Mi=C5=82ecki > >=20 > > Pass whole NVMEM cell struct and length pointer as arguments to > callb= ack > > functions. > >=20 > > This allows: > >=20 > > 1. Cells content to be modified based on more info > > Some cells (identified by their names) contain specific data that > > needs further processing. This can be e.g. MAC address stored in an > > ASCII format. NVMEM consumers expect MAC to be read in a binary > fo= rm. > > More complex cells may be additionally described in DT. This change > > allows also accessing relevant DT nodes and reading extra info. > >=20 > > 2. Adjusting data length > > If cell processing results in reformatting it, it's required to > > adjust length. This again applies e.g. to the MAC format change from > > ASCII to the byte-based. Michael's series brings read_post_process, isn't what you need here? > >=20 > > Later on we may consider more cleanups & features like: > > 1. Dropping "const char *id" and just using NVMEM cell name > > 2. Adding extra argument for cells providing multiple values > >=20 > > Signed-off-by: Rafa=C5=82 Mi=C5=82ecki > > --- > > This solution conflicts with 1 part of Michael's work: > > [PATCH v2 00/20] nvmem: core: introduce NVMEM layouts > > https://lore.kernel.org/linux-arm-kernel/20220901221857.2600340-1-micha= el@walle.cc/ > >=20 > > Instead of: > > 1. Adding NVMEM cell-level post_process callback > > 2. Adding callback (.fixup_cell_info()) for setting callbacks > > 3. Dropping NVMEM device-level post_process callback > > I decided to refactor existing callback. > >=20 > > Michael's work on adding #nvmem-cell-cells should be possible to easily > > rebase on top of those changes. =20 Yeah, I guess since Michael's series has been out for 2 years and we finally agreed on the bindings plus some implementation points, I would expect it to be merged very soon (I don't know if Srinivas still plans to take it for this release or for the next?) unless someone speaks up against it. > As yours should be easily added on top of my series. I've showed that > providing a global post process hook is bad because that way you need > to have *all* cells of your device read-only. >=20 > -michael Thanks, Miqu=C3=A8l