From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-14.0 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,INCLUDES_CR_TRAILER,INCLUDES_PATCH,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id A74B0C433FE for ; Sat, 12 Dec 2020 23:43:45 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 5EA5C22D6E for ; Sat, 12 Dec 2020 23:43:45 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 5EA5C22D6E Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Type:Cc: List-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: In-Reply-To:MIME-Version:References:Message-ID:Subject:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=CpKrz9Sbq23XOdHdEv/GA0iNg7+Nn5SXMGQ5g4XiLNk=; b=xfF9iJHs3japgevCGbhfEAcaI 4NUmH80nVHNNCjps+i6QPLUw8TJCxES8ufGpvkCkGsgbxO44CALt7nydf+eb6nUMyjyTXQOvtjGXo zqZQ+hI3MvGcux2PYZ+k1/XkMTl+msJQtGOMpFOEsSx/eivTXUKEzI+ro3yrIb9v4V7MuPDhymNRR IBDga9w8aGtVzNSaOGf2hvbNC2KV2B4Tb5t6RtcTQQ7ujRbAEAcFlsBCRRfbM/ndy5U0qb2qL0SDX DglW758eD5wnw4FEjEc9WIphdsPkcb3i0wVLMRHcon1wHga4En+lopp3VswsIiSNcNAVdVpxFNui+ zU4+92n1w==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1koEW1-0002GX-7w; Sat, 12 Dec 2020 23:41:29 +0000 Received: from mail.kernel.org ([198.145.29.99]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1koEVt-0002Ft-Ez for linux-arm-kernel@lists.infradead.org; Sat, 12 Dec 2020 23:41:22 +0000 Date: Sun, 13 Dec 2020 00:41:16 +0100 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1607816478; bh=6QWq4Fbv5EWiMVzfXArjywVZkwvzqc/ihW7BC3cGPZw=; h=From:To:Cc:Subject:References:In-Reply-To:From; b=ZKZHCT1fodBHM9AI1zj/DzR8ac82ETq/u1TSCumpJSmKZ+tIT1Z22LFhlIw7p14El iM9Z3+q3byFwqk9Kg+I1FZ1kKoi2FzstUjIKU/AHMUUI6yjyo08f8fWN53yLr3g/72 JSYGdV8CG9tDsGXf7DbmLtLUCxIDdzaICnnLF6Xp4Avg9QohmDE15ErIiakReev2K1 RYh6H3z4Gy4HLyV+6s9Jbno6qt4JetLF5BvkXjPKTBJvqYxEL0cnVSSBKpjujpiAcV r49J/s9KEEUqwoKGwUcxE61sKQ7UzcKKTvMapTV5Ut4jhMCqXB7aSy6ZpNOx/ji+xT 2J+6X5B/pbiAA== From: Sebastian Reichel To: Michael Klein Subject: Re: [PATCH v5 1/3] power: reset: new driver regulator-poweroff Message-ID: <20201212234116.cddx5yur7ox7itxv@earth.universe> References: <20201211151445.115943-1-michael@fossekall.de> <20201211151445.115943-2-michael@fossekall.de> MIME-Version: 1.0 In-Reply-To: <20201211151445.115943-2-michael@fossekall.de> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20201212_184121_916828_23A6A374 X-CRM114-Status: GOOD ( 27.76 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: devicetree@vger.kernel.org, Jernej Skrabec , linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org, Maxime Ripard , Chen-Yu Tsai , Rob Herring , linux-arm-kernel@lists.infradead.org Content-Type: multipart/mixed; boundary="===============8074970511435551146==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============8074970511435551146== Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="bqkjnicvkmtr26ev" Content-Disposition: inline --bqkjnicvkmtr26ev Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Hi, On Fri, Dec 11, 2020 at 04:14:43PM +0100, Michael Klein wrote: > This driver registers a pm_power_off function to turn off the board > by force-disabling a devicetree-defined regulator. >=20 > Signed-off-by: Michael Klein > --- Thanks, queued. -- Sebastian > drivers/power/reset/Kconfig | 7 ++ > drivers/power/reset/Makefile | 1 + > drivers/power/reset/regulator-poweroff.c | 82 ++++++++++++++++++++++++ > 3 files changed, 90 insertions(+) > create mode 100644 drivers/power/reset/regulator-poweroff.c >=20 > diff --git a/drivers/power/reset/Kconfig b/drivers/power/reset/Kconfig > index d55b3727e00e..b22c4fdb2561 100644 > --- a/drivers/power/reset/Kconfig > +++ b/drivers/power/reset/Kconfig > @@ -177,6 +177,13 @@ config POWER_RESET_QNAP > =20 > Say Y if you have a QNAP NAS. > =20 > +config POWER_RESET_REGULATOR > + bool "Regulator subsystem power-off driver" > + depends on OF && REGULATOR > + help > + This driver supports turning off your board by disabling a > + power regulator defined in the devicetree. > + > config POWER_RESET_RESTART > bool "Restart power-off driver" > help > diff --git a/drivers/power/reset/Makefile b/drivers/power/reset/Makefile > index c51eceba9ea3..9dc49d3a57ff 100644 > --- a/drivers/power/reset/Makefile > +++ b/drivers/power/reset/Makefile > @@ -19,6 +19,7 @@ obj-$(CONFIG_POWER_RESET_OCELOT_RESET) +=3D ocelot-rese= t.o > obj-$(CONFIG_POWER_RESET_PIIX4_POWEROFF) +=3D piix4-poweroff.o > obj-$(CONFIG_POWER_RESET_LTC2952) +=3D ltc2952-poweroff.o > obj-$(CONFIG_POWER_RESET_QNAP) +=3D qnap-poweroff.o > +obj-$(CONFIG_POWER_RESET_REGULATOR) +=3D regulator-poweroff.o > obj-$(CONFIG_POWER_RESET_RESTART) +=3D restart-poweroff.o > obj-$(CONFIG_POWER_RESET_ST) +=3D st-poweroff.o > obj-$(CONFIG_POWER_RESET_VERSATILE) +=3D arm-versatile-reboot.o > diff --git a/drivers/power/reset/regulator-poweroff.c b/drivers/power/res= et/regulator-poweroff.c > new file mode 100644 > index 000000000000..f697088e0ad1 > --- /dev/null > +++ b/drivers/power/reset/regulator-poweroff.c > @@ -0,0 +1,82 @@ > +// SPDX-License-Identifier: GPL-2.0-only > +/* > + * Force-disables a regulator to power down a device > + * > + * Michael Klein > + * > + * Copyright (C) 2020 Michael Klein > + * > + * Based on the gpio-poweroff driver. > + */ > +#include > +#include > +#include > +#include > +#include > +#include > + > +#define TIMEOUT_MS 3000 > + > +/* > + * Hold configuration here, cannot be more than one instance of the driv= er > + * since pm_power_off itself is global. > + */ > +static struct regulator *cpu_regulator; > + > +static void regulator_poweroff_do_poweroff(void) > +{ > + if (cpu_regulator && regulator_is_enabled(cpu_regulator)) > + regulator_force_disable(cpu_regulator); > + > + /* give it some time */ > + mdelay(TIMEOUT_MS); > + > + WARN_ON(1); > +} > + > +static int regulator_poweroff_probe(struct platform_device *pdev) > +{ > + /* If a pm_power_off function has already been added, leave it alone */ > + if (pm_power_off !=3D NULL) { > + dev_err(&pdev->dev, > + "%s: pm_power_off function already registered\n", > + __func__); > + return -EBUSY; > + } > + > + cpu_regulator =3D devm_regulator_get(&pdev->dev, "cpu"); > + if (IS_ERR(cpu_regulator)) > + return PTR_ERR(cpu_regulator); > + > + pm_power_off =3D ®ulator_poweroff_do_poweroff; > + return 0; > +} > + > +static int regulator_poweroff_remove(__maybe_unused struct platform_devi= ce *pdev) > +{ > + if (pm_power_off =3D=3D ®ulator_poweroff_do_poweroff) > + pm_power_off =3D NULL; > + > + return 0; > +} > + > +static const struct of_device_id of_regulator_poweroff_match[] =3D { > + { .compatible =3D "regulator-poweroff", }, > + {}, > +}; > + > +static struct platform_driver regulator_poweroff_driver =3D { > + .probe =3D regulator_poweroff_probe, > + .remove =3D regulator_poweroff_remove, > + .driver =3D { > + .name =3D "poweroff-regulator", > + .of_match_table =3D of_regulator_poweroff_match, > + }, > +}; > + > +module_platform_driver(regulator_poweroff_driver); > + > +MODULE_AUTHOR("Michael Klein "); > +MODULE_DESCRIPTION("Regulator poweroff driver"); > +MODULE_LICENSE("GPL v2"); > +MODULE_ALIAS("platform:poweroff-regulator"); > --=20 > 2.29.2 >=20 >=20 > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel --bqkjnicvkmtr26ev Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAABCgAdFiEE72YNB0Y/i3JqeVQT2O7X88g7+poFAl/VVRwACgkQ2O7X88g7 +pobnQ/8DRqZP9pJQbIdOkTSWtKX6e/WJ+umvhdCGxjT2q3vfU9X0MwDtnBQzOk5 igA02V7pduBfKRWR/V9J/4OtHvvY2iCAKRyqvnT57FEA1M9nFz21DhUpDKWEzczj mdR4DMM7g7OXdc0+FcbSCRqBEmatQM2N4xmLTTNmoQBWzMC04TdAPE0i12ZHQoIF YD7od3zfenYjg2g1C8Od2vjbIXx6tstDxRZlpaRSE1WGBHZ0g8hUjsBCWwtrJ983 Fqs8RAj5BmkvNdJjaIzlGc+8MypzL9da9t7L+2Pnr/0W5LLEQn1nkwxzRMzGPqlf Nlk0kFHYXHqGzdgK1fid3zgFzHLZL1fZA14/LRrsqCwDlEwICL9cR0Jmzp1FKiPg o7VmHvrYiG/6rt0Gxz2Gg7Ka+S7hkWe9pjrVhkc6h62F2it0s+8eNXl+p+H7DooA QDb166OvIhyoEzz7RkeQ6EKmXF7qEWFoqZfKJuEa53FkhYjKsGzXD3x5C6WIpp3O ghCB3ZOT8BehiXG4eZ7OmZEzMdT8dBihoK2k2h3K7mvf25gvjYkJfQJ+ezJ9F5Om hCp3A5s7/d9TWEU9KLoC0bol17cvR+APjCexV+fn0zCUR1T1PNLqvjnz5/eG+qr/ NU/F41U3f+XIjXmRtOudxi7MrufmVjC90WRhxzjQacrLCc5GEhE= =l5NK -----END PGP SIGNATURE----- --bqkjnicvkmtr26ev-- --===============8074970511435551146== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel --===============8074970511435551146==--