Commit 393a089
fbgc: fbgc2: Modify sensor thresholds and fix sensor names
Summary:
# Description
- Update HDD temperature threshold from 61°C to 60°C
- Correct PTB voltage and current sensor thresholds according to hardware specifications
- dpb thresholds are taken from expander
- Fix server DIMM sensor naming from DIMMA0/DIMMA4 to DIMMA2/DIMMA3/DIMMA6/DIMMA7
# Motivation
- Match ours latest sensor table
X-link: https://github.com/facebookexternal/openbmc.wiwynn/pull/3490
Test Plan:
- Verify all HDD temperature sensors report correct threshold (60°C)
- Verify PTB sensors show updated thresholds
- Verify server DIMM sensor names are correctly displayed
- Check all sensors report ok status within normal operating range
Log:
root@bmc-oob:~# sensor-util dpb --threshold | grep HDD | grep TEMP
HDD_SMART_TEMP_00 (0x60) : 36.000 C | (ok) | UCR: 60.000 | UNC: NA | UNR: NA | LCR: NA | LNC: NA | LNR: NA
HDD_SMART_TEMP_01 (0x61) : 37.000 C | (ok) | UCR: 60.000 | UNC: NA | UNR: NA | LCR: NA | LNC: NA | LNR: NA
HDD_SMART_TEMP_02 (0x62) : 37.000 C | (ok) | UCR: 60.000 | UNC: NA | UNR: NA | LCR: NA | LNC: NA | LNR: NA
HDD_SMART_TEMP_03 (0x63) : 37.000 C | (ok) | UCR: 60.000 | UNC: NA | UNR: NA | LCR: NA | LNC: NA | LNR: NA
HDD_SMART_TEMP_04 (0x64) : 37.000 C | (ok) | UCR: 60.000 | UNC: NA | UNR: NA | LCR: NA | LNC: NA | LNR: NA
HDD_SMART_TEMP_05 (0x65) : 36.000 C | (ok) | UCR: 60.000 | UNC: NA | UNR: NA | LCR: NA | LNC: NA | LNR: NA
HDD_SMART_TEMP_06 (0x66) : 36.000 C | (ok) | UCR: 60.000 | UNC: NA | UNR: NA | LCR: NA | LNC: NA | LNR: NA
HDD_SMART_TEMP_07 (0x67) : 37.000 C | (ok) | UCR: 60.000 | UNC: NA | UNR: NA | LCR: NA | LNC: NA | LNR: NA
HDD_SMART_TEMP_08 (0x68) : 37.000 C | (ok) | UCR: 60.000 | UNC: NA | UNR: NA | LCR: NA | LNC: NA | LNR: NA
HDD_SMART_TEMP_09 (0x69) : 37.000 C | (ok) | UCR: 60.000 | UNC: NA | UNR: NA | LCR: NA | LNC: NA | LNR: NA
HDD_SMART_TEMP_10 (0x6A) : 37.000 C | (ok) | UCR: 60.000 | UNC: NA | UNR: NA | LCR: NA | LNC: NA | LNR: NA
HDD_SMART_TEMP_11 (0x6B) : 36.000 C | (ok) | UCR: 60.000 | UNC: NA | UNR: NA | LCR: NA | LNC: NA | LNR: NA
HDD_SMART_TEMP_12 (0x6C) : 36.000 C | (ok) | UCR: 60.000 | UNC: NA | UNR: NA | LCR: NA | LNC: NA | LNR: NA
HDD_SMART_TEMP_13 (0x6D) : 36.000 C | (ok) | UCR: 60.000 | UNC: NA | UNR: NA | LCR: NA | LNC: NA | LNR: NA
HDD_SMART_TEMP_14 (0x6E) : 36.000 C | (ok) | UCR: 60.000 | UNC: NA | UNR: NA | LCR: NA | LNC: NA | LNR: NA
HDD_SMART_TEMP_15 (0x6F) : 36.000 C | (ok) | UCR: 60.000 | UNC: NA | UNR: NA | LCR: NA | LNC: NA | LNR: NA
HDD_SMART_TEMP_16 (0x70) : 36.000 C | (ok) | UCR: 60.000 | UNC: NA | UNR: NA | LCR: NA | LNC: NA | LNR: NA
HDD_SMART_TEMP_17 (0x71) : 36.000 C | (ok) | UCR: 60.000 | UNC: NA | UNR: NA | LCR: NA | LNC: NA | LNR: NA
HDD_SMART_TEMP_18 (0x72) : 36.000 C | (ok) | UCR: 60.000 | UNC: NA | UNR: NA | LCR: NA | LNC: NA | LNR: NA
HDD_SMART_TEMP_19 (0x73) : 36.000 C | (ok) | UCR: 60.000 | UNC: NA | UNR: NA | LCR: NA | LNC: NA | LNR: NA
HDD_SMART_TEMP_20 (0x74) : 36.000 C | (ok) | UCR: 60.000 | UNC: NA | UNR: NA | LCR: NA | LNC: NA | LNR: NA
HDD_SMART_TEMP_21 (0x75) : 36.000 C | (ok) | UCR: 60.000 | UNC: NA | UNR: NA | LCR: NA | LNC: NA | LNR: NA
HDD_SMART_TEMP_22 (0x76) : 36.000 C | (ok) | UCR: 60.000 | UNC: NA | UNR: NA | LCR: NA | LNC: NA | LNR: NA
HDD_SMART_TEMP_23 (0x77) : 36.000 C | (ok) | UCR: 60.000 | UNC: NA | UNR: NA | LCR: NA | LNC: NA | LNR: NA
HDD_SMART_TEMP_24 (0x78) : 35.000 C | (ok) | UCR: 60.000 | UNC: NA | UNR: NA | LCR: NA | LNC: NA | LNR: NA
HDD_SMART_TEMP_25 (0x79) : 35.000 C | (ok) | UCR: 60.000 | UNC: NA | UNR: NA | LCR: NA | LNC: NA | LNR: NA
HDD_SMART_TEMP_26 (0x7A) : 35.000 C | (ok) | UCR: 60.000 | UNC: NA | UNR: NA | LCR: NA | LNC: NA | LNR: NA
HDD_SMART_TEMP_27 (0x7B) : 35.000 C | (ok) | UCR: 60.000 | UNC: NA | UNR: NA | LCR: NA | LNC: NA | LNR: NA
HDD_SMART_TEMP_28 (0x7C) : 35.000 C | (ok) | UCR: 60.000 | UNC: NA | UNR: NA | LCR: NA | LNC: NA | LNR: NA
HDD_SMART_TEMP_29 (0x7D) : 35.000 C | (ok) | UCR: 60.000 | UNC: NA | UNR: NA | LCR: NA | LNC: NA | LNR: NA
HDD_SMART_TEMP_30 (0x7E) : 35.000 C | (ok) | UCR: 60.000 | UNC: NA | UNR: NA | LCR: NA | LNC: NA | LNR: NA
HDD_SMART_TEMP_31 (0x7F) : 35.000 C | (ok) | UCR: 60.000 | UNC: NA | UNR: NA | LCR: NA | LNC: NA | LNR: NA
HDD_SMART_TEMP_32 (0x80) : 35.000 C | (ok) | UCR: 60.000 | UNC: NA | UNR: NA | LCR: NA | LNC: NA | LNR: NA
HDD_SMART_TEMP_33 (0x81) : 35.000 C | (ok) | UCR: 60.000 | UNC: NA | UNR: NA | LCR: NA | LNC: NA | LNR: NA
HDD_SMART_TEMP_34 (0x82) : 35.000 C | (ok) | UCR: 60.000 | UNC: NA | UNR: NA | LCR: NA | LNC: NA | LNR: NA
HDD_SMART_TEMP_35 (0x83) : 35.000 C | (ok) | UCR: 60.000 | UNC: NA | UNR: NA | LCR: NA | LNC: NA | LNR: NA
root@bmc-oob:~# sensor-util dpb --threshold | grep PTB
PTB_P48V_AUX (0xDC) : 50.660 Volts | (ok) | UCR: 52.800 | UNC: NA | UNR: NA | LCR: 43.200 | LNC: NA | LNR: NA
PTB_P12V_PU2_DC_MODULE (0xDD) : 12.230 Volts | (ok) | UCR: 13.200 | UNC: NA | UNR: NA | LCR: 10.800 | LNC: NA | LNR: NA
PTB_P12V_PU3_DC_MODULE (0xDE) : 12.220 Volts | (ok) | UCR: 13.200 | UNC: NA | UNR: NA | LCR: 10.800 | LNC: NA | LNR: NA
PTB_U19_ADC_MONITOR (0xDF) : 62.900 Volts | (ucr) | UCR: 52.800 | UNC: NA | UNR: NA | LCR: 43.200 | LNC: NA | LNR: NA
PTB_U20_ADC_MONITOR (0xE0) : 191.760 Volts | (ok) | UCR: NA | UNC: NA | UNR: NA | LCR: NA | LNC: NA | LNR: NA
PTB_P48V_AUX_Current (0xE1) : 8.550 Amps | (ok) | UCR: 165.000 | UNC: NA | UNR: NA | LCR: NA | LNC: NA | LNR: NA
PTB_P12V_PU2_DC_MODULE_Current (0xE2) : 33.500 Amps | (ok) | UCR: 41.250 | UNC: NA | UNR: NA | LCR: NA | LNC: NA | LNR: NA
PTB_P12V_PU3_DC_MODULE_Current (0xE3) : 35.500 Amps | (ok) | UCR: 41.250 | UNC: NA | UNR: NA | LCR: NA | LNC: NA | LNR: NA
PTB_P48V_AUX_Power (0xE4) : 433.000 Watts | (ok) | UCR: NA | UNC: NA | UNR: NA | LCR: NA | LNC: NA | LNR: NA
PTB_P12V_PU2_DC_MODULE_Power (0xE5) : 433.000 Watts | (ok) | UCR: NA | UNC: NA | UNR: NA | LCR: NA | LNC: NA | LNR: NA
PTB_P12V_PU3_DC_MODULE_Power (0xE6) : 433.000 Watts | (ok) | UCR: NA | UNC: NA | UNR: NA | LCR: NA | LNC: NA | LNR: NA
root@bmc-oob:~# sensor-util server --threshold | grep DIMM | grep TEMP
MB_DIMMA2_TEMP_C (0x6) : 33.000 C | (ok) | UCR: 85.000 | UNC: NA | UNR: NA | LCR: NA | LNC: NA | LNR: NA
MB_DIMMA3_TEMP_C (0x7) : 33.000 C | (ok) | UCR: 85.000 | UNC: NA | UNR: NA | LCR: NA | LNC: NA | LNR: NA
MB_DIMMA6_TEMP_C (0xA) : 33.000 C | (ok) | UCR: 85.000 | UNC: NA | UNR: NA | LCR: NA | LNC: NA | LNR: NA
MB_DIMMA7_TEMP_C (0xB) : 33.000 C | (ok) | UCR: 85.000 | UNC: NA | UNR: NA | LCR: NA | LNC: NA | LNR: NA
root@bmc-oob:~# sensor-util server --threshold | grep DIMM | grep PWR
MB_VR_DIMMA2_PMIC_PWR_W (0x40) : 0.560 Watts | (ok) | UCR: 32.200 | UNC: 31.500 | UNR: NA | LCR: NA | LNC: NA | LNR: NA
MB_VR_DIMMA3_PMIC_PWR_W (0x41) : 0.980 Watts | (ok) | UCR: 32.200 | UNC: 31.500 | UNR: NA | LCR: NA | LNC: NA | LNR: NA
MB_VR_DIMMA6_PMIC_PWR_W (0x42) : 0.420 Watts | (ok) | UCR: 32.200 | UNC: 31.500 | UNR: NA | LCR: NA | LNC: NA | LNR: NA
MB_VR_DIMMA7_PMIC_PWR_W (0x43) : 0.840 Watts | (ok) | UCR: 32.200 | UNC: 31.500 | UNR: NA | LCR: NA | LNC: NA | LNR: NA
Reviewed By: waffle2k
Differential Revision: D90217684
fbshipit-source-id: fb3702be12e889aad5b6d5b18c27b5c7bf5160a51 parent 80da032 commit 393a089
File tree
3 files changed
+61
-65
lines changed- meta-facebook/meta-grandcanyon
- meta-grandcanyon2/recipes-grandcanyon2/fscd/fscd
- recipes-grandcanyon/plat-libs/files/pal
3 files changed
+61
-65
lines changedLines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
30 | 30 | | |
31 | 31 | | |
32 | 32 | | |
33 | | - | |
| 33 | + | |
Lines changed: 51 additions & 54 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
82 | 82 | | |
83 | 83 | | |
84 | 84 | | |
85 | | - | |
86 | | - | |
| 85 | + | |
87 | 86 | | |
88 | | - | |
| 87 | + | |
89 | 88 | | |
90 | | - | |
| 89 | + | |
91 | 90 | | |
92 | | - | |
| 91 | + | |
93 | 92 | | |
94 | | - | |
| 93 | + | |
95 | 94 | | |
96 | | - | |
| 95 | + | |
97 | 96 | | |
98 | | - | |
| 97 | + | |
99 | 98 | | |
100 | | - | |
| 99 | + | |
101 | 100 | | |
102 | | - | |
| 101 | + | |
103 | 102 | | |
104 | | - | |
| 103 | + | |
105 | 104 | | |
106 | | - | |
| 105 | + | |
107 | 106 | | |
108 | | - | |
| 107 | + | |
109 | 108 | | |
110 | | - | |
| 109 | + | |
111 | 110 | | |
112 | | - | |
| 111 | + | |
113 | 112 | | |
114 | | - | |
| 113 | + | |
115 | 114 | | |
116 | | - | |
| 115 | + | |
117 | 116 | | |
118 | | - | |
| 117 | + | |
119 | 118 | | |
120 | | - | |
| 119 | + | |
121 | 120 | | |
122 | | - | |
| 121 | + | |
123 | 122 | | |
124 | | - | |
| 123 | + | |
125 | 124 | | |
126 | | - | |
| 125 | + | |
127 | 126 | | |
128 | | - | |
| 127 | + | |
129 | 128 | | |
130 | | - | |
| 129 | + | |
131 | 130 | | |
132 | | - | |
| 131 | + | |
133 | 132 | | |
134 | | - | |
| 133 | + | |
135 | 134 | | |
136 | | - | |
| 135 | + | |
137 | 136 | | |
138 | | - | |
| 137 | + | |
139 | 138 | | |
140 | | - | |
| 139 | + | |
141 | 140 | | |
142 | | - | |
| 141 | + | |
143 | 142 | | |
144 | | - | |
| 143 | + | |
145 | 144 | | |
146 | | - | |
| 145 | + | |
147 | 146 | | |
148 | | - | |
| 147 | + | |
149 | 148 | | |
150 | | - | |
| 149 | + | |
151 | 150 | | |
152 | | - | |
| 151 | + | |
153 | 152 | | |
154 | | - | |
| 153 | + | |
155 | 154 | | |
156 | | - | |
| 155 | + | |
157 | 156 | | |
158 | 157 | | |
159 | 158 | | |
| |||
323 | 322 | | |
324 | 323 | | |
325 | 324 | | |
326 | | - | |
| 325 | + | |
327 | 326 | | |
328 | | - | |
| 327 | + | |
329 | 328 | | |
330 | | - | |
| 329 | + | |
331 | 330 | | |
332 | | - | |
| 331 | + | |
333 | 332 | | |
334 | | - | |
| 333 | + | |
335 | 334 | | |
336 | | - | |
| 335 | + | |
337 | 336 | | |
338 | | - | |
| 337 | + | |
339 | 338 | | |
340 | | - | |
| 339 | + | |
341 | 340 | | |
342 | | - | |
| 341 | + | |
343 | 342 | | |
344 | | - | |
| 343 | + | |
345 | 344 | | |
346 | | - | |
| 345 | + | |
347 | 346 | | |
348 | 347 | | |
349 | 348 | | |
| |||
633 | 632 | | |
634 | 633 | | |
635 | 634 | | |
636 | | - | |
637 | 635 | | |
638 | 636 | | |
639 | 637 | | |
| |||
714 | 712 | | |
715 | 713 | | |
716 | 714 | | |
717 | | - | |
718 | 715 | | |
719 | | - | |
| 716 | + | |
720 | 717 | | |
| 718 | + | |
721 | 719 | | |
722 | 720 | | |
723 | 721 | | |
| |||
756 | 754 | | |
757 | 755 | | |
758 | 756 | | |
759 | | - | |
760 | 757 | | |
761 | | - | |
| 758 | + | |
762 | 759 | | |
| 760 | + | |
763 | 761 | | |
764 | 762 | | |
765 | 763 | | |
| |||
808 | 806 | | |
809 | 807 | | |
810 | 808 | | |
811 | | - | |
812 | 809 | | |
813 | 810 | | |
814 | 811 | | |
| |||
Lines changed: 9 additions & 10 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
97 | 97 | | |
98 | 98 | | |
99 | 99 | | |
| 100 | + | |
100 | 101 | | |
101 | 102 | | |
102 | 103 | | |
103 | 104 | | |
104 | 105 | | |
105 | 106 | | |
106 | | - | |
107 | | - | |
108 | | - | |
109 | | - | |
| 107 | + | |
| 108 | + | |
| 109 | + | |
| 110 | + | |
110 | 111 | | |
111 | 112 | | |
112 | 113 | | |
| |||
145 | 146 | | |
146 | 147 | | |
147 | 148 | | |
148 | | - | |
149 | | - | |
150 | | - | |
151 | | - | |
| 149 | + | |
| 150 | + | |
| 151 | + | |
| 152 | + | |
152 | 153 | | |
153 | 154 | | |
154 | 155 | | |
| |||
196 | 197 | | |
197 | 198 | | |
198 | 199 | | |
199 | | - | |
200 | 200 | | |
201 | | - | |
202 | 201 | | |
203 | 202 | | |
204 | 203 | | |
| |||
0 commit comments