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 alsa0.perex.cz (alsa0.perex.cz [77.48.224.243]) (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 A8DCCC0015E for ; Tue, 25 Jul 2023 11:53:24 +0000 (UTC) Received: from alsa1.perex.cz (alsa1.perex.cz [207.180.221.201]) (using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by alsa0.perex.cz (Postfix) with ESMTPS id 453E81513; Tue, 25 Jul 2023 13:52:32 +0200 (CEST) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa0.perex.cz 453E81513 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=alsa-project.org; s=default; t=1690286002; bh=R1cMTZX1PUU2KkohyDFYTgVjZODSeThTo605Qizu2/o=; h=Date:From:To:Cc:Subject:In-Reply-To:References:List-Id: List-Archive:List-Help:List-Owner:List-Post:List-Subscribe: List-Unsubscribe:From; b=k4rF1TD0PJqK5zr1vV4XlyTuuQGjaHr/A1dyTanyIbnxHmfrJW3ES50tu5NKLJdrt yub1DIVmgxNCQMbbO2CCmFW5ft9jG+cEtlR6hWBpn2djSydM6M91vWZhQmMKMdg5EY LsX4XM+frcaGFcER4eEl81Vd3DRgdlfqkApTosBc= Received: by alsa1.perex.cz (Postfix, from userid 50401) id D9B20F80163; Tue, 25 Jul 2023 13:51:38 +0200 (CEST) Received: from mailman-core.alsa-project.org (mailman-core.alsa-project.org [10.254.200.10]) by alsa1.perex.cz (Postfix) with ESMTP id 8C068F8019B; Tue, 25 Jul 2023 13:51:37 +0200 (CEST) Received: by alsa1.perex.cz (Postfix, from userid 50401) id 29EC7F8019B; Mon, 24 Jul 2023 09:09:27 +0200 (CEST) Received: from relay1-d.mail.gandi.net (relay1-d.mail.gandi.net [IPv6:2001:4b98:dc4:8::221]) (using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by alsa1.perex.cz (Postfix) with ESMTPS id F0FB0F800C7 for ; Mon, 24 Jul 2023 09:09:13 +0200 (CEST) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa1.perex.cz F0FB0F800C7 Authentication-Results: alsa1.perex.cz; dkim=pass (2048-bit key, unprotected) header.d=bootlin.com header.i=@bootlin.com header.a=rsa-sha256 header.s=gm1 header.b=iC5ZIY44 Received: by mail.gandi.net (Postfix) with ESMTPSA id 6C2E524000E; Mon, 24 Jul 2023 07:09:04 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1690182552; 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=0OTdgWQ6Vo/4tl1ZM7En0YdjITPLKGIP2IIxkkPFJRM=; b=iC5ZIY44VFrCSUNoeMIdYmOyqSrlrW7N4tyY+Hq53oXBVph3jCQ5ij8fsDynsYh6NybKfF 47iBUwR964ggu52YB13yaU1E18wok2aDQAXdbpXfZnCVzqeybMcvNBEm04M8MJv49vz912 60p20TKBhztddQMV6EZbSo0vTLzFb3Ok+FWErn92iDiDFjDyPmqRh1JKdYw5xT3hwpwVlV u4k2GthKUnBFuRNrRP/XoPSerupU1HIaiVySzO/QpWk35FaiLI3dwdIhQ2brWGTyOPPUCM 8E3/Ier83smwYkYCv/LsZ20OWFpB5I2g8A5epoX659Pa/Omr3Rckh60nqnZ+5A== Date: Mon, 24 Jul 2023 09:09:02 +0200 From: Miquel Raynal To: Andy Shevchenko Cc: nikita.shubin@maquefel.me, Hartley Sweeten , Lennert Buytenhek , Alexander Sverdlin , Russell King , Lukasz Majewski , Linus Walleij , Bartosz Golaszewski , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Michael Turquette , Stephen Boyd , Daniel Lezcano , Thomas Gleixner , Alessandro Zummo , Alexandre Belloni , Wim Van Sebroeck , Guenter Roeck , Sebastian Reichel , Thierry Reding , Uwe =?UTF-8?B?S2xlaW5lLUvDtm5pZw==?= , Mark Brown , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Vinod Koul , Richard Weinberger , Vignesh Raghavendra , Damien Le Moal , Sergey Shtylyov , Dmitry Torokhov , Arnd Bergmann , Olof Johansson , soc@kernel.org, Liam Girdwood , Jaroslav Kysela , Takashi Iwai , Michael Peters , Kris Bahnsen , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-gpio@vger.kernel.org, devicetree@vger.kernel.org, linux-clk@vger.kernel.org, linux-rtc@vger.kernel.org, linux-watchdog@vger.kernel.org, linux-pm@vger.kernel.org, linux-pwm@vger.kernel.org, linux-spi@vger.kernel.org, netdev@vger.kernel.org, dmaengine@vger.kernel.org, linux-mtd@lists.infradead.org, linux-ide@vger.kernel.org, linux-input@vger.kernel.org, alsa-devel@alsa-project.org Subject: Re: [PATCH v3 24/42] mtd: nand: add support for ts72xx Message-ID: <20230724090902.679ea56d@xps-13> In-Reply-To: References: <20230605-ep93xx-v3-0-3d63a5f1103e@maquefel.me> <20230605-ep93xx-v3-24-3d63a5f1103e@maquefel.me> 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 X-GND-Sasl: miquel.raynal@bootlin.com X-MailFrom: miquel.raynal@bootlin.com X-Mailman-Rule-Hits: max-recipients X-Mailman-Rule-Misses: dmarc-mitigation; no-senders; approved; emergency; loop; banned-address; member-moderation; header-match-alsa-devel.alsa-project.org-0; header-match-alsa-devel.alsa-project.org-1; nonmember-moderation; administrivia; implicit-dest; max-size; news-moderation; no-subject; digests; suspicious-header Message-ID-Hash: JVCHUMNHXNIZ2L65HRXD7ECL7QXVGQXZ X-Message-ID-Hash: JVCHUMNHXNIZ2L65HRXD7ECL7QXVGQXZ X-Mailman-Approved-At: Tue, 25 Jul 2023 11:51:32 +0000 X-Mailman-Version: 3.3.8 Precedence: list List-Id: "Alsa-devel mailing list for ALSA developers - http://www.alsa-project.org" Archived-At: List-Archive: List-Help: List-Owner: List-Post: List-Subscribe: List-Unsubscribe: Hi Andy, > > +static int ts72xx_nand_attach_chip(struct nand_chip *chip) > > +{ > > + switch (chip->ecc.engine_type) { > > + case NAND_ECC_ENGINE_TYPE_SOFT: > > + if (chip->ecc.algo =3D=3D NAND_ECC_ALGO_UNKNOWN) > > + chip->ecc.algo =3D NAND_ECC_ALGO_HAMMING; > > + break; > > + case NAND_ECC_ENGINE_TYPE_ON_HOST: > > + return -EINVAL; > > + default: =20 >=20 > > + break; =20 >=20 > Here it will return 0, is it a problem? Seems ok, there are two other situations: on-die ECC engine and no ECC engine, both do not require any specific handling on the controller side. >=20 > > + } > > + > > + return 0; > > +} =20 >=20 > ... >=20 > > +static void ts72xx_nand_remove(struct platform_device *pdev) > > +{ > > + struct ts72xx_nand_data *data =3D platform_get_drvdata(pdev); > > + struct nand_chip *chip =3D &data->chip; > > + int ret; > > + > > + ret =3D mtd_device_unregister(nand_to_mtd(chip)); =20 >=20 > > + WARN_ON(ret); =20 >=20 > Why?! Is it like this in other MTD drivers? Yes, we did not yet change the internal machinery to return void, and we don't want people to think getting errors there is normal. > > + nand_cleanup(chip); > > +} =20 >=20 Thanks, Miqu=C3=A8l 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 07773C001B0 for ; Mon, 24 Jul 2023 07:09:47 +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=sAG0PnUe72areBQeejVc6FHzTrlN4WaL6dp9C5xItQ8=; b=wQ4vLmyjhw4zIc +M1+AyfrQ/blRtP718kTO9ObOm0f2hzDYtjLNT1P75kniZkMVTS3DGnmKNi6eOTHVqB0KYPmTxR/e YX8vHSxSSDEq2MMc4+xWCIdoo3NXuY44+RB9CuWicSTXJ9NC57NaPaA6NmDDUhvgIaxMiFyDbrdXO ng9vJJ2Cma0wM3nDGR4NiKp135EVH4fmBm+LCVmvANbRqk1foro9E+d0K3OvjEnRSC53sUfdZ253N muzWrENottHCWRH7eKWwOL5tE6dC8VkR5p9pSfMjvEF6HBYVoSX1uqee0v1hRDsDB7n73gZD0FcQz G+U8Vf5c8d0tcnzuLuaQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qNph8-0033u2-0g; Mon, 24 Jul 2023 07:09:26 +0000 Received: from relay1-d.mail.gandi.net ([2001:4b98:dc4:8::221]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qNph1-0033sF-2r; Mon, 24 Jul 2023 07:09:22 +0000 Received: by mail.gandi.net (Postfix) with ESMTPSA id 6C2E524000E; Mon, 24 Jul 2023 07:09:04 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1690182552; 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=0OTdgWQ6Vo/4tl1ZM7En0YdjITPLKGIP2IIxkkPFJRM=; b=iC5ZIY44VFrCSUNoeMIdYmOyqSrlrW7N4tyY+Hq53oXBVph3jCQ5ij8fsDynsYh6NybKfF 47iBUwR964ggu52YB13yaU1E18wok2aDQAXdbpXfZnCVzqeybMcvNBEm04M8MJv49vz912 60p20TKBhztddQMV6EZbSo0vTLzFb3Ok+FWErn92iDiDFjDyPmqRh1JKdYw5xT3hwpwVlV u4k2GthKUnBFuRNrRP/XoPSerupU1HIaiVySzO/QpWk35FaiLI3dwdIhQ2brWGTyOPPUCM 8E3/Ier83smwYkYCv/LsZ20OWFpB5I2g8A5epoX659Pa/Omr3Rckh60nqnZ+5A== Date: Mon, 24 Jul 2023 09:09:02 +0200 From: Miquel Raynal To: Andy Shevchenko Cc: nikita.shubin@maquefel.me, Hartley Sweeten , Lennert Buytenhek , Alexander Sverdlin , Russell King , Lukasz Majewski , Linus Walleij , Bartosz Golaszewski , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Michael Turquette , Stephen Boyd , Daniel Lezcano , Thomas Gleixner , Alessandro Zummo , Alexandre Belloni , Wim Van Sebroeck , Guenter Roeck , Sebastian Reichel , Thierry Reding , Uwe =?UTF-8?B?S2xlaW5lLUvDtm5pZw==?= , Mark Brown , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Vinod Koul , Richard Weinberger , Vignesh Raghavendra , Damien Le Moal , Sergey Shtylyov , Dmitry Torokhov , Arnd Bergmann , Olof Johansson , soc@kernel.org, Liam Girdwood , Jaroslav Kysela , Takashi Iwai , Michael Peters , Kris Bahnsen , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-gpio@vger.kernel.org, devicetree@vger.kernel.org, linux-clk@vger.kernel.org, linux-rtc@vger.kernel.org, linux-watchdog@vger.kernel.org, linux-pm@vger.kernel.org, linux-pwm@vger.kernel.org, linux-spi@vger.kernel.org, netdev@vger.kernel.org, dmaengine@vger.kernel.org, linux-mtd@lists.infradead.org, linux-ide@vger.kernel.org, linux-input@vger.kernel.org, alsa-devel@alsa-project.org Subject: Re: [PATCH v3 24/42] mtd: nand: add support for ts72xx Message-ID: <20230724090902.679ea56d@xps-13> In-Reply-To: References: <20230605-ep93xx-v3-0-3d63a5f1103e@maquefel.me> <20230605-ep93xx-v3-24-3d63a5f1103e@maquefel.me> 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-20230724_000920_393551_2D0EFBE0 X-CRM114-Status: GOOD ( 18.94 ) 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 SGkgQW5keSwKCj4gPiArc3RhdGljIGludCB0czcyeHhfbmFuZF9hdHRhY2hfY2hpcChzdHJ1Y3Qg bmFuZF9jaGlwICpjaGlwKQo+ID4gK3sKPiA+ICsJc3dpdGNoIChjaGlwLT5lY2MuZW5naW5lX3R5 cGUpIHsKPiA+ICsJY2FzZSBOQU5EX0VDQ19FTkdJTkVfVFlQRV9TT0ZUOgo+ID4gKwkJaWYgKGNo aXAtPmVjYy5hbGdvID09IE5BTkRfRUNDX0FMR09fVU5LTk9XTikKPiA+ICsJCQljaGlwLT5lY2Mu YWxnbyA9IE5BTkRfRUNDX0FMR09fSEFNTUlORzsKPiA+ICsJCWJyZWFrOwo+ID4gKwljYXNlIE5B TkRfRUNDX0VOR0lORV9UWVBFX09OX0hPU1Q6Cj4gPiArCQlyZXR1cm4gLUVJTlZBTDsKPiA+ICsJ ZGVmYXVsdDogIAo+IAo+ID4gKwkJYnJlYWs7ICAKPiAKPiBIZXJlIGl0IHdpbGwgcmV0dXJuIDAs IGlzIGl0IGEgcHJvYmxlbT8KClNlZW1zIG9rLCB0aGVyZSBhcmUgdHdvIG90aGVyIHNpdHVhdGlv bnM6IG9uLWRpZSBFQ0MgZW5naW5lIGFuZCBubyBFQ0MKZW5naW5lLCBib3RoIGRvIG5vdCByZXF1 aXJlIGFueSBzcGVjaWZpYyBoYW5kbGluZyBvbiB0aGUgY29udHJvbGxlcgpzaWRlLgoKPiAKPiA+ ICsJfQo+ID4gKwo+ID4gKwlyZXR1cm4gMDsKPiA+ICt9ICAKPiAKPiAuLi4KPiAKPiA+ICtzdGF0 aWMgdm9pZCB0czcyeHhfbmFuZF9yZW1vdmUoc3RydWN0IHBsYXRmb3JtX2RldmljZSAqcGRldikK PiA+ICt7Cj4gPiArCXN0cnVjdCB0czcyeHhfbmFuZF9kYXRhICpkYXRhID0gcGxhdGZvcm1fZ2V0 X2RydmRhdGEocGRldik7Cj4gPiArCXN0cnVjdCBuYW5kX2NoaXAgKmNoaXAgPSAmZGF0YS0+Y2hp cDsKPiA+ICsJaW50IHJldDsKPiA+ICsKPiA+ICsJcmV0ID0gbXRkX2RldmljZV91bnJlZ2lzdGVy KG5hbmRfdG9fbXRkKGNoaXApKTsgIAo+IAo+ID4gKwlXQVJOX09OKHJldCk7ICAKPiAKPiBXaHk/ ISAgSXMgaXQgbGlrZSB0aGlzIGluIG90aGVyIE1URCBkcml2ZXJzPwoKWWVzLCB3ZSBkaWQgbm90 IHlldCBjaGFuZ2UgdGhlIGludGVybmFsIG1hY2hpbmVyeSB0byByZXR1cm4gdm9pZCwgYW5kCndl IGRvbid0IHdhbnQgcGVvcGxlIHRvIHRoaW5rIGdldHRpbmcgZXJyb3JzIHRoZXJlIGlzIG5vcm1h bC4KCj4gPiArCW5hbmRfY2xlYW51cChjaGlwKTsKPiA+ICt9ICAKPiAKClRoYW5rcywKTWlxdcOo bAoKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f CkxpbnV4IE1URCBkaXNjdXNzaW9uIG1haWxpbmcgbGlzdApodHRwOi8vbGlzdHMuaW5mcmFkZWFk Lm9yZy9tYWlsbWFuL2xpc3RpbmZvL2xpbnV4LW10ZC8K