net/ice/base: add NVM helper functions
[dpdk.git] / drivers / net / ice / base / ice_nvm.c
index e009425..bedfbcb 100644 (file)
@@ -1,5 +1,5 @@
 /* SPDX-License-Identifier: BSD-3-Clause
- * Copyright(c) 2001-2019
+ * Copyright(c) 2001-2020 Intel Corporation
  */
 
 #include "ice_common.h"
@@ -17,7 +17,7 @@
  *
  * Read the NVM using the admin queue commands (0x0701)
  */
-static enum ice_status
+enum ice_status
 ice_aq_read_nvm(struct ice_hw *hw, u16 module_typeid, u32 offset, u16 length,
                void *data, bool last_command, bool read_shadow_ram,
                struct ice_sq_cd *cd)
@@ -29,8 +29,7 @@ ice_aq_read_nvm(struct ice_hw *hw, u16 module_typeid, u32 offset, u16 length,
 
        cmd = &desc.params.nvm;
 
-       /* In offset the highest byte must be zeroed. */
-       if (offset & 0xFF000000)
+       if (offset > ICE_AQC_NVM_MAX_OFFSET)
                return ICE_ERR_PARAM;
 
        ice_fill_dflt_direct_cmd_desc(&desc, ice_aqc_opc_nvm_read);
@@ -50,68 +49,70 @@ ice_aq_read_nvm(struct ice_hw *hw, u16 module_typeid, u32 offset, u16 length,
 }
 
 /**
- * ice_check_sr_access_params - verify params for Shadow RAM R/W operations.
- * @hw: pointer to the HW structure
- * @offset: offset in words from module start
- * @words: number of words to access
+ * ice_read_flat_nvm - Read portion of NVM by flat offset
+ * @hw: pointer to the HW struct
+ * @offset: offset from beginning of NVM
+ * @length: (in) number of bytes to read; (out) number of bytes actually read
+ * @data: buffer to return data in (sized to fit the specified length)
+ * @read_shadow_ram: if true, read from shadow RAM instead of NVM
+ *
+ * Reads a portion of the NVM, as a flat memory space. This function correctly
+ * breaks read requests across Shadow RAM sectors and ensures that no single
+ * read request exceeds the maximum 4Kb read for a single AdminQ command.
+ *
+ * Returns a status code on failure. Note that the data pointer may be
+ * partially updated if some reads succeed before a failure.
  */
-static enum ice_status
-ice_check_sr_access_params(struct ice_hw *hw, u32 offset, u16 words)
+enum ice_status
+ice_read_flat_nvm(struct ice_hw *hw, u32 offset, u32 *length, u8 *data,
+                 bool read_shadow_ram)
 {
-       if ((offset + words) > hw->nvm.sr_words) {
-               ice_debug(hw, ICE_DBG_NVM,
-                         "NVM error: offset beyond SR lmt.\n");
-               return ICE_ERR_PARAM;
-       }
+       enum ice_status status;
+       u32 inlen = *length;
+       u32 bytes_read = 0;
+       bool last_cmd;
 
-       if (words > ICE_SR_SECTOR_SIZE_IN_WORDS) {
-               /* We can access only up to 4KB (one sector), in one AQ write */
-               ice_debug(hw, ICE_DBG_NVM,
-                         "NVM error: tried to access %d words, limit is %d.\n",
-                         words, ICE_SR_SECTOR_SIZE_IN_WORDS);
-               return ICE_ERR_PARAM;
-       }
+       ice_debug(hw, ICE_DBG_TRACE, "%s\n", __func__);
+
+       *length = 0;
 
-       if (((offset + (words - 1)) / ICE_SR_SECTOR_SIZE_IN_WORDS) !=
-           (offset / ICE_SR_SECTOR_SIZE_IN_WORDS)) {
-               /* A single access cannot spread over two sectors */
+       /* Verify the length of the read if this is for the Shadow RAM */
+       if (read_shadow_ram && ((offset + inlen) > (hw->nvm.sr_words * 2u))) {
                ice_debug(hw, ICE_DBG_NVM,
-                         "NVM error: cannot spread over two sectors.\n");
+                         "NVM error: requested data is beyond Shadow RAM limit\n");
                return ICE_ERR_PARAM;
        }
 
-       return ICE_SUCCESS;
-}
-
-/**
- * ice_read_sr_aq - Read Shadow RAM.
- * @hw: pointer to the HW structure
- * @offset: offset in words from module start
- * @words: number of words to read
- * @data: buffer for words reads from Shadow RAM
- * @last_command: tells the AdminQ that this is the last command
- *
- * Reads 16-bit word buffers from the Shadow RAM using the admin command.
- */
-static enum ice_status
-ice_read_sr_aq(struct ice_hw *hw, u32 offset, u16 words, u16 *data,
-              bool last_command)
-{
-       enum ice_status status;
+       do {
+               u32 read_size, sector_offset;
 
-       ice_debug(hw, ICE_DBG_TRACE, "%s\n", __func__);
+               /* ice_aq_read_nvm cannot read more than 4Kb at a time.
+                * Additionally, a read from the Shadow RAM may not cross over
+                * a sector boundary. Conveniently, the sector size is also
+                * 4Kb.
+                */
+               sector_offset = offset % ICE_AQ_MAX_BUF_LEN;
+               read_size = MIN_T(u32, ICE_AQ_MAX_BUF_LEN - sector_offset,
+                                 inlen - bytes_read);
 
-       status = ice_check_sr_access_params(hw, offset, words);
+               last_cmd = !(bytes_read + read_size < inlen);
 
-       /* values in "offset" and "words" parameters are sized as words
-        * (16 bits) but ice_aq_read_nvm expects these values in bytes.
-        * So do this conversion while calling ice_aq_read_nvm.
-        */
-       if (!status)
+               /* ice_aq_read_nvm takes the length as a u16. Our read_size is
+                * calculated using a u32, but the ICE_AQ_MAX_BUF_LEN maximum
+                * size guarantees that it will fit within the 2 bytes.
+                */
                status = ice_aq_read_nvm(hw, ICE_AQC_NVM_START_POINT,
-                                        2 * offset, 2 * words, data,
-                                        last_command, true, NULL);
+                                        offset, (u16)read_size,
+                                        data + bytes_read, last_cmd,
+                                        read_shadow_ram, NULL);
+               if (status)
+                       break;
 
+               bytes_read += read_size;
+               offset += read_size;
+       } while (!last_cmd);
+
+       *length = bytes_read;
        return status;
 }
 
