Re: [PATCH v1 2/7] regulator: tps65215: Update platform_device_id table

[Christophe JAILLET <christophe.jaillet@xxxxxxxxxx>]

Le 26/12/2024 à 22:54, Shree Ramamoorthy a écrit :
> Add TI TPS65215 PMIC to the existing platform_device_id struct, so the
> regulator probe() can match which PMIC chip_data information.
>
> Signed-off-by: Shree Ramamoorthy <s-ramamoorthy@xxxxxx>
> ---
>   drivers/regulator/tps65219-regulator.c | 3 ++-
>   1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/regulator/tps65219-regulator.c b/drivers/regulator/tps65219-regulator.c
> index aa65077f9d41..b8a178ae6b42 100644
> --- a/drivers/regulator/tps65219-regulator.c
> +++ b/drivers/regulator/tps65219-regulator.c
> @@ -344,7 +344,8 @@ static int tps65219_regulator_probe(struct platform_device *pdev)
>   }
>   
>   static const struct platform_device_id tps65219_regulator_id_table[] = {
> -	{ "tps65219-regulator", },
> +	{ "tps65219-regulator", TPS65219 },
> +	{ "tps65215-regulator", TPS65215 },

Maybe keep alphabetical order? TPS65215, then TPS65219.

CJ

>   	{ /* sentinel */ }
>   };
>   MODULE_DEVICE_TABLE(platform, tps65219_regulator_id_table);