sfc: use IS_ENABLED() checks for CONFIG_SFC_SRIOV

One local variable has become unused after a recent change:

drivers/net/ethernet/sfc/ef100_nic.c: In function 'ef100_probe_netdev_pf':
drivers/net/ethernet/sfc/ef100_nic.c:1155:21: error: unused variable 'net_dev' [-Werror=unused-variable]
  struct net_device *net_dev = efx->net_dev;
                     ^~~~~~~

The variable is still used in an #ifdef. Replace the #ifdef with
an if(IS_ENABLED()) check that lets the compiler see where it is
used, rather than adding another #ifdef.

This also fixes an uninitialized return value in ef100_probe_netdev_pf()
that gcc did not spot.

Fixes: 7e056e2360 ("sfc: obtain device mac address based on firmware handle for ef100")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Reviewed-by: Simon Horman <simon.horman@corigine.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
Arnd Bergmann 2023-02-17 10:56:39 +01:00 committed by David S. Miller
parent 8173c2f9a1
commit a59f832a71
1 changed files with 10 additions and 17 deletions

View File

@ -399,14 +399,14 @@ static int ef100_filter_table_up(struct efx_nic *efx)
* filter insertion will need to take the lock for read. * filter insertion will need to take the lock for read.
*/ */
up_write(&efx->filter_sem); up_write(&efx->filter_sem);
#ifdef CONFIG_SFC_SRIOV if (IS_ENABLED(CONFIG_SFC_SRIOV))
rc = efx_tc_insert_rep_filters(efx); rc = efx_tc_insert_rep_filters(efx);
/* Rep filter failure is nonfatal */ /* Rep filter failure is nonfatal */
if (rc) if (rc)
netif_warn(efx, drv, efx->net_dev, netif_warn(efx, drv, efx->net_dev,
"Failed to insert representor filters, rc %d\n", "Failed to insert representor filters, rc %d\n",
rc); rc);
#endif
return 0; return 0;
fail_vlan0: fail_vlan0:
@ -419,9 +419,8 @@ fail_unspec:
static void ef100_filter_table_down(struct efx_nic *efx) static void ef100_filter_table_down(struct efx_nic *efx)
{ {
#ifdef CONFIG_SFC_SRIOV if (IS_ENABLED(CONFIG_SFC_SRIOV))
efx_tc_remove_rep_filters(efx); efx_tc_remove_rep_filters(efx);
#endif
down_write(&efx->filter_sem); down_write(&efx->filter_sem);
efx_mcdi_filter_del_vlan(efx, 0); efx_mcdi_filter_del_vlan(efx, 0);
efx_mcdi_filter_del_vlan(efx, EFX_FILTER_VID_UNSPEC); efx_mcdi_filter_del_vlan(efx, EFX_FILTER_VID_UNSPEC);
@ -737,7 +736,6 @@ static unsigned int efx_ef100_recycle_ring_size(const struct efx_nic *efx)
return 10 * EFX_RECYCLE_RING_SIZE_10G; return 10 * EFX_RECYCLE_RING_SIZE_10G;
} }
#ifdef CONFIG_SFC_SRIOV
static int efx_ef100_get_base_mport(struct efx_nic *efx) static int efx_ef100_get_base_mport(struct efx_nic *efx)
{ {
struct ef100_nic_data *nic_data = efx->nic_data; struct ef100_nic_data *nic_data = efx->nic_data;
@ -773,7 +771,6 @@ static int efx_ef100_get_base_mport(struct efx_nic *efx)
return 0; return 0;
} }
#endif
static int compare_versions(const char *a, const char *b) static int compare_versions(const char *a, const char *b)
{ {
@ -1155,10 +1152,9 @@ int ef100_probe_netdev_pf(struct efx_nic *efx)
struct net_device *net_dev = efx->net_dev; struct net_device *net_dev = efx->net_dev;
int rc; int rc;
if (!nic_data->grp_mae) if (!IS_ENABLED(CONFIG_SFC_SRIOV) || !nic_data->grp_mae)
return 0; return 0;
#ifdef CONFIG_SFC_SRIOV
rc = efx_init_struct_tc(efx); rc = efx_init_struct_tc(efx);
if (rc) if (rc)
return rc; return rc;
@ -1193,7 +1189,6 @@ int ef100_probe_netdev_pf(struct efx_nic *efx)
net_dev->features |= NETIF_F_HW_TC; net_dev->features |= NETIF_F_HW_TC;
efx->fixed_features |= NETIF_F_HW_TC; efx->fixed_features |= NETIF_F_HW_TC;
} }
#endif
return rc; return rc;
} }
@ -1206,12 +1201,11 @@ void ef100_remove(struct efx_nic *efx)
{ {
struct ef100_nic_data *nic_data = efx->nic_data; struct ef100_nic_data *nic_data = efx->nic_data;
#ifdef CONFIG_SFC_SRIOV if (IS_ENABLED(CONFIG_SFC_SRIOV) && efx->mae) {
if (efx->mae) {
efx_ef100_fini_reps(efx); efx_ef100_fini_reps(efx);
efx_fini_mae(efx); efx_fini_mae(efx);
} }
#endif
efx_mcdi_detach(efx); efx_mcdi_detach(efx);
efx_mcdi_fini(efx); efx_mcdi_fini(efx);
if (nic_data) if (nic_data)
@ -1304,9 +1298,8 @@ const struct efx_nic_type ef100_pf_nic_type = {
.update_stats = ef100_update_stats, .update_stats = ef100_update_stats,
.pull_stats = efx_mcdi_mac_pull_stats, .pull_stats = efx_mcdi_mac_pull_stats,
.stop_stats = efx_mcdi_mac_stop_stats, .stop_stats = efx_mcdi_mac_stop_stats,
#ifdef CONFIG_SFC_SRIOV .sriov_configure = IS_ENABLED(CONFIG_SFC_SRIOV) ?
.sriov_configure = efx_ef100_sriov_configure, efx_ef100_sriov_configure : NULL,
#endif
/* Per-type bar/size configuration not used on ef100. Location of /* Per-type bar/size configuration not used on ef100. Location of
* registers is defined by extended capabilities. * registers is defined by extended capabilities.