This patch consists of semantic/formatting changes. It also includes
comment additions.

Signed-off-by: Rasesh Mody <rasesh.m...@cavium.com>
---
 drivers/net/qede/base/common_hsi.h       |   5 +-
 drivers/net/qede/base/ecore_dev.c        |  53 ++++---
 drivers/net/qede/base/ecore_hsi_common.h |  14 +-
 drivers/net/qede/base/ecore_hw.c         |   4 +-
 drivers/net/qede/base/ecore_init_ops.c   |  23 ++-
 drivers/net/qede/base/ecore_int.c        |   6 +-
 drivers/net/qede/base/ecore_mcp.c        |   5 +-
 drivers/net/qede/base/ecore_spq.c        |   3 +-
 drivers/net/qede/base/ecore_sriov.c      |   8 +-
 drivers/net/qede/base/mcp_public.h       | 262 +++++++++++++++----------------
 10 files changed, 186 insertions(+), 197 deletions(-)

diff --git a/drivers/net/qede/base/common_hsi.h 
b/drivers/net/qede/base/common_hsi.h
index 4083e86d..2f84148e 100644
--- a/drivers/net/qede/base/common_hsi.h
+++ b/drivers/net/qede/base/common_hsi.h
@@ -721,8 +721,7 @@ union event_ring_data {
        u8 bytes[8] /* Byte Array */;
        struct vf_pf_channel_eqe_data vf_pf_channel /* VF-PF Channel data */;
        struct iscsi_eqe_data iscsi_info /* Dedicated fields to iscsi data */;
-           /* Dedicated field for RoCE affiliated asynchronous error */;
-       struct regpair roceHandle;
+       struct regpair roceHandle /* Dedicated field for RDMA data */;
        struct malicious_vf_eqe_data malicious_vf /* Malicious VF data */;
        struct initial_cleanup_eqe_data vf_init_cleanup
            /* VF Initial Cleanup data */;
@@ -766,6 +765,8 @@ enum protocol_type {
        MAX_PROTOCOL_TYPE
 };
 
+
+
 /*
  * Ustorm Queue Zone
  */
diff --git a/drivers/net/qede/base/ecore_dev.c 
b/drivers/net/qede/base/ecore_dev.c
index 15db09fc..b7540286 100644
--- a/drivers/net/qede/base/ecore_dev.c
+++ b/drivers/net/qede/base/ecore_dev.c
@@ -70,28 +70,26 @@ static u32 ecore_hw_bar_size(struct ecore_hwfn *p_hwfn, 
enum BAR_ID bar_id)
        }
 
        val = ecore_rd(p_hwfn, p_hwfn->p_main_ptt, bar_reg);
+       if (val)
+               return 1 << (val + 15);
 
        /* The above registers were updated in the past only in CMT mode. Since
         * they were found to be useful MFW started updating them from 8.7.7.0.
         * In older MFW versions they are set to 0 which means disabled.
         */
