commaai / commaai/openpilot

hardwared: gracefully handle no thermal readings

Open
#36,690 0 comments 0 reactions 0 assignees View on GitHub
bug
Dominant language
Python
Stars
63.7k
Forks
11.4k
Avg merge
17h 53m
Merged PRs (30d)
171

Description

unclear if this is a hardware issue or not, but it crashes hardwared

`22b0b51eb1991fe6/00000003--cd41c126da`
```
[18.147382] MAIN 0 kernel - spmi spmi-0: pmic_arb_wait_for_done: transaction failed (0x3)
[18.147695] MAIN 0 kernel - qcom,qpnp-temp-alarm c440000.qcom,spmi:qcom,pm8005@4:qcom,temp-alarm@2400: qpnp_tm_read: regmap_bulk_readl failed. sid=4, addr=2408, len=1, rc=-5
```

temperature should probably be NaN or inf in the log, then everything downstream needs to handle those values

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.