-
Notifications
You must be signed in to change notification settings - Fork 53
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
YV4: sd: Revise sensor name to comply naming rule #2031
YV4: sd: Revise sensor name to comply naming rule #2031
Conversation
@facebook-github-bot has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
Description: - Revise the descriptor part of sensor name. - Revise the fru name of the sensor name to be in uppercase. Motivation: - Revise sensor name to comply naming rules. Note: - Related JIRA link: https://metainfra.atlassian.net/browse/YV4T1M-1070 Test Plan: - Build code: Pass - Check sensor name: Pass Test Log: - Revised sensor name root@bmc:~# busctl tree xyz.openbmc_project.PLDM | grep SENTINEL_DOME_SLOT_1_ | wc -l 73 root@bmc:~# root@bmc:~# busctl tree xyz.openbmc_project.PLDM | grep SENTINEL_DOME_SLOT_1_ │ │ ├─ /xyz/openbmc_project/sensors/current/SENTINEL_DOME_SLOT_1_MB_PMON_E1S_BOOT_CURR_A │ │ ├─ /xyz/openbmc_project/sensors/current/SENTINEL_DOME_SLOT_1_MB_PMON_E1S_DATA_CURR_A │ │ ├─ /xyz/openbmc_project/sensors/current/SENTINEL_DOME_SLOT_1_MB_PMON_X8_RTM_CURR_A │ │ ├─ /xyz/openbmc_project/sensors/current/SENTINEL_DOME_SLOT_1_MB_VR_CPU0_CURR_A │ │ ├─ /xyz/openbmc_project/sensors/current/SENTINEL_DOME_SLOT_1_MB_VR_CPU1_CURR_A │ │ ├─ /xyz/openbmc_project/sensors/current/SENTINEL_DOME_SLOT_1_MB_VR_PVDD11_CURR_A │ │ ├─ /xyz/openbmc_project/sensors/current/SENTINEL_DOME_SLOT_1_MB_VR_PVDDIO_CURR_A │ │ ├─ /xyz/openbmc_project/sensors/current/SENTINEL_DOME_SLOT_1_MB_VR_SOC_CURR_A │ │ ├─ /xyz/openbmc_project/sensors/power/SENTINEL_DOME_SLOT_1_MB_CPU_PWR_W │ │ ├─ /xyz/openbmc_project/sensors/power/SENTINEL_DOME_SLOT_1_MB_DIMM_A_PWR_W │ │ ├─ /xyz/openbmc_project/sensors/power/SENTINEL_DOME_SLOT_1_MB_DIMM_B_PWR_W │ │ ├─ /xyz/openbmc_project/sensors/power/SENTINEL_DOME_SLOT_1_MB_DIMM_C_PWR_W │ │ ├─ /xyz/openbmc_project/sensors/power/SENTINEL_DOME_SLOT_1_MB_DIMM_D_PWR_W │ │ ├─ /xyz/openbmc_project/sensors/power/SENTINEL_DOME_SLOT_1_MB_DIMM_E_PWR_W │ │ ├─ /xyz/openbmc_project/sensors/power/SENTINEL_DOME_SLOT_1_MB_DIMM_F_PWR_W │ │ ├─ /xyz/openbmc_project/sensors/power/SENTINEL_DOME_SLOT_1_MB_DIMM_G_PWR_W │ │ ├─ /xyz/openbmc_project/sensors/power/SENTINEL_DOME_SLOT_1_MB_DIMM_H_PWR_W │ │ ├─ /xyz/openbmc_project/sensors/power/SENTINEL_DOME_SLOT_1_MB_DIMM_I_PWR_W │ │ ├─ /xyz/openbmc_project/sensors/power/SENTINEL_DOME_SLOT_1_MB_DIMM_J_PWR_W │ │ ├─ /xyz/openbmc_project/sensors/power/SENTINEL_DOME_SLOT_1_MB_DIMM_K_PWR_W │ │ ├─ /xyz/openbmc_project/sensors/power/SENTINEL_DOME_SLOT_1_MB_DIMM_L_PWR_W │ │ ├─ /xyz/openbmc_project/sensors/power/SENTINEL_DOME_SLOT_1_MB_PMON_E1S_BOOT_PWR_W │ │ ├─ /xyz/openbmc_project/sensors/power/SENTINEL_DOME_SLOT_1_MB_PMON_E1S_DATA_PWR_W │ │ ├─ /xyz/openbmc_project/sensors/power/SENTINEL_DOME_SLOT_1_MB_PMON_X8_RTM_PWR_W │ │ ├─ /xyz/openbmc_project/sensors/power/SENTINEL_DOME_SLOT_1_MB_VR_CPU0_PWR_W │ │ ├─ /xyz/openbmc_project/sensors/power/SENTINEL_DOME_SLOT_1_MB_VR_CPU1_PWR_W │ │ ├─ /xyz/openbmc_project/sensors/power/SENTINEL_DOME_SLOT_1_MB_VR_PVDD11_PWR_W │ │ ├─ /xyz/openbmc_project/sensors/power/SENTINEL_DOME_SLOT_1_MB_VR_PVDDIO_PWR_W │ │ ├─ /xyz/openbmc_project/sensors/power/SENTINEL_DOME_SLOT_1_MB_VR_SOC_PWR_W │ │ ├─ /xyz/openbmc_project/sensors/temperature/SENTINEL_DOME_SLOT_1_MB_CPU_TEMP_C │ │ ├─ /xyz/openbmc_project/sensors/temperature/SENTINEL_DOME_SLOT_1_MB_DIMM_A_TEMP_C │ │ ├─ /xyz/openbmc_project/sensors/temperature/SENTINEL_DOME_SLOT_1_MB_DIMM_B_TEMP_C │ │ ├─ /xyz/openbmc_project/sensors/temperature/SENTINEL_DOME_SLOT_1_MB_DIMM_C_TEMP_C │ │ ├─ /xyz/openbmc_project/sensors/temperature/SENTINEL_DOME_SLOT_1_MB_DIMM_D_TEMP_C │ │ ├─ /xyz/openbmc_project/sensors/temperature/SENTINEL_DOME_SLOT_1_MB_DIMM_E_TEMP_C │ │ ├─ /xyz/openbmc_project/sensors/temperature/SENTINEL_DOME_SLOT_1_MB_DIMM_F_TEMP_C │ │ ├─ /xyz/openbmc_project/sensors/temperature/SENTINEL_DOME_SLOT_1_MB_DIMM_G_TEMP_C │ │ ├─ /xyz/openbmc_project/sensors/temperature/SENTINEL_DOME_SLOT_1_MB_DIMM_H_TEMP_C │ │ ├─ /xyz/openbmc_project/sensors/temperature/SENTINEL_DOME_SLOT_1_MB_DIMM_I_TEMP_C │ │ ├─ /xyz/openbmc_project/sensors/temperature/SENTINEL_DOME_SLOT_1_MB_DIMM_J_TEMP_C │ │ ├─ /xyz/openbmc_project/sensors/temperature/SENTINEL_DOME_SLOT_1_MB_DIMM_K_TEMP_C │ │ ├─ /xyz/openbmc_project/sensors/temperature/SENTINEL_DOME_SLOT_1_MB_DIMM_L_TEMP_C │ │ ├─ /xyz/openbmc_project/sensors/temperature/SENTINEL_DOME_SLOT_1_MB_FIO_TEMP_C │ │ ├─ /xyz/openbmc_project/sensors/temperature/SENTINEL_DOME_SLOT_1_MB_INLET_TEMP_C │ │ ├─ /xyz/openbmc_project/sensors/temperature/SENTINEL_DOME_SLOT_1_MB_OUTLET_TEMP_C │ │ ├─ /xyz/openbmc_project/sensors/temperature/SENTINEL_DOME_SLOT_1_MB_SSD_BOOT_TEMP_C │ │ ├─ /xyz/openbmc_project/sensors/temperature/SENTINEL_DOME_SLOT_1_MB_SSD_DATA_TEMP_C │ │ ├─ /xyz/openbmc_project/sensors/temperature/SENTINEL_DOME_SLOT_1_MB_VR_CPU0_TEMP_C │ │ ├─ /xyz/openbmc_project/sensors/temperature/SENTINEL_DOME_SLOT_1_MB_VR_CPU1_TEMP_C │ │ ├─ /xyz/openbmc_project/sensors/temperature/SENTINEL_DOME_SLOT_1_MB_VR_PVDD11_TEMP_C │ │ ├─ /xyz/openbmc_project/sensors/temperature/SENTINEL_DOME_SLOT_1_MB_VR_PVDDIO_TEMP_C │ │ ├─ /xyz/openbmc_project/sensors/temperature/SENTINEL_DOME_SLOT_1_MB_VR_SOC_TEMP_C │ │ ├─ /xyz/openbmc_project/sensors/temperature/SENTINEL_DOME_SLOT_1_MB_X8_RTM_TEMP_C │ ├─ /xyz/openbmc_project/sensors/voltage/SENTINEL_DOME_SLOT_1_MB_ADC_DIMM0_P12V_VOLT_V │ ├─ /xyz/openbmc_project/sensors/voltage/SENTINEL_DOME_SLOT_1_MB_ADC_DIMM1_P12V_VOLT_V │ ├─ /xyz/openbmc_project/sensors/voltage/SENTINEL_DOME_SLOT_1_MB_ADC_P12V_STBY_VOLT_V │ ├─ /xyz/openbmc_project/sensors/voltage/SENTINEL_DOME_SLOT_1_MB_ADC_P1V2_STBY_VOLT_V │ ├─ /xyz/openbmc_project/sensors/voltage/SENTINEL_DOME_SLOT_1_MB_ADC_P1V8_STBY_VOLT_V │ ├─ /xyz/openbmc_project/sensors/voltage/SENTINEL_DOME_SLOT_1_MB_ADC_P3V3_STBY_VOLT_V │ ├─ /xyz/openbmc_project/sensors/voltage/SENTINEL_DOME_SLOT_1_MB_ADC_P3V_BAT_VOLT_V │ ├─ /xyz/openbmc_project/sensors/voltage/SENTINEL_DOME_SLOT_1_MB_ADC_P5V_STBY_VOLT_V │ ├─ /xyz/openbmc_project/sensors/voltage/SENTINEL_DOME_SLOT_1_MB_ADC_PVDD11_S3_VOLT_V │ ├─ /xyz/openbmc_project/sensors/voltage/SENTINEL_DOME_SLOT_1_MB_ADC_PVDD18_S5_VOLT_V │ ├─ /xyz/openbmc_project/sensors/voltage/SENTINEL_DOME_SLOT_1_MB_ADC_PVDD33_S5_VOLT_V │ ├─ /xyz/openbmc_project/sensors/voltage/SENTINEL_DOME_SLOT_1_MB_ADC_SIDECAR_DETECT_VOLT_V │ ├─ /xyz/openbmc_project/sensors/voltage/SENTINEL_DOME_SLOT_1_MB_ADC_SLOT_DETECT_VOLT_V │ ├─ /xyz/openbmc_project/sensors/voltage/SENTINEL_DOME_SLOT_1_MB_PMON_E1S_BOOT_VOLT_V │ ├─ /xyz/openbmc_project/sensors/voltage/SENTINEL_DOME_SLOT_1_MB_PMON_E1S_DATA_VOLT_V │ ├─ /xyz/openbmc_project/sensors/voltage/SENTINEL_DOME_SLOT_1_MB_VR_CPU0_VOLT_V │ ├─ /xyz/openbmc_project/sensors/voltage/SENTINEL_DOME_SLOT_1_MB_VR_CPU1_VOLT_V │ ├─ /xyz/openbmc_project/sensors/voltage/SENTINEL_DOME_SLOT_1_MB_VR_PVDD11_VOLT_V │ ├─ /xyz/openbmc_project/sensors/voltage/SENTINEL_DOME_SLOT_1_MB_VR_PVDDIO_VOLT_V │ ├─ /xyz/openbmc_project/sensors/voltage/SENTINEL_DOME_SLOT_1_MB_VR_SOC_VOLT_V
526f1b3
to
28b21bf
Compare
@DelphineChiu has updated the pull request. You must reimport the pull request before landing. |
@facebook-github-bot has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
Based on the discussion results of sensor naming rules, it is uniformly changed to BLAH. Unless there is a special case to distinguish the main/second source, it will be displayed as 'BLAH_'. |
This pull request has been merged in a33ab76. |
Description:
Motivation:
Note:
Test Plan:
Test Log:
# busctl tree xyz.openbmc_project.PLDM | grep SENTINEL_DOME_SLOT_1_ | wc -l 73#root@bmc:
root@bmc:~# busctl tree xyz.openbmc_project.PLDM | grep SENTINEL_DOME_SLOT_1_
│ │ ├─ /xyz/openbmc_project/sensors/current/SENTINEL_DOME_SLOT_1_MB_PMON_E1S_BOOT_CURR_A
│ │ ├─ /xyz/openbmc_project/sensors/current/SENTINEL_DOME_SLOT_1_MB_PMON_E1S_DATA_CURR_A
│ │ ├─ /xyz/openbmc_project/sensors/current/SENTINEL_DOME_SLOT_1_MB_PMON_X8_RTM_CURR_A
│ │ ├─ /xyz/openbmc_project/sensors/current/SENTINEL_DOME_SLOT_1_MB_VR_CPU0_CURR_A
│ │ ├─ /xyz/openbmc_project/sensors/current/SENTINEL_DOME_SLOT_1_MB_VR_CPU1_CURR_A
│ │ ├─ /xyz/openbmc_project/sensors/current/SENTINEL_DOME_SLOT_1_MB_VR_PVDD11_CURR_A
│ │ ├─ /xyz/openbmc_project/sensors/current/SENTINEL_DOME_SLOT_1_MB_VR_PVDDIO_CURR_A
│ │ ├─ /xyz/openbmc_project/sensors/current/SENTINEL_DOME_SLOT_1_MB_VR_SOC_CURR_A
│ │ ├─ /xyz/openbmc_project/sensors/power/SENTINEL_DOME_SLOT_1_MB_CPU_PWR_W
│ │ ├─ /xyz/openbmc_project/sensors/power/SENTINEL_DOME_SLOT_1_MB_DIMM_A_PWR_W
│ │ ├─ /xyz/openbmc_project/sensors/power/SENTINEL_DOME_SLOT_1_MB_DIMM_B_PWR_W
│ │ ├─ /xyz/openbmc_project/sensors/power/SENTINEL_DOME_SLOT_1_MB_DIMM_C_PWR_W
│ │ ├─ /xyz/openbmc_project/sensors/power/SENTINEL_DOME_SLOT_1_MB_DIMM_D_PWR_W
│ │ ├─ /xyz/openbmc_project/sensors/power/SENTINEL_DOME_SLOT_1_MB_DIMM_E_PWR_W
│ │ ├─ /xyz/openbmc_project/sensors/power/SENTINEL_DOME_SLOT_1_MB_DIMM_F_PWR_W
│ │ ├─ /xyz/openbmc_project/sensors/power/SENTINEL_DOME_SLOT_1_MB_DIMM_G_PWR_W
│ │ ├─ /xyz/openbmc_project/sensors/power/SENTINEL_DOME_SLOT_1_MB_DIMM_H_PWR_W
│ │ ├─ /xyz/openbmc_project/sensors/power/SENTINEL_DOME_SLOT_1_MB_DIMM_I_PWR_W
│ │ ├─ /xyz/openbmc_project/sensors/power/SENTINEL_DOME_SLOT_1_MB_DIMM_J_PWR_W
│ │ ├─ /xyz/openbmc_project/sensors/power/SENTINEL_DOME_SLOT_1_MB_DIMM_K_PWR_W
│ │ ├─ /xyz/openbmc_project/sensors/power/SENTINEL_DOME_SLOT_1_MB_DIMM_L_PWR_W
│ │ ├─ /xyz/openbmc_project/sensors/power/SENTINEL_DOME_SLOT_1_MB_PMON_E1S_BOOT_PWR_W
│ │ ├─ /xyz/openbmc_project/sensors/power/SENTINEL_DOME_SLOT_1_MB_PMON_E1S_DATA_PWR_W
│ │ ├─ /xyz/openbmc_project/sensors/power/SENTINEL_DOME_SLOT_1_MB_PMON_X8_RTM_PWR_W
│ │ ├─ /xyz/openbmc_project/sensors/power/SENTINEL_DOME_SLOT_1_MB_VR_CPU0_PWR_W
│ │ ├─ /xyz/openbmc_project/sensors/power/SENTINEL_DOME_SLOT_1_MB_VR_CPU1_PWR_W
│ │ ├─ /xyz/openbmc_project/sensors/power/SENTINEL_DOME_SLOT_1_MB_VR_PVDD11_PWR_W
│ │ ├─ /xyz/openbmc_project/sensors/power/SENTINEL_DOME_SLOT_1_MB_VR_PVDDIO_PWR_W
│ │ ├─ /xyz/openbmc_project/sensors/power/SENTINEL_DOME_SLOT_1_MB_VR_SOC_PWR_W
│ │ ├─ /xyz/openbmc_project/sensors/temperature/SENTINEL_DOME_SLOT_1_MB_CPU_TEMP_C
│ │ ├─ /xyz/openbmc_project/sensors/temperature/SENTINEL_DOME_SLOT_1_MB_DIMM_A_TEMP_C
│ │ ├─ /xyz/openbmc_project/sensors/temperature/SENTINEL_DOME_SLOT_1_MB_DIMM_B_TEMP_C
│ │ ├─ /xyz/openbmc_project/sensors/temperature/SENTINEL_DOME_SLOT_1_MB_DIMM_C_TEMP_C
│ │ ├─ /xyz/openbmc_project/sensors/temperature/SENTINEL_DOME_SLOT_1_MB_DIMM_D_TEMP_C
│ │ ├─ /xyz/openbmc_project/sensors/temperature/SENTINEL_DOME_SLOT_1_MB_DIMM_E_TEMP_C
│ │ ├─ /xyz/openbmc_project/sensors/temperature/SENTINEL_DOME_SLOT_1_MB_DIMM_F_TEMP_C
│ │ ├─ /xyz/openbmc_project/sensors/temperature/SENTINEL_DOME_SLOT_1_MB_DIMM_G_TEMP_C
│ │ ├─ /xyz/openbmc_project/sensors/temperature/SENTINEL_DOME_SLOT_1_MB_DIMM_H_TEMP_C
│ │ ├─ /xyz/openbmc_project/sensors/temperature/SENTINEL_DOME_SLOT_1_MB_DIMM_I_TEMP_C
│ │ ├─ /xyz/openbmc_project/sensors/temperature/SENTINEL_DOME_SLOT_1_MB_DIMM_J_TEMP_C
│ │ ├─ /xyz/openbmc_project/sensors/temperature/SENTINEL_DOME_SLOT_1_MB_DIMM_K_TEMP_C
│ │ ├─ /xyz/openbmc_project/sensors/temperature/SENTINEL_DOME_SLOT_1_MB_DIMM_L_TEMP_C
│ │ ├─ /xyz/openbmc_project/sensors/temperature/SENTINEL_DOME_SLOT_1_MB_FIO_TEMP_C
│ │ ├─ /xyz/openbmc_project/sensors/temperature/SENTINEL_DOME_SLOT_1_MB_INLET_TEMP_C
│ │ ├─ /xyz/openbmc_project/sensors/temperature/SENTINEL_DOME_SLOT_1_MB_OUTLET_TEMP_C
│ │ ├─ /xyz/openbmc_project/sensors/temperature/SENTINEL_DOME_SLOT_1_MB_SSD_BOOT_TEMP_C
│ │ ├─ /xyz/openbmc_project/sensors/temperature/SENTINEL_DOME_SLOT_1_MB_SSD_DATA_TEMP_C
│ │ ├─ /xyz/openbmc_project/sensors/temperature/SENTINEL_DOME_SLOT_1_MB_VR_CPU0_TEMP_C
│ │ ├─ /xyz/openbmc_project/sensors/temperature/SENTINEL_DOME_SLOT_1_MB_VR_CPU1_TEMP_C
│ │ ├─ /xyz/openbmc_project/sensors/temperature/SENTINEL_DOME_SLOT_1_MB_VR_PVDD11_TEMP_C
│ │ ├─ /xyz/openbmc_project/sensors/temperature/SENTINEL_DOME_SLOT_1_MB_VR_PVDDIO_TEMP_C
│ │ ├─ /xyz/openbmc_project/sensors/temperature/SENTINEL_DOME_SLOT_1_MB_VR_SOC_TEMP_C
│ │ ├─ /xyz/openbmc_project/sensors/temperature/SENTINEL_DOME_SLOT_1_MB_X8_RTM_TEMP_C
│ ├─ /xyz/openbmc_project/sensors/voltage/SENTINEL_DOME_SLOT_1_MB_ADC_DIMM_0_P12V_VOLT_V
│ ├─ /xyz/openbmc_project/sensors/voltage/SENTINEL_DOME_SLOT_1_MB_ADC_DIMM_1_P12V_VOLT_V
│ ├─ /xyz/openbmc_project/sensors/voltage/SENTINEL_DOME_SLOT_1_MB_ADC_P12V_STBY_VOLT_V
│ ├─ /xyz/openbmc_project/sensors/voltage/SENTINEL_DOME_SLOT_1_MB_ADC_P1V2_STBY_VOLT_V
│ ├─ /xyz/openbmc_project/sensors/voltage/SENTINEL_DOME_SLOT_1_MB_ADC_P1V8_STBY_VOLT_V
│ ├─ /xyz/openbmc_project/sensors/voltage/SENTINEL_DOME_SLOT_1_MB_ADC_P3V3_STBY_VOLT_V
│ ├─ /xyz/openbmc_project/sensors/voltage/SENTINEL_DOME_SLOT_1_MB_ADC_P3V_BAT_VOLT_V
│ ├─ /xyz/openbmc_project/sensors/voltage/SENTINEL_DOME_SLOT_1_MB_ADC_P5V_STBY_VOLT_V
│ ├─ /xyz/openbmc_project/sensors/voltage/SENTINEL_DOME_SLOT_1_MB_ADC_PVDD11_S3_VOLT_V
│ ├─ /xyz/openbmc_project/sensors/voltage/SENTINEL_DOME_SLOT_1_MB_ADC_PVDD18_S5_VOLT_V
│ ├─ /xyz/openbmc_project/sensors/voltage/SENTINEL_DOME_SLOT_1_MB_ADC_PVDD33_S5_VOLT_V
│ ├─ /xyz/openbmc_project/sensors/voltage/SENTINEL_DOME_SLOT_1_MB_ADC_SIDECAR_DETECT_VOLT_V
│ ├─ /xyz/openbmc_project/sensors/voltage/SENTINEL_DOME_SLOT_1_MB_ADC_SLOT_DETECT_VOLT_V
│ ├─ /xyz/openbmc_project/sensors/voltage/SENTINEL_DOME_SLOT_1_MB_PMON_E1S_BOOT_VOLT_V
│ ├─ /xyz/openbmc_project/sensors/voltage/SENTINEL_DOME_SLOT_1_MB_PMON_E1S_DATA_VOLT_V
│ ├─ /xyz/openbmc_project/sensors/voltage/SENTINEL_DOME_SLOT_1_MB_VR_CPU0_VOLT_V
│ ├─ /xyz/openbmc_project/sensors/voltage/SENTINEL_DOME_SLOT_1_MB_VR_CPU1_VOLT_V
│ ├─ /xyz/openbmc_project/sensors/voltage/SENTINEL_DOME_SLOT_1_MB_VR_PVDD11_VOLT_V
│ ├─ /xyz/openbmc_project/sensors/voltage/SENTINEL_DOME_SLOT_1_MB_VR_PVDDIO_VOLT_V
│ ├─ /xyz/openbmc_project/sensors/voltage/SENTINEL_DOME_SLOT_1_MB_VR_SOC_VOLT_V