iavf: remove redundant ret variable
[ Upstream commit c3fec56e12678c3ad68084048a73818a7968d6b8 ] 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: 227d8d2f7f22 ("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
@ -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…
x
Reference in New Issue
Block a user