accel/habanalabs: rename fw_{major/minor}_version to fw_inner_{major/minor}_ver
authorDafna Hirschfeld <dhirschfeld@habana.ai>
Sun, 2 Apr 2023 09:05:43 +0000 (12:05 +0300)
committerOded Gabbay <ogabbay@kernel.org>
Mon, 5 Jun 2023 12:31:33 +0000 (15:31 +0300)
We later want to add fields for Firmware SW version. The current
extracted FW version is the inner FW versioning so the new name
is better and also better differentiate from the FW's SW version.

Signed-off-by: Dafna Hirschfeld <dhirschfeld@habana.ai>
Reviewed-by: Oded Gabbay <ogabbay@kernel.org>
Signed-off-by: Oded Gabbay <ogabbay@kernel.org>
drivers/accel/habanalabs/common/firmware_if.c
drivers/accel/habanalabs/common/habanalabs.h

index 1400a4430045543141ff21b913cc706e2791d5ee..6150ab6ba810472a7b63949a5b9656d6390d3194 100644 (file)
@@ -105,7 +105,7 @@ static char *extract_u32_until_given_char(char *str, u32 *ver_num, char given_ch
  */
 static int hl_get_preboot_major_minor(struct hl_device *hdev, char *preboot_ver)
 {
-       preboot_ver = extract_u32_until_given_char(preboot_ver, &hdev->fw_major_version, '.');
+       preboot_ver = extract_u32_until_given_char(preboot_ver, &hdev->fw_inner_major_ver, '.');
        if (!preboot_ver) {
                dev_err(hdev->dev, "Error parsing preboot major version\n");
                goto err_zero_ver;
@@ -113,7 +113,7 @@ static int hl_get_preboot_major_minor(struct hl_device *hdev, char *preboot_ver)
 
        preboot_ver++;
 
-       preboot_ver = extract_u32_until_given_char(preboot_ver, &hdev->fw_minor_version, '.');
+       preboot_ver = extract_u32_until_given_char(preboot_ver, &hdev->fw_inner_minor_ver, '.');
        if (!preboot_ver) {
                dev_err(hdev->dev, "Error parsing preboot minor version\n");
                goto err_zero_ver;
@@ -121,8 +121,8 @@ static int hl_get_preboot_major_minor(struct hl_device *hdev, char *preboot_ver)
        return 0;
 
 err_zero_ver:
-       hdev->fw_major_version = 0;
-       hdev->fw_minor_version = 0;
+       hdev->fw_inner_major_ver = 0;
+       hdev->fw_inner_minor_ver = 0;
        return -EINVAL;
 }
 
index eaae69a9f8178fb0ce9931f07fb8d1e0622dea41..57661cb516218d658e91c17186ecced4cb62d05e 100644 (file)
@@ -3225,8 +3225,8 @@ struct hl_reset_info {
  * @captured_err_info: holds information about errors.
  * @reset_info: holds current device reset information.
  * @stream_master_qid_arr: pointer to array with QIDs of master streams.
- * @fw_major_version: major version of current loaded preboot.
- * @fw_minor_version: minor version of current loaded preboot.
+ * @fw_inner_major_ver: the major of current loaded preboot inner version.
+ * @fw_inner_minor_ver: the minor of current loaded preboot inner version.
  * @dram_used_mem: current DRAM memory consumption.
  * @memory_scrub_val: the value to which the dram will be scrubbed to using cb scrub_device_dram
  * @timeout_jiffies: device CS timeout value.
@@ -3412,8 +3412,8 @@ struct hl_device {
        struct hl_reset_info            reset_info;
 
        u32                             *stream_master_qid_arr;
-       u32                             fw_major_version;
-       u32                             fw_minor_version;
+       u32                             fw_inner_major_ver;
+       u32                             fw_inner_minor_ver;
        atomic64_t                      dram_used_mem;
        u64                             memory_scrub_val;
        u64                             timeout_jiffies;
@@ -3549,7 +3549,7 @@ struct hl_ioctl_desc {
 
 static inline bool hl_is_fw_ver_below_1_9(struct hl_device *hdev)
 {
-       return (hdev->fw_major_version < 42);
+       return (hdev->fw_inner_major_ver < 42);
 }
 
 /*