Re: [PATCH net-next v2 2/3] net: pse-pd: pse_core: Fix pse regulator type

From: Jiri Pirko
Date: Mon Apr 22 2024 - 11:02:32 EST


Mon, Apr 22, 2024 at 03:35:47PM CEST, kory.maincent@xxxxxxxxxxx wrote:
>From: Kory Maincent (Dent Project) <kory.maincent@xxxxxxxxxxx>
>
>Clarify PSE regulator as voltage regulator, not current.
>The PSE (Power Sourcing Equipment) regulator is defined as a voltage
>regulator, maintaining fixed voltage while accommodating varying current.
>
>Reviewed-by: Andrew Lunn <andrew@xxxxxxx>
>Signed-off-by: Kory Maincent <kory.maincent@xxxxxxxxxxx>

This looks like a fix. Can you provide "Fixes" tag please and perhaps
send this to -net tree?

Thanks!

>---
> drivers/net/pse-pd/pse_core.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/drivers/net/pse-pd/pse_core.c b/drivers/net/pse-pd/pse_core.c
>index bad29eaa4d01..795ab264eaf2 100644
>--- a/drivers/net/pse-pd/pse_core.c
>+++ b/drivers/net/pse-pd/pse_core.c
>@@ -294,7 +294,7 @@ devm_pse_pi_regulator_register(struct pse_controller_dev *pcdev,
> */
> rdesc->id = id;
> rdesc->name = name;
>- rdesc->type = REGULATOR_CURRENT;
>+ rdesc->type = REGULATOR_VOLTAGE;
> rdesc->ops = &pse_pi_ops;
> rdesc->owner = pcdev->owner;
>
>
>--
>2.34.1
>
>