Skip to content
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

MPH-30 shows wrong temperature #2043

Open
PaulKissinger opened this issue Jan 5, 2025 · 4 comments · May be fixed by #2094
Open

MPH-30 shows wrong temperature #2043

PaulKissinger opened this issue Jan 5, 2025 · 4 comments · May be fixed by #2094
Assignees

Comments

@PaulKissinger
Copy link

Describe the bug
Shown temperature is 100 °C to high. MPH-30 shows 120 °C instead of 20 °C

To Reproduce

  1. Flash dev build
  2. Plug USB-C power into cold MHP-30
  3. See error --> 120 °C instead of 20 °C

Expected behavior
correct temperature measurement

Details of your device:

  • Device: MPH-30
  • Release: 2.23D.000EBA7B 2024-12-26
  • Power adapter being used: USB-C PD (QC-92) (Part of MHP-30 delivery)

Additional context
Correct temp measurment with V2.22E.C308FE8C

@Ralim
Copy link
Owner

Ralim commented Jan 5, 2025

Oh yay, this could be a regression from trying to fix the temperature reading for the newer models of the MHP-30. Will likely need someone to dig into more what its going on with the temp sense hardware.

@PaulKissinger
Copy link
Author

If you have an hint for me how where to start, i'll do this :D

@PaulKissinger PaulKissinger changed the title MPH-30 shows shows wrong temperature MPH-30 shows wrong temperature Jan 6, 2025
@wu9226
Copy link

wu9226 commented Feb 3, 2025

Had the same issue on v2.23

Ralim added a commit that referenced this issue Mar 4, 2025
@Ralim Ralim linked a pull request Mar 4, 2025 that will close this issue
1 task
@Ralim
Copy link
Owner

Ralim commented Mar 4, 2025

Hia, if you have a chance could you test on the builds from this PR: #2094

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants