Re: [PATCH 5/6] power: reset: sc27xx: Add platform_device_id table

[Sebastian Reichel]

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@xxxxxxxxx>
> 
> [1]: https://lore.kernel.org/all/20251002025344.GA2958334-robh@xxxxxxxxxx/
> ---

Acked-by: Sebastian Reichel <sebastian.reichel@xxxxxxxxxxxxx>

-- 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: signature.asc
Description: PGP signature