net/sfc/base: use simpler EF10 family run-time checks
authorAndrew Rybchenko <arybchenko@solarflare.com>
Tue, 28 Apr 2020 13:37:25 +0000 (14:37 +0100)
committerFerruh Yigit <ferruh.yigit@intel.com>
Tue, 5 May 2020 13:54:26 +0000 (15:54 +0200)
Fixes: 4625c4f5277d ("net/sfc/base: factor out upstream port vAdaptor allocation")
Fixes: 4f12e20c85dc ("net/sfc/base: introduce EVB module for SR-IOV")
Fixes: 18c8e84d7726 ("net/sfc/base: support proxy auth operations for SR-IOV")
Cc: stable@dpdk.org
Signed-off-by: Andrew Rybchenko <arybchenko@solarflare.com>
drivers/net/sfc/base/ef10_evb.c
drivers/net/sfc/base/ef10_nic.c
drivers/net/sfc/base/ef10_proxy.c

index f290339..d541db4 100644 (file)
@@ -15,9 +15,7 @@
 ef10_evb_init(
        __in            efx_nic_t *enp)
 {
-       EFSYS_ASSERT(enp->en_family == EFX_FAMILY_HUNTINGTON ||
-               enp->en_family == EFX_FAMILY_MEDFORD ||
-               enp->en_family == EFX_FAMILY_MEDFORD2);
+       EFSYS_ASSERT(EFX_FAMILY_IS_EF10(enp));
 
        return (0);
 }
@@ -26,9 +24,7 @@ ef10_evb_init(
 ef10_evb_fini(
        __in            efx_nic_t *enp)
 {
-       EFSYS_ASSERT(enp->en_family == EFX_FAMILY_HUNTINGTON ||
-               enp->en_family == EFX_FAMILY_MEDFORD ||
-               enp->en_family == EFX_FAMILY_MEDFORD2);
+       EFSYS_ASSERT(EFX_FAMILY_IS_EF10(enp));
 }
 
        __checkReturn   efx_rc_t
index d1802da..34fa45e 100644 (file)
@@ -2288,9 +2288,7 @@ ef10_nic_init(
        efx_rc_t rc;
        boolean_t alloc_vadaptor = B_TRUE;
 
-       EFSYS_ASSERT(enp->en_family == EFX_FAMILY_HUNTINGTON ||
-           enp->en_family == EFX_FAMILY_MEDFORD ||
-           enp->en_family == EFX_FAMILY_MEDFORD2);
+       EFSYS_ASSERT(EFX_FAMILY_IS_EF10(enp));
 
        /* Enable reporting of some events (e.g. link change) */
        if ((rc = efx_mcdi_log_ctrl(enp)) != 0)
index 9be9e22..19c11c6 100644 (file)
@@ -13,9 +13,7 @@
 ef10_proxy_auth_init(
        __in            efx_nic_t *enp)
 {
-       EFSYS_ASSERT(enp->en_family == EFX_FAMILY_HUNTINGTON ||
-               enp->en_family == EFX_FAMILY_MEDFORD ||
-               enp->en_family == EFX_FAMILY_MEDFORD2);
+       EFSYS_ASSERT(EFX_FAMILY_IS_EF10(enp));
 
        return (0);
 }
@@ -24,9 +22,7 @@ ef10_proxy_auth_init(
 ef10_proxy_auth_fini(
        __in            efx_nic_t *enp)
 {
-       EFSYS_ASSERT(enp->en_family == EFX_FAMILY_HUNTINGTON ||
-               enp->en_family == EFX_FAMILY_MEDFORD ||
-               enp->en_family == EFX_FAMILY_MEDFORD2);
+       EFSYS_ASSERT(EFX_FAMILY_IS_EF10(enp));
 }
 
 static __checkReturn   efx_rc_t