From patchwork Wed Dec 27 16:26:24 2023 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: 13505388 Received: from metis.whiteo.stw.pengutronix.de (metis.whiteo.stw.pengutronix.de [185.203.201.7]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 7956B4988F for ; Wed, 27 Dec 2023 16:27:22 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=pengutronix.de Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=pengutronix.de Received: from drehscheibe.grey.stw.pengutronix.de ([2a0a:edc0:0:c01:1d::a2]) by metis.whiteo.stw.pengutronix.de with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1rIWk9-00059l-Ry; Wed, 27 Dec 2023 17:26:53 +0100 Received: from [2a0a:edc0:0:900:1d::77] (helo=ptz.office.stw.pengutronix.de) by drehscheibe.grey.stw.pengutronix.de with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1rIWjw-001tZH-HS; Wed, 27 Dec 2023 17:26:41 +0100 Received: from ukl by ptz.office.stw.pengutronix.de with local (Exim 4.96) (envelope-from ) id 1rIWjx-001Wmd-1d; Wed, 27 Dec 2023 17:26:41 +0100 From: =?utf-8?q?Uwe_Kleine-K=C3=B6nig?= To: Sudeep Holla , Tzung-Bi Shih , Shawn Guo , Sascha Hauer , Daniel Baluta , Thierry Reding , Hao Ge , Rob Herring , Christophe JAILLET , Matthias Brugger , AngeloGioacchino Del Regno , Arnd Bergmann , Gabriel Somlo , "Michael S. Tsirkin" , Florian Fainelli , Kieran Bingham , Dinh Nguyen , =?utf-8?q?Marek_Beh=C3=BAn?= , Michal Simek , Greg Kroah-Hartman , Jay Buddhabhatti , Sai Krishna Potthuri , Linus Walleij , Nava kishore Manne , Rajan Vaja , Dhaval Shah , Marek Vasut Cc: kernel@pengutronix.de, Cristian Marussi , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Brian Norris , Julius Werner , chrome-platform@lists.linux.dev, Fabio Estevam , NXP Linux Team , linux-mediatek@lists.infradead.org, qemu-devel@nongnu.org, Broadcom internal kernel review list , linux-rpi-kernel@lists.infradead.org Subject: [PATCH 00/11] firmware: Convert to platform remove callback returning void Date: Wed, 27 Dec 2023 17:26:24 +0100 Message-ID: X-Mailer: git-send-email 2.43.0 Precedence: bulk X-Mailing-List: chrome-platform@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-Developer-Signature: v=1; a=openpgp-sha256; l=2285; i=u.kleine-koenig@pengutronix.de; h=from:subject:message-id; bh=tI3zkKx4Ui/NNKJl8C0bvrZtoud3+4uqGjgMG21LlTs=; b=owGbwMvMwMXY3/A7olbonx/jabUkhtSeAKNH8tevhnB8dlzNPIGlymlxs/6VfQ7Zsacbwi4eS m/jlu3vZDRmYWDkYpAVU2Sxb1yTaVUlF9m59t9lmEGsTCBTGLg4BWAiCa/Z/9cn838+oXFr/z2L FyUtR2cw+haX73S48WmTZ9FOuTgxrrTr8lYvFvFx7OX2yc6QnvUtSviR/N5ft82FOnsDUv9/tlj eskLcWs9N+PqrufauLGUqGWY6t7dqca28cfdQXvW8eTafm1bPVv/j/rpm78XrGxiCbzm0Nz68YO xYtWSa6gaNrIybLSzGNhHeZXLT1Xi9D6/vOe708zk3d33rrux1X9LsXZa8jhJnuhPyVLryz8MFO mk27O+F0mfK/qlxOduSf3JesLWJY8PtiEbdO8E/TrfNvvdeuetgsmG4TjtXXIrBqWN7Lir2hcbv 2R3tYjRPK7tZ8KNkyfbGmcUVh0wfbwlTWRz6K+jbg5pZAA== X-Developer-Key: i=u.kleine-koenig@pengutronix.de; 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.whiteo.stw.pengutronix.de); SAEximRunCond expanded to false X-PTX-Original-Recipient: chrome-platform@lists.linux.dev Hello, this series converts all platform drivers below drivers/firmware that make use of .remove() to use .remove_new() instead. See commit 5c5a7680e67b ("platform: Provide a remove callback that returns no value") for an extended explanation and the eventual goal. The TL;DR; is to make it harder for driver authors to leak resources without noticing. The drivers here get it right though and so can be converted trivially. This is merge window material. There doesn't seem to be a maintainer for all of drivers/firmware and I don't know how patch application works there usually. All patches are pairwise independent, so they can be applied individually. Best regards Uwe Uwe Kleine-König (11): firmware: arm_scmi: Convert to platform remove callback returning void firmware: arm_scpi: Convert to platform remove callback returning void firmware: coreboot_table: Convert to platform remove callback returning void firmware: imx-dsp: Convert to platform remove callback returning void firmware: mtk-adsp-ipc: Convert to platform remove callback returning void firmware: qemu_fw_cfg: Convert to platform remove callback returning void firmware: raspberrypi: Convert to platform remove callback returning void firmware: stratix10-rsu: Convert to platform remove callback returning void firmware: stratix10-svc: Convert to platform remove callback returning void firmware: turris-mox-rwtm: Convert to platform remove callback returning void firmware: zynqmp: Convert to platform remove callback returning void drivers/firmware/arm_scmi/driver.c | 6 ++---- drivers/firmware/arm_scpi.c | 6 ++---- drivers/firmware/google/coreboot_table.c | 5 ++--- drivers/firmware/imx/imx-dsp.c | 6 ++---- drivers/firmware/mtk-adsp-ipc.c | 6 ++---- drivers/firmware/qemu_fw_cfg.c | 5 ++--- drivers/firmware/raspberrypi.c | 6 ++---- drivers/firmware/stratix10-rsu.c | 5 ++--- drivers/firmware/stratix10-svc.c | 6 ++---- drivers/firmware/turris-mox-rwtm.c | 6 ++---- drivers/firmware/xilinx/zynqmp.c | 6 ++---- 11 files changed, 22 insertions(+), 41 deletions(-) base-commit: 39676dfe52331dba909c617f213fdb21015c8d10