return 0;
 }
 
+static int qed_nvm_get_image(struct qed_dev *cdev, enum qed_nvm_images type,
+                            u8 *buf, u16 len)
+{
+       struct qed_hwfn *hwfn = QED_LEADING_HWFN(cdev);
+       struct qed_ptt *ptt = qed_ptt_acquire(hwfn);
+       int rc;
+
+       if (!ptt)
+               return -EAGAIN;
+
+       rc = qed_mcp_get_nvm_image(hwfn, ptt, type, buf, len);
+       qed_ptt_release(hwfn, ptt);
+       return rc;
+}
+
 static void qed_get_coalesce(struct qed_dev *cdev, u16 *rx_coal, u16 *tx_coal)
 {
        *rx_coal = cdev->rx_coalesce_usecs;
        .dbg_all_data_size = &qed_dbg_all_data_size,
        .chain_alloc = &qed_chain_alloc,
        .chain_free = &qed_chain_free,
+       .nvm_get_image = &qed_nvm_get_image,
        .get_coalesce = &qed_get_coalesce,
        .set_coalesce = &qed_set_coalesce,
        .set_led = &qed_set_led,
 
        return rc;
 }
 
+static int
+qed_mcp_get_nvm_image_att(struct qed_hwfn *p_hwfn,
+                         struct qed_ptt *p_ptt,
+                         enum qed_nvm_images image_id,
+                         struct qed_nvm_image_att *p_image_att)
+{
+       struct bist_nvm_image_att mfw_image_att;
+       enum nvm_image_type type;
+       u32 num_images, i;
+       int rc;
+
+       /* Translate image_id into MFW definitions */
+       switch (image_id) {
+       case QED_NVM_IMAGE_ISCSI_CFG:
+               type = NVM_TYPE_ISCSI_CFG;
+               break;
+       case QED_NVM_IMAGE_FCOE_CFG:
+               type = NVM_TYPE_FCOE_CFG;
+               break;
+       default:
+               DP_NOTICE(p_hwfn, "Unknown request of image_id %08x\n",
+                         image_id);
+               return -EINVAL;
+       }
+
+       /* Learn number of images, then traverse and see if one fits */
+       rc = qed_mcp_bist_nvm_test_get_num_images(p_hwfn, p_ptt, &num_images);
+       if (rc || !num_images)
+               return -EINVAL;
+
+       for (i = 0; i < num_images; i++) {
+               rc = qed_mcp_bist_nvm_test_get_image_att(p_hwfn, p_ptt,
+                                                        &mfw_image_att, i);
+               if (rc)
+                       return rc;
+
+               if (type == mfw_image_att.image_type)
+                       break;
+       }
+       if (i == num_images) {
+               DP_VERBOSE(p_hwfn, QED_MSG_STORAGE,
+                          "Failed to find nvram image of type %08x\n",
+                          image_id);
+               return -EINVAL;
+       }
+
+       p_image_att->start_addr = mfw_image_att.nvm_start_addr;
+       p_image_att->length = mfw_image_att.len;
+
+       return 0;
+}
+
+int qed_mcp_get_nvm_image(struct qed_hwfn *p_hwfn,
+                         struct qed_ptt *p_ptt,
+                         enum qed_nvm_images image_id,
+                         u8 *p_buffer, u32 buffer_len)
+{
+       struct qed_nvm_image_att image_att;
+       int rc;
+
+       memset(p_buffer, 0, buffer_len);
+
+       rc = qed_mcp_get_nvm_image_att(p_hwfn, p_ptt, image_id, &image_att);
+       if (rc)
+               return rc;
+
+       /* Validate sizes - both the image's and the supplied buffer's */
+       if (image_att.length <= 4) {
+               DP_VERBOSE(p_hwfn, QED_MSG_STORAGE,
+                          "Image [%d] is too small - only %d bytes\n",
+                          image_id, image_att.length);
+               return -EINVAL;
+       }
+
+       /* Each NVM image is suffixed by CRC; Upper-layer has no need for it */
+       image_att.length -= 4;
+
+       if (image_att.length > buffer_len) {
+               DP_VERBOSE(p_hwfn,
+                          QED_MSG_STORAGE,
+                          "Image [%d] is too big - %08x bytes where only %08x are available\n",
+                          image_id, image_att.length, buffer_len);
+               return -ENOMEM;
+       }
+
+       return qed_mcp_nvm_read(p_hwfn->cdev, image_att.start_addr,
+                               p_buffer, image_att.length);
+}
+
 static enum resource_id_enum qed_mcp_get_mfw_res_id(enum qed_resources res_id)
 {
        enum resource_id_enum mfw_res_id = RESOURCE_NUM_INVALID;
 
  */
 int qed_mcp_nvm_read(struct qed_dev *cdev, u32 addr, u8 *p_buf, u32 len);
 
+struct qed_nvm_image_att {
+       u32 start_addr;
+       u32 length;
+};
+
+/**
+ * @brief Allows reading a whole nvram image
+ *
+ * @param p_hwfn
+ * @param p_ptt
+ * @param image_id - image requested for reading
+ * @param p_buffer - allocated buffer into which to fill data
+ * @param buffer_len - length of the allocated buffer.
+ *
+ * @return 0 iff p_buffer now contains the nvram image.
+ */
+int qed_mcp_get_nvm_image(struct qed_hwfn *p_hwfn,
+                         struct qed_ptt *p_ptt,
+                         enum qed_nvm_images image_id,
+                         u8 *p_buffer, u32 buffer_len);
+
 /**
  * @brief Bist register test
  *
 
        struct qed_dcbx_admin_params local;
 };
 
+enum qed_nvm_images {
+       QED_NVM_IMAGE_ISCSI_CFG,
+       QED_NVM_IMAGE_FCOE_CFG,
+};
+
 enum qed_led_mode {
        QED_LED_MODE_OFF,
        QED_LED_MODE_ON,
        void            (*chain_free)(struct qed_dev *cdev,
                                      struct qed_chain *p_chain);
 
+/**
+ * @brief nvm_get_image - reads an entire image from nvram
+ *
+ * @param cdev
+ * @param type - type of the request nvram image
+ * @param buf - preallocated buffer to fill with the image
+ * @param len - length of the allocated buffer
+ *
+ * @return 0 on success, error otherwise
+ */
+       int (*nvm_get_image)(struct qed_dev *cdev,
+                            enum qed_nvm_images type, u8 *buf, u16 len);
+
 /**
  * @brief get_coalesce - Get coalesce parameters in usec
  *