#define FM10K_MBX_STATS_LEN ARRAY_SIZE(fm10k_gstrings_mbx_stats)
 
 #define FM10K_QUEUE_STATS_LEN(_n) \
-       ( (_n) * 2 * (sizeof(struct fm10k_queue_stats) / sizeof(u64)))
+       ((_n) * 2 * (sizeof(struct fm10k_queue_stats) / sizeof(u64)))
 
 #define FM10K_STATIC_STATS_LEN (FM10K_GLOBAL_STATS_LEN + \
                                FM10K_NETDEV_STATS_LEN + \
        if ((interface->flags & FM10K_FLAG_DEBUG_STATS) && iov_data) {
                for (i = 0; i < iov_data->num_vfs; i++) {
                        struct fm10k_vf_info *vf_info;
+
                        vf_info = &iov_data->vf_info[i];
 
                        /* skip stats if we don't have a vf info */
        return 0;
 }
 
-
 static u32 fm10k_get_reta_size(struct net_device __always_unused *netdev)
 {
        return FM10K_RETA_SIZE * FM10K_RETA_ENTRIES_PER_REG;
 
        /* set timestamping bits */
        if (unlikely(skb_shinfo(skb)->tx_flags & SKBTX_HW_TSTAMP) &&
            likely(skb_shinfo(skb)->tx_flags & SKBTX_IN_PROGRESS))
-                       desc_flags |= FM10K_TXD_FLAG_TIME;
+               desc_flags |= FM10K_TXD_FLAG_TIME;
 
        /* set checksum offload bits */
        desc_flags |= FM10K_SET_FLAG(tx_flags, FM10K_TX_FLAGS_CSUM,
         * allow the budget to go below 1 because we'll exit polling
         */
        if (q_vector->rx.count > 1)
-               per_ring_budget = max(budget/q_vector->rx.count, 1);
+               per_ring_budget = max(budget / q_vector->rx.count, 1);
        else
                per_ring_budget = budget;
 
 
 {
        mbx->mbx_reg = FM10K_GMBX;
        mbx->mbmem_reg = FM10K_MBMEM_PF(0);
+
        /* start out in closed state */
        mbx->state = FM10K_STATE_CLOSED;
 
 
        unsigned int r_idx = skb->queue_mapping;
        int err;
 
-       if ((skb->protocol ==  htons(ETH_P_8021Q)) &&
+       if ((skb->protocol == htons(ETH_P_8021Q)) &&
            !skb_vlan_tag_present(skb)) {
                /* FM10K only supports hardware tagging, any tags in frame
                 * are considered 2nd level or "outer" tags
        } else {
                netif_info(interface, drv, netdev,
                           "Fake Tx hang detected with timeout of %d seconds\n",
-                          netdev->watchdog_timeo/HZ);
+                          netdev->watchdog_timeo / HZ);
 
                /* fake Tx hang - increase the kernel timeout */
                if (netdev->watchdog_timeo < TX_TIMEO_LIMIT)
 
 #endif
 #define FM10K_ERR_MSG(type) case (type): error = #type; break
 static void fm10k_handle_fault(struct fm10k_intfc *interface, int type,
-                             struct fm10k_fault *fault)
+                              struct fm10k_fault *fault)
 {
        struct pci_dev *pdev = interface->pdev;
        struct fm10k_hw *hw = &interface->hw;
  * The OS initialization, configuring of the interface private structure,
  * and a hardware reset occur.
  **/
-static int fm10k_probe(struct pci_dev *pdev,
-                      const struct pci_device_id *ent)
+static int fm10k_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
 {
        struct net_device *netdev;
        struct fm10k_intfc *interface;
 
 
        if (mode > FM10K_XCAST_MODE_NONE)
                return FM10K_ERR_PARAM;
+
        /* if glort is not valid return error */
        if (!fm10k_glort_valid_pf(hw, glort))
                return FM10K_ERR_PARAM;
 
 
        if (mode > FM10K_XCAST_MODE_NONE)
                return FM10K_ERR_PARAM;
+
        /* generate message requesting to change xcast mode */
        fm10k_tlv_msg_init(msg, FM10K_VF_MSG_ID_LPORT_STATE);
        fm10k_tlv_attr_put_u8(msg, FM10K_LPORT_STATE_MSG_XCAST_MODE, mode);