Skip to content

Conversation

kwd-doodling
Copy link
Contributor

They're previously developed against zephyr 3.7 and out of date. Expose them in header file iut_zephyr.h.
They'll be used in incoming pm tests.

Copilot

This comment was marked as outdated.

@kwd-doodling kwd-doodling force-pushed the dwang4_iut_shell_suspend branch from 91b2138 to fb15ca6 Compare July 31, 2025 08:44
@kwd-doodling kwd-doodling requested a review from Copilot July 31, 2025 08:44
Copilot

This comment was marked as outdated.

@kwd-doodling kwd-doodling force-pushed the dwang4_iut_shell_suspend branch from fb15ca6 to 9d47dbc Compare August 1, 2025 00:33
likongintel
likongintel previously approved these changes Aug 1, 2025
Copy link

@Copilot 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 the shell suspend/resume functionality to work with the latest Zephyr version, migrating from Zephyr 3.7 compatibility. The code adapts to changes in the Zephyr shell UART backend structure and exposes the functions in a new header file for upcoming power management tests.

  • Updates shell UART backend structure from shell_uart to shell_uart_polling
  • Changes timer field access from direct pointer to structure member reference
  • Exposes shell suspend/resume functions in new header file iut_zephyr.h

Reviewed Changes

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

File Description
zephyr/iut_test/src/shell_iut.c Updates shell UART backend structure type and timer field access for latest Zephyr compatibility
zephyr/iut_test/include/iut_zephyr.h New header file exposing shell suspend/resume functions with conditional compilation

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

They're previously developed against zephyr 3.7 and out of date.
Expose them in header file iut_zephyr.h.
They'll be used in incoming pm tests.

Signed-off-by: Dong Wang <[email protected]>
Copy link

@Copilot 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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@kwd-doodling kwd-doodling requested a review from Copilot August 26, 2025 03:24
Copy link

@Copilot 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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@kwd-doodling kwd-doodling merged commit 1f3ef8b into main Aug 26, 2025
4 checks passed
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 this pull request may close these issues.

2 participants