Skip to content

RDKEMW-21159: avoid thermal call in deepsleep (#548)#567

Merged
nhanasi merged 1 commit into
support/4.5.2from
RDKEMW-21157-avoid-thermal-call-in-deepsleep
Jul 7, 2026
Merged

RDKEMW-21159: avoid thermal call in deepsleep (#548)#567
nhanasi merged 1 commit into
support/4.5.2from
RDKEMW-21157-avoid-thermal-call-in-deepsleep

Conversation

@apatel859

Copy link
Copy Markdown
Contributor
  • RDKEMW-21159: avoid thermal call in deepsleep

  • Apply suggestions from code review


* RDKEMW-21159: avoid thermal call in deepsleep

Signed-off-by: apatel859 <Amit_Patel5@comcast.com>

* Apply suggestions from code review

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

---------

Signed-off-by: apatel859 <Amit_Patel5@comcast.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings July 7, 2026 18:28
@apatel859 apatel859 requested a review from a team as a code owner July 7, 2026 18:28

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates RDK temperature telemetry collection to avoid reading the thermal sysfs node while the device is in DEEPSLEEP, which can be problematic on some platforms/power modes.

Changes:

  • Query the current power state before reading /sys/class/thermal/thermal_zone0/temp.
  • Skip thermal reads in DEEPSLEEP and emit a placeholder value instead.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread lib/rdk/temperature-telemetry.sh
@nhanasi nhanasi merged commit 023f3ea into support/4.5.2 Jul 7, 2026
3 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 7, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants