* [PATCH net-next v2 0/3] net: pse-pd: Fixes for few small issues
@ 2024-04-22 13:35 Kory Maincent (Dent Project)
2024-04-22 13:35 ` [PATCH net-next v2 1/3] net: pse-pd: pse_core: Add missing kdoc return description Kory Maincent (Dent Project)
` (2 more replies)
0 siblings, 3 replies; 9+ messages in thread
From: Kory Maincent (Dent Project) @ 2024-04-22 13:35 UTC (permalink / raw
To: Oleksij Rempel, David S. Miller, Eric Dumazet, Jakub Kicinski,
Paolo Abeni, Andrew Lunn
Cc: Thomas Petazzoni, netdev, linux-kernel, Kyle Swenson,
Kory Maincent, kernel test robot
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="utf-8", Size: 950 bytes --]
From: Kory Maincent (Dent Project) <kory.maincent@bootlin.com>
This patch series fix few issues in PSE net subsystem like Regulator
dependency, PSE regulator type and kernel Documentation.
Signed-off-by: Kory Maincent <kory.maincent@bootlin.com>
---
Changes in v2:
- Remove "Dent project" from my sob.
- Link to v1: https://lore.kernel.org/r/20240422-fix_poe-v1-0-811c8c0b9da7@bootlin.com
---
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
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH net-next v2 1/3] net: pse-pd: pse_core: Add missing kdoc return description
2024-04-22 13:35 [PATCH net-next v2 0/3] net: pse-pd: Fixes for few small issues Kory Maincent (Dent Project)
@ 2024-04-22 13:35 ` Kory Maincent (Dent Project)
2024-04-23 8:49 ` Oleksij Rempel
2024-04-22 13:35 ` [PATCH net-next v2 2/3] net: pse-pd: pse_core: Fix pse regulator type Kory Maincent (Dent Project)
2024-04-22 13:35 ` [PATCH net-next v2 3/3] net: pse-pd: Kconfig: Add missing Regulator API dependency Kory Maincent (Dent Project)
2 siblings, 1 reply; 9+ messages in thread
From: Kory Maincent (Dent Project) @ 2024-04-22 13:35 UTC (permalink / raw
To: Oleksij Rempel, David S. Miller, Eric Dumazet, Jakub Kicinski,
Paolo Abeni, Andrew Lunn
Cc: Thomas Petazzoni, netdev, linux-kernel, Kyle Swenson,
Kory Maincent
From: Kory Maincent (Dent Project) <kory.maincent@bootlin.com>
Add missing kernel documentation return description.
This allows to remove all warning from kernel-doc test script.
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: Kory Maincent <kory.maincent@bootlin.com>
---
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
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH net-next v2 2/3] net: pse-pd: pse_core: Fix pse regulator type
2024-04-22 13:35 [PATCH net-next v2 0/3] net: pse-pd: Fixes for few small issues Kory Maincent (Dent Project)
2024-04-22 13:35 ` [PATCH net-next v2 1/3] net: pse-pd: pse_core: Add missing kdoc return description Kory Maincent (Dent Project)
@ 2024-04-22 13:35 ` Kory Maincent (Dent Project)
2024-04-22 15:02 ` Jiri Pirko
2024-04-22 13:35 ` [PATCH net-next v2 3/3] net: pse-pd: Kconfig: Add missing Regulator API dependency Kory Maincent (Dent Project)
2 siblings, 1 reply; 9+ messages in thread
From: Kory Maincent (Dent Project) @ 2024-04-22 13:35 UTC (permalink / raw
To: Oleksij Rempel, David S. Miller, Eric Dumazet, Jakub Kicinski,
Paolo Abeni, Andrew Lunn
Cc: Thomas Petazzoni, netdev, linux-kernel, Kyle Swenson,
Kory Maincent
From: Kory Maincent (Dent Project) <kory.maincent@bootlin.com>
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@lunn.ch>
Signed-off-by: Kory Maincent <kory.maincent@bootlin.com>
---
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
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH net-next v2 3/3] net: pse-pd: Kconfig: Add missing Regulator API dependency
2024-04-22 13:35 [PATCH net-next v2 0/3] net: pse-pd: Fixes for few small issues Kory Maincent (Dent Project)
2024-04-22 13:35 ` [PATCH net-next v2 1/3] net: pse-pd: pse_core: Add missing kdoc return description Kory Maincent (Dent Project)
2024-04-22 13:35 ` [PATCH net-next v2 2/3] net: pse-pd: pse_core: Fix pse regulator type Kory Maincent (Dent Project)
@ 2024-04-22 13:35 ` Kory Maincent (Dent Project)
2024-04-23 8:50 ` Oleksij Rempel
2 siblings, 1 reply; 9+ messages in thread
From: Kory Maincent (Dent Project) @ 2024-04-22 13:35 UTC (permalink / raw
To: Oleksij Rempel, David S. Miller, Eric Dumazet, Jakub Kicinski,
Paolo Abeni, Andrew Lunn
Cc: Thomas Petazzoni, netdev, linux-kernel, Kyle Swenson,
Kory Maincent, kernel test robot
From: Kory Maincent (Dent Project) <kory.maincent@bootlin.com>
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 <lkp@intel.com>
Closes: https://lore.kernel.org/oe-kbuild-all/202404201020.mqX2IOu7-lkp@intel.com/
Closes: https://lore.kernel.org/oe-kbuild-all/202404200036.D8ap1Mf5-lkp@intel.com/
Fixes: d83e13761d5b ("net: pse-pd: Use regulator framework within PSE framework")
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: Kory Maincent <kory.maincent@bootlin.com>
---
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
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH net-next v2 2/3] net: pse-pd: pse_core: Fix pse regulator type
2024-04-22 13:35 ` [PATCH net-next v2 2/3] net: pse-pd: pse_core: Fix pse regulator type Kory Maincent (Dent Project)
@ 2024-04-22 15:02 ` Jiri Pirko
2024-04-22 16:20 ` Kory Maincent
0 siblings, 1 reply; 9+ messages in thread
From: Jiri Pirko @ 2024-04-22 15:02 UTC (permalink / raw
To: Kory Maincent (Dent Project)
Cc: Oleksij Rempel, David S. Miller, Eric Dumazet, Jakub Kicinski,
Paolo Abeni, Andrew Lunn, Thomas Petazzoni, netdev, linux-kernel,
Kyle Swenson
Mon, Apr 22, 2024 at 03:35:47PM CEST, kory.maincent@bootlin.com wrote:
>From: Kory Maincent (Dent Project) <kory.maincent@bootlin.com>
>
>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@lunn.ch>
>Signed-off-by: Kory Maincent <kory.maincent@bootlin.com>
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
>
>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH net-next v2 2/3] net: pse-pd: pse_core: Fix pse regulator type
2024-04-22 15:02 ` Jiri Pirko
@ 2024-04-22 16:20 ` Kory Maincent
2024-04-23 8:48 ` Oleksij Rempel
0 siblings, 1 reply; 9+ messages in thread
From: Kory Maincent @ 2024-04-22 16:20 UTC (permalink / raw
To: Jiri Pirko
Cc: Oleksij Rempel, David S. Miller, Eric Dumazet, Jakub Kicinski,
Paolo Abeni, Andrew Lunn, Thomas Petazzoni, netdev, linux-kernel,
Kyle Swenson
On Mon, 22 Apr 2024 17:02:18 +0200
Jiri Pirko <jiri@resnulli.us> wrote:
> Mon, Apr 22, 2024 at 03:35:47PM CEST, kory.maincent@bootlin.com wrote:
> >From: Kory Maincent (Dent Project) <kory.maincent@bootlin.com>
> >
> >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@lunn.ch>
> >Signed-off-by: Kory Maincent <kory.maincent@bootlin.com>
>
> This looks like a fix. Can you provide "Fixes" tag please and perhaps
> send this to -net tree?
Indeed I should have used the Fixes tag.
The PoE patch series that introduce this issue is currently in net-next.
Regards,
--
Köry Maincent, Bootlin
Embedded Linux and kernel engineering
https://bootlin.com
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH net-next v2 2/3] net: pse-pd: pse_core: Fix pse regulator type
2024-04-22 16:20 ` Kory Maincent
@ 2024-04-23 8:48 ` Oleksij Rempel
0 siblings, 0 replies; 9+ messages in thread
From: Oleksij Rempel @ 2024-04-23 8:48 UTC (permalink / raw
To: Kory Maincent
Cc: Jiri Pirko, David S. Miller, Eric Dumazet, Jakub Kicinski,
Paolo Abeni, Andrew Lunn, Thomas Petazzoni, netdev, linux-kernel,
Kyle Swenson
On Mon, Apr 22, 2024 at 06:20:30PM +0200, Kory Maincent wrote:
> On Mon, 22 Apr 2024 17:02:18 +0200
> Jiri Pirko <jiri@resnulli.us> wrote:
>
> > Mon, Apr 22, 2024 at 03:35:47PM CEST, kory.maincent@bootlin.com wrote:
> > >From: Kory Maincent (Dent Project) <kory.maincent@bootlin.com>
> > >
> > >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@lunn.ch>
> > >Signed-off-by: Kory Maincent <kory.maincent@bootlin.com>
> >
> > This looks like a fix. Can you provide "Fixes" tag please and perhaps
> > send this to -net tree?
>
> Indeed I should have used the Fixes tag.
> The PoE patch series that introduce this issue is currently in net-next.
With Fixes tag:
Acked-by: Oleksij Rempel <o.rempel@pengutronix.de>
--
Pengutronix e.K. | |
Steuerwalder Str. 21 | http://www.pengutronix.de/ |
31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH net-next v2 1/3] net: pse-pd: pse_core: Add missing kdoc return description
2024-04-22 13:35 ` [PATCH net-next v2 1/3] net: pse-pd: pse_core: Add missing kdoc return description Kory Maincent (Dent Project)
@ 2024-04-23 8:49 ` Oleksij Rempel
0 siblings, 0 replies; 9+ messages in thread
From: Oleksij Rempel @ 2024-04-23 8:49 UTC (permalink / raw
To: Kory Maincent (Dent Project)
Cc: David S. Miller, Eric Dumazet, Jakub Kicinski, Paolo Abeni,
Andrew Lunn, Thomas Petazzoni, netdev, linux-kernel, Kyle Swenson
On Mon, Apr 22, 2024 at 03:35:46PM +0200, Kory Maincent (Dent Project) wrote:
> From: Kory Maincent (Dent Project) <kory.maincent@bootlin.com>
>
> Add missing kernel documentation return description.
> This allows to remove all warning from kernel-doc test script.
>
> Reviewed-by: Andrew Lunn <andrew@lunn.ch>
> Signed-off-by: Kory Maincent <kory.maincent@bootlin.com>
Acked-by: Oleksij Rempel <o.rempel@pengutronix.de>
--
Pengutronix e.K. | |
Steuerwalder Str. 21 | http://www.pengutronix.de/ |
31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH net-next v2 3/3] net: pse-pd: Kconfig: Add missing Regulator API dependency
2024-04-22 13:35 ` [PATCH net-next v2 3/3] net: pse-pd: Kconfig: Add missing Regulator API dependency Kory Maincent (Dent Project)
@ 2024-04-23 8:50 ` Oleksij Rempel
0 siblings, 0 replies; 9+ messages in thread
From: Oleksij Rempel @ 2024-04-23 8:50 UTC (permalink / raw
To: Kory Maincent (Dent Project)
Cc: David S. Miller, Eric Dumazet, Jakub Kicinski, Paolo Abeni,
Andrew Lunn, Thomas Petazzoni, netdev, linux-kernel, Kyle Swenson,
kernel test robot
On Mon, Apr 22, 2024 at 03:35:48PM +0200, Kory Maincent (Dent Project) wrote:
> From: Kory Maincent (Dent Project) <kory.maincent@bootlin.com>
>
> 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 <lkp@intel.com>
> Closes: https://lore.kernel.org/oe-kbuild-all/202404201020.mqX2IOu7-lkp@intel.com/
> Closes: https://lore.kernel.org/oe-kbuild-all/202404200036.D8ap1Mf5-lkp@intel.com/
> Fixes: d83e13761d5b ("net: pse-pd: Use regulator framework within PSE framework")
> Reviewed-by: Andrew Lunn <andrew@lunn.ch>
> Signed-off-by: Kory Maincent <kory.maincent@bootlin.com>
Acked-by: Oleksij Rempel <o.rempel@pengutronix.de>
--
Pengutronix e.K. | |
Steuerwalder Str. 21 | http://www.pengutronix.de/ |
31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2024-04-23 8:50 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-04-22 13:35 [PATCH net-next v2 0/3] net: pse-pd: Fixes for few small issues Kory Maincent (Dent Project)
2024-04-22 13:35 ` [PATCH net-next v2 1/3] net: pse-pd: pse_core: Add missing kdoc return description Kory Maincent (Dent Project)
2024-04-23 8:49 ` Oleksij Rempel
2024-04-22 13:35 ` [PATCH net-next v2 2/3] net: pse-pd: pse_core: Fix pse regulator type Kory Maincent (Dent Project)
2024-04-22 15:02 ` Jiri Pirko
2024-04-22 16:20 ` Kory Maincent
2024-04-23 8:48 ` Oleksij Rempel
2024-04-22 13:35 ` [PATCH net-next v2 3/3] net: pse-pd: Kconfig: Add missing Regulator API dependency Kory Maincent (Dent Project)
2024-04-23 8:50 ` Oleksij Rempel
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.