iavf: remove redundant ret variable
[ Upstream commitc3fec56e12] Return value directly instead of taking this in another redundant variable. Reported-by: Zeal Robot <zealci@zte.com.cn> Signed-off-by: Minghao Chi <chi.minghao@zte.com.cn> Signed-off-by: CGEL ZTE <cgel.zte@gmail.com> Tested-by: Konrad Jankowski <konrad0.jankowski@intel.com> Signed-off-by: Tony Nguyen <anthony.l.nguyen@intel.com> Stable-dep-of:227d8d2f7f("iavf: Fix error handling in iavf_init_module()") Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
parent
fd4960ea53
commit
d389a4c698
1 changed files with 2 additions and 7 deletions
|
|
@ -1318,7 +1318,6 @@ static void iavf_fill_rss_lut(struct iavf_adapter *adapter)
|
|||
static int iavf_init_rss(struct iavf_adapter *adapter)
|
||||
{
|
||||
struct iavf_hw *hw = &adapter->hw;
|
||||
int ret;
|
||||
|
||||
if (!RSS_PF(adapter)) {
|
||||
/* Enable PCTYPES for RSS, TCP/UDP with IPv4/IPv6 */
|
||||
|
|
@ -1334,9 +1333,8 @@ static int iavf_init_rss(struct iavf_adapter *adapter)
|
|||
|
||||
iavf_fill_rss_lut(adapter);
|
||||
netdev_rss_key_fill((void *)adapter->rss_key, adapter->rss_key_size);
|
||||
ret = iavf_config_rss(adapter);
|
||||
|
||||
return ret;
|
||||
return iavf_config_rss(adapter);
|
||||
}
|
||||
|
||||
/**
|
||||
|
|
@ -4028,8 +4026,6 @@ static struct pci_driver iavf_driver = {
|
|||
**/
|
||||
static int __init iavf_init_module(void)
|
||||
{
|
||||
int ret;
|
||||
|
||||
pr_info("iavf: %s\n", iavf_driver_string);
|
||||
|
||||
pr_info("%s\n", iavf_copyright);
|
||||
|
|
@ -4040,8 +4036,7 @@ static int __init iavf_init_module(void)
|
|||
pr_err("%s: Failed to create workqueue\n", iavf_driver_name);
|
||||
return -ENOMEM;
|
||||
}
|
||||
ret = pci_register_driver(&iavf_driver);
|
||||
return ret;
|
||||
return pci_register_driver(&iavf_driver);
|
||||
}
|
||||
|
||||
module_init(iavf_init_module);
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue