Received-SPF: pass (google.com: domain of linux-kernel+bounces-195183-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
Date: Thu, 30 May 2024 11:33:41 +0200
From: Kory Maincent <kory.maincent@bootlin.com>
To: Andrew Lunn <andrew@lunn.ch>
Cc: "David S. Miller" <davem@davemloft.net>, Eric Dumazet
 <edumazet@google.com>, Jakub Kicinski <kuba@kernel.org>, Paolo Abeni
 <pabeni@redhat.com>, Donald Hunter <donald.hunter@gmail.com>, Oleksij
 Rempel <o.rempel@pengutronix.de>, Thomas Petazzoni
 <thomas.petazzoni@bootlin.com>, linux-kernel@vger.kernel.org,
 netdev@vger.kernel.org, Dent Project <dentproject@linuxfoundation.org>,
 kernel@pengutronix.de
Subject: Re: [PATCH 4/8] net: pse-pd: pd692x0: Expand ethtool status message
Message-ID: <20240530113341.36865f09@kmaincent-XPS-13-7390>
In-Reply-To: <d6aab44f-5e9a-4281-8c67-4b890b726337@lunn.ch>
References: <20240529-feature_poe_power_cap-v1-0-0c4b1d5953b8@bootlin.com>
	<20240529-feature_poe_power_cap-v1-4-0c4b1d5953b8@bootlin.com>
	<d6aab44f-5e9a-4281-8c67-4b890b726337@lunn.ch>
Organization: bootlin
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

Thanks for the review!

On Thu, 30 May 2024 01:13:59 +0200
Andrew Lunn <andrew@lunn.ch> wrote:

> > +static const struct pd692x0_status_msg pd692x0_status_msg_list[] =3D {
> > +	{.id =3D 0x06, .msg =3D "Port is off: Main supply voltage is high."},
> > +	{.id =3D 0x07, .msg =3D "Port is off: Main supply voltage is low."},
> > +	{.id =3D 0x08, .msg =3D "Port is off: Disable all ports pin is
> > active."},
> > +	{.id =3D 0x0C, .msg =3D "Port is off: Non-existing port number."},
> > +	{.id =3D 0x11, .msg =3D "Port is yet undefined."},
> > +	{.id =3D 0x12, .msg =3D "Port is off: Internal hardware fault."},
> > +	{.id =3D 0x1A, .msg =3D "Port is off: User setting."},
> > +	{.id =3D 0x1B, .msg =3D "Port is off: Detection is in process."},
> > +	{.id =3D 0x1C, .msg =3D "Port is off: Non-802.3AF/AT powered device."=
},
> > +	{.id =3D 0x1E, .msg =3D "Port is off: Underload state."},
> > +	{.id =3D 0x1F, .msg =3D "Port is off: Overload state."},
> > +	{.id =3D 0x20, .msg =3D "Port is off: Power budget exceeded."},
> > +	{.id =3D 0x21, .msg =3D "Port is off: Internal hardware routing
> > error."},
> > +	{.id =3D 0x22, .msg =3D "Port is off: Configuration change."},
> > +	{.id =3D 0x24, .msg =3D "Port is off: Voltage injection into the
> > port."},
> > +	{.id =3D 0x25, .msg =3D "Port is off: Improper Capacitor Detection"},
> > +	{.id =3D 0x26, .msg =3D "Port is off: Discharged load."}, =20
>=20
> I don't know of any other driver returning strings like this. Have you
> seen any other PSE driver with anything similar?

We would like to be able to return the failure reason but there is nothing
generic in the IEEE 802.3 standard to be able to add it to the UAPI.
The TI controller has SUPPLY and FAULT EVENT Register which could report few
messages. I am not aware of other PoE controller and how they deal with it.
We could add sysfs for reporting the status messages for all the ports but I
don't think it is a better idea.

> > +	{.id =3D 0x34, .msg =3D "Port is off: Short condition."},
> > +	{.id =3D 0x35, .msg =3D "Port is off: Over temperature at the port."},
> > +	{.id =3D 0x36, .msg =3D "Port is off: Device is too hot."},
> > +	{.id =3D 0x37, .msg =3D "Unknown device port status."},
> > +	{.id =3D 0x3C, .msg =3D "Power Management-Static."},
> > +	{.id =3D 0x3D, .msg =3D "Power Management-Static\u2014OVL."}, =20
>=20
> Is there something going on with UTF here? the \u2014 ?

Some copy paste of the messages bring a non utf-8 character :/
Will fix it, thanks for spotting it.

Regards,
--=20
K=C3=B6ry Maincent, Bootlin
Embedded Linux and kernel engineering
https://bootlin.com