diff options
| author | Tom Rix <trix@redhat.com> | 2022-03-05 07:06:42 -0800 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2022-03-16 13:20:26 +0100 |
| commit | 8078fff82254062f03d3b764e158980a7115a55a (patch) | |
| tree | 17092a71b2ba8df9bfef6f5ba05cc3f4d2f6b8b1 /include/linux/seq_buf.h | |
| parent | cf7a578096fe6b615934c2b0e1031306a7fc224f (diff) | |
qed: return status of qed_iov_get_link
[ Upstream commit d9dc0c84ad2d4cc911ba252c973d1bf18d5eb9cf ]
Clang static analysis reports this issue
qed_sriov.c:4727:19: warning: Assigned value is
garbage or undefined
ivi->max_tx_rate = tx_rate ? tx_rate : link.speed;
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
link is only sometimes set by the call to qed_iov_get_link()
qed_iov_get_link fails without setting link or returning
status. So change the decl to return status.
Fixes: 73390ac9d82b ("qed*: support ndo_get_vf_config")
Signed-off-by: Tom Rix <trix@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Sasha Levin <sashal@kernel.org>
Diffstat (limited to 'include/linux/seq_buf.h')
0 files changed, 0 insertions, 0 deletions