Re: [PATCH] soc: Switch back to struct platform_driver::remove()

From: Herve Codina <herve.codina@bootlin.com>
To: "Uwe Kleine-König" <u.kleine-koenig@baylibre.com>
Cc: "Arnd Bergmann" <arnd@arndb.de>,
	"Olof Johansson" <olof@lixom.net>,
	"Joel Stanley" <joel@jms.id.au>,
	"Andrew Jeffery" <andrew@codeconstruct.com.au>,
	"Christophe Leroy" <christophe.leroy@csgroup.eu>,
	"Qiang Zhao" <qiang.zhao@nxp.com>,
	"Hitomi Hasegawa" <hasegawa-hitomi@fujitsu.com>,
	"Huisong Li" <lihuisong@huawei.com>,
	"Linus Walleij" <linusw@kernel.org>,
	"Imre Kaloz" <kaloz@openwrt.org>,
	"Karol Gugala" <kgugala@antmicro.com>,
	"Mateusz Holenko" <mholenko@antmicro.com>,
	"Gabriel Somlo" <gsomlo@gmail.com>,
	"Yinbo Zhu" <zhuyinbo@loongson.cn>,
	"Matthias Brugger" <matthias.bgg@gmail.com>,
	"AngeloGioacchino Del Regno"
	<angelogioacchino.delregno@collabora.com>,
	"Conor Dooley" <conor.dooley@microchip.com>,
	"Daire McNamara" <daire.mcnamara@microchip.com>,
	"Krzysztof Kozlowski" <krzysztof.kozlowski@linaro.org>,
	"Bjorn Andersson" <andersson@kernel.org>,
	"Konrad Dybcio" <konradybcio@kernel.org>,
	"Heiko Stuebner" <heiko@sntech.de>,
	"Alim Akhtar" <alim.akhtar@samsung.com>,
	"Thierry Reding" <thierry.reding@gmail.com>,
	"Jonathan Hunter" <jonathanh@nvidia.com>,
	"Nishanth Menon" <nm@ti.com>,
	"Santosh Shilimkar" <ssantosh@kernel.org>,
	"Michal Simek" <michal.simek@amd.com>,
	"Andy Shevchenko" <andriy.shevchenko@linux.intel.com>,
	"Duje Mihanović" <duje.mihanovic@skole.hr>,
	"Mark Brown" <broonie@kernel.org>,
	"David Wu" <david.wu@rock-chips.com>,
	"Jianqun Xu" <jay.xu@rock-chips.com>,
	"Jay Buddhabhatti" <jay.buddhabhatti@amd.com>,
	"Radhey Shyam Pandey" <radhey.shyam.pandey@amd.com>,
	"Izhar Ameer Shaikh" <izhar.ameer.shaikh@amd.com>,
	"Naman Trivedi Manojbhai" <naman.trivedimanojbhai@amd.com>,
	linux-arm-kernel@lists.infradead.org, soc@lists.linux.dev,
	linux-aspeed@lists.ozlabs.org, linux-kernel@vger.kernel.org,
	linuxppc-dev@lists.ozlabs.org, loongarch@lists.linux.dev,
	linux-mediatek@lists.infradead.org,
	linux-riscv@lists.infradead.org, linux-arm-msm@vger.kernel.org,
	linux-rockchip@lists.infradead.org,
	linux-samsung-soc@vger.kernel.org, linux-tegra@vger.kernel.org,
	linux-pm@vger.kernel.org
Subject: Re: [PATCH] soc: Switch back to struct platform_driver::remove()
Date: Tue, 29 Oct 2024 10:03:47 +0100	[thread overview]
Message-ID: <20241029100347.542b56d4@bootlin.com> (raw)
In-Reply-To: <20241029074859.509587-2-u.kleine-koenig@baylibre.com>

Hi Uwe,

On Tue, 29 Oct 2024 08:48:58 +0100
Uwe Kleine-König <u.kleine-koenig@baylibre.com> wrote:

> After commit 0edb555a65d1 ("platform: Make platform_driver::remove()
> return void") .remove() is (again) the right callback to implement for
> platform drivers.
> 
> Convert all platform drivers below drivers/soc to use .remove(), with
> the eventual goal to drop struct platform_driver::remove_new(). As
> .remove() and .remove_new() have the same prototypes, conversion is done
> by just changing the structure member name in the driver initializer.
> 
> On the way do a few whitespace changes to make indention consistent.
> 
> Signed-off-by: Uwe Kleine-König <u.kleine-koenig@baylibre.com>
> ---
...
>  drivers/soc/fsl/qe/qmc.c                    | 2 +-
>  drivers/soc/fsl/qe/tsa.c                    | 2 +-

Acked-by: Herve Codina <herve.codina@bootlin.com> # for fsl/qe/{qmc,tsa}.c

Best regards,
Hervé