-       if (!val) {
-               if (p_hwfn->p_dev->num_hwfns > 1) {
-                       DP_NOTICE(p_hwfn, false,
-                                 "BAR size not configured. Assuming BAR size");
-                       DP_NOTICE(p_hwfn, false,
-                                 "of 256kB for GRC and 512kB for DB\n");
-                       return BAR_ID_0 ? 256 * 1024 : 512 * 1024;
-               } else {
-                       DP_NOTICE(p_hwfn, false,
-                                 "BAR size not configured. Assuming BAR size");
-                       DP_NOTICE(p_hwfn, false,
-                                 "of 512kB for GRC and 512kB for DB\n");
-                       return 512 * 1024;
-               }
+       if (p_hwfn->p_dev->num_hwfns > 1) {
+               DP_NOTICE(p_hwfn, false,
+                         "BAR size not configured. Assuming BAR size of 256kB"
+                         " for GRC and 512kB for DB\n");
+               val = BAR_ID_0 ? 256 * 1024 : 512 * 1024;
+       } else {
+               DP_NOTICE(p_hwfn, false,
+                         "BAR size not configured. Assuming BAR size of 512kB"
+                         " for GRC and 512kB for DB\n");
+               val = 512 * 1024;
        }
 
-       return 1 << (val + 15);
+       return val;
 }
 
 void ecore_init_dp(struct ecore_dev *p_dev,
@@ -1623,7 +1621,8 @@ enum _ecore_status_t ecore_hw_init(struct ecore_dev 
*p_dev,
        u32 load_code, param;
        int i;
 
-       if (p_params->int_mode == ECORE_INT_MODE_MSI && p_dev->num_hwfns > 1) {
+       if ((p_params->int_mode == ECORE_INT_MODE_MSI) &&
+           (p_dev->num_hwfns > 1)) {
                DP_NOTICE(p_dev, false,
                          "MSI mode is not supported for CMT devices\n");
                return ECORE_INVAL;
@@ -2784,11 +2783,14 @@ ecore_get_hw_info(struct ecore_hwfn *p_hwfn, struct 
ecore_ptt *p_ptt,
                ecore_mcp_cmd_port_init(p_hwfn, p_ptt);
        }
 
-       if (personality != ECORE_PCI_DEFAULT)
+       if (personality != ECORE_PCI_DEFAULT) {
                p_hwfn->hw_info.personality = personality;
-       else if (ecore_mcp_is_init(p_hwfn))
-               p_hwfn->hw_info.personality =
-                   p_hwfn->mcp_info->func_info.protocol;
+       } else if (ecore_mcp_is_init(p_hwfn)) {
+               enum ecore_pci_personality protocol;
+
+               protocol = p_hwfn->mcp_info->func_info.protocol;
+               p_hwfn->hw_info.personality = protocol;
+       }
 
 #ifndef ASIC_ONLY
        /* To overcome ILT lack for emulation, until at least until we'll have
@@ -2937,8 +2939,9 @@ void ecore_prepare_hibernate(struct ecore_dev *p_dev)
 #endif
 
 static enum _ecore_status_t
-ecore_hw_prepare_single(struct ecore_hwfn *p_hwfn, void OSAL_IOMEM *p_regview,
-                       void OSAL_IOMEM *p_doorbells,
+ecore_hw_prepare_single(struct ecore_hwfn *p_hwfn,
+                       void OSAL_IOMEM * p_regview,
+                       void OSAL_IOMEM * p_doorbells,
                        struct ecore_hw_prepare_params *p_params)
 {
        struct ecore_dev *p_dev = p_hwfn->p_dev;
@@ -3280,8 +3283,8 @@ ecore_chain_alloc_next_ptr(struct ecore_dev *p_dev, 
struct ecore_chain *p_chain)
 static enum _ecore_status_t
 ecore_chain_alloc_single(struct ecore_dev *p_dev, struct ecore_chain *p_chain)
 {
-       void *p_virt = OSAL_NULL;
        dma_addr_t p_phys = 0;
+       void *p_virt = OSAL_NULL;
 
        p_virt = OSAL_DMA_ALLOC_COHERENT(p_dev, &p_phys, ECORE_CHAIN_PAGE_SIZE);
        if (!p_virt) {
@@ -3809,10 +3812,10 @@ enum _ecore_status_t ecore_set_rxq_coalesce(struct 
ecore_hwfn *p_hwfn,
                                            u16 coalesce, u8 qid, u16 sb_id)
 {
        struct ustorm_eth_queue_zone eth_qzone;
+       u8 timeset, timer_res;
        u16 fw_qid = 0;
        u32 address;
        enum _ecore_status_t rc;
-       u8 timeset, timer_res;
 
        /* Coalesce = (timeset << timer-resolution), timeset is 7bit wide */
        if (coalesce <= 0x7F) {
@@ -3852,10 +3855,10 @@ enum _ecore_status_t ecore_set_txq_coalesce(struct 
ecore_hwfn *p_hwfn,
                                            u16 coalesce, u8 qid, u16 sb_id)
 {
        struct xstorm_eth_queue_zone eth_qzone;
+       u8 timeset, timer_res;
        u16 fw_qid = 0;
        u32 address;
        enum _ecore_status_t rc;
-       u8 timeset, timer_res;
 
        /* Coalesce = (timeset << timer-resolution), timeset is 7bit wide */
        if (coalesce <= 0x7F) {
diff --git a/drivers/net/qede/base/ecore_hsi_common.h 
b/drivers/net/qede/base/ecore_hsi_common.h
index 6ddbe1a3..d978bb08 100644
--- a/drivers/net/qede/base/ecore_hsi_common.h
+++ b/drivers/net/qede/base/ecore_hsi_common.h
@@ -1320,9 +1320,13 @@ enum personality_type {
  * tunnel configuration
  */
 struct pf_start_tunnel_config {
-/* Set VXLAN tunnel UDP destination port. */
+/* Set VXLAN tunnel UDP destination port to vxlan_udp_port. If not set -
+ * FW will use a default port
+ */
        u8 set_vxlan_udp_port_flg;
-/* Set GENEVE tunnel UDP destination port. */
+/* Set GENEVE tunnel UDP destination port to geneve_udp_port. If not set -
+ * FW will use a default port
+ */
        u8 set_geneve_udp_port_flg;
        u8 tx_enable_vxlan /* If set, enable VXLAN tunnel in TX path. */;
 /* If set, enable l2 GENEVE tunnel in TX path. */
@@ -1338,8 +1342,10 @@ struct pf_start_tunnel_config {
        u8 tunnel_clss_ipgeneve;
        u8 tunnel_clss_l2gre /* Classification scheme for l2 GRE tunnel. */;
        u8 tunnel_clss_ipgre /* Classification scheme for ip GRE tunnel. */;
-       __le16 vxlan_udp_port /* VXLAN tunnel UDP destination port. */;
-       __le16 geneve_udp_port /* GENEVE tunnel UDP destination port. */;
+/* VXLAN tunnel UDP destination port. Valid if set_vxlan_udp_port_flg=1 */
+       __le16 vxlan_udp_port;
+/* GENEVE tunnel UDP destination port. Valid if set_geneve_udp_port_flg=1 */
+       __le16 geneve_udp_port;
 };
 
 /*
diff --git a/drivers/net/qede/base/ecore_hw.c b/drivers/net/qede/base/ecore_hw.c
index 8abe60a9..22da415e 100644
--- a/drivers/net/qede/base/ecore_hw.c
+++ b/drivers/net/qede/base/ecore_hw.c
@@ -496,8 +496,8 @@ static u32 ecore_dmae_idx_to_go_cmd(u8 idx)
        return DMAE_REG_GO_C0 + (idx << 2);
 }
 
-static enum _ecore_status_t
-ecore_dmae_post_command(struct ecore_hwfn *p_hwfn, struct ecore_ptt *p_ptt)
+static enum _ecore_status_t ecore_dmae_post_command(struct ecore_hwfn *p_hwfn,
+                                                   struct ecore_ptt *p_ptt)
 {
        struct dmae_cmd *p_command = p_hwfn->dmae_info.p_dmae_cmd;
        u8 idx_cmd = p_hwfn->dmae_info.channel, i;
diff --git a/drivers/net/qede/base/ecore_init_ops.c 
b/drivers/net/qede/base/ecore_init_ops.c
index faeca685..b907a95e 100644
--- a/drivers/net/qede/base/ecore_init_ops.c
+++ b/drivers/net/qede/base/ecore_init_ops.c
@@ -63,8 +63,8 @@ static enum _ecore_status_t ecore_init_rt(struct ecore_hwfn 
*p_hwfn,
 {
        u32 *p_init_val = &p_hwfn->rt_data.init_val[rt_offset];
        bool *p_valid = &p_hwfn->rt_data.b_valid[rt_offset];
-       enum _ecore_status_t rc = ECORE_SUCCESS;
        u16 i, segment;
+       enum _ecore_status_t rc = ECORE_SUCCESS;
 
        /* Since not all RT entries are initialized, go over the RT and
         * for each segment of initialized values use DMA.
@@ -190,19 +190,19 @@ static enum _ecore_status_t ecore_init_cmd_array(struct 
ecore_hwfn *p_hwfn,
                                                 bool b_must_dmae,
                                                 bool b_can_dmae)
 {
+       u32 dmae_array_offset = OSAL_LE32_TO_CPU(cmd->args.array_offset);
+       u32 data = OSAL_LE32_TO_CPU(cmd->data);
+       u32 addr = GET_FIELD(data, INIT_WRITE_OP_ADDRESS) << 2;
 #ifdef CONFIG_ECORE_ZIPPED_FW
        u32 offset, output_len, input_len, max_size;
 #endif
-       u32 dmae_array_offset = OSAL_LE32_TO_CPU(cmd->args.array_offset);
        struct ecore_dev *p_dev = p_hwfn->p_dev;
-       enum _ecore_status_t rc = ECORE_SUCCESS;
        union init_array_hdr *hdr;
        const u32 *array_data;
-       u32 size, addr, data;
+       enum _ecore_status_t rc = ECORE_SUCCESS;
+       u32 size;
 
        array_data = p_dev->fw_data->arr_data;
-       data = OSAL_LE32_TO_CPU(cmd->data);
-       addr = GET_FIELD(data, INIT_WRITE_OP_ADDRESS) << 2;
 
        hdr = (union init_array_hdr *)
                (uintptr_t)(array_data + dmae_array_offset);
@@ -272,13 +272,10 @@ static enum _ecore_status_t ecore_init_cmd_wr(struct 
ecore_hwfn *p_hwfn,
                                              struct init_write_op *p_cmd,
                                              bool b_can_dmae)
 {
+       u32 data = OSAL_LE32_TO_CPU(p_cmd->data);
+       bool b_must_dmae = GET_FIELD(data, INIT_WRITE_OP_WIDE_BUS);
+       u32 addr = GET_FIELD(data, INIT_WRITE_OP_ADDRESS) << 2;
        enum _ecore_status_t rc = ECORE_SUCCESS;
-       bool b_must_dmae;
-       u32 addr, data;
-
-       data = OSAL_LE32_TO_CPU(p_cmd->data);
-       b_must_dmae = GET_FIELD(data, INIT_WRITE_OP_WIDE_BUS);
-       addr = GET_FIELD(data, INIT_WRITE_OP_ADDRESS) << 2;
 
        /* Sanitize */
        if (b_must_dmae && !b_can_dmae) {
@@ -452,10 +449,10 @@ enum _ecore_status_t ecore_init_run(struct ecore_hwfn 
*p_hwfn,
                                    int phase, int phase_id, int modes)
 {
        struct ecore_dev *p_dev = p_hwfn->p_dev;
-       enum _ecore_status_t rc = ECORE_SUCCESS;
        u32 cmd_num, num_init_ops;
        union init_op *init_ops;
        bool b_dmae = false;
+       enum _ecore_status_t rc = ECORE_SUCCESS;
 
        num_init_ops = p_dev->fw_data->init_ops_size;
        init_ops = p_dev->fw_data->init_ops;
diff --git a/drivers/net/qede/base/ecore_int.c 
b/drivers/net/qede/base/ecore_int.c
index 6fb037df..96f283ba 100644
--- a/drivers/net/qede/base/ecore_int.c
+++ b/drivers/net/qede/base/ecore_int.c
@@ -1234,7 +1234,7 @@ static enum _ecore_status_t 
ecore_int_sb_attn_alloc(struct ecore_hwfn *p_hwfn,
        p_sb = OSAL_ALLOC(p_dev, GFP_KERNEL, sizeof(*p_sb));
        if (!p_sb) {
                DP_NOTICE(p_dev, true,
-                         "Failed to allocate `struct ecore_sb_attn_info'");
+                         "Failed to allocate `struct ecore_sb_attn_info'\n");
                return ECORE_NOMEM;
        }
 
@@ -1243,7 +1243,7 @@ static enum _ecore_status_t 
ecore_int_sb_attn_alloc(struct ecore_hwfn *p_hwfn,
                                         SB_ATTN_ALIGNED_SIZE(p_hwfn));
        if (!p_virt) {
                DP_NOTICE(p_dev, true,
-                         "Failed to allocate status block (attentions)");
+                         "Failed to allocate status block (attentions)\n");
                OSAL_FREE(p_dev, p_sb);
                return ECORE_NOMEM;
        }
@@ -2127,8 +2127,8 @@ enum _ecore_status_t ecore_int_set_timer_res(struct 
ecore_hwfn *p_hwfn,
                                             struct ecore_ptt *p_ptt,
                                             u8 timer_res, u16 sb_id, bool tx)
 {
-       enum _ecore_status_t rc;
        struct cau_sb_entry sb_entry;
+       enum _ecore_status_t rc;
 
        if (!p_hwfn->hw_init_done) {
                DP_ERR(p_hwfn, "hardware not initialized yet\n");
diff --git a/drivers/net/qede/base/ecore_mcp.c 
b/drivers/net/qede/base/ecore_mcp.c
index e641a77f..bb13828d 100644
--- a/drivers/net/qede/base/ecore_mcp.c
+++ b/drivers/net/qede/base/ecore_mcp.c
@@ -945,9 +945,8 @@ static void ecore_mcp_send_protocol_stats(struct ecore_hwfn 
*p_hwfn,
        ecore_mcp_cmd_and_union(p_hwfn, p_ptt, &mb_params);
 }
 
-static void
-ecore_read_pf_bandwidth(struct ecore_hwfn *p_hwfn,
-                       struct public_func *p_shmem_info)
+static void ecore_read_pf_bandwidth(struct ecore_hwfn *p_hwfn,
+                                   struct public_func *p_shmem_info)
 {
        struct ecore_mcp_function_info *p_info;
 
diff --git a/drivers/net/qede/base/ecore_spq.c 
b/drivers/net/qede/base/ecore_spq.c
index e3714925..9f5fdf88 100644
--- a/drivers/net/qede/base/ecore_spq.c
+++ b/drivers/net/qede/base/ecore_spq.c
@@ -380,8 +380,7 @@ struct ecore_eq *ecore_eq_alloc(struct ecore_hwfn *p_hwfn, 
u16 num_elem)
        }
 
        /* register EQ completion on the SP SB */
-       ecore_int_register_cb(p_hwfn,
-                             ecore_eq_completion,
+       ecore_int_register_cb(p_hwfn, ecore_eq_completion,
                              p_eq, &p_eq->eq_sb_index, &p_eq->p_fw_cons);
 
        return p_eq;
diff --git a/drivers/net/qede/base/ecore_sriov.c 
b/drivers/net/qede/base/ecore_sriov.c
index e8f1ebe6..4c1a0787 100644
--- a/drivers/net/qede/base/ecore_sriov.c
+++ b/drivers/net/qede/base/ecore_sriov.c
@@ -1675,11 +1675,9 @@ ecore_iov_reconfigure_unicast_vlan(struct ecore_hwfn 
*p_hwfn,
                DP_VERBOSE(p_hwfn, ECORE_MSG_IOV,
                           "Reconfiguring VLAN [0x%04x] for VF [%04x]\n",
                           filter.vlan, p_vf->relative_vf_id);
-               rc = ecore_sp_eth_filter_ucast(p_hwfn,
-                                              p_vf->opaque_fid,
-                                              &filter,
-                                              ECORE_SPQ_MODE_CB,
-                                                      OSAL_NULL);
+               rc = ecore_sp_eth_filter_ucast(p_hwfn, p_vf->opaque_fid,
+                                              &filter, ECORE_SPQ_MODE_CB,
+                                              OSAL_NULL);
                if (rc) {
                        DP_NOTICE(p_hwfn, true,
                                  "Failed to configure VLAN [%04x]"
diff --git a/drivers/net/qede/base/mcp_public.h 
b/drivers/net/qede/base/mcp_public.h
index b8a9ae3a..81567d1d 100644
--- a/drivers/net/qede/base/mcp_public.h
+++ b/drivers/net/qede/base/mcp_public.h
@@ -584,23 +584,20 @@ struct public_port {
 #define MCP_VALIDITY_ACTIVE_MFW_NONE            0x000001c0
 
        u32 link_status;
-#define LINK_STATUS_LINK_UP                                    0x00000001
-#define LINK_STATUS_SPEED_AND_DUPLEX_MASK                      0x0000001e
-#define LINK_STATUS_SPEED_AND_DUPLEX_1000THD           (1 << 1)
-#define LINK_STATUS_SPEED_AND_DUPLEX_1000TFD           (2 << 1)
+#define LINK_STATUS_LINK_UP                            0x00000001
+#define LINK_STATUS_SPEED_AND_DUPLEX_MASK              0x0000001e
+#define LINK_STATUS_SPEED_AND_DUPLEX_1000THD                   (1 << 1)
+#define LINK_STATUS_SPEED_AND_DUPLEX_1000TFD                   (2 << 1)
 #define LINK_STATUS_SPEED_AND_DUPLEX_10G                       (3 << 1)
 #define LINK_STATUS_SPEED_AND_DUPLEX_20G                       (4 << 1)
 #define LINK_STATUS_SPEED_AND_DUPLEX_40G                       (5 << 1)
 #define LINK_STATUS_SPEED_AND_DUPLEX_50G                       (6 << 1)
 #define LINK_STATUS_SPEED_AND_DUPLEX_100G                      (7 << 1)
 #define LINK_STATUS_SPEED_AND_DUPLEX_25G                       (8 << 1)
-
-#define LINK_STATUS_AUTO_NEGOTIATE_ENABLED                     0x00000020
-
-#define LINK_STATUS_AUTO_NEGOTIATE_COMPLETE                    0x00000040
-#define LINK_STATUS_PARALLEL_DETECTION_USED                    0x00000080
-
-#define LINK_STATUS_PFC_ENABLED                                        
0x00000100
+#define LINK_STATUS_AUTO_NEGOTIATE_ENABLED             0x00000020
+#define LINK_STATUS_AUTO_NEGOTIATE_COMPLETE            0x00000040
+#define LINK_STATUS_PARALLEL_DETECTION_USED            0x00000080
+#define LINK_STATUS_PFC_ENABLED                                0x00000100
 #define LINK_STATUS_LINK_PARTNER_1000TFD_CAPABLE       0x00000200
 #define LINK_STATUS_LINK_PARTNER_1000THD_CAPABLE       0x00000400
 #define LINK_STATUS_LINK_PARTNER_10G_CAPABLE           0x00000800
@@ -609,22 +606,19 @@ struct public_port {
 #define LINK_STATUS_LINK_PARTNER_50G_CAPABLE           0x00004000
 #define LINK_STATUS_LINK_PARTNER_100G_CAPABLE          0x00008000
 #define LINK_STATUS_LINK_PARTNER_25G_CAPABLE           0x00010000
-
 #define LINK_STATUS_LINK_PARTNER_FLOW_CONTROL_MASK     0x000C0000
-#define LINK_STATUS_LINK_PARTNER_NOT_PAUSE_CAPABLE     (0 << 18)
-#define LINK_STATUS_LINK_PARTNER_SYMMETRIC_PAUSE       (1 << 18)
-#define LINK_STATUS_LINK_PARTNER_ASYMMETRIC_PAUSE      (2 << 18)
+#define LINK_STATUS_LINK_PARTNER_NOT_PAUSE_CAPABLE             (0 << 18)
+#define LINK_STATUS_LINK_PARTNER_SYMMETRIC_PAUSE               (1 << 18)
+#define LINK_STATUS_LINK_PARTNER_ASYMMETRIC_PAUSE              (2 << 18)
 #define LINK_STATUS_LINK_PARTNER_BOTH_PAUSE                    (3 << 18)
-
-#define LINK_STATUS_SFP_TX_FAULT                               0x00100000
-#define LINK_STATUS_TX_FLOW_CONTROL_ENABLED                    0x00200000
-#define LINK_STATUS_RX_FLOW_CONTROL_ENABLED                    0x00400000
-#define LINK_STATUS_RX_SIGNAL_PRESENT               0x00800000
-#define LINK_STATUS_MAC_LOCAL_FAULT                 0x01000000
-#define LINK_STATUS_MAC_REMOTE_FAULT                0x02000000
-#define LINK_STATUS_UNSUPPORTED_SPD_REQ                                
0x04000000
-
-#define LINK_STATUS_FEC_MODE_MASK                              0x38000000
+#define LINK_STATUS_SFP_TX_FAULT                       0x00100000
+#define LINK_STATUS_TX_FLOW_CONTROL_ENABLED            0x00200000
+#define LINK_STATUS_RX_FLOW_CONTROL_ENABLED            0x00400000
+#define LINK_STATUS_RX_SIGNAL_PRESENT                  0x00800000
+#define LINK_STATUS_MAC_LOCAL_FAULT                    0x01000000
+#define LINK_STATUS_MAC_REMOTE_FAULT                   0x02000000
+#define LINK_STATUS_UNSUPPORTED_SPD_REQ                        0x04000000
+#define LINK_STATUS_FEC_MODE_MASK                      0x38000000
 #define LINK_STATUS_FEC_MODE_NONE                              (0 << 27)
 #define LINK_STATUS_FEC_MODE_FIRECODE_CL74                     (1 << 27)
 #define LINK_STATUS_FEC_MODE_RS_CL91                           (2 << 27)
@@ -686,45 +680,47 @@ struct public_port {
        u32 fc_npiv_nvram_tbl_addr;
        u32 fc_npiv_nvram_tbl_size;
        u32 transceiver_data;
-#define ETH_TRANSCEIVER_STATE_MASK             0x000000FF
-#define ETH_TRANSCEIVER_STATE_SHIFT            0x00000000
-#define ETH_TRANSCEIVER_STATE_UNPLUGGED                0x00000000
-#define ETH_TRANSCEIVER_STATE_PRESENT          0x00000001
-#define ETH_TRANSCEIVER_STATE_VALID            0x00000003
-#define ETH_TRANSCEIVER_STATE_UPDATING         0x00000008
-#define ETH_TRANSCEIVER_TYPE_MASK              0x0000FF00
-#define ETH_TRANSCEIVER_TYPE_SHIFT             0x00000008
-#define ETH_TRANSCEIVER_TYPE_NONE              0x00000000
-#define ETH_TRANSCEIVER_TYPE_UNKNOWN           0x000000FF
+#define ETH_TRANSCEIVER_STATE_MASK                     0x000000FF
+#define ETH_TRANSCEIVER_STATE_SHIFT                    0x00000000
+#define ETH_TRANSCEIVER_STATE_UNPLUGGED                        0x00000000
+#define ETH_TRANSCEIVER_STATE_PRESENT                  0x00000001
+#define ETH_TRANSCEIVER_STATE_VALID                    0x00000003
+#define ETH_TRANSCEIVER_STATE_UPDATING                 0x00000008
+#define ETH_TRANSCEIVER_TYPE_MASK                      0x0000FF00
+#define ETH_TRANSCEIVER_TYPE_SHIFT                     0x00000008
+#define ETH_TRANSCEIVER_TYPE_NONE                      0x00000000
+#define ETH_TRANSCEIVER_TYPE_UNKNOWN                   0x000000FF
 /* 1G Passive copper cable */
-#define ETH_TRANSCEIVER_TYPE_1G_PCC            0x01
+#define ETH_TRANSCEIVER_TYPE_1G_PCC                    0x01
 /* 1G Active copper cable  */
-#define ETH_TRANSCEIVER_TYPE_1G_ACC            0x02
-#define ETH_TRANSCEIVER_TYPE_1G_LX             0x03
-#define ETH_TRANSCEIVER_TYPE_1G_SX             0x04
-#define ETH_TRANSCEIVER_TYPE_10G_SR            0x05
-#define ETH_TRANSCEIVER_TYPE_10G_LR            0x06
-#define ETH_TRANSCEIVER_TYPE_10G_LRM           0x07
-#define ETH_TRANSCEIVER_TYPE_10G_ER            0x08
+#define ETH_TRANSCEIVER_TYPE_1G_ACC                    0x02
+#define ETH_TRANSCEIVER_TYPE_1G_LX                     0x03
+#define ETH_TRANSCEIVER_TYPE_1G_SX                     0x04
+#define ETH_TRANSCEIVER_TYPE_10G_SR                    0x05
+#define ETH_TRANSCEIVER_TYPE_10G_LR                    0x06
+#define ETH_TRANSCEIVER_TYPE_10G_LRM                   0x07
+#define ETH_TRANSCEIVER_TYPE_10G_ER                    0x08
 /* 10G Passive copper cable */
-#define ETH_TRANSCEIVER_TYPE_10G_PCC           0x09
+#define ETH_TRANSCEIVER_TYPE_10G_PCC                   0x09
 /* 10G Active copper cable  */
-#define ETH_TRANSCEIVER_TYPE_10G_ACC           0x0a
-#define ETH_TRANSCEIVER_TYPE_XLPPI             0x0b
-#define ETH_TRANSCEIVER_TYPE_40G_LR4           0x0c
-#define ETH_TRANSCEIVER_TYPE_40G_SR4           0x0d
-#define ETH_TRANSCEIVER_TYPE_40G_CR4           0x0e
-#define ETH_TRANSCEIVER_TYPE_100G_AOC          0x0f /* Active optical cable */
-#define ETH_TRANSCEIVER_TYPE_100G_SR4          0x10
-#define ETH_TRANSCEIVER_TYPE_100G_LR4          0x11
-#define ETH_TRANSCEIVER_TYPE_100G_ER4          0x12
-#define ETH_TRANSCEIVER_TYPE_100G_ACC          0x13 /* Active copper cable */
-#define ETH_TRANSCEIVER_TYPE_100G_CR4          0x14
-#define ETH_TRANSCEIVER_TYPE_4x10G_SR          0x15
+#define ETH_TRANSCEIVER_TYPE_10G_ACC                   0x0a
+#define ETH_TRANSCEIVER_TYPE_XLPPI                     0x0b
+#define ETH_TRANSCEIVER_TYPE_40G_LR4                   0x0c
+#define ETH_TRANSCEIVER_TYPE_40G_SR4                   0x0d
+#define ETH_TRANSCEIVER_TYPE_40G_CR4                   0x0e
+/* Active optical cable */
+#define ETH_TRANSCEIVER_TYPE_100G_AOC                  0x0f
+#define ETH_TRANSCEIVER_TYPE_100G_SR4                  0x10
+#define ETH_TRANSCEIVER_TYPE_100G_LR4                  0x11
+#define ETH_TRANSCEIVER_TYPE_100G_ER4                  0x12
+/* Active copper cable */
+#define ETH_TRANSCEIVER_TYPE_100G_ACC                  0x13
+#define ETH_TRANSCEIVER_TYPE_100G_CR4                  0x14
+#define ETH_TRANSCEIVER_TYPE_4x10G_SR                  0x15
 /* 25G Passive copper cable - short */
-#define ETH_TRANSCEIVER_TYPE_25G_CA_N          0x16
+#define ETH_TRANSCEIVER_TYPE_25G_CA_N                  0x16
 /* 25G Active copper cable  - short */
-#define ETH_TRANSCEIVER_TYPE_25G_ACC_S         0x17
+#define ETH_TRANSCEIVER_TYPE_25G_ACC_S                 0x17
 /* 25G Passive copper cable - medium */
 #define ETH_TRANSCEIVER_TYPE_25G_CA_S                  0x18
 /* 25G Active copper cable  - medium */
@@ -1153,6 +1149,23 @@ struct public_drv_mb {
  * MCP_REG_CPU_STATE/MCP_REG_CPU_MODE registers.
  */
 #define DRV_MSG_CODE_MCP_HALT                  0x00100000
+/* Set virtual mac address, params [31:6] - reserved, [5:4] - type,
+ * [3:0] - func, drv_data[7:0] - MAC/WWNN/WWPN
+ */
+#define DRV_MSG_CODE_SET_VMAC                   0x00110000
+/* Set virtual mac address, params [31:6] - reserved, [5:4] - type,
+ * [3:0] - func, drv_data[7:0] - MAC/WWNN/WWPN
+ */
+#define DRV_MSG_CODE_GET_VMAC                   0x00120000
+       #define DRV_MSG_CODE_VMAC_TYPE_MAC              1
+       #define DRV_MSG_CODE_VMAC_TYPE_WWNN             2
+       #define DRV_MSG_CODE_VMAC_TYPE_WWPN             3
+/* Get statistics from pf, params [31:4] - reserved, [3:0] - stats type */
+#define DRV_MSG_CODE_GET_STATS                  0x00130000
+       #define DRV_MSG_CODE_STATS_TYPE_LAN             1
+       #define DRV_MSG_CODE_STATS_TYPE_FCOE            2
+       #define DRV_MSG_CODE_STATS_TYPE_ISCSI           3
+       #define DRV_MSG_CODE_STATS_TYPE_RDMA            4
 /* Host shall provide buffer and size for MFW  */
 #define DRV_MSG_CODE_PMD_DIAG_DUMP             0x00140000
 /* Host shall provide buffer and size for MFW  */
@@ -1165,29 +1178,8 @@ struct public_drv_mb {
  * [16:31] - offset
  */
 #define DRV_MSG_CODE_TRANSCEIVER_WRITE         0x00170000
-
-/* Set virtual mac address, params [31:6] - reserved, [5:4] - type,
- * [3:0] - func, drv_data[7:0] - MAC/WWNN/WWPN
- */
-#define DRV_MSG_CODE_SET_VMAC                   0x00110000
-/* Set virtual mac address, params [31:6] - reserved, [5:4] - type,
- * [3:0] - func, drv_data[7:0] - MAC/WWNN/WWPN
- */
-#define DRV_MSG_CODE_GET_VMAC                   0x00120000
-#define DRV_MSG_CODE_VMAC_TYPE_MAC              1
-#define DRV_MSG_CODE_VMAC_TYPE_WWNN             2
-#define DRV_MSG_CODE_VMAC_TYPE_WWPN             3
-
-/* Get statistics from pf, params [31:4] - reserved, [3:0] - stats type */
-#define DRV_MSG_CODE_GET_STATS                  0x00130000
-#define DRV_MSG_CODE_STATS_TYPE_LAN             1
-#define DRV_MSG_CODE_STATS_TYPE_FCOE            2
-#define DRV_MSG_CODE_STATS_TYPE_ISCSI           3
-#define DRV_MSG_CODE_STATS_TYPE_RDMA           4
-
 /* indicate OCBB related information */
 #define DRV_MSG_CODE_OCBB_DATA                 0x00180000
-
 /* Set function BW, params[15:8] - min, params[7:0] - max */
 #define DRV_MSG_CODE_SET_BW                    0x00190000
 #define BW_MAX_MASK                            0x000000ff
@@ -1201,16 +1193,12 @@ struct public_drv_mb {
 #define DRV_MSG_CODE_MASK_PARITIES             0x001a0000
 /* param[0] - Simulate fan failure,  param[1] - simulate over temp. */
 #define DRV_MSG_CODE_INDUCE_FAILURE            0x001b0000
-#define DRV_MSG_FAN_FAILURE_TYPE               (1 << 0)
-#define DRV_MSG_TEMPERATURE_FAILURE_TYPE       (1 << 1)
-
+       #define DRV_MSG_FAN_FAILURE_TYPE                (1 << 0)
+       #define DRV_MSG_TEMPERATURE_FAILURE_TYPE        (1 << 1)
 /* Param: [0:15] - gpio number */
 #define DRV_MSG_CODE_GPIO_READ                 0x001c0000
 /* Param: [0:15] - gpio number, [16:31] - gpio value */
 #define DRV_MSG_CODE_GPIO_WRITE                        0x001d0000
-/* Param: [0:15] - gpio number */
-#define DRV_MSG_CODE_GPIO_INFO             0x00270000
-
 /* Param: [0:7] - test enum, [8:15] - image index, [16:31] - reserved */
 #define DRV_MSG_CODE_BIST_TEST                 0x001e0000
 #define DRV_MSG_CODE_GET_TEMPERATURE            0x001f0000
@@ -1227,55 +1215,53 @@ struct public_drv_mb {
  * param[15:8] - age
  */
 #define DRV_MSG_CODE_RESOURCE_CMD              0x00230000
-
-/* request resource ownership with default aging */
-#define RESOURCE_OPCODE_REQ                    1
-/* request resource ownership without aging */
-#define RESOURCE_OPCODE_REQ_WO_AGING           2
-/* request resource ownership with specific aging timer (in seconds) */
-#define RESOURCE_OPCODE_REQ_W_AGING            3
-#define RESOURCE_OPCODE_RELEASE                        4 /* release resource */
-#define RESOURCE_OPCODE_FORCE_RELEASE          5 /* force resource release */
-
-/* resource is free and granted to requester */
-#define RESOURCE_OPCODE_GNT                    1
-/* resource is busy, param[7:0] indicates owner as follow 0-15 = PF0-15,
- * 16 = MFW, 17 = diag over serial
- */
-#define RESOURCE_OPCODE_BUSY                   2
-/* indicate release request was acknowledged */
-#define RESOURCE_OPCODE_RELEASED               3
-/* indicate release request was previously received by other owner */
-#define RESOURCE_OPCODE_RELEASED_PREVIOUS      4
-/* indicate wrong owner during release */
-#define RESOURCE_OPCODE_WRONG_OWNER            5
-#define RESOURCE_OPCODE_UNKNOWN_CMD            255
-/* dedicate resource 0 for dump */
-#define RESOURCE_DUMP                          (1 << 0)
-
+       /* request resource ownership with default aging */
+       #define RESOURCE_OPCODE_REQ                     1
+       /* request resource ownership without aging */
+       #define RESOURCE_OPCODE_REQ_WO_AGING            2
+       /* request resource ownership with specific aging timer (in seconds) */
+       #define RESOURCE_OPCODE_REQ_W_AGING             3
+       #define RESOURCE_OPCODE_RELEASE                 4 /* release resource */
+       /* force resource release */
+       #define RESOURCE_OPCODE_FORCE_RELEASE           5
+       /* resource is free and granted to requester */
+       #define RESOURCE_OPCODE_GNT                     1
+       /* resource is busy, param[7:0] indicates owner as follow 0-15 = PF0-15,
+        * 16 = MFW, 17 = diag over serial
+        */
+       #define RESOURCE_OPCODE_BUSY                    2
+       /* indicate release request was acknowledged */
+       #define RESOURCE_OPCODE_RELEASED                3
+       /* indicate release request was previously received by other owner */
+       #define RESOURCE_OPCODE_RELEASED_PREVIOUS       4
+       /* indicate wrong owner during release */
+       #define RESOURCE_OPCODE_WRONG_OWNER             5
+       #define RESOURCE_OPCODE_UNKNOWN_CMD             255
+       /* dedicate resource 0 for dump */
+       #define RESOURCE_DUMP                           (1 << 0)
 #define DRV_MSG_CODE_GET_MBA_VERSION           0x00240000 /* Get MBA version */
-
 /* Send crash dump commands with param[3:0] - opcode */
 #define DRV_MSG_CODE_MDUMP_CMD                 0x00250000
-#define MDUMP_DRV_PARAM_OPCODE_MASK            0x0000000f
-/* acknowledge reception of error indication */
-#define DRV_MSG_CODE_MDUMP_ACK                 0x01
-/* set epoc and personality as follow: drv_data[3:0] - epoch,
- * drv_data[7:4] - personality
- */
-#define DRV_MSG_CODE_MDUMP_SET_VALUES          0x02
-/* trigger crash dump procedure */
-#define DRV_MSG_CODE_MDUMP_TRIGGER             0x03
-/* Request valid logs and config words */
-#define DRV_MSG_CODE_MDUMP_GET_CONFIG          0x04
-/* Set triggers mask. drv_mb_param should indicate (bitwise) which trigger
- * enabled
- */
-#define DRV_MSG_CODE_MDUMP_SET_ENABLE          0x05
-#define DRV_MSG_CODE_MDUMP_CLEAR_LOGS          0x06 /* Clear all logs */
-
-
+       #define MDUMP_DRV_PARAM_OPCODE_MASK             0x0000000f
+       /* acknowledge reception of error indication */
+       #define DRV_MSG_CODE_MDUMP_ACK                  0x01
+       /* set epoc and personality as follow: drv_data[3:0] - epoch,
+        * drv_data[7:4] - personality
+        */
+       #define DRV_MSG_CODE_MDUMP_SET_VALUES           0x02
+       /* trigger crash dump procedure */
+       #define DRV_MSG_CODE_MDUMP_TRIGGER              0x03
+       /* Request valid logs and config words */
+       #define DRV_MSG_CODE_MDUMP_GET_CONFIG           0x04
+       /* Set triggers mask. drv_mb_param should indicate (bitwise) which
+        * trigger enabled
+        */
+       #define DRV_MSG_CODE_MDUMP_SET_ENABLE           0x05
+       /* Clear all logs */
+       #define DRV_MSG_CODE_MDUMP_CLEAR_LOGS           0x06
 #define DRV_MSG_CODE_MEM_ECC_EVENTS            0x00260000 /* Param: None */
+/* Param: [0:15] - gpio number */
+#define DRV_MSG_CODE_GPIO_INFO                 0x00270000
 /* Value will be placed in union */
 #define DRV_MSG_CODE_EXT_PHY_READ              0x00280000
 /* Value should be placed in union */
@@ -1502,22 +1488,22 @@ struct public_drv_mb {
 #define FW_MSG_MODE_PHY_PRIVILEGE_ERROR                0x00150000
 #define FW_MSG_CODE_OK                         0x00160000
 #define FW_MSG_CODE_LED_MODE_INVALID           0x00170000
-#define FW_MSG_CODE_PHY_DIAG_OK           0x00160000
-#define FW_MSG_CODE_PHY_DIAG_ERROR        0x00170000
+#define FW_MSG_CODE_PHY_DIAG_OK                        0x00160000
+#define FW_MSG_CODE_PHY_DIAG_ERROR             0x00170000
 #define FW_MSG_CODE_INIT_HW_FAILED_TO_ALLOCATE_PAGE    0x00040000
 #define FW_MSG_CODE_INIT_HW_FAILED_BAD_STATE    0x00170000
 #define FW_MSG_CODE_INIT_HW_FAILED_TO_SET_WINDOW 0x000d0000
 #define FW_MSG_CODE_INIT_HW_FAILED_NO_IMAGE    0x000c0000
 #define FW_MSG_CODE_INIT_HW_FAILED_VERSION_MISMATCH    0x00100000
-#define FW_MSG_CODE_TRANSCEIVER_DIAG_OK           0x00160000
-#define FW_MSG_CODE_TRANSCEIVER_DIAG_ERROR        0x00170000
+#define FW_MSG_CODE_TRANSCEIVER_DIAG_OK                        0x00160000
+#define FW_MSG_CODE_TRANSCEIVER_DIAG_ERROR             0x00170000
 #define FW_MSG_CODE_TRANSCEIVER_NOT_PRESENT            0x00020000
-#define FW_MSG_CODE_TRANSCEIVER_BAD_BUFFER_SIZE        0x000f0000
-#define FW_MSG_CODE_GPIO_OK           0x00160000
-#define FW_MSG_CODE_GPIO_DIRECTION_ERR        0x00170000
+#define FW_MSG_CODE_TRANSCEIVER_BAD_BUFFER_SIZE                0x000f0000
+#define FW_MSG_CODE_GPIO_OK                    0x00160000
+#define FW_MSG_CODE_GPIO_DIRECTION_ERR         0x00170000
 #define FW_MSG_CODE_GPIO_CTRL_ERR              0x00020000
 #define FW_MSG_CODE_GPIO_INVALID               0x000f0000
-#define FW_MSG_CODE_GPIO_INVALID_VALUE 0x00050000
+#define FW_MSG_CODE_GPIO_INVALID_VALUE         0x00050000
 #define FW_MSG_CODE_BIST_TEST_INVALID          0x000f0000
 #define FW_MSG_CODE_EXTPHY_INVALID_IMAGE_HEADER        0x00700000
 #define FW_MSG_CODE_EXTPHY_INVALID_PHY_TYPE    0x00710000
-- 
2.11.0.rc1

Reply via email to