Skip to content

[MIMXRT1180]Add fsl_gpt.c to build for clock timer feature#7

Open
HonestQiao wants to merge 1 commit into
RT-Thread-packages:masterfrom
HonestQiao:imxrt1180-nxp-evk-timer-v1
Open

[MIMXRT1180]Add fsl_gpt.c to build for clock timer feature#7
HonestQiao wants to merge 1 commit into
RT-Thread-packages:masterfrom
HonestQiao:imxrt1180-nxp-evk-timer-v1

Conversation

@HonestQiao

@HonestQiao HonestQiao commented Jul 3, 2026

Copy link
Copy Markdown

Summary by CodeRabbit

  • New Features
    • Added support for the clock timer when the related board option is enabled, allowing the timer driver to be included automatically.
  • Bug Fixes
    • Improved build configuration so the correct timer support is picked up only when needed.

@coderabbitai

coderabbitai Bot commented Jul 3, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 98364924-1104-42b5-813f-1af8d1349a05

📥 Commits

Reviewing files that changed from the base of the PR and between 8b90dd5 and 11e4853.

📒 Files selected for processing (1)
  • MIMXRT1180/SConscript

Walkthrough

Adds a conditional check in the MIMXRT1180 SConscript build file to include the GPT driver source file when the BSP_USING_CLOCK_TIMER dependency is enabled.

Changes

Build configuration update

Layer / File(s) Summary
Conditional GPT driver inclusion
MIMXRT1180/SConscript
Adds a GetDepend(['BSP_USING_CLOCK_TIMER']) check that appends MIMXRT1189/drivers/fsl_gpt.c to the src list when enabled.

Estimated code review effort: 1 (Trivial) | ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main change: adding fsl_gpt.c to the MIMXRT1180 build for the clock timer feature.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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.

1 participant