From: Beilei Xing <beilei.x...@intel.com> Update the cpchnl2 function type according to the definition in MEV 1.0 release.
Signed-off-by: Beilei Xing <beilei.x...@intel.com> --- drivers/net/cpfl/cpfl_cpchnl.h | 19 ++++++++++--------- drivers/net/cpfl/cpfl_ethdev.h | 2 +- drivers/net/cpfl/cpfl_representor.c | 20 ++++++++++---------- 3 files changed, 21 insertions(+), 20 deletions(-) diff --git a/drivers/net/cpfl/cpfl_cpchnl.h b/drivers/net/cpfl/cpfl_cpchnl.h index 2eefcbcc10..667fadcac4 100644 --- a/drivers/net/cpfl/cpfl_cpchnl.h +++ b/drivers/net/cpfl/cpfl_cpchnl.h @@ -22,9 +22,6 @@ enum cpchnl2_ops { #define CPCHNL2_ETH_LENGTH_OF_ADDRESS 6 -#define CPCHNL2_FUNC_TYPE_PF 0 -#define CPCHNL2_FUNC_TYPE_SRIOV 1 - /* vport statuses - must match the DB ones - see enum cp_vport_status*/ #define CPCHNL2_VPORT_STATUS_CREATED 0 #define CPCHNL2_VPORT_STATUS_ENABLED 1 @@ -136,8 +133,10 @@ CPCHNL2_CHECK_STRUCT_LEN(3792, cpchnl2_queue_groups); * @brief function types */ enum cpchnl2_func_type { - CPCHNL2_FTYPE_LAN_PF = 0, - CPCHNL2_FTYPE_LAN_VF = 1, + CPCHNL2_FTYPE_LAN_VF = 0x0, + CPCHNL2_FTYPE_LAN_VM = 0x1, + CPCHNL2_FTYPE_LAN_PF = 0x2, + CPCHNL2_FTYPE_LAN_IMC_BMC = 0x3, CPCHNL2_FTYPE_LAN_MAX }; @@ -176,7 +175,7 @@ struct cpchnl2_vport_info { */ u16 vsi_id; u8 vport_status; /* enum cpchnl2_vport_status */ - /* 0 - LAN PF, 1 - LAN VF. Rest - reserved. Can be later expanded to other PEs */ + /* 0 - LAN VF, 2 - LAN PF. Rest - reserved. Can be later expanded to other PEs */ u8 func_type; /* Valid only if "type" above is VF, indexing is relative to PF specified above. */ u16 vf_id; @@ -216,7 +215,9 @@ struct cpchnl2_vport_info { u16 default_rx_qid; /* Default LAN RX Queue ID */ u16 vport_flags; /* see: VPORT_FLAGS */ u8 egress_port; - u8 pad_reserved[5]; + /* Host LAN APF: 0; ACC LAN APF: 4; IMC LAN APF: 5; ACC LAN CPF: 4; IMC LAN CPF: 5 */ + u8 host_id; + u8 pad_reserved[4]; }; CPCHNL2_CHECK_STRUCT_LEN(96, cpchnl2_vport_info); @@ -226,7 +227,7 @@ CPCHNL2_CHECK_STRUCT_LEN(96, cpchnl2_vport_info); /** * @brief Used for CPCHNL2_OP_GET_VPORT_LIST opcode request - * @param func_type Func type: 0 - LAN_PF, 1 - LAN_VF. Rest - reserved (see enum cpchnl2_func_type) + * @param func_type Func type: 0 - LAN_VF, 2 - LAN_PF. Rest - reserved (see enum cpchnl2_func_type) * @param pf_id Always relevant, indexing is according to LAN PE 0-15, while only 0-4 APFs and 8-12 * CPFs are valid * @param vf_id Valid only if "type" above is VF, indexing is relative to PF specified above @@ -241,7 +242,7 @@ CPCHNL2_CHECK_STRUCT_LEN(8, cpchnl2_get_vport_list_request); /** * @brief Used for CPCHNL2_OP_GET_VPORT_LIST opcode response - * @param func_type Func type: 0 - LAN_PF, 1 - LAN_VF. Rest - reserved. Can be later extended to + * @param func_type Func type: 0 - LAN_VF, 2 - LAN_PF. Rest - reserved. Can be later extended to * other PE types * @param pf_id Always relevant, indexing is according to LAN PE 0-15, while only 0-4 APFs and 8-12 * CPFs are valid diff --git a/drivers/net/cpfl/cpfl_ethdev.h b/drivers/net/cpfl/cpfl_ethdev.h index efb0eb5251..bb53fca7c0 100644 --- a/drivers/net/cpfl/cpfl_ethdev.h +++ b/drivers/net/cpfl/cpfl_ethdev.h @@ -296,7 +296,7 @@ cpfl_get_vsi_id(struct cpfl_itf *itf) } else if (itf->type == CPFL_ITF_TYPE_VPORT) { vport_id = ((struct cpfl_vport *)itf)->base.vport_id; - vport_identity.func_type = CPCHNL2_FUNC_TYPE_PF; + vport_identity.func_type = CPCHNL2_FTYPE_LAN_PF; /* host: CPFL_HOST0_CPF_ID, acc: CPFL_ACC_CPF_ID */ vport_identity.pf_id = CPFL_ACC_CPF_ID; vport_identity.vf_id = 0; diff --git a/drivers/net/cpfl/cpfl_representor.c b/drivers/net/cpfl/cpfl_representor.c index de3b426727..cb253f7af4 100644 --- a/drivers/net/cpfl/cpfl_representor.c +++ b/drivers/net/cpfl/cpfl_representor.c @@ -348,12 +348,12 @@ cpfl_repr_link_update(struct rte_eth_dev *ethdev, if (wait_to_complete) { if (repr->repr_id.type == RTE_ETH_REPRESENTOR_PF) { /* PF */ - vi.func_type = CPCHNL2_FUNC_TYPE_PF; + vi.func_type = CPCHNL2_FTYPE_LAN_PF; vi.pf_id = cpfl_func_id_get(repr->repr_id.host_id, repr->repr_id.pf_id); vi.vf_id = 0; } else { /* VF */ - vi.func_type = CPCHNL2_FUNC_TYPE_SRIOV; + vi.func_type = CPCHNL2_FTYPE_LAN_VF; vi.pf_id = CPFL_HOST0_APF; vi.vf_id = repr->repr_id.vf_id; } @@ -455,14 +455,14 @@ cpfl_match_repr_with_vport(const struct cpfl_repr_id *repr_id, int func_id; if (repr_id->type == RTE_ETH_REPRESENTOR_PF && - info->func_type == CPFL_VPORT_LAN_PF) { + info->func_type == CPCHNL2_FTYPE_LAN_PF) { func_id = cpfl_func_id_get(repr_id->host_id, repr_id->pf_id); if (func_id < 0 || func_id != info->pf_id) return false; else return true; } else if (repr_id->type == RTE_ETH_REPRESENTOR_VF && - info->func_type == CPFL_VPORT_LAN_VF) { + info->func_type == CPCHNL2_FTYPE_LAN_VF) { if (repr_id->vf_id == info->vf_id) return true; } @@ -480,12 +480,12 @@ cpfl_repr_vport_list_query(struct cpfl_adapter_ext *adapter, if (repr_id->type == RTE_ETH_REPRESENTOR_PF) { /* PF */ - vi.func_type = CPCHNL2_FUNC_TYPE_PF; + vi.func_type = CPCHNL2_FTYPE_LAN_PF; vi.pf_id = cpfl_func_id_get(repr_id->host_id, repr_id->pf_id); vi.vf_id = 0; } else { /* VF */ - vi.func_type = CPCHNL2_FUNC_TYPE_SRIOV; + vi.func_type = CPCHNL2_FTYPE_LAN_VF; vi.pf_id = CPFL_HOST0_APF; vi.vf_id = repr_id->vf_id; } @@ -506,12 +506,12 @@ cpfl_repr_vport_info_query(struct cpfl_adapter_ext *adapter, if (repr_id->type == RTE_ETH_REPRESENTOR_PF) { /* PF */ - vi.func_type = CPCHNL2_FUNC_TYPE_PF; + vi.func_type = CPCHNL2_FTYPE_LAN_PF; vi.pf_id = cpfl_func_id_get(repr_id->host_id, repr_id->pf_id); vi.vf_id = 0; } else { /* VF */ - vi.func_type = CPCHNL2_FUNC_TYPE_SRIOV; + vi.func_type = CPCHNL2_FTYPE_LAN_VF; vi.pf_id = CPFL_HOST0_APF; vi.vf_id = repr_id->vf_id; } @@ -532,11 +532,11 @@ cpfl_repr_vport_map_update(struct cpfl_adapter_ext *adapter, vi.vport_id = vport_id; if (repr_id->type == RTE_ETH_REPRESENTOR_PF) { /* PF */ - vi.func_type = CPCHNL2_FUNC_TYPE_PF; + vi.func_type = CPCHNL2_FTYPE_LAN_VF; vi.pf_id = cpfl_func_id_get(repr_id->host_id, repr_id->pf_id); } else { /* VF */ - vi.func_type = CPCHNL2_FUNC_TYPE_SRIOV; + vi.func_type = CPCHNL2_FTYPE_LAN_VF; vi.pf_id = CPFL_HOST0_APF; vi.vf_id = repr_id->vf_id; } -- 2.34.1