SWDEV-381227 - [AMDSMI][Linux][BM] SMILIB returns wrong temperature value

Signed-off-by: Marko Oblak <Marko.Oblak@amd.com>
Change-Id: Idc9929d1cfd882bb33abf040378587f68d22b31a

README.md

@@ -135,7 +135,7 @@ int main() {
       int64_t val_i64 = 0;
       ret =  amdsmi_dev_get_temp_metric(device_handles[j], TEMPERATURE_TYPE_EDGE,
               AMDSMI_TEMP_CURRENT, &val_i64);
-      std::cout << "\t\tTemperature: " << val_i64/1000 << "C" << std::endl;
+      std::cout << "\t\tTemperature: " << val_i64 << "C" << std::endl;
     }
   }
  

example/amd_smi_drm_example.cc

@@ -640,7 +640,7 @@ int main() {
                                              AMDSMI_TEMP_CURRENT, &val_i64);
             CHK_AMDSMI_RET(ret)
             printf("    Output of  amdsmi_dev_get_temp_metric:\n");
-            std::cout << "\t\tTemperature: " << val_i64 / 1000 << "C"
+            std::cout << "\t\tTemperature: " << val_i64 << "C"
                       << "\n\n";
 
             // Get frame buffer

example/amd_smi_nodrm_example.cc

@@ -304,7 +304,7 @@ int main() {
                                              AMDSMI_TEMP_CURRENT, &val_i64);
             CHK_AMDSMI_RET(ret)
             printf("    Output of  amdsmi_dev_get_temp_metric:\n");
-            std::cout << "\t\tTemperature: " << val_i64 / 1000 << "C"
+            std::cout << "\t\tTemperature: " << val_i64 << "C"
                       << "\n\n";
 
             // Get frame buffer

src/amd_smi/amd_smi.cc

@@ -336,14 +336,14 @@ amdsmi_status_t  amdsmi_dev_get_temp_metric(amdsmi_device_handle device_handle,
                 device_handle, &metric_info);
         if (r_status != AMDSMI_STATUS_SUCCESS)
             return r_status;
-
         *temperature = metric_info.temperature_vrsoc;
         return r_status;
     }
-
-    return rsmi_wrapper(rsmi_dev_temp_metric_get, device_handle,
+    amdsmi_status_t amdsmi_status = rsmi_wrapper(rsmi_dev_temp_metric_get, device_handle,
             static_cast<uint32_t>(sensor_type),
             static_cast<rsmi_temperature_metric_t>(metric), temperature);
+    *temperature /= 1000;
+    return amdsmi_status;
 }
 
 amdsmi_status_t amdsmi_get_vram_usage(amdsmi_device_handle device_handle,