Skip to content

common: fixed many of VR Vol sensors (xdpe12284c) showing LCR on yv3-dl#480

Closed
DelphineChiu wants to merge 1 commit into
facebook:mainfrom
Wiwynn:Ian/fby3-fixed_VR-Vol-show-LCR
Closed

common: fixed many of VR Vol sensors (xdpe12284c) showing LCR on yv3-dl#480
DelphineChiu wants to merge 1 commit into
facebook:mainfrom
Wiwynn:Ian/fby3-fixed_VR-Vol-show-LCR

Conversation

@DelphineChiu

Copy link
Copy Markdown

Summary:

  • Fixed VCCIN VR Vol (0x27), P3V3_STBY VR Vol (0x2A), VDDQ_ABC VR Vol (0x2C),
    VDDQ_DEF VR Vol (0x2D) showing LCR. It's caused by mV did not turn to V.

Test Plan:

  • sensor-util showed normal sensor value. - passed

Log:

root@bmc-oob:~# sensor-util slot4 | grep "VR Vol"
VCCIN VR Vol                 (0x27) :    1.70 Volts | (ok)
VCCSA VR Vol                 (0x28) :    0.83 Volts | (ok)
VCCIO VR Vol                 (0x29) :    1.01 Volts | (ok)
P3V3_STBY VR Vol             (0x2A) :    3.30 Volts | (ok)
VDDQ_ABC VR Vol              (0x2C) :    1.24 Volts | (ok)
VDDQ_DEF VR Vol              (0x2D) :    1.24 Volts | (ok)
root@bmc-oob:~#
Summary:
- Fixed VCCIN VR Vol (0x27), P3V3_STBY VR Vol (0x2A), VDDQ_ABC VR Vol (0x2C),
  VDDQ_DEF VR Vol (0x2D) showing LCR. It's caused by mV did not turn to V.

Test Plan:
- sensor-util showed normal sensor value. - passed

Log:
```
root@bmc-oob:~# sensor-util slot4 | grep "VR Vol"
VCCIN VR Vol                 (0x27) :    1.70 Volts | (ok)
VCCSA VR Vol                 (0x28) :    0.83 Volts | (ok)
VCCIO VR Vol                 (0x29) :    1.01 Volts | (ok)
P3V3_STBY VR Vol             (0x2A) :    3.30 Volts | (ok)
VDDQ_ABC VR Vol              (0x2C) :    1.24 Volts | (ok)
VDDQ_DEF VR Vol              (0x2D) :    1.24 Volts | (ok)
root@bmc-oob:~#
```
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Aug 10, 2022
@facebook-github-bot

Copy link
Copy Markdown
Contributor

@GoldenBug has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

3 participants