From patchwork Fri Jun 3 21:07:44 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Uwe_Kleine-K=C3=B6nig?= X-Patchwork-Id: 12869413 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 50926C43334 for ; Fri, 3 Jun 2022 21:12:18 +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: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:In-Reply-To:References: List-Owner; bh=eFoZ0vHIjH8N4BbzOzwx6Hztgdw6PdoywZK3aTrpDj0=; b=tb4Rp2AgSrkK2n d9RYD1p/X3efhN5PU5zQ0MckCpOWNFaJ0HfIccP7qjnvwoZ/Yj7AoE+Hm8zZBRJ4PIe4fupOO78o6 okOQBzGwSOHY0x5KyPgK4ikNJ2EYeH7vhgev6JRwdG9njwsl0uPY3omg/c1So7P3JQtebb7dhxl5h 2hq/Ht2dGZZe/xAQqfTtjFv3YOZWuaWA4EAk5vH3QvR6Z+RMVb9VNxLpyaVegZ+Q48vlb5BArshi9 DY4g+FPbJ4ZJftM1Q4d3vFRLgW44RbqVBvpKGnY1ov9Oe3WZIrccPZIg32T+OM1370kx7U0NrGnQG Baz9nl/ns8UjYbgHqtJw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nxEab-008k7R-4m; Fri, 03 Jun 2022 21:12:13 +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 1nxEXc-008hak-Gy for linux-amlogic@lists.infradead.org; Fri, 03 Jun 2022 21:09:09 +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 ) id 1nxEWg-0003ql-Rk; Fri, 03 Jun 2022 23:08:10 +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.94.2) (envelope-from ) id 1nxEWb-006Hpk-CA; Fri, 03 Jun 2022 23:08:04 +0200 Received: from ukl by ptz.office.stw.pengutronix.de with local (Exim 4.94.2) (envelope-from ) id 1nxEWZ-00E1H5-6s; Fri, 03 Jun 2022 23:08:03 +0200 From: =?utf-8?q?Uwe_Kleine-K=C3=B6nig?= To: Vignesh Raghavendra , Miquel Raynal , Richard Weinberger , Tudor Ambarus , Pratyush Yadav , Joel Stanley , Michael Ellerman , Nicolas Ferre , Alexandre Belloni , Claudiu Beznea , Stefan Agner , Lucas Stach , Thierry Reding , Jonathan Hunter , Liang Yang , Neil Armstrong , Kevin Hilman Cc: kernel@pengutronix.de, linux-mtd@lists.infradead.org, Michael Walle , Andrew Jeffery , linux-arm-kernel@lists.infradead.org, linux-aspeed@lists.ozlabs.org, Benjamin Herrenschmidt , Paul Mackerras , linuxppc-dev@lists.ozlabs.org, linux-tegra@vger.kernel.org, Jerome Brunet , Martin Blumenstingl , linux-amlogic@lists.infradead.org Subject: [PATCH 00/14] mtd: Fix platform remove callbacks to always return 0 Date: Fri, 3 Jun 2022 23:07:44 +0200 Message-Id: <20220603210758.148493-1-u.kleine-koenig@pengutronix.de> X-Mailer: git-send-email 2.36.1 MIME-Version: 1.0 X-Developer-Signature: v=1; a=openpgp-sha256; l=2310; i=uwe@kleine-koenig.org; h=from:subject; bh=q2ycDgelp/nO6ViqBZJgFoKvH7fVSGWxa2BGxqpYD+s=; b=owEBbQGS/pANAwAKAcH8FHityuwJAcsmYgBimnfE5fVVOvpqL4Xqzddai1jCcQBM7b5jo5W1QZw8 iIwB8NmJATMEAAEKAB0WIQR+cioWkBis/z50pAvB/BR4rcrsCQUCYpp3xAAKCRDB/BR4rcrsCXt2B/ 4qWz6olnMyidYhZ7g/qZaoPMgkHVnZRaB+OnSUr+L4Jm1IodQP4MrwiltmKlSyqmdwnvxJzxtnT0zb s0TX8Y6bInlnxd/SRRBKvGKHvHFYpztNxRZkvHHz9U4M6YMqj5zZeMjx9xgdm7bQ2o3UTQTFks2VAw kS3U2+mKRRm/1zgoPawD0hqmCMMbidTgGK/2vZHrF/V1JRMy5bcK1ByBNeXwoSXt72M2W72LAdU5XA mI7wUQR/g3qkW0tGS11qyaC5ZAS76ZpOBS5GaLlS2NwqdsSmKlZ+rWdlrNH0aBd3XjiWq8HeT31nQx srBqXHcGBUJS922WV6Z9YIm/CKnDXI X-Developer-Key: i=uwe@kleine-koenig.org; a=openpgp; fpr=0D2511F322BFAB1C1580266BE2DCDD9132669BD6 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-amlogic@lists.infradead.org X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220603_140908_612001_BE6F39CA X-CRM114-Status: GOOD ( 11.12 ) X-BeenThere: linux-amlogic@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-amlogic" Errors-To: linux-amlogic-bounces+linux-amlogic=archiver.kernel.org@lists.infradead.org Hello, this series prepares to make platform remove callbacks return void. Therefor first update them to always return 0. The rationale is that the Linux device model doesn't handle failures on remove and if a remove callback returns an error, it just emits a quite generic error message and still removes the device. Best regards Uwe Uwe Kleine-König (14): mtd: hyperbus: Make hyperbus_unregister_device() return void mtd: spi-nor: aspeed-smc: Make aspeed_smc_unregister() return void mtd: powernv_flash: Warn about failure to unregister mtd device mtd: st-spi_fsm: Warn about failure to unregister mtd device mtd: lpddr2_nvm: Warn about failure to unregister mtd device mtd: spear_smi: Don't skip cleanup after mtd_device_unregister() failed mtd: spear_smi: Drop if with an always false condition mtd: rawnand: atmel: Warn about failure to unregister mtd device mtd: rawnand: omap2: Suppress error message after WARN in .remove() mtd: rawnand: tegra: Don't skip cleanup after mtd_device_unregister() failed mtd: rawnand: meson: Don't skip cleanup after mtd_device_unregister() failed mtd: rawnand: meson: Drop cleaning platform data in .remove() mtd: physmap: Don't skip cleanup after mtd_device_unregister() failed mtd: physmap: Drop if with an always false condition drivers/mtd/devices/powernv_flash.c | 4 +++- drivers/mtd/devices/spear_smi.c | 10 ++-------- drivers/mtd/devices/st_spi_fsm.c | 4 +++- drivers/mtd/hyperbus/hbmc-am654.c | 6 +++--- drivers/mtd/hyperbus/hyperbus-core.c | 8 ++------ drivers/mtd/hyperbus/rpc-if.c | 5 +++-- drivers/mtd/lpddr/lpddr2_nvm.c | 4 +++- drivers/mtd/maps/physmap-core.c | 13 +++---------- drivers/mtd/nand/raw/atmel/nand-controller.c | 5 ++++- drivers/mtd/nand/raw/meson_nand.c | 16 +++------------- drivers/mtd/nand/raw/omap2.c | 6 ++---- drivers/mtd/nand/raw/tegra_nand.c | 5 +---- drivers/mtd/spi-nor/controllers/aspeed-smc.c | 8 ++++---- include/linux/mtd/hyperbus.h | 4 +--- 14 files changed, 37 insertions(+), 61 deletions(-) base-commit: 4b0986a3613c92f4ec1bdc7f60ec66fea135991f