[PATCH net-next 03/16] net: pcs: xpcs: Return EINVAL in the internal methods
Andrew Lunn
andrew at lunn.ch
Wed Dec 6 00:34:32 AEDT 2023
On Tue, Dec 05, 2023 at 01:35:24PM +0300, Serge Semin wrote:
> In particular the xpcs_soft_reset() and xpcs_do_config() functions
> currently return -1 if invalid auto-negotiation mode is specified. That
> value can be then passed to the generic kernel subsystems which require a
> standard kernel errno value. Even though the error conditions are very
> specific (memory corruption or buggy implementation) using a hard-coded -1
> literal doesn't seem correct anyway.
>
> Signed-off-by: Serge Semin <fancer.lancer at gmail.com>
Tested-by: Andrew Lunn <andrew at lunn.ch>
Andrew
More information about the openbmc
mailing list