BCC: [email protected]
CC: [email protected]
CC: [email protected]
TO: Martin Habets <[email protected]>
CC: Jakub Kicinski <[email protected]>

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 
master
head:   521a547ced6477c54b4b0cc206000406c221b4d6
commit: c5a13c319e10e795850b61bc7e3447b08024be2e sfc: Add a basic Siena module
date:   4 months ago
:::::: branch date: 23 hours ago
:::::: commit date: 4 months ago
config: csky-randconfig-m041-20220918 
(https://download.01.org/0day-ci/archive/20220920/[email protected]/config)
compiler: csky-linux-gcc (GCC) 12.1.0

If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot <[email protected]>
Reported-by: Dan Carpenter <[email protected]>

smatch warnings:
drivers/net/ethernet/sfc/siena/efx_common.c:767 efx_siena_reset_up() warn: 
missing error code 'rc'

vim +/rc +767 drivers/net/ethernet/sfc/siena/efx_common.c

6e173d3b4af9e88 Martin Habets 2022-05-09  743  
6e173d3b4af9e88 Martin Habets 2022-05-09  744  /* This function will always 
ensure that the locks acquired in
71ad88f661253f5 Martin Habets 2022-05-09  745   * efx_siena_reset_down() are 
released. A failure return code indicates
6e173d3b4af9e88 Martin Habets 2022-05-09  746   * that we were unable to 
reinitialise the hardware, and the
6e173d3b4af9e88 Martin Habets 2022-05-09  747   * driver should be disabled. If 
ok is false, then the rx and tx
6e173d3b4af9e88 Martin Habets 2022-05-09  748   * engines are not restarted, 
pending a RESET_DISABLE.
6e173d3b4af9e88 Martin Habets 2022-05-09  749   */
71ad88f661253f5 Martin Habets 2022-05-09  750  int efx_siena_reset_up(struct 
efx_nic *efx, enum reset_type method, bool ok)
6e173d3b4af9e88 Martin Habets 2022-05-09  751  {
6e173d3b4af9e88 Martin Habets 2022-05-09  752   int rc;
6e173d3b4af9e88 Martin Habets 2022-05-09  753  
6e173d3b4af9e88 Martin Habets 2022-05-09  754   
EFX_ASSERT_RESET_SERIALISED(efx);
6e173d3b4af9e88 Martin Habets 2022-05-09  755  
6e173d3b4af9e88 Martin Habets 2022-05-09  756   if (method == 
RESET_TYPE_MCDI_TIMEOUT)
6e173d3b4af9e88 Martin Habets 2022-05-09  757           
efx->type->finish_flr(efx);
6e173d3b4af9e88 Martin Habets 2022-05-09  758  
6e173d3b4af9e88 Martin Habets 2022-05-09  759   /* Ensure that SRAM is 
initialised even if we're disabling the device */
6e173d3b4af9e88 Martin Habets 2022-05-09  760   rc = efx->type->init(efx);
6e173d3b4af9e88 Martin Habets 2022-05-09  761   if (rc) {
6e173d3b4af9e88 Martin Habets 2022-05-09  762           netif_err(efx, drv, 
efx->net_dev, "failed to initialise NIC\n");
6e173d3b4af9e88 Martin Habets 2022-05-09  763           goto fail;
6e173d3b4af9e88 Martin Habets 2022-05-09  764   }
6e173d3b4af9e88 Martin Habets 2022-05-09  765  
6e173d3b4af9e88 Martin Habets 2022-05-09  766   if (!ok)
6e173d3b4af9e88 Martin Habets 2022-05-09 @767           goto fail;
6e173d3b4af9e88 Martin Habets 2022-05-09  768  
6e173d3b4af9e88 Martin Habets 2022-05-09  769   if (efx->port_initialized && 
method != RESET_TYPE_INVISIBLE &&
6e173d3b4af9e88 Martin Habets 2022-05-09  770       method != 
RESET_TYPE_DATAPATH) {
4d49e5cd4b095cd Martin Habets 2022-05-09  771           rc = 
efx_siena_mcdi_port_reconfigure(efx);
6e173d3b4af9e88 Martin Habets 2022-05-09  772           if (rc && rc != -EPERM)
6e173d3b4af9e88 Martin Habets 2022-05-09  773                   netif_err(efx, 
drv, efx->net_dev,
6e173d3b4af9e88 Martin Habets 2022-05-09  774                             
"could not restore PHY settings\n");
6e173d3b4af9e88 Martin Habets 2022-05-09  775   }
6e173d3b4af9e88 Martin Habets 2022-05-09  776  
71ad88f661253f5 Martin Habets 2022-05-09  777   rc = 
efx_siena_enable_interrupts(efx);
6e173d3b4af9e88 Martin Habets 2022-05-09  778   if (rc)
6e173d3b4af9e88 Martin Habets 2022-05-09  779           goto fail;
6e173d3b4af9e88 Martin Habets 2022-05-09  780  

:::::: The code at line 767 was first introduced by commit
:::::: 6e173d3b4af9e8804ebdbdb7a4afd7ed8f96220b sfc: Copy shared files needed 
for Siena (part 1)

:::::: TO: Martin Habets <[email protected]>
:::::: CC: Jakub Kicinski <[email protected]>

-- 
0-DAY CI Kernel Test Service
https://01.org/lkp
_______________________________________________
kbuild mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to