Use ICE_MAC_GENERIC_3K type for C825X devices and fix E825X
name to C825X.

Signed-off-by: Grzegorz Nitka <grzegorz.ni...@intel.com>
Signed-off-by: Qiming Yang <qiming.y...@intel.com>
---
 drivers/net/ice/base/ice_common.c | 13 +++++++++----
 drivers/net/ice/base/ice_ddp.c    |  8 +++++++-
 drivers/net/ice/base/ice_ddp.h    |  1 +
 drivers/net/ice/base/ice_devids.h | 11 ++++-------
 drivers/net/ice/base/ice_type.h   |  1 +
 drivers/net/ice/ice_ethdev.c      |  3 +--
 6 files changed, 23 insertions(+), 14 deletions(-)

diff --git a/drivers/net/ice/base/ice_common.c 
b/drivers/net/ice/base/ice_common.c
index ed822afc30..e4b25321db 100644
--- a/drivers/net/ice/base/ice_common.c
+++ b/drivers/net/ice/base/ice_common.c
@@ -241,13 +241,17 @@ static enum ice_status ice_set_mac_type(struct ice_hw *hw)
        case ICE_DEV_ID_E823C_QSFP:
        case ICE_DEV_ID_E823C_SFP:
        case ICE_DEV_ID_E823C_SGMII:
+               hw->mac_type = ICE_MAC_GENERIC;
+               break;
        case ICE_DEV_ID_E824S:
+               hw->mac_type = ICE_MAC_GENERIC_3K;
+               break;
        case ICE_DEV_ID_E825C_BACKPLANE:
        case ICE_DEV_ID_E825C_QSFP:
        case ICE_DEV_ID_E825C_SFP:
-       case ICE_DEV_ID_E825C_1GBE:
-       case ICE_DEV_ID_E825X:
-               hw->mac_type = ICE_MAC_GENERIC;
+       case ICE_DEV_ID_C825X:
+       case ICE_DEV_ID_E825C_SGMII:
+               hw->mac_type = ICE_MAC_GENERIC_3K_E825;
                break;
        case ICE_DEV_ID_E830_BACKPLANE:
        case ICE_DEV_ID_E830_QSFP56:
@@ -273,7 +277,8 @@ static enum ice_status ice_set_mac_type(struct ice_hw *hw)
 bool ice_is_generic_mac(struct ice_hw *hw)
 {
        return (hw->mac_type == ICE_MAC_GENERIC ||
-               hw->mac_type == ICE_MAC_GENERIC_3K);
+               hw->mac_type == ICE_MAC_GENERIC_3K ||
+               hw->mac_type == ICE_MAC_GENERIC_3K_E825);
 }
 
 /**
diff --git a/drivers/net/ice/base/ice_ddp.c b/drivers/net/ice/base/ice_ddp.c
index e3c1f413dd..e43aab981d 100644
--- a/drivers/net/ice/base/ice_ddp.c
+++ b/drivers/net/ice/base/ice_ddp.c
@@ -444,6 +444,7 @@ static u32 ice_get_pkg_segment_id(enum ice_mac_type 
mac_type)
                break;
        case ICE_MAC_GENERIC:
        case ICE_MAC_GENERIC_3K:
+       case ICE_MAC_GENERIC_3K_E825:
        default:
                seg_id = SEGMENT_TYPE_ICE_E810;
                break;
@@ -467,6 +468,9 @@ static u32 ice_get_pkg_sign_type(enum ice_mac_type mac_type)
        case ICE_MAC_GENERIC_3K:
                sign_type = SEGMENT_SIGN_TYPE_RSA3K;
                break;
+       case ICE_MAC_GENERIC_3K_E825:
+               sign_type = SEGMENT_SIGN_TYPE_RSA3K_E825;
+               break;
        case ICE_MAC_GENERIC:
        default:
                sign_type = SEGMENT_SIGN_TYPE_RSA2K;
@@ -1632,8 +1636,10 @@ ice_get_sw_fv_list(struct ice_hw *hw, struct 
ice_prot_lkup_ext *lkups,
                        }
                }
        } while (fv);
-       if (LIST_EMPTY(fv_list))
+       if (LIST_EMPTY(fv_list)) {
+               ice_warn(hw, "Required profiles not found in currently loaded 
DDP package");
                return ICE_ERR_CFG;
+       }
        return ICE_SUCCESS;
 
 err:
diff --git a/drivers/net/ice/base/ice_ddp.h b/drivers/net/ice/base/ice_ddp.h
index 57b39c72ca..6c87f11972 100644
--- a/drivers/net/ice/base/ice_ddp.h
+++ b/drivers/net/ice/base/ice_ddp.h
@@ -99,6 +99,7 @@ struct ice_pkg_hdr {
 #define SEGMENT_SIGN_TYPE_RSA2K                0x00000001
 #define SEGMENT_SIGN_TYPE_RSA3K                0x00000002
 #define SEGMENT_SIGN_TYPE_RSA3K_SBB    0x00000003 /* Secure Boot Block */
