Re: [PATCH net] sfc: fix error code in efx_devlink_info_running_versions()
Edward Cree <[email protected]>
| Newsgroups | org.kernel.vger.kernel-janitors,org.kernel.vger.linux-kernel,org.kernel.vger.netdev |
|---|---|
| Message-ID | <[email protected]> |
On 25/04/2026 10:09, Dan Carpenter wrote:
> Return -EINVAL if efx_mcdi_rpc() doesn't return enough space.
>
> Fixes: 14743ddd2495 ("sfc: add devlink info support for ef100")
> Signed-off-by: Dan Carpenter <[email protected]>
> ---
> drivers/net/ethernet/sfc/efx_devlink.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/net/ethernet/sfc/efx_devlink.c b/drivers/net/ethernet/sfc/efx_devlink.c
> index d842c60dfc10..45f412bb9b5d 100644
> --- a/drivers/net/ethernet/sfc/efx_devlink.c
> +++ b/drivers/net/ethernet/sfc/efx_devlink.c
> @@ -531,7 +531,7 @@ static int efx_devlink_info_running_versions(struct efx_nic *efx,
> if (rc || outlength < MC_CMD_GET_VERSION_OUT_LEN) {
> netif_err(efx, drv, efx->net_dev,
> "mcdi MC_CMD_GET_VERSION failed\n");
> - return rc;
> + return rc ?: -EINVAL;
> }
>
> /* Handle previous output */
Usual practice in sfc is to use -EIO for 'MC response was too short'; it
signifies misbehaviour from the hardware, not any invalid input argument
as -EINVAL would suggest.
Other than that, looks fine.
-ed