[dpdk-dev] [PATCH 08/11] net/hns3: fix ring vector related mailbox command format

[Wei Hu (Xavier)]

From: "Wei Hu (Xavier)" <xavier.hu...@huawei.com>

The format of the ring vector related mailbox commands between driver and
firmware is different from those of other mailbox commands in hns3 network
engine.

This patch fixes the error mailbox command format about the vector of the
rings, the related command opcode as below:
HNS3_MBX_MAP_RING_TO_VECTOR
HNS3_MBX_UNMAP_RING_TO_VECTOR
HNS3_MBX_GET_RING_VECTOR_MAP

Fixes: 463e748964f5 ("net/hns3: support mailbox")
Cc: sta...@dpdk.org

Signed-off-by: Chengwen Feng <fengcheng...@huawei.com>
Signed-off-by: Wei Hu (Xavier) <xavier.hu...@huawei.com>
---
 drivers/net/hns3/hns3_mbx.c | 14 +++++++++++---
 drivers/net/hns3/hns3_mbx.h |  1 +
 2 files changed, 12 insertions(+), 3 deletions(-)

diff --git a/drivers/net/hns3/hns3_mbx.c b/drivers/net/hns3/hns3_mbx.c
index 26807bc4b..0d03f5064 100644
--- a/drivers/net/hns3/hns3_mbx.c
+++ b/drivers/net/hns3/hns3_mbx.c
@@ -150,6 +150,8 @@ hns3_send_mbx_msg(struct hns3_hw *hw, uint16_t code, 
uint16_t subcode,
 {
        struct hns3_mbx_vf_to_pf_cmd *req;
        struct hns3_cmd_desc desc;
+       bool is_ring_vector_msg;
+       int offset;
        int ret;
 
        req = (struct hns3_mbx_vf_to_pf_cmd *)desc.data;
@@ -164,9 +166,15 @@ hns3_send_mbx_msg(struct hns3_hw *hw, uint16_t code, 
uint16_t subcode,
 
        hns3_cmd_setup_basic_desc(&desc, HNS3_OPC_MBX_VF_TO_PF, false);
        req->msg[0] = code;
-       req->msg[1] = subcode;
-       if (msg_data)
-               memcpy(&req->msg[HNS3_CMD_CODE_OFFSET], msg_data, msg_len);
+       is_ring_vector_msg = (code == HNS3_MBX_MAP_RING_TO_VECTOR) ||
+                            (code == HNS3_MBX_UNMAP_RING_TO_VECTOR) ||
+                            (code == HNS3_MBX_GET_RING_VECTOR_MAP);
+       if (!is_ring_vector_msg)
+               req->msg[1] = subcode;
+       if (msg_data) {
+               offset = is_ring_vector_msg ? 1 : HNS3_CMD_CODE_OFFSET;
+               memcpy(&req->msg[offset], msg_data, msg_len);
+       }
 
        /* synchronous send */
        if (need_resp) {
diff --git a/drivers/net/hns3/hns3_mbx.h b/drivers/net/hns3/hns3_mbx.h
index 3722c8760..b01eaacc3 100644
--- a/drivers/net/hns3/hns3_mbx.h
+++ b/drivers/net/hns3/hns3_mbx.h
@@ -41,6 +41,7 @@ enum HNS3_MBX_OPCODE {
        HNS3_MBX_GET_QID_IN_PF,         /* (VF -> PF) get queue id in pf */
 
        HNS3_MBX_HANDLE_VF_TBL = 38,    /* (VF -> PF) store/clear hw cfg tbl */
+       HNS3_MBX_GET_RING_VECTOR_MAP,   /* (VF -> PF) get ring-to-vector map */
        HNS3_MBX_PUSH_LINK_STATUS = 201, /* (IMP -> PF) get port link status */
 };
 
-- 
2.23.0