Current driver implementation for Sideband Queue supports a
fixed flag (ICE_AQ_FLAG_RD). To retrieve FEC statistics from
firmware, Sideband Queue command is used with a different flag.

Extend API for Sideband Queue command to use 'flag' as input
argument.

Reviewed-by: Anthony L Nguyen <anthony.l.ngu...@intel.com>
Reviewed-by: Jesse Brandeburg <jesse.brandeb...@intel.com>
Signed-off-by: Anil Samal <anil.sa...@intel.com>
---
 drivers/net/ethernet/intel/ice/ice_common.c |  5 +++--
 drivers/net/ethernet/intel/ice/ice_common.h |  2 +-
 drivers/net/ethernet/intel/ice/ice_ptp_hw.c | 16 ++++++++--------
 3 files changed, 12 insertions(+), 11 deletions(-)

diff --git a/drivers/net/ethernet/intel/ice/ice_common.c 
b/drivers/net/ethernet/intel/ice/ice_common.c
index 5649b257e631..9a0a533613ff 100644
--- a/drivers/net/ethernet/intel/ice/ice_common.c
+++ b/drivers/net/ethernet/intel/ice/ice_common.c
@@ -1473,8 +1473,9 @@ ice_sbq_send_cmd(struct ice_hw *hw, struct 
ice_sbq_cmd_desc *desc,
  * ice_sbq_rw_reg - Fill Sideband Queue command
  * @hw: pointer to the HW struct
  * @in: message info to be filled in descriptor
+ * @flag: flag to fill desc structure
  */
-int ice_sbq_rw_reg(struct ice_hw *hw, struct ice_sbq_msg_input *in)
+int ice_sbq_rw_reg(struct ice_hw *hw, struct ice_sbq_msg_input *in, u16 flag)
 {
        struct ice_sbq_cmd_desc desc = {0};
        struct ice_sbq_msg_req msg = {0};
@@ -1498,7 +1499,7 @@ int ice_sbq_rw_reg(struct ice_hw *hw, struct 
ice_sbq_msg_input *in)
                 */
                msg_len -= sizeof(msg.data);
 
-       desc.flags = cpu_to_le16(ICE_AQ_FLAG_RD);
+       desc.flags = cpu_to_le16(flag);
        desc.opcode = cpu_to_le16(ice_sbq_opc_neigh_dev_req);
        desc.param0.cmd_len = cpu_to_le16(msg_len);
        status = ice_sbq_send_cmd(hw, &desc, &msg, msg_len, NULL);
diff --git a/drivers/net/ethernet/intel/ice/ice_common.h 
b/drivers/net/ethernet/intel/ice/ice_common.h
index ffb22c7ce28b..42cda1bbbaab 100644
--- a/drivers/net/ethernet/intel/ice/ice_common.h
+++ b/drivers/net/ethernet/intel/ice/ice_common.h
@@ -201,7 +201,7 @@ int ice_replay_vsi(struct ice_hw *hw, u16 vsi_handle);
 void ice_replay_post(struct ice_hw *hw);
 struct ice_q_ctx *
 ice_get_lan_q_ctx(struct ice_hw *hw, u16 vsi_handle, u8 tc, u16 q_handle);
-int ice_sbq_rw_reg(struct ice_hw *hw, struct ice_sbq_msg_input *in);
+int ice_sbq_rw_reg(struct ice_hw *hw, struct ice_sbq_msg_input *in, u16 flag);
 int
 ice_aq_get_cgu_abilities(struct ice_hw *hw,
                         struct ice_aqc_get_cgu_abilities *abilities);
diff --git a/drivers/net/ethernet/intel/ice/ice_ptp_hw.c 
b/drivers/net/ethernet/intel/ice/ice_ptp_hw.c
index 2b9423a173bb..e97b73d1b0f4 100644
--- a/drivers/net/ethernet/intel/ice/ice_ptp_hw.c
+++ b/drivers/net/ethernet/intel/ice/ice_ptp_hw.c
@@ -433,7 +433,7 @@ ice_read_phy_reg_e82x(struct ice_hw *hw, u8 port, u16 
offset, u32 *val)
        ice_fill_phy_msg_e82x(&msg, port, offset);
        msg.opcode = ice_sbq_msg_rd;
 
-       err = ice_sbq_rw_reg(hw, &msg);
+       err = ice_sbq_rw_reg(hw, &msg, ICE_AQ_FLAG_RD);
        if (err) {
                ice_debug(hw, ICE_DBG_PTP, "Failed to send message to PHY, err 
%d\n",
                          err);
@@ -511,7 +511,7 @@ ice_write_phy_reg_e82x(struct ice_hw *hw, u8 port, u16 
offset, u32 val)
        msg.opcode = ice_sbq_msg_wr;
        msg.data = val;
 
-       err = ice_sbq_rw_reg(hw, &msg);
+       err = ice_sbq_rw_reg(hw, &msg, ICE_AQ_FLAG_RD);
        if (err) {
                ice_debug(hw, ICE_DBG_PTP, "Failed to send message to PHY, err 
%d\n",
                          err);
@@ -667,7 +667,7 @@ ice_read_quad_reg_e82x(struct ice_hw *hw, u8 quad, u16 
offset, u32 *val)
 
        msg.opcode = ice_sbq_msg_rd;
 
-       err = ice_sbq_rw_reg(hw, &msg);
+       err = ice_sbq_rw_reg(hw, &msg, ICE_AQ_FLAG_RD);
        if (err) {
                ice_debug(hw, ICE_DBG_PTP, "Failed to send message to PHY, err 
%d\n",
                          err);
@@ -702,7 +702,7 @@ ice_write_quad_reg_e82x(struct ice_hw *hw, u8 quad, u16 
offset, u32 val)
        msg.opcode = ice_sbq_msg_wr;
        msg.data = val;
 
-       err = ice_sbq_rw_reg(hw, &msg);
+       err = ice_sbq_rw_reg(hw, &msg, ICE_AQ_FLAG_RD);
        if (err) {
                ice_debug(hw, ICE_DBG_PTP, "Failed to send message to PHY, err 
%d\n",
                          err);
@@ -840,7 +840,7 @@ ice_read_cgu_reg_e82x(struct ice_hw *hw, u32 addr, u32 *val)
        cgu_msg.msg_addr_low = addr;
        cgu_msg.msg_addr_high = 0x0;
 
-       err = ice_sbq_rw_reg(hw, &cgu_msg);
+       err = ice_sbq_rw_reg(hw, &cgu_msg, ICE_AQ_FLAG_RD);
        if (err) {
                ice_debug(hw, ICE_DBG_PTP, "Failed to read CGU register 0x%04x, 
err %d\n",
                          addr, err);
@@ -873,7 +873,7 @@ ice_write_cgu_reg_e82x(struct ice_hw *hw, u32 addr, u32 val)
        cgu_msg.msg_addr_high = 0x0;
        cgu_msg.data = val;
 
-       err = ice_sbq_rw_reg(hw, &cgu_msg);
+       err = ice_sbq_rw_reg(hw, &cgu_msg, ICE_AQ_FLAG_RD);
        if (err) {
                ice_debug(hw, ICE_DBG_PTP, "Failed to write CGU register 
0x%04x, err %d\n",
                          addr, err);
@@ -2660,7 +2660,7 @@ static int ice_read_phy_reg_e810(struct ice_hw *hw, u32 
addr, u32 *val)
        msg.opcode = ice_sbq_msg_rd;
        msg.dest_dev = rmn_0;
 
-       err = ice_sbq_rw_reg(hw, &msg);
+       err = ice_sbq_rw_reg(hw, &msg, ICE_AQ_FLAG_RD);
        if (err) {
                ice_debug(hw, ICE_DBG_PTP, "Failed to send message to PHY, err 
%d\n",
                          err);
@@ -2691,7 +2691,7 @@ static int ice_write_phy_reg_e810(struct ice_hw *hw, u32 
addr, u32 val)
        msg.dest_dev = rmn_0;
        msg.data = val;
 
-       err = ice_sbq_rw_reg(hw, &msg);
+       err = ice_sbq_rw_reg(hw, &msg, ICE_AQ_FLAG_RD);
        if (err) {
                ice_debug(hw, ICE_DBG_PTP, "Failed to send message to PHY, err 
%d\n",
                          err);
-- 
2.44.0

Reply via email to