@@ -121,20 +122,28 @@ ice_read_sr_aq(struct ice_hw *hw, u32 offset, u16 words, u16 *data,
  * @offset: offset of the Shadow RAM word to read (0x000000 - 0x001FFF)
  * @data: word read from the Shadow RAM
  *
- * Reads one 16 bit word from the Shadow RAM using the ice_read_sr_aq method.
+ * Reads one 16 bit word from the Shadow RAM using ice_read_flat_nvm.
  */
 static enum ice_status
 ice_read_sr_word_aq(struct ice_hw *hw, u16 offset, u16 *data)
 {
+       u32 bytes = sizeof(u16);
        enum ice_status status;
+       __le16 data_local;
 
        ice_debug(hw, ICE_DBG_TRACE, "%s\n", __func__);
 
-       status = ice_read_sr_aq(hw, offset, 1, data, true);
-       if (!status)
-               *data = LE16_TO_CPU(*(_FORCE_ __le16 *)data);
+       /* Note that ice_read_flat_nvm checks if the read is past the Shadow
+        * RAM size, and ensures we don't read across a Shadow RAM sector
+        * boundary
+        */
+       status = ice_read_flat_nvm(hw, offset * sizeof(u16), &bytes,
+                                  (u8 *)&data_local, true);
+       if (status)
+               return status;
 
-       return status;
+       *data = LE16_TO_CPU(data_local);
+       return ICE_SUCCESS;
 }
 
 /**
@@ -144,55 +153,29 @@ ice_read_sr_word_aq(struct ice_hw *hw, u16 offset, u16 *data)
  * @words: (in) number of words to read; (out) number of words actually read
  * @data: words read from the Shadow RAM
  *
- * Reads 16 bit words (data buf) from the SR using the ice_read_sr_aq
- * method. Ownership of the NVM is taken before reading the buffer and later
- * released.
+ * Reads 16 bit words (data buf) from the Shadow RAM. Ownership of the NVM is
+ * taken before reading the buffer and later released.
  */
 static enum ice_status
 ice_read_sr_buf_aq(struct ice_hw *hw, u16 offset, u16 *words, u16 *data)
 {
+       u32 bytes = *words * 2, i;
        enum ice_status status;
-       bool last_cmd = false;
-       u16 words_read = 0;
-       u16 i = 0;
 
        ice_debug(hw, ICE_DBG_TRACE, "%s\n", __func__);
 
-       do {
-               u16 read_size, off_w;
-
-               /* Calculate number of bytes we should read in this step.
-                * It's not allowed to read more than one page at a time or
-                * to cross page boundaries.
-                */
-               off_w = offset % ICE_SR_SECTOR_SIZE_IN_WORDS;
-               read_size = off_w ?
-                       MIN_T(u16, *words,
-                             (ICE_SR_SECTOR_SIZE_IN_WORDS - off_w)) :
-                       MIN_T(u16, (*words - words_read),
-                             ICE_SR_SECTOR_SIZE_IN_WORDS);
-
-               /* Check if this is last command, if so set proper flag */
-               if ((words_read + read_size) >= *words)
-                       last_cmd = true;
-
-               status = ice_read_sr_aq(hw, offset, read_size,
-                                       data + words_read, last_cmd);
-               if (status)
-                       goto read_nvm_buf_aq_exit;
+       /* ice_read_flat_nvm takes into account the 4Kb AdminQ and Shadow RAM
+        * sector restrictions necessary when reading from the NVM.
+        */
+       status = ice_read_flat_nvm(hw, offset * 2, &bytes, (u8 *)data, true);
 
-               /* Increment counter for words already read and move offset to
-                * new read location
-                */
-               words_read += read_size;
-               offset += read_size;
-       } while (words_read < *words);
+       /* Report the number of words successfully read */
+       *words = bytes / 2;
 
+       /* Byte swap the words up to the amount we actually read */
        for (i = 0; i < *words; i++)
                data[i] = LE16_TO_CPU(((_FORCE_ __le16 *)data)[i]);
 
-read_nvm_buf_aq_exit:
-       *words = words_read;
        return status;
 }
 
@@ -203,7 +186,7 @@ read_nvm_buf_aq_exit:
  *
  * This function will request NVM ownership.
  */
-static enum ice_status
+enum ice_status
 ice_acquire_nvm(struct ice_hw *hw, enum ice_aq_res_access_type access)
 {
        ice_debug(hw, ICE_DBG_TRACE, "%s\n", __func__);
@@ -220,7 +203,7 @@ ice_acquire_nvm(struct ice_hw *hw, enum ice_aq_res_access_type access)
  *
  * This function will release NVM ownership.
  */
-static void ice_release_nvm(struct ice_hw *hw)
+void ice_release_nvm(struct ice_hw *hw)
 {
        ice_debug(hw, ICE_DBG_TRACE, "%s\n", __func__);
 
@@ -251,6 +234,243 @@ enum ice_status ice_read_sr_word(struct ice_hw *hw, u16 offset, u16 *data)
        return status;
 }
 
+/**
+ * ice_get_pfa_module_tlv - Reads sub module TLV from NVM PFA
+ * @hw: pointer to hardware structure
+ * @module_tlv: pointer to module TLV to return
+ * @module_tlv_len: pointer to module TLV length to return
+ * @module_type: module type requested
+ *
+ * Finds the requested sub module TLV type from the Preserved Field
+ * Area (PFA) and returns the TLV pointer and length. The caller can
+ * use these to read the variable length TLV value.
+ */
+enum ice_status
+ice_get_pfa_module_tlv(struct ice_hw *hw, u16 *module_tlv, u16 *module_tlv_len,
+                      u16 module_type)
+{
+       enum ice_status status;
+       u16 pfa_len, pfa_ptr;
+       u16 next_tlv;
+
+       status = ice_read_sr_word(hw, ICE_SR_PFA_PTR, &pfa_ptr);
+       if (status != ICE_SUCCESS) {
+               ice_debug(hw, ICE_DBG_INIT, "Preserved Field Array pointer.\n");
+               return status;
+       }
+       status = ice_read_sr_word(hw, pfa_ptr, &pfa_len);
+       if (status != ICE_SUCCESS) {
+               ice_debug(hw, ICE_DBG_INIT, "Failed to read PFA length.\n");
+               return status;
+       }
+       /* Starting with first TLV after PFA length, iterate through the list
+        * of TLVs to find the requested one.
+        */
+       next_tlv = pfa_ptr + 1;
+       while (next_tlv < pfa_ptr + pfa_len) {
+               u16 tlv_sub_module_type;
+               u16 tlv_len;
+
+               /* Read TLV type */
+               status = ice_read_sr_word(hw, next_tlv, &tlv_sub_module_type);
+               if (status != ICE_SUCCESS) {
+                       ice_debug(hw, ICE_DBG_INIT, "Failed to read TLV type.\n");
+                       break;
+               }
+               /* Read TLV length */
+               status = ice_read_sr_word(hw, next_tlv + 1, &tlv_len);
+               if (status != ICE_SUCCESS) {
+                       ice_debug(hw, ICE_DBG_INIT, "Failed to read TLV length.\n");
+                       break;
+               }
+               if (tlv_sub_module_type == module_type) {
+                       if (tlv_len) {
+                               *module_tlv = next_tlv;
+                               *module_tlv_len = tlv_len;
+                               return ICE_SUCCESS;
+                       }
+                       return ICE_ERR_INVAL_SIZE;
+               }
+               /* Check next TLV, i.e. current TLV pointer + length + 2 words
+                * (for current TLV's type and length)
+                */
+               next_tlv = next_tlv + tlv_len + 2;
+       }
+       /* Module does not exist */
+       return ICE_ERR_DOES_NOT_EXIST;
+}
+
+/**
+ * ice_read_pba_string - Reads part number string from NVM
+ * @hw: pointer to hardware structure
+ * @pba_num: stores the part number string from the NVM
+ * @pba_num_size: part number string buffer length
+ *
+ * Reads the part number string from the NVM.
+ */
+enum ice_status
+ice_read_pba_string(struct ice_hw *hw, u8 *pba_num, u32 pba_num_size)
+{
+       u16 pba_tlv, pba_tlv_len;
+       enum ice_status status;
+       u16 pba_word, pba_size;
+       u16 i;
+
+       status = ice_get_pfa_module_tlv(hw, &pba_tlv, &pba_tlv_len,
+                                       ICE_SR_PBA_BLOCK_PTR);
+       if (status != ICE_SUCCESS) {
+               ice_debug(hw, ICE_DBG_INIT, "Failed to read PBA Block TLV.\n");
+               return status;
+       }
+
+       /* pba_size is the next word */
+       status = ice_read_sr_word(hw, (pba_tlv + 2), &pba_size);
+       if (status != ICE_SUCCESS) {
+               ice_debug(hw, ICE_DBG_INIT, "Failed to read PBA Section size.\n");
+               return status;
+       }
+
+       if (pba_tlv_len < pba_size) {
+               ice_debug(hw, ICE_DBG_INIT, "Invalid PBA Block TLV size.\n");
+               return ICE_ERR_INVAL_SIZE;
+       }
+
+       /* Subtract one to get PBA word count (PBA Size word is included in
+        * total size)
+        */
+       pba_size--;
+       if (pba_num_size < (((u32)pba_size * 2) + 1)) {
+               ice_debug(hw, ICE_DBG_INIT,
+                         "Buffer too small for PBA data.\n");
+               return ICE_ERR_PARAM;
+       }
+
+       for (i = 0; i < pba_size; i++) {
+               status = ice_read_sr_word(hw, (pba_tlv + 2 + 1) + i, &pba_word);
+               if (status != ICE_SUCCESS) {
+                       ice_debug(hw, ICE_DBG_INIT,
+                                 "Failed to read PBA Block word %d.\n", i);
+                       return status;
+               }
+
+               pba_num[(i * 2)] = (pba_word >> 8) & 0xFF;
+               pba_num[(i * 2) + 1] = pba_word & 0xFF;
+       }
+       pba_num[(pba_size * 2)] = '\0';
+
+       return status;
+}
+
+/**
+ * ice_get_orom_ver_info - Read Option ROM version information
+ * @hw: pointer to the HW struct
+ *
+ * Read the Combo Image version data from the Boot Configuration TLV and fill
+ * in the option ROM version data.
+ */
+static enum ice_status ice_get_orom_ver_info(struct ice_hw *hw)
+{
+       u16 combo_hi, combo_lo, boot_cfg_tlv, boot_cfg_tlv_len;
+       struct ice_orom_info *orom = &hw->nvm.orom;
+       enum ice_status status;
+       u32 combo_ver;
+
+       status = ice_get_pfa_module_tlv(hw, &boot_cfg_tlv, &boot_cfg_tlv_len,
+                                       ICE_SR_BOOT_CFG_PTR);
+       if (status) {
+               ice_debug(hw, ICE_DBG_INIT,
+                         "Failed to read Boot Configuration Block TLV.\n");
+               return status;
+       }
+
+       /* Boot Configuration Block must have length at least 2 words
+        * (Combo Image Version High and Combo Image Version Low)
+        */
+       if (boot_cfg_tlv_len < 2) {
+               ice_debug(hw, ICE_DBG_INIT,
+                         "Invalid Boot Configuration Block TLV size.\n");
+               return ICE_ERR_INVAL_SIZE;
+       }
+
+       status = ice_read_sr_word(hw, (boot_cfg_tlv + ICE_NVM_OROM_VER_OFF),
+                                 &combo_hi);
+       if (status) {
+               ice_debug(hw, ICE_DBG_INIT, "Failed to read OROM_VER hi.\n");
+               return status;
+       }
+
+       status = ice_read_sr_word(hw, (boot_cfg_tlv + ICE_NVM_OROM_VER_OFF + 1),
+                                 &combo_lo);
+       if (status) {
+               ice_debug(hw, ICE_DBG_INIT, "Failed to read OROM_VER lo.\n");
+               return status;
+       }
+
+       combo_ver = ((u32)combo_hi << 16) | combo_lo;
+
+       orom->major = (u8)((combo_ver & ICE_OROM_VER_MASK) >>
+                          ICE_OROM_VER_SHIFT);
+       orom->patch = (u8)(combo_ver & ICE_OROM_VER_PATCH_MASK);
+       orom->build = (u16)((combo_ver & ICE_OROM_VER_BUILD_MASK) >>
+                           ICE_OROM_VER_BUILD_SHIFT);
+
+       return ICE_SUCCESS;
+}
+
+/**
+ * ice_discover_flash_size - Discover the available flash size.
+ * @hw: pointer to the HW struct
+ *
+ * The device flash could be up to 16MB in size. However, it is possible that
+ * the actual size is smaller. Use bisection to determine the accessible size
+ * of flash memory.
+ */
+static enum ice_status ice_discover_flash_size(struct ice_hw *hw)
+{
+       u32 min_size = 0, max_size = ICE_AQC_NVM_MAX_OFFSET + 1;
+       enum ice_status status;
+
+       ice_debug(hw, ICE_DBG_TRACE, "%s\n", __func__);
+
+       status = ice_acquire_nvm(hw, ICE_RES_READ);
+       if (status)
+               return status;
+
+       while ((max_size - min_size) > 1) {
+               u32 offset = (max_size + min_size) / 2;
+               u32 len = 1;
+               u8 data;
+
+               status = ice_read_flat_nvm(hw, offset, &len, &data, false);
+               if (status == ICE_ERR_AQ_ERROR &&
+                   hw->adminq.sq_last_status == ICE_AQ_RC_EINVAL) {
+                       ice_debug(hw, ICE_DBG_NVM,
+                                 "%s: New upper bound of %u bytes\n",
+                                 __func__, offset);
+                       status = ICE_SUCCESS;
+                       max_size = offset;
+               } else if (!status) {
+                       ice_debug(hw, ICE_DBG_NVM,
+                                 "%s: New lower bound of %u bytes\n",
+                                 __func__, offset);
+                       min_size = offset;
+               } else {
+                       /* an unexpected error occurred */
+                       goto err_read_flat_nvm;
+               }
+       }
+
+       ice_debug(hw, ICE_DBG_NVM,
+                 "Predicted flash size is %u bytes\n", max_size);
+
+       hw->nvm.flash_size = max_size;
+
+err_read_flat_nvm:
+       ice_release_nvm(hw);
+
+       return status;
+}
+
 /**
  * ice_init_nvm - initializes NVM setting
  * @hw: pointer to the HW struct
@@ -261,8 +481,7 @@ enum ice_status ice_read_sr_word(struct ice_hw *hw, u16 offset, u16 *data)
 enum ice_status ice_init_nvm(struct ice_hw *hw)
 {
        struct ice_nvm_info *nvm = &hw->nvm;
-       u16 oem_hi, oem_lo, boot_cfg_tlv, boot_cfg_tlv_len;
-       u16 eetrack_lo, eetrack_hi;
+       u16 eetrack_lo, eetrack_hi, ver;
        enum ice_status status;
        u32 fla, gens_stat;
        u8 sr_size;
@@ -290,12 +509,14 @@ enum ice_status ice_init_nvm(struct ice_hw *hw)
                return ICE_ERR_NVM_BLANK_MODE;
        }
 
-       status = ice_read_sr_word(hw, ICE_SR_NVM_DEV_STARTER_VER, &nvm->ver);
+       status = ice_read_sr_word(hw, ICE_SR_NVM_DEV_STARTER_VER, &ver);
        if (status) {
                ice_debug(hw, ICE_DBG_INIT,
                          "Failed to read DEV starter version.\n");
                return status;
        }
+       nvm->major_ver = (ver & ICE_NVM_VER_HI_MASK) >> ICE_NVM_VER_HI_SHIFT;
+       nvm->minor_ver = (ver & ICE_NVM_VER_LO_MASK) >> ICE_NVM_VER_LO_SHIFT;
 
        status = ice_read_sr_word(hw, ICE_SR_NVM_EETRACK_LO, &eetrack_lo);
        if (status) {
@@ -310,45 +531,35 @@ enum ice_status ice_init_nvm(struct ice_hw *hw)
 
        nvm->eetrack = (eetrack_hi << 16) | eetrack_lo;
 
-       /* the following devices do not have boot_cfg_tlv yet */
-       if (hw->device_id == ICE_DEV_ID_C822N_BACKPLANE ||
-           hw->device_id == ICE_DEV_ID_C822N_QSFP ||
-           hw->device_id == ICE_DEV_ID_C822N_SFP)
-               return status;
-
-       status = ice_get_pfa_module_tlv(hw, &boot_cfg_tlv, &boot_cfg_tlv_len,
-                                       ICE_SR_BOOT_CFG_PTR);
+       status = ice_discover_flash_size(hw);
        if (status) {
-               ice_debug(hw, ICE_DBG_INIT,
-                         "Failed to read Boot Configuration Block TLV.\n");
+               ice_debug(hw, ICE_DBG_NVM,
+                         "NVM init error: failed to discover flash size.\n");
                return status;
        }
 
-       /* Boot Configuration Block must have length at least 2 words
-        * (Combo Image Version High and Combo Image Version Low)
-        */
-       if (boot_cfg_tlv_len < 2) {
-               ice_debug(hw, ICE_DBG_INIT,
-                         "Invalid Boot Configuration Block TLV size.\n");
-               return ICE_ERR_INVAL_SIZE;
-       }
-
-       status = ice_read_sr_word(hw, (boot_cfg_tlv + ICE_NVM_OEM_VER_OFF),
-                                 &oem_hi);
-       if (status) {
-               ice_debug(hw, ICE_DBG_INIT, "Failed to read OEM_VER hi.\n");
+       switch (hw->device_id) {
+       /* the following devices do not have boot_cfg_tlv yet */
+       case ICE_DEV_ID_E822C_BACKPLANE:
+       case ICE_DEV_ID_E822C_QSFP:
+       case ICE_DEV_ID_E822C_10G_BASE_T:
+       case ICE_DEV_ID_E822C_SGMII:
+       case ICE_DEV_ID_E822C_SFP:
+       case ICE_DEV_ID_E822L_BACKPLANE:
+       case ICE_DEV_ID_E822L_SFP:
+       case ICE_DEV_ID_E822L_10G_BASE_T:
+       case ICE_DEV_ID_E822L_SGMII:
                return status;
+       default:
+               break;
        }
 
-       status = ice_read_sr_word(hw, (boot_cfg_tlv + ICE_NVM_OEM_VER_OFF + 1),
-                                 &oem_lo);
+       status = ice_get_orom_ver_info(hw);
        if (status) {
-               ice_debug(hw, ICE_DBG_INIT, "Failed to read OEM_VER lo.\n");
+               ice_debug(hw, ICE_DBG_INIT, "Failed to read Option ROM info.\n");
                return status;
        }
 
-       nvm->oem_ver = ((u32)oem_hi << 16) | oem_lo;
-
        return ICE_SUCCESS;
 }
 
@@ -504,6 +715,7 @@ ice_validate_nvm_rw_reg(struct ice_nvm_access_cmd *cmd)
        case GL_FWSTS:
        case GL_MNG_FWSM:
        case GLGEN_CSR_DEBUG_C:
+       case GLGEN_RSTAT:
        case GLPCI_LBARCTRL:
        case GLNVM_GENS:
        case GLNVM_FLA:
@@ -579,9 +791,14 @@ ice_nvm_access_write(struct ice_hw *hw, struct ice_nvm_access_cmd *cmd,
        if (status)
                return status;
 
-       /* The HICR_EN register is read-only */
-       if (cmd->offset == GL_HICR_EN)
+       /* Reject requests to write to read-only registers */
+       switch (cmd->offset) {
+       case GL_HICR_EN:
+       case GLGEN_RSTAT:
                return ICE_ERR_OUT_OF_RANGE;
+       default:
+               break;
+       }
 
        ice_debug(hw, ICE_DBG_NVM,
                  "NVM access: writing register %08x with value %08x\n",