From nobody Wed Dec 17 12:06:47 2025 Received: from mail-wm1-f46.google.com (mail-wm1-f46.google.com [209.85.128.46]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id EAB3F20D518 for ; Tue, 12 Nov 2024 08:35:58 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.46 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1731400561; cv=none; b=aozX9skzK2CFQ3OuhxleDfItnD2egV+kHAsnDtEDHF037KXq8W4k8X3LTWfuOMLDYaNTsbLDnnGfk3Cv+15HiV/u46VmW8EoSDIhYK85l1kjuCzvLUCBApFUtgY4YXX9HaGizHiH9lIi4OykDRWRCdL+mTGTX660P7rIM0vbjNw= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1731400561; c=relaxed/simple; bh=Ogd04ibt1boR9D5geKyDS0rBfBdR3YmD0RQ58WcqbMs=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version:Content-Type; b=bTsW0JGPyL3MMqd5furA+y0ebEHH+kbTeUpAZ1M4JhiyA5ORhyQzQkxQIyW29C1cbkgM3FIVMhakh7mQf/yeQhi1EfHrSqTF7D2zYAX4WDuyoilZxM/N1Nq/HON7ySR4X5CAlGP3GtwtQKl0VrdqpBca8NFzii8XURZ+xfZeRy0= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=baylibre.com; spf=pass smtp.mailfrom=baylibre.com; dkim=pass (2048-bit key) header.d=baylibre-com.20230601.gappssmtp.com header.i=@baylibre-com.20230601.gappssmtp.com header.b=MCZI6DmB; arc=none smtp.client-ip=209.85.128.46 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=baylibre.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=baylibre.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=baylibre-com.20230601.gappssmtp.com header.i=@baylibre-com.20230601.gappssmtp.com header.b="MCZI6DmB" Received: by mail-wm1-f46.google.com with SMTP id 5b1f17b1804b1-4315df7b43fso46628025e9.0 for ; Tue, 12 Nov 2024 00:35:58 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre-com.20230601.gappssmtp.com; s=20230601; t=1731400557; x=1732005357; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=vzBUP4IjhpgilYzc3QP5HxQ10f2Yq/2yZCvHZ4g8xvA=; b=MCZI6DmBCZWPXvq7fTh79CivdABB55wD4dEx9UxKtsQ+wxq9OGHt2y27+DDsQK6Qrh LcigIQ/zHZdHNp9f5tj0jx8xQyd1iO1YIhEAp3356ujYtYQxcyUo4bXsycDscxaU7zkK JN1T4Wyd0GGlcFjY0jitbCRNom5QX6LM8GjpRheg5O3i8AMRp/pfx78CeFMfBijWx/Z+ FPrTYrI9SAl3MTQswO6KzxHyNVTkJZ9OrhrygmcLWl65uMrxtaWVkFJJ+NRt9JNEmLXb A8k08PUCC4d1cCtgUaDhZeuClKgBuSE7/Jz5IvkKL3ZzL863AAsL2PmewMPcz1kCythp bx/w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1731400557; x=1732005357; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=vzBUP4IjhpgilYzc3QP5HxQ10f2Yq/2yZCvHZ4g8xvA=; b=QT+P3+nabk9z86E2Q2ptyOeJ38aM72l9rcY23CZgQHt4LuxP8GQ7WjYuacPGtNGuFe KOCt4hIqQxHFPA95zwUXEC54MfzM/7ZHu8M4d0ojfCUUFdWAKNXFo8qp2Ud2yMnPD4B+ UDRd9UMVBtZYYbS28izTIhlXstnFgE9NpM1qoTA/35WgBMiRTEJQ1SkX8KebYIk597as ufEUB19Bk8JlGBJg2G5igHExXXxpiQN7KmQ4oueW2rn2YQTlt0uIyOKo4m+LOjakampL pYFcA6PIKLLvOQXFEdDeRk/eG8IqCvCsMDIVr38anbKitme7OWeUkwNb88Mk15e2rwud jv/g== X-Forwarded-Encrypted: i=1; AJvYcCWRTF1oaioD11gwusc2VQmoTXATkoIp9Ex8xSApSr2cHm5hzniTTcuphqBUFVlpd9J8lkAhAEonQcmZi34=@vger.kernel.org X-Gm-Message-State: AOJu0Yw7lLIHG6tjmkTUtrUrHLSl9JMHVw3/njl+482/ZWSXl/K4iwid Su6TjWSRayXDjOeoVviRsAKsELGmpru+YFE2NyEpBOwuIsJiurfjj1xh/UhFrwY= X-Google-Smtp-Source: AGHT+IHJ9ujCbbhIsj7TrIvtnmafDAChYmP23T6s6gb+JxDcqoE9J2hSsnZv1j2TrI4f4DKJmqsWhA== X-Received: by 2002:a05:600c:19c8:b0:42f:7ed4:4c26 with SMTP id 5b1f17b1804b1-432b75002b7mr138902825e9.12.1731400557221; Tue, 12 Nov 2024 00:35:57 -0800 (PST) Received: from localhost (p509159f1.dip0.t-ipconnect.de. [80.145.89.241]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-432aa709ec7sm242209895e9.35.2024.11.12.00.35.56 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 12 Nov 2024 00:35:56 -0800 (PST) From: =?UTF-8?q?Uwe=20Kleine-K=C3=B6nig?= To: Liam Girdwood , Mark Brown Cc: Marek Vasut , Maxime Coquelin , Alexandre Torgue , Kunihiko Hayashi , Masami Hiramatsu , patches@opensource.cirrus.com, linux-kernel@vger.kernel.org, linux-renesas-soc@vger.kernel.org, linux-stm32@st-md-mailman.stormreply.com, linux-arm-kernel@lists.infradead.org Subject: [PATCH] regulator: Switch back to struct platform_driver::remove() Date: Tue, 12 Nov 2024 09:35:21 +0100 Message-ID: X-Mailer: git-send-email 2.45.2 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" X-Developer-Signature: v=1; a=openpgp-sha256; l=6809; i=u.kleine-koenig@baylibre.com; h=from:subject:message-id; bh=Ogd04ibt1boR9D5geKyDS0rBfBdR3YmD0RQ58WcqbMs=; b=owEBbQGS/pANAwAKAY+A+1h9Ev5OAcsmYgBnMxNbdHLjDOnU3k+1ISOuzbFCoPl7yMqU5CS7W aALrvuQhoSJATMEAAEKAB0WIQQ/gaxpOnoeWYmt/tOPgPtYfRL+TgUCZzMTWwAKCRCPgPtYfRL+ TsWYCACaD+qvsw1AJCAzT6ixsV4Mxdp3Z0Vy/Ye08kXC/NvejBKyAgZBh98uiHT1XNztZrRFSX6 1B3710YaALWB7SDDlHxXhY6E0HIVGrMAu3PkdmaxX0N6bUeGyn9qQVp9w6YD2oruut0akzt+V0W gssa5Pa9C3jwsgkHkKR82GkbeTVxK9Zk213tc9cvE5VQYu+tcxArPAHFZVgo2VZe8yLgPa/klLz /mbHEuK6R4IWJdHy+vMBAFPrOtPGDrR0UjkCiVqKQiHH/5CpHP6IiGunLTrHaXNtQfKMnQGm+CJ rMJj77T1fadlyr5CUG1HN7ZIaOnSPd0ErA1wUiZT18D1wohr X-Developer-Key: i=u.kleine-koenig@baylibre.com; a=openpgp; fpr=0D2511F322BFAB1C1580266BE2DCDD9132669BD6 Content-Transfer-Encoding: quoted-printable 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/regulator 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. A few whitespace changes are done en passant to make indention consistent. Signed-off-by: Uwe Kleine-K=C3=B6nig --- Hello, I did a single patch for all of drivers/regulator. While I usually prefer to do one logical change per patch, this seems to be overengineering here as the individual changes are really trivial and shouldn't be much in the way for stable backports. But I'll happily split the patch if you prefer it split. Also if you object the indentation stuff, I can rework that. This is based on yesterday's next, if conflicts arise when you apply it at some later time and don't want to resolve them, feel free to just drop the changes to the conflicting files. I'll notice and followup at a later time then. Or ask me for a fixed resend. (Having said that, I recommend b4 am -3 + git am -3 which should resolve most conflicts just fine.) Best regards Uwe drivers/regulator/arizona-ldo1.c | 12 ++++++------ drivers/regulator/bd9571mwv-regulator.c | 2 +- drivers/regulator/db8500-prcmu.c | 2 +- drivers/regulator/stm32-vrefbuf.c | 2 +- drivers/regulator/uniphier-regulator.c | 2 +- drivers/regulator/userspace-consumer.c | 2 +- drivers/regulator/virtual.c | 2 +- drivers/regulator/wm8350-regulator.c | 6 +++--- 8 files changed, 15 insertions(+), 15 deletions(-) diff --git a/drivers/regulator/arizona-ldo1.c b/drivers/regulator/arizona-l= do1.c index 4b54068d4f59..501843996faa 100644 --- a/drivers/regulator/arizona-ldo1.c +++ b/drivers/regulator/arizona-ldo1.c @@ -375,18 +375,18 @@ static int madera_ldo1_probe(struct platform_device *= pdev) =20 static struct platform_driver arizona_ldo1_driver =3D { .probe =3D arizona_ldo1_probe, - .remove_new =3D arizona_ldo1_remove, - .driver =3D { - .name =3D "arizona-ldo1", + .remove =3D arizona_ldo1_remove, + .driver =3D { + .name =3D "arizona-ldo1", .probe_type =3D PROBE_FORCE_SYNCHRONOUS, }, }; =20 static struct platform_driver madera_ldo1_driver =3D { .probe =3D madera_ldo1_probe, - .remove_new =3D arizona_ldo1_remove, - .driver =3D { - .name =3D "madera-ldo1", + .remove =3D arizona_ldo1_remove, + .driver =3D { + .name =3D "madera-ldo1", .probe_type =3D PROBE_FORCE_SYNCHRONOUS, }, }; diff --git a/drivers/regulator/bd9571mwv-regulator.c b/drivers/regulator/bd= 9571mwv-regulator.c index c7ceba56e7dc..209beabb5c37 100644 --- a/drivers/regulator/bd9571mwv-regulator.c +++ b/drivers/regulator/bd9571mwv-regulator.c @@ -356,7 +356,7 @@ static struct platform_driver bd9571mwv_regulator_drive= r =3D { .pm =3D DEV_PM_OPS, }, .probe =3D bd9571mwv_regulator_probe, - .remove_new =3D bd9571mwv_regulator_remove, + .remove =3D bd9571mwv_regulator_remove, .id_table =3D bd9571mwv_regulator_id_table, }; module_platform_driver(bd9571mwv_regulator_driver); diff --git a/drivers/regulator/db8500-prcmu.c b/drivers/regulator/db8500-pr= cmu.c index 1e2d54da1b9a..1ec2e1348891 100644 --- a/drivers/regulator/db8500-prcmu.c +++ b/drivers/regulator/db8500-prcmu.c @@ -480,7 +480,7 @@ static struct platform_driver db8500_regulator_driver = =3D { .probe_type =3D PROBE_PREFER_ASYNCHRONOUS, }, .probe =3D db8500_regulator_probe, - .remove_new =3D db8500_regulator_remove, + .remove =3D db8500_regulator_remove, }; =20 static int __init db8500_regulator_init(void) diff --git a/drivers/regulator/stm32-vrefbuf.c b/drivers/regulator/stm32-vr= efbuf.c index 40855105dd33..a85ea94f0673 100644 --- a/drivers/regulator/stm32-vrefbuf.c +++ b/drivers/regulator/stm32-vrefbuf.c @@ -280,7 +280,7 @@ MODULE_DEVICE_TABLE(of, stm32_vrefbuf_of_match); =20 static struct platform_driver stm32_vrefbuf_driver =3D { .probe =3D stm32_vrefbuf_probe, - .remove_new =3D stm32_vrefbuf_remove, + .remove =3D stm32_vrefbuf_remove, .driver =3D { .name =3D "stm32-vrefbuf", .probe_type =3D PROBE_PREFER_ASYNCHRONOUS, diff --git a/drivers/regulator/uniphier-regulator.c b/drivers/regulator/uni= phier-regulator.c index 5f868042392f..74939b7fcd81 100644 --- a/drivers/regulator/uniphier-regulator.c +++ b/drivers/regulator/uniphier-regulator.c @@ -207,7 +207,7 @@ MODULE_DEVICE_TABLE(of, uniphier_regulator_match); =20 static struct platform_driver uniphier_regulator_driver =3D { .probe =3D uniphier_regulator_probe, - .remove_new =3D uniphier_regulator_remove, + .remove =3D uniphier_regulator_remove, .driver =3D { .name =3D "uniphier-regulator", .probe_type =3D PROBE_PREFER_ASYNCHRONOUS, diff --git a/drivers/regulator/userspace-consumer.c b/drivers/regulator/use= rspace-consumer.c index 6153d0295b6d..72bb5ffb49a8 100644 --- a/drivers/regulator/userspace-consumer.c +++ b/drivers/regulator/userspace-consumer.c @@ -210,7 +210,7 @@ MODULE_DEVICE_TABLE(of, regulator_userspace_consumer_of= _match); =20 static struct platform_driver regulator_userspace_consumer_driver =3D { .probe =3D regulator_userspace_consumer_probe, - .remove_new =3D regulator_userspace_consumer_remove, + .remove =3D regulator_userspace_consumer_remove, .driver =3D { .name =3D "reg-userspace-consumer", .probe_type =3D PROBE_PREFER_ASYNCHRONOUS, diff --git a/drivers/regulator/virtual.c b/drivers/regulator/virtual.c index 0a0ee186c6af..218a0d66a152 100644 --- a/drivers/regulator/virtual.c +++ b/drivers/regulator/virtual.c @@ -357,7 +357,7 @@ static void regulator_virtual_remove(struct platform_de= vice *pdev) =20 static struct platform_driver regulator_virtual_consumer_driver =3D { .probe =3D regulator_virtual_probe, - .remove_new =3D regulator_virtual_remove, + .remove =3D regulator_virtual_remove, .driver =3D { .name =3D "reg-virt-consumer", .probe_type =3D PROBE_PREFER_ASYNCHRONOUS, diff --git a/drivers/regulator/wm8350-regulator.c b/drivers/regulator/wm835= 0-regulator.c index 9939a5d2cbec..d09864bae5ef 100644 --- a/drivers/regulator/wm8350-regulator.c +++ b/drivers/regulator/wm8350-regulator.c @@ -1304,9 +1304,9 @@ EXPORT_SYMBOL_GPL(wm8350_register_led); =20 static struct platform_driver wm8350_regulator_driver =3D { .probe =3D wm8350_regulator_probe, - .remove_new =3D wm8350_regulator_remove, - .driver =3D { - .name =3D "wm8350-regulator", + .remove =3D wm8350_regulator_remove, + .driver =3D { + .name =3D "wm8350-regulator", .probe_type =3D PROBE_PREFER_ASYNCHRONOUS, }, }; base-commit: 6d59cab07b8d74d0f0422b750038123334f6ecc2 --=20 2.45.2