From: Sebastian Reichel <sebastian.reichel@collabora.com>
To: "Otto Pflüger" <otto.pflueger@abscue.de>
Cc: Alexandre Belloni <alexandre.belloni@bootlin.com>,
Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
Orson Zhai <orsonzhai@gmail.com>,
Baolin Wang <baolin.wang@linux.alibaba.com>,
Chunyan Zhang <zhang.lyra@gmail.com>, Lee Jones <lee@kernel.org>,
Pavel Machek <pavel@kernel.org>,
Liam Girdwood <lgirdwood@gmail.com>,
Mark Brown <broonie@kernel.org>,
linux-rtc@vger.kernel.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org, linux-leds@vger.kernel.org,
linux-pm@vger.kernel.org
Subject: Re: [PATCH 5/6] power: reset: sc27xx: Add platform_device_id table
Date: Tue, 3 Mar 2026 01:00:45 +0100 [thread overview]
Message-ID: <aaYkhIGP2pexbxt7@venus> (raw)
In-Reply-To: <20260222-sc27xx-mfd-cells-v1-5-69526fe74c77@abscue.de>
[-- Attachment #1: Type: text/plain, Size: 1848 bytes --]
Hi,
On Sun, Feb 22, 2026 at 02:16:49PM +0100, Otto Pflüger wrote:
> Make the poweroff driver for SC27xx-series PMICs probe automatically.
> Since the device representing the poweroff functionality of the SC27xx
> PMIC is not supposed to have a dedicated device tree node without any
> corresponding DT resources [1], an of_device_id table cannot be used
> here. Instead, use a platform_device_id table to match the poweroff
> sub-device instantiated by the parent MFD driver.
>
> Signed-off-by: Otto Pflüger <otto.pflueger@abscue.de>
>
> [1]: https://lore.kernel.org/all/20251002025344.GA2958334-robh@kernel.org/
> ---
Acked-by: Sebastian Reichel <sebastian.reichel@collabora.com>
-- Sebastian
> drivers/power/reset/sc27xx-poweroff.c | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
> diff --git a/drivers/power/reset/sc27xx-poweroff.c b/drivers/power/reset/sc27xx-poweroff.c
> index 393bd1c33b73..6376706bf561 100644
> --- a/drivers/power/reset/sc27xx-poweroff.c
> +++ b/drivers/power/reset/sc27xx-poweroff.c
> @@ -6,6 +6,7 @@
>
> #include <linux/cpu.h>
> #include <linux/kernel.h>
> +#include <linux/mod_devicetable.h>
> #include <linux/module.h>
> #include <linux/platform_device.h>
> #include <linux/pm.h>
> @@ -70,11 +71,18 @@ static int sc27xx_poweroff_probe(struct platform_device *pdev)
> return 0;
> }
>
> +static const struct platform_device_id sc27xx_poweroff_id_table[] = {
> + { "sc2731-poweroff" },
> + { }
> +};
> +MODULE_DEVICE_TABLE(platform, sc27xx_poweroff_id_table);
> +
> static struct platform_driver sc27xx_poweroff_driver = {
> .probe = sc27xx_poweroff_probe,
> .driver = {
> .name = "sc27xx-poweroff",
> },
> + .id_table = sc27xx_poweroff_id_table,
> };
> module_platform_driver(sc27xx_poweroff_driver);
>
>
> --
> 2.51.0
>
>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
next prev parent reply other threads:[~2026-03-03 0:01 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-02-22 13:16 [PATCH 0/6] mfd: sc27xx: Use MFD cells and devm_mfd_add_devices() Otto Pflüger
2026-02-22 13:16 ` [PATCH 1/6] dt-bindings: rtc: sc2731: Add compatible for SC2730 Otto Pflüger
2026-03-05 23:51 ` Rob Herring (Arm)
2026-02-22 13:16 ` [PATCH 2/6] dt-bindings: leds: " Otto Pflüger
2026-03-05 23:51 ` Rob Herring (Arm)
2026-03-09 18:54 ` (subset) " Lee Jones
2026-02-22 13:16 ` [PATCH 3/6] regulator: dt-bindings: sc2731: Deprecate compatible property Otto Pflüger
2026-03-02 16:28 ` Mark Brown
2026-03-05 23:53 ` Rob Herring (Arm)
2026-02-22 13:16 ` [PATCH 4/6] mfd: sprd-sc27xx: Switch to devm_mfd_add_devices() Otto Pflüger
2026-03-09 18:58 ` Lee Jones
2026-03-20 19:41 ` Otto Pflüger
2026-03-25 11:22 ` Lee Jones
2026-02-22 13:16 ` [PATCH 5/6] power: reset: sc27xx: Add platform_device_id table Otto Pflüger
2026-03-03 0:00 ` Sebastian Reichel [this message]
2026-02-22 13:16 ` [PATCH 6/6] regulator: sc2731: " Otto Pflüger
2026-03-02 16:06 ` Mark Brown
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=aaYkhIGP2pexbxt7@venus \
--to=sebastian.reichel@collabora.com \
--cc=alexandre.belloni@bootlin.com \
--cc=baolin.wang@linux.alibaba.com \
--cc=broonie@kernel.org \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=krzk+dt@kernel.org \
--cc=lee@kernel.org \
--cc=lgirdwood@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-leds@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=linux-rtc@vger.kernel.org \
--cc=orsonzhai@gmail.com \
--cc=otto.pflueger@abscue.de \
--cc=pavel@kernel.org \
--cc=robh@kernel.org \
--cc=zhang.lyra@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.