Message ID | b5a455a64f774adc18dfe2eec7a54413e0cfb2e2.1700740705.git.petrm@nvidia.com (mailing list archive) |
---|---|
State | Accepted |
Commit | 9f1f6111fd5d553ec175c4e81d7ebf6932aaeca0 |
Delegated to: | Netdev Maintainers |
Headers | show |
Series | [net-next] mlxsw: pci: Fix missing error checking | expand |
On Thu, Nov 23, 2023 at 01:01:35PM +0100, Petr Machata wrote: > From: Ido Schimmel <idosch@nvidia.com> > > I accidentally removed the error checking after issuing the reset. > Restore it. > > Fixes: f257c73e5356 ("mlxsw: pci: Add support for new reset flow") > Reported-by: Coverity Scan <scan-admin@coverity.com> > Signed-off-by: Ido Schimmel <idosch@nvidia.com> > Reviewed-by: Jiri Pirko <jiri@nvidia.com> > Signed-off-by: Petr Machata <petrm@nvidia.com> Reviewed-by: Simon Horman <horms@kernel.org>
Hello: This patch was applied to netdev/net-next.git (main) by David S. Miller <davem@davemloft.net>: On Thu, 23 Nov 2023 13:01:35 +0100 you wrote: > From: Ido Schimmel <idosch@nvidia.com> > > I accidentally removed the error checking after issuing the reset. > Restore it. > > Fixes: f257c73e5356 ("mlxsw: pci: Add support for new reset flow") > Reported-by: Coverity Scan <scan-admin@coverity.com> > Signed-off-by: Ido Schimmel <idosch@nvidia.com> > Reviewed-by: Jiri Pirko <jiri@nvidia.com> > Signed-off-by: Petr Machata <petrm@nvidia.com> > > [...] Here is the summary with links: - [net-next] mlxsw: pci: Fix missing error checking https://git.kernel.org/netdev/net-next/c/9f1f6111fd5d You are awesome, thank you!
diff --git a/drivers/net/ethernet/mellanox/mlxsw/pci.c b/drivers/net/ethernet/mellanox/mlxsw/pci.c index 0d58f13a7c7d..af99bf17eb36 100644 --- a/drivers/net/ethernet/mellanox/mlxsw/pci.c +++ b/drivers/net/ethernet/mellanox/mlxsw/pci.c @@ -1561,6 +1561,8 @@ mlxsw_pci_reset(struct mlxsw_pci *mlxsw_pci, const struct pci_device_id *id) pci_dbg(pdev, "Starting software reset flow\n"); err = mlxsw_pci_reset_sw(mlxsw_pci); } + if (err) + return err; err = mlxsw_pci_sys_ready_wait(mlxsw_pci, id, &sys_status); if (err) {