2024-04-22 12:55:03

by Kory Maincent

[permalink] [raw]
Subject: [PATCH net-next 0/3] net: pse-pd: Fixes for few small issues

From: Kory Maincent (Dent Project) <[email protected]>

This patch series fix few issues in PSE net subsystem like Regulator
dependency, PSE regulator type and kernel Documentation.

Signed-off-by: Kory Maincent (Dent Project) <[email protected]>
---
Kory Maincent (Dent Project) (3):
net: pse-pd: pse_core: Add missing kdoc return description
net: pse-pd: pse_core: Fix pse regulator type
net: pse-pd: Kconfig: Add missing Regulator API dependency

drivers/net/pse-pd/Kconfig | 2 +-
drivers/net/pse-pd/pse_core.c | 10 +++++++++-
2 files changed, 10 insertions(+), 2 deletions(-)
---
base-commit: 6a57f091622a1251c2826f7380577049199b80ea
change-id: 20240422-fix_poe-d0fb292c9d8e

Best regards,
--
K?ry Maincent, Bootlin
Embedded Linux and kernel engineering
https://bootlin.com



2024-04-22 12:55:15

by Kory Maincent

[permalink] [raw]
Subject: [PATCH net-next 1/3] net: pse-pd: pse_core: Add missing kdoc return description

From: Kory Maincent (Dent Project) <[email protected]>

Add missing kernel documentation return description.
This allows to remove all warning from kernel-doc test script.

Signed-off-by: Kory Maincent (Dent Project) <[email protected]>
---
drivers/net/pse-pd/pse_core.c | 8 ++++++++
1 file changed, 8 insertions(+)

