Skip to content

Fix imxrt nxstyle issues - #20134

Merged
acassis merged 8 commits into
apache:masterfrom
tiiuae:fix_imxrt_nxstyle_issues
Sep 14, 2026
Merged

acassis merged 8 commits into
apache:masterfrom
tiiuae:fix_imxrt_nxstyle_issues

Conversation

@jlaitine

@jlaitine jlaitine commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

Summary

While working on #20114 , I found several nxstyle issues currently present in imxrt.

This PR fixes the style issues for those files, which the PR mentioned above touches. There are more style issues in other files in the same directory, which may need to be addressed in the future if/when those files need to be touched.

Note: In the github diff the imxrt_usbdev.c diff looks horrible. In reality it is not that big, I used another diff program (beyond compare) to check that there are no actual code changes, just whitespace changes and added braces.

Impact

No functional impact, only nxstyle issues fixed.

Testing

Tested that the nxstyle now passes for the files.

Add a missing blank line

Signed-off-by: Jukka Laitinen <jukka.laitinen@tii.ae>
Fix alignment issues

Signed-off-by: Jukka Laitinen <jukka.laitinen@tii.ae>
Add blank lines, fix alignment and add braces to switch-case

Signed-off-by: Jukka Laitinen <jukka.laitinen@tii.ae>
Add braces to switch-case

Signed-off-by: Jukka Laitinen <jukka.laitinen@tii.ae>
Add a missing blank line

Signed-off-by: Jukka Laitinen <jukka.laitinen@tii.ae>
Fix alignment in multiple places

Signed-off-by: Jukka Laitinen <jukka.laitinen@tii.ae>
Fix alignment, add blank lines and add braces where missing.

Signed-off-by: Jukka Laitinen <jukka.laitinen@tii.ae>
Add blank lines, fix alignment and add braces to switch-case

Signed-off-by: Jukka Laitinen <jukka.laitinen@tii.ae>
@github-actions github-actions Bot added Arch: arm Issues related to ARM (32-bit) architecture Size: XL The size of the change in this PR is very large. Consider breaking down the PR into smaller pieces. labels Sep 14, 2026
@github-actions

Copy link
Copy Markdown

MemBrowse Memory Report

No memory changes detected for:

@acassis
acassis merged commit 885bdef into apache:master Sep 14, 2026
38 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Arch: arm Issues related to ARM (32-bit) architecture Size: XL The size of the change in this PR is very large. Consider breaking down the PR into smaller pieces.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants