Skip to content

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

Merged
nhanasi merged 1 commit into
developfrom
RDKEMW-21159-avoid-thermal-call-in-deepsleep
Jul 7, 2026
Merged

RDKEMW-21159: avoid thermal call in deepsleep (#548)#566
nhanasi merged 1 commit into
developfrom
RDKEMW-21159-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

Updates the temperature telemetry script to avoid reading the kernel thermal sysfs node when the device is in deep sleep, aligning telemetry collection with power-state constraints.

Changes:

  • Query current power state before attempting to read /sys/class/thermal/thermal_zone0/temp.
  • Skip thermal-zone read during deep sleep and emit a placeholder temperature string 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 400a9c4 into develop Jul 7, 2026
7 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