Message ID | 20211012153945.2651412-17-u.kleine-koenig@pengutronix.de |
---|---|
State | Changes Requested |
Headers | show
Return-Path: <linux-mtd-bounces+incoming=patchwork.ozlabs.org@lists.infradead.org> X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Authentication-Results: bilbo.ozlabs.org; dkim=pass (2048-bit key; secure) header.d=lists.infradead.org header.i=@lists.infradead.org header.a=rsa-sha256 header.s=bombadil.20210309 header.b=jMZHTLof; dkim-atps=neutral Authentication-Results: ozlabs.org; spf=none (no SPF record) smtp.mailfrom=lists.infradead.org (client-ip=2607:7c80:54:e::133; helo=bombadil.infradead.org; envelope-from=linux-mtd-bounces+incoming=patchwork.ozlabs.org@lists.infradead.org; receiver=<UNKNOWN>) Received: from bombadil.infradead.org (bombadil.infradead.org [IPv6:2607:7c80:54:e::133]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by bilbo.ozlabs.org (Postfix) with ESMTPS id 4HTKdD0Kxpz9t0p for <incoming@patchwork.ozlabs.org>; Wed, 13 Oct 2021 02:41:04 +1100 (AEDT) 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:Date:Subject:Cc:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=uebfITYbESR2gVl43stwKvzOu6suv6XEN3RWjW5lNYo=; b=jMZHTLofwsLbxS ZOePiTLifvXcftbmgERuu03UPyW5hJSr8bzUQ7Bft67SVPYoogaVg++O5u0i0s8Xc6vJQ7Oke6Yza b/ZFNwdN0hyyCVx5VFuEMt6UkeyCa8KoiFBSxydhKoMzxubqxpd9qdH2eTszzab4iymHUVMVNGCqe IJm1uwfYWoamv3i4zk7KM4a9QpDmk/3cUZ9cHPJIO06fI5evN7ygGkUQnpA7llX1w0CnLMHKz8Vu/ gYdiOXnI/mVOkBFQpOcCPt8gPmyRLPLxRA4d8qZ+P+rb6kVQZGpvdECUCmbJQa5fmHCDTK7q8gie7 H/y32ZFOXFsdGPh4oVsQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1maJt2-00DN2j-KF; Tue, 12 Oct 2021 15:40:16 +0000 Received: from metis.ext.pengutronix.de ([2001:67c:670:201:290:27ff:fe1d:cc33]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1maJsp-00DMwK-4f for linux-mtd@lists.infradead.org; Tue, 12 Oct 2021 15:40:04 +0000 Received: from drehscheibe.grey.stw.pengutronix.de ([2a0a:edc0:0:c01:1d::a2]) by metis.ext.pengutronix.de with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from <ukl@pengutronix.de>) id 1maJse-0008NW-M2; Tue, 12 Oct 2021 17:39:52 +0200 Received: from [2a0a:edc0:0:900:1d::77] (helo=ptz.office.stw.pengutronix.de) by drehscheibe.grey.stw.pengutronix.de with esmtp (Exim 4.92) (envelope-from <ukl@pengutronix.de>) id 1maJsd-0004in-Ps; Tue, 12 Oct 2021 17:39:51 +0200 Received: from ukl by ptz.office.stw.pengutronix.de with local (Exim 4.92) (envelope-from <ukl@pengutronix.de>) id 1maJsd-0004Yp-Om; Tue, 12 Oct 2021 17:39:51 +0200 From: =?utf-8?q?Uwe_Kleine-K=C3=B6nig?= <u.kleine-koenig@pengutronix.de> To: Miquel Raynal <miquel.raynal@bootlin.com>, Richard Weinberger <richard@nod.at>, Vignesh Raghavendra <vigneshr@ti.com> Cc: Mark Brown <broonie@kernel.org>, kernel@pengutronix.de, linux-mtd@lists.infradead.org, linux-spi@vger.kernel.org Subject: [PATCH v2 16/20] mtd: sst25l: Warn about failure to unregister mtd device Date: Tue, 12 Oct 2021 17:39:41 +0200 Message-Id: <20211012153945.2651412-17-u.kleine-koenig@pengutronix.de> X-Mailer: git-send-email 2.30.2 In-Reply-To: <20211012153945.2651412-1-u.kleine-koenig@pengutronix.de> References: <20211012153945.2651412-1-u.kleine-koenig@pengutronix.de> MIME-Version: 1.0 X-Patch-Hashes: v=1; h=sha256; i=chTrsNwtyvGFDidu8JfqkR8D1NeBCtiv8G0Xdn/Msdg=; m=aWXlVn4zGl3/VNxE8xQ1auTXzJiWk3xyQCYCHspYcNI=; p=TyuS/iMN/2qfQLKMz56PvUq4wtcc0B7f+zs42TknICw=; g=c6b8458784ebb8ccbecde25a4af573d5e8fd863b X-Patch-Sig: m=pgp; i=u.kleine-koenig@pengutronix.de; s=0x0D2511F322BFAB1C1580266BE2DCDD9132669BD6; b=iQEzBAABCgAdFiEEfnIqFpAYrP8+dKQLwfwUeK3K7AkFAmFlrCkACgkQwfwUeK3K7AnDkQf+Nb3 3j5Jgm1majYjJ+sScbZ2HOkWGFVOQWeTvYfeytVvtxml2BbCSUhRa60oI3pnRl8KZpf1LSPtBx0Lv gw0p6PqXhs0kpkFsu1zukS4yPR4O6U+11E+3KrCQvgpXZtpIEuMYwdqirjME5jbPloH2LftzFCXeT C85UknBWop/d3AD0ivfVGL7M8x7iMrhPacg6H8/x6CoOp9UyiWor0X9qahA3FpnSQBd1SmA8HhPoO 2jnNcag0IBYeCs08IYABkj1kdMk3CFurya0dEyMO774G3TwA+fP+nWl2cZRZKf6iKGPzY98B9xCVZ sh77HoYmLIWw5+szDsL7QgTVpm7ueDA== X-SA-Exim-Connect-IP: 2a0a:edc0:0:c01:1d::a2 X-SA-Exim-Mail-From: ukl@pengutronix.de X-SA-Exim-Scanned: No (on metis.ext.pengutronix.de); SAEximRunCond expanded to false X-PTX-Original-Recipient: linux-mtd@lists.infradead.org X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20211012_084003_206400_46DFE41A X-CRM114-Status: GOOD ( 13.70 ) X-Spam-Score: -2.3 (--) X-Spam-Report: =?unknown-8bit?q?Spam_detection_software=2C_running_on_the_sy?= =?unknown-8bit?q?stem_=22bombadil=2Einfradead=2Eorg=22=2C?= =?unknown-8bit?q?_has_NOT_identified_this_incoming_email_as_spam=2E__The_ori?= =?unknown-8bit?q?ginal?= =?unknown-8bit?q?_message_has_been_attached_to_this_so_you_can_view_it_or_la?= =?unknown-8bit?q?bel?= =?unknown-8bit?q?_similar_future_email=2E__If_you_have_any_questions=2C_see?= =?unknown-8bit?q?_the_administrator_of_that_system_for_details=2E?= =?unknown-8bit?q?_?= =?unknown-8bit?q?_Content_preview=3A__When_an_spi_driver=27s_remove_function?= =?unknown-8bit?q?_returns_a_non-zero_error?= =?unknown-8bit?q?_code_nothing_happens_apart_from_emitting_a_generic_error_m?= =?unknown-8bit?q?essage=2E_Make_this?= =?unknown-8bit?q?_error_message_more_device_specific_and_return_zero_instead?= =?unknown-8bit?q?=2E_Signed-off-by=3A?= =?unknown-8bit?q?_Uwe_Kleine-K=C3=B6nig_=3Cu=2Ekleine-koenig=40pengutronix?= =?unknown-8bit?q?=2Ede=3E_---_drivers/mtd/devices/sst25l=2Ec?= =?unknown-8bit?q?_=7C_8_+++++++-_1_file_changed=2C_7_insertions=28+=29=2C_1_?= =?unknown-8bit?q?deletion=28-=29_?= =?unknown-8bit?q?_?= =?unknown-8bit?q?_Content_analysis_details=3A___=28-2=2E3_points=2C_5=2E0_re?= =?unknown-8bit?q?quired=29?= =?unknown-8bit?q?_?= =?unknown-8bit?q?_pts_rule_name______________description?= =?unknown-8bit?q?_----_----------------------_------------------------------?= =?unknown-8bit?q?--------------------?= =?unknown-8bit?q?_-2=2E3_RCVD=5FIN=5FDNSWL=5FMED______RBL=3A_Sender_listed_a?= =?unknown-8bit?q?t_https=3A//www=2Ednswl=2Eorg/=2C?= =?unknown-8bit?q?_medium_trust?= =?unknown-8bit?b?IFsyMDAxOjY3Yzo2NzA6MjAxOjI5MDoyN2ZmOmZlMWQ6Y2MzMyBsaXN0?= =?unknown-8bit?b?ZWQgaW5d?= =?unknown-8bit?b?IFtsaXN0LmRuc3dsLm9yZ10=?= =?unknown-8bit?q?_-0=2E0_SPF=5FPASS_______________SPF=3A_sender_matches_SPF_?= =?unknown-8bit?q?record?= =?unknown-8bit?q?_0=2E0_SPF=5FHELO=5FNONE__________SPF=3A_HELO_does_not_publ?= =?unknown-8bit?q?ish_an_SPF_Record?= X-BeenThere: linux-mtd@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Linux MTD discussion mailing list <linux-mtd.lists.infradead.org> List-Unsubscribe: <http://lists.infradead.org/mailman/options/linux-mtd>, <mailto:linux-mtd-request@lists.infradead.org?subject=unsubscribe> List-Archive: <http://lists.infradead.org/pipermail/linux-mtd/> List-Post: <mailto:linux-mtd@lists.infradead.org> List-Help: <mailto:linux-mtd-request@lists.infradead.org?subject=help> List-Subscribe: <http://lists.infradead.org/mailman/listinfo/linux-mtd>, <mailto:linux-mtd-request@lists.infradead.org?subject=subscribe> Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Sender: "linux-mtd" <linux-mtd-bounces@lists.infradead.org> Errors-To: linux-mtd-bounces+incoming=patchwork.ozlabs.org@lists.infradead.org |
Series |
Make some spi device drivers return zero in .remove()
|
expand
|
diff --git a/drivers/mtd/devices/sst25l.c b/drivers/mtd/devices/sst25l.c index b81c3f0b85f9..04062a1f54c1 100644 --- a/drivers/mtd/devices/sst25l.c +++ b/drivers/mtd/devices/sst25l.c @@ -401,8 +401,14 @@ static int sst25l_probe(struct spi_device *spi) static int sst25l_remove(struct spi_device *spi) { struct sst25l_flash *flash = spi_get_drvdata(spi); + int ret; + + ret = mtd_device_unregister(&flash->mtd); + if (ret) + dev_warn(&spi->dev, "Failed to unregister mtd device (%pe)\n", + ERR_PTR(ret)); - return mtd_device_unregister(&flash->mtd); + return 0; } static struct spi_driver sst25l_driver = {
When an spi driver's remove function returns a non-zero error code nothing happens apart from emitting a generic error message. Make this error message more device specific and return zero instead. Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de> --- drivers/mtd/devices/sst25l.c | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-)