+#define SEGMENT_SIGN_TYPE_RSA3K_E825   0x00000005
 
 /* generic segment */
 struct ice_generic_seg_hdr {
diff --git a/drivers/net/ice/base/ice_devids.h 
b/drivers/net/ice/base/ice_devids.h
index 9ea915b967..0e0a20542b 100644
--- a/drivers/net/ice/base/ice_devids.h
+++ b/drivers/net/ice/base/ice_devids.h
@@ -76,13 +76,10 @@
 /* Intel(R) Ethernet Connection E825-C for backplane */
 #define ICE_DEV_ID_E825C_BACKPLANE     0x579C
 /* Intel(R) Ethernet Connection E825-C for QSFP */
-#define ICE_DEV_ID_E825C_QSFP          0x579D
+#define ICE_DEV_ID_E825C_QSFP          0x579D
 /* Intel(R) Ethernet Connection E825-C for SFP */
-#define ICE_DEV_ID_E825C_SFP           0x579E
+#define ICE_DEV_ID_E825C_SFP           0x579E
 /* Intel(R) Ethernet Connection E825-C 1GbE */
-#define ICE_DEV_ID_E825C_1GBE          0x579F
-/* Intel(R) Ethernet Connection E825-X */
-#define ICE_DEV_ID_E825X               0x0DCD
-
-
+#define ICE_DEV_ID_E825C_SGMII         0x579F
+#define ICE_DEV_ID_C825X               0x0DCD
 #endif /* _ICE_DEVIDS_H_ */
diff --git a/drivers/net/ice/base/ice_type.h b/drivers/net/ice/base/ice_type.h
index d072b0bfe2..9651720470 100644
--- a/drivers/net/ice/base/ice_type.h
+++ b/drivers/net/ice/base/ice_type.h
@@ -225,6 +225,7 @@ enum ice_mac_type {
        ICE_MAC_E830,
        ICE_MAC_GENERIC,
        ICE_MAC_GENERIC_3K,
+       ICE_MAC_GENERIC_3K_E825,
 };
 
 /* Media Types */
diff --git a/drivers/net/ice/ice_ethdev.c b/drivers/net/ice/ice_ethdev.c
index a5bf8317a7..2a4073c4d1 100644
--- a/drivers/net/ice/ice_ethdev.c
+++ b/drivers/net/ice/ice_ethdev.c
@@ -196,8 +196,7 @@ static const struct rte_pci_id pci_id_ice_map[] = {
        { RTE_PCI_DEVICE(ICE_INTEL_VENDOR_ID, ICE_DEV_ID_E825C_BACKPLANE) },
        { RTE_PCI_DEVICE(ICE_INTEL_VENDOR_ID, ICE_DEV_ID_E825C_QSFP) },
        { RTE_PCI_DEVICE(ICE_INTEL_VENDOR_ID, ICE_DEV_ID_E825C_SFP) },
-       { RTE_PCI_DEVICE(ICE_INTEL_VENDOR_ID, ICE_DEV_ID_E825C_1GBE) },
-       { RTE_PCI_DEVICE(ICE_INTEL_VENDOR_ID, ICE_DEV_ID_E825X) },
+       { RTE_PCI_DEVICE(ICE_INTEL_VENDOR_ID, ICE_DEV_ID_C825X) },
        { .vendor_id = 0, /* sentinel */ },
 };
 
-- 
2.25.1

Reply via email to