gc2: es: Add E1S boot sensors #2644
Closed
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
[Task Description]
Added E1.S Boot device sensors to enhance system monitoring coverage and complete the sensor list for comprehensive power and thermal management.
[Motivation]
The E1.S Boot device lacked power telemetry monitoring (voltage, current, power) in the existing sensor configuration. Adding these sensors enables:
[Design]
Added E1.S Boot INA233 power sensors (0x2F, 0x36, 0x44):
MB_INA233_E1S_Boot_VOLT_V (0x2F): E1.S Boot voltage monitoring
MB_INA233_E1S_Boot_CURR_A (0x36): E1.S Boot current monitoring
MB_INA233_E1S_Boot_PWR_W (0x44): E1.S Boot power monitoring
All sensors use INA233 driver with vr_access handler and ina233_init_args[2] configuration (current_lsb=0.001, r_shunt=0.002Ω)
Added E1.S temperature sensor (0x0D):
[Test Result]
Tested on GC2-ES platform, all E1.S sensors reading correctly with proper thresholds:
root@bmc-oob:~# sensor-util server --threshold | grep E1S
MB_E1S_TEMP_C (0xD) : 58.000 C | (ok) | UCR: 75.000 | UNC: NA | UNR: NA | LCR: NA | LNC: NA | LNR: NA
MB_INA233_E1S_Boot_VOLT_V (0x2F) : 12.100 Volts | (ok) | UCR: 13.600 | UNC: 13.400 | UNR: 14.300 | LCR: 10.400 | LNC: 10.500 | LNR: 10.100
MB_INA233_E1S_Boot_CURR_A (0x36) : 0.290 Amps | (ok) | UCR: 1.830 | UNC: 1.750 | UNR: 2.200 | LCR: NA | LNC: NA | LNR: NA
MB_INA233_E1S_Boot_PWR_W (0x44) : 3.500 Watts | (ok) | UCR: 20.600 | UNC: 20.400 | UNR: 20.800 | LCR: NA | LNC: NA | LNR: NA