From 3c829750a8b12c95c060d4e2c8fd3079bb54308c Mon Sep 17 00:00:00 2001 From: "Bill(Shuzhou) Liu" Date: Mon, 13 Mar 2023 09:27:19 -0500 Subject: [PATCH] Fix cppcheck static analysis report warning Fix some warning from static anaysis tool. Change-Id: I7e8c2f5d6f79aff5fdcad81b1fd832900f213c47 [ROCm/rocm_smi_lib commit: 1b7eb4e1f4b325b1eec6498a6eb58aec32732301] --- projects/rocm-smi-lib/oam/example/oam_example.c | 2 +- projects/rocm-smi-lib/oam/src/amd_oam.cc | 8 ++++---- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/projects/rocm-smi-lib/oam/example/oam_example.c b/projects/rocm-smi-lib/oam/example/oam_example.c index 232fd205af..8d8a106a91 100644 --- a/projects/rocm-smi-lib/oam/example/oam_example.c +++ b/projects/rocm-smi-lib/oam/example/oam_example.c @@ -51,7 +51,7 @@ int main() // amd_oam_ops.get_mapi_version(&version); if (!amd_oam_ops.discover_devices(&dev_cnt)) - printf("%d AMD devices are discovered\n", dev_cnt); + printf("%u AMD devices are discovered\n", dev_cnt); if (!dev_cnt) { printf("No devices are found.\n"); return amd_oam_ops.free(); diff --git a/projects/rocm-smi-lib/oam/src/amd_oam.cc b/projects/rocm-smi-lib/oam/src/amd_oam.cc index 1262fcc9e8..8d63d94f40 100755 --- a/projects/rocm-smi-lib/oam/src/amd_oam.cc +++ b/projects/rocm-smi-lib/oam/src/amd_oam.cc @@ -250,7 +250,7 @@ int amdoam_get_sensors_info(uint32_t device_id, oam_sensor_type_t type, case OAM_SENSOR_TYPE_POWER: for (i = 0; i < num_sensors; i++) { snprintf(sensor_info[i].sensor_name, OAM_SENSOR_NAME_MAX, - "POWER_SENSOR_%d", i+1); + "POWER_SENSOR_%u", i+1); sensor_info[i].sensor_type = type; status = rsmi_dev_power_ave_get(device_id, i, reinterpret_cast(&sensor_info[i].value)); @@ -262,7 +262,7 @@ int amdoam_get_sensors_info(uint32_t device_id, oam_sensor_type_t type, case OAM_SENSOR_TYPE_VOLTAGE: for (i = 0; i < num_sensors; i++) { snprintf(sensor_info[i].sensor_name, OAM_SENSOR_NAME_MAX, - "VOLTAGE_SENSOR_%d", i); + "VOLTAGE_SENSOR_%u", i); sensor_info[i].sensor_type = type; status = rsmi_dev_volt_metric_get(device_id, RSMI_VOLT_TYPE_VDDGFX, RSMI_VOLT_CURRENT, &sensor_info[i].value); @@ -274,7 +274,7 @@ int amdoam_get_sensors_info(uint32_t device_id, oam_sensor_type_t type, case OAM_SENSOR_TYPE_TEMP: for (i = 0; i < num_sensors; i++) { snprintf(sensor_info[i].sensor_name, OAM_SENSOR_NAME_MAX, - "TEMP_SENSOR_%d", i+1); + "TEMP_SENSOR_%u", i+1); sensor_info[i].sensor_type = type; status = rsmi_dev_temp_metric_get(device_id, i, RSMI_TEMP_CURRENT, &sensor_info[i].value); @@ -286,7 +286,7 @@ int amdoam_get_sensors_info(uint32_t device_id, oam_sensor_type_t type, case OAM_SENSOR_TYPE_FAN_SPEED: for (i = 0; i < num_sensors; i++) { snprintf(sensor_info[i].sensor_name, OAM_SENSOR_NAME_MAX, - "FAN_SENSOR_%d", i+1); + "FAN_SENSOR_%u", i+1); sensor_info[i].sensor_type = type; status = rsmi_dev_fan_speed_get(device_id, i, &sensor_info[i].value); if (status != RSMI_STATUS_SUCCESS)