Re: [PATCH net-next] Documentation: networking: Fix missing PSE documentation issue

From: Kory Maincent
Date: Thu Sep 12 2024 - 04:59:39 EST


On Thu, 12 Sep 2024 09:09:29 +0100
Simon Horman <horms@xxxxxxxxxx> wrote:

> On Wed, Sep 11, 2024 at 04:47:11PM +0200, Kory Maincent wrote:
> > Fix a missing end of phrase in the documentation. It describes the
> > ETHTOOL_A_C33_PSE_ACTUAL_PW attribute, which was not fully explained.
> >
> > Signed-off-by: Kory Maincent <kory.maincent@xxxxxxxxxxx>
> > ---
> > Documentation/networking/ethtool-netlink.rst | 5 +++--
> > 1 file changed, 3 insertions(+), 2 deletions(-)
> >
> > diff --git a/Documentation/networking/ethtool-netlink.rst
> > b/Documentation/networking/ethtool-netlink.rst index
> > ba90457b8b2d..b1390878ba84 100644 ---
> > a/Documentation/networking/ethtool-netlink.rst +++
> > b/Documentation/networking/ethtool-netlink.rst @@ -1801,8 +1801,9 @@ the
> > PSE and the PD. This option is corresponding to ``IEEE 802.3-2022``
> > 30.9.1.1.8 aPSEPowerClassification.
> > When set, the optional ``ETHTOOL_A_C33_PSE_ACTUAL_PW`` attribute identifies
> > -This option is corresponding to ``IEEE 802.3-2022`` 30.9.1.1.23
> > aPSEActualPower. -Actual power is reported in mW.
> > +the actual power drawn by the C33 PSE. This option is corresponding to
>
> nit: While we are here, perhaps we can also update the grammar.
>
> This attribute corresponds to...

Yes, indeed. Thanks.

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