diff --git a/drivers/net/pse-pd/pse_core.c b/drivers/net/pse-pd/pse_core.c
index 31f23c454678..bad29eaa4d01 100644
--- a/drivers/net/pse-pd/pse_core.c
+++ b/drivers/net/pse-pd/pse_core.c
@@ -320,6 +320,8 @@ devm_pse_pi_regulator_register(struct pse_controller_dev *pcdev,
/**
* pse_controller_register - register a PSE controller device
* @pcdev: a pointer to the initialized PSE controller device
+ *
+ * Return: 0 on success and failure value on error
*/
int pse_controller_register(struct pse_controller_dev *pcdev)
{
@@ -401,6 +403,8 @@ static void devm_pse_controller_release(struct device *dev, void *res)
* Managed pse_controller_register(). For PSE controllers registered by
* this function, pse_controller_unregister() is automatically called on
* driver detach. See pse_controller_register() for more information.
+ *
+ * Return: 0 on success and failure value on error
*/
int devm_pse_controller_register(struct device *dev,
struct pse_controller_dev *pcdev)
@@ -627,6 +631,8 @@ EXPORT_SYMBOL_GPL(of_pse_control_get);
* @psec: PSE control pointer
* @extack: extack for reporting useful error messages
* @status: struct to store PSE status
+ *
+ * Return: 0 on success and failure value on error
*/
int pse_ethtool_get_status(struct pse_control *psec,
struct netlink_ext_ack *extack,
@@ -704,6 +710,8 @@ static int pse_ethtool_podl_set_config(struct pse_control *psec,
* @psec: PSE control pointer
* @extack: extack for reporting useful error messages
* @config: Configuration of the test to run
+ *
+ * Return: 0 on success and failure value on error
*/
int pse_ethtool_set_config(struct pse_control *psec,
struct netlink_ext_ack *extack,

--
2.34.1


2024-04-22 12:55:20

by Kory Maincent

[permalink] [raw]
Subject: [PATCH net-next 2/3] net: pse-pd: pse_core: Fix pse regulator type

From: Kory Maincent (Dent Project) <[email protected]>

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.

Signed-off-by: Kory Maincent (Dent Project) <[email protected]>
---
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


2024-04-22 13:03:11

by Kory Maincent

[permalink] [raw]
Subject: [PATCH net-next 3/3] net: pse-pd: Kconfig: Add missing Regulator API dependency

From: Kory Maincent (Dent Project) <[email protected]>

The PSE (Power Sourcing Equipment) API now relies on the Regulator API.
However, the Regulator dependency was missing from Kconfig. This patch
adds the necessary dependency, resolving the issue of the missing
dependency and ensuring proper functionality of the PSE API.

Reported-by: kernel test robot <[email protected]>
Closes: https://lore.kernel.org/oe-kbuild-all/[email protected]/
Closes: https://lore.kernel.org/oe-kbuild-all/[email protected]/
Fixes: d83e13761d5b ("net: pse-pd: Use regulator framework within PSE framework")
Signed-off-by: Kory Maincent (Dent Project) <[email protected]>
---
drivers/net/pse-pd/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/pse-pd/Kconfig b/drivers/net/pse-pd/Kconfig
index 80cf373a5a0e..577ea904b3d9 100644
--- a/drivers/net/pse-pd/Kconfig
+++ b/drivers/net/pse-pd/Kconfig
@@ -5,6 +5,7 @@

menuconfig PSE_CONTROLLER
bool "Ethernet Power Sourcing Equipment Support"
+ depends on REGULATOR
help
Generic Power Sourcing Equipment Controller support.

@@ -14,7 +15,6 @@ if PSE_CONTROLLER

config PSE_REGULATOR
tristate "Regulator based PSE controller"
- depends on REGULATOR || COMPILE_TEST
help
This module provides support for simple regulator based Ethernet Power
Sourcing Equipment without automatic classification support. For

--
2.34.1


2024-04-22 13:04:25

by Kory Maincent

[permalink] [raw]
Subject: Re: [PATCH net-next 0/3] net: pse-pd: Fixes for few small issues

On Mon, 22 Apr 2024 14:50:47 +0200
"Kory Maincent (Dent Project)" <[email protected]> wrote:

> From: Kory Maincent (Dent Project) <[email protected]>
>
> This patch series fix few issues in PSE net subsystem like Regulator
> dependency, PSE regulator type and kernel Documentation.
>
> Signed-off-by: Kory Maincent (Dent Project) <[email protected]>

Found out I had a git configuration that adds "Dent Project" to my sob. I don't
want that, and will send a v2 without it.
I will wait a week for review before the v2, so please do not merge this
version.
Sorry for the noise.

Regards,
--
Köry Maincent, Bootlin
Embedded Linux and kernel engineering
https://bootlin.com

2024-04-22 13:06:08

by Andrew Lunn

[permalink] [raw]
Subject: Re: [PATCH net-next 0/3] net: pse-pd: Fixes for few small issues

On Mon, Apr 22, 2024 at 03:02:34PM +0200, Kory Maincent wrote:
> On Mon, 22 Apr 2024 14:50:47 +0200
> "Kory Maincent (Dent Project)" <[email protected]> wrote:
>
> > From: Kory Maincent (Dent Project) <[email protected]>
> >
> > This patch series fix few issues in PSE net subsystem like Regulator
> > dependency, PSE regulator type and kernel Documentation.
> >
> > Signed-off-by: Kory Maincent (Dent Project) <[email protected]>
>
> Found out I had a git configuration that adds "Dent Project" to my sob. I don't
> want that, and will send a v2 without it.
> I will wait a week for review before the v2, so please do not merge this
> version.

Hi Kory

You should be able to send:

pw-bot: cr

to your own patch series and patchwork will mark the series are Change
Requested. That will stop it from being merged.

Andrew

2024-04-22 13:12:01

by Kory Maincent

[permalink] [raw]
Subject: Re: [PATCH net-next 0/3] net: pse-pd: Fixes for few small issues

On Mon, 22 Apr 2024 15:02:34 +0200
Kory Maincent <[email protected]> wrote:

> On Mon, 22 Apr 2024 14:50:47 +0200
> "Kory Maincent (Dent Project)" <[email protected]> wrote:
>
> > From: Kory Maincent (Dent Project) <[email protected]>
> >
> > This patch series fix few issues in PSE net subsystem like Regulator
> > dependency, PSE regulator type and kernel Documentation.
> >
> > Signed-off-by: Kory Maincent (Dent Project) <[email protected]>
>
> Found out I had a git configuration that adds "Dent Project" to my sob. I
> don't want that, and will send a v2 without it.
> I will wait a week for review before the v2, so please do not merge this
> version.
> Sorry for the noise.

---
pw-bot: cr

--
Köry Maincent, Bootlin
Embedded Linux and kernel engineering
https://bootlin.com

2024-04-22 13:12:58

by Kory Maincent

[permalink] [raw]
Subject: Re: [PATCH net-next 0/3] net: pse-pd: Fixes for few small issues

On Mon, 22 Apr 2024 15:05:39 +0200
Andrew Lunn <[email protected]> wrote:

> On Mon, Apr 22, 2024 at 03:02:34PM +0200, Kory Maincent wrote:
> > On Mon, 22 Apr 2024 14:50:47 +0200
> > "Kory Maincent (Dent Project)" <[email protected]> wrote:
> >
> > > From: Kory Maincent (Dent Project) <[email protected]>
> > >
> > > This patch series fix few issues in PSE net subsystem like Regulator
> > > dependency, PSE regulator type and kernel Documentation.
> > >
> > > Signed-off-by: Kory Maincent (Dent Project) <[email protected]>
> >
> > Found out I had a git configuration that adds "Dent Project" to my sob. I
> > don't want that, and will send a v2 without it.
> > I will wait a week for review before the v2, so please do not merge this
> > version.
>
> Hi Kory

Hi Andrew,

> You should be able to send:
>
> pw-bot: cr
>
> to your own patch series and patchwork will mark the series are Change
> Requested. That will stop it from being merged.

Oh thanks, I didn't know that.

Regards,
--
Köry Maincent, Bootlin
Embedded Linux and kernel engineering
https://bootlin.com

2024-04-22 13:14:02

by Andrew Lunn

[permalink] [raw]
Subject: Re: [PATCH net-next 1/3] net: pse-pd: pse_core: Add missing kdoc return description

On Mon, Apr 22, 2024 at 02:50:48PM +0200, Kory Maincent (Dent Project) wrote:
> From: Kory Maincent (Dent Project) <[email protected]>
>
> Add missing kernel documentation return description.
> This allows to remove all warning from kernel-doc test script.
>
> Signed-off-by: Kory Maincent (Dent Project) <[email protected]>

Reviewed-by: Andrew Lunn <[email protected]>

Andrew

2024-04-22 13:14:35

by Andrew Lunn

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

On Mon, Apr 22, 2024 at 02:50:49PM +0200, Kory Maincent (Dent Project) wrote:
> From: Kory Maincent (Dent Project) <[email protected]>
>
> 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.
>
> Signed-off-by: Kory Maincent (Dent Project) <[email protected]>

Reviewed-by: Andrew Lunn <[email protected]>

Andrew

2024-04-22 13:18:10

by Andrew Lunn

[permalink] [raw]
Subject: Re: [PATCH net-next 3/3] net: pse-pd: Kconfig: Add missing Regulator API dependency

On Mon, Apr 22, 2024 at 02:50:50PM +0200, Kory Maincent (Dent Project) wrote:
> From: Kory Maincent (Dent Project) <[email protected]>
>
> The PSE (Power Sourcing Equipment) API now relies on the Regulator API.
> However, the Regulator dependency was missing from Kconfig. This patch
> adds the necessary dependency, resolving the issue of the missing
> dependency and ensuring proper functionality of the PSE API.
>
> Reported-by: kernel test robot <[email protected]>
> Closes: https://lore.kernel.org/oe-kbuild-all/[email protected]/
> Closes: https://lore.kernel.org/oe-kbuild-all/[email protected]/
> Fixes: d83e13761d5b ("net: pse-pd: Use regulator framework within PSE framework")
> Signed-off-by: Kory Maincent (Dent Project) <[email protected]>

Reviewed-by: Andrew Lunn <[email protected]>

Andrew

2024-04-22 13:32:56

by Kory Maincent

[permalink] [raw]
Subject: Re: [PATCH net-next 3/3] net: pse-pd: Kconfig: Add missing Regulator API dependency

On Mon, 22 Apr 2024 15:17:27 +0200
Andrew Lunn <[email protected]> wrote:

> On Mon, Apr 22, 2024 at 02:50:50PM +0200, Kory Maincent (Dent Project) wrote:
> > From: Kory Maincent (Dent Project) <[email protected]>
> >
> > The PSE (Power Sourcing Equipment) API now relies on the Regulator API.
> > However, the Regulator dependency was missing from Kconfig. This patch
> > adds the necessary dependency, resolving the issue of the missing
> > dependency and ensuring proper functionality of the PSE API.
> >
> > Reported-by: kernel test robot <[email protected]>
> > Closes:
> > https://lore.kernel.org/oe-kbuild-all/[email protected]/
> > Closes:
> > https://lore.kernel.org/oe-kbuild-all/[email protected]/
> > Fixes: d83e13761d5b ("net: pse-pd: Use regulator framework within PSE
> > framework") Signed-off-by: Kory Maincent (Dent Project)
> > <[email protected]>
>
> Reviewed-by: Andrew Lunn <[email protected]>

Oh you already put your reviewed-by on the whole series!
So efficient!! Thank you!
I can send the v2 then.

Regards,
--
Köry Maincent, Bootlin
Embedded Linux and kernel engineering
https://bootlin.com