Skip to content

RDKEMW-18793a: avoid thermal call in deepsleep#548

Merged
nhanasi merged 2 commits into
support/4.2.1from
RDKEMW-18793-avoid-thermal-call-in-deepsleep
May 20, 2026
Merged

RDKEMW-18793a: avoid thermal call in deepsleep#548
nhanasi merged 2 commits into
support/4.2.1from
RDKEMW-18793-avoid-thermal-call-in-deepsleep

Conversation

@apatel859
Copy link
Copy Markdown

No description provided.

Signed-off-by: apatel859 <Amit_Patel5@comcast.com>
Copilot AI review requested due to automatic review settings May 19, 2026 19:35
@apatel859 apatel859 requested a review from a team as a code owner May 19, 2026 19:35
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

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 temperature telemetry collection to avoid reading the thermal sysfs node while the device is in deep sleep.

Changes:

  • Queries the device power state before collecting board temperature.
  • Reports a deep-sleep placeholder instead of reading /sys/class/thermal/thermal_zone0/temp when in deep sleep.

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

Comment thread lib/rdk/temperature-telemetry.sh Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings May 20, 2026 16:46
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.

Comment thread lib/rdk/temperature-telemetry.sh
Comment thread lib/rdk/temperature-telemetry.sh
@nhanasi nhanasi merged commit 9509729 into support/4.2.1 May 20, 2026
8 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators May 20, 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