Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

cpu/samd5x/cpu.c: reformat code #20864

Merged
merged 1 commit into from
Sep 19, 2024
Merged

cpu/samd5x/cpu.c: reformat code #20864

merged 1 commit into from
Sep 19, 2024

Conversation

maribu
Copy link
Member

@maribu maribu commented Sep 17, 2024

Contribution description

This formats the code to improve readability and apply the coding convention. This is a whitespace only change that will not change generated binaries.

Testing procedure

Confirm that only white space is changed. (I did sneak in dropping an unused header. But other than that.)

Issues/PRs references

None

@maribu
cpu/samd5x/cpu.c: reformat code
20e2186 
This formats the code to improve readability and apply the coding
convention. This is a whitespace only change that will not change
generated binaries.
@maribu maribu added Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Sep 17, 2024
@github-actions github-actions bot added Platform: ARM Platform: This PR/issue effects ARM-based platforms Area: cpu Area: CPU/MCU ports labels Sep 17, 2024
@riot-ci
Copy link

riot-ci commented Sep 17, 2024

Murdock results

✔️ PASSED

20e2186 cpu/samd5x/cpu.c: reformat code

Success Failures Total Runtime
10197 0 10197 14m:38s

Artifacts

@mguetschow
Copy link
Contributor

Did you generate those automatically with #20865 ?

@maribu maribu mentioned this pull request Sep 18, 2024
Merged
mguetschow
mguetschow approved these changes Sep 19, 2024
View reviewed changes
Copy link
Contributor

@mguetschow mguetschow left a comment

Choose a reason for hiding this comment

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

LGTM then, thanks!

@maribu maribu added this pull request to the merge queue Sep 19, 2024
Merged via the queue into RIOT-OS:master with commit 20f7328 Sep 19, 2024
34 checks passed
@maribu maribu deleted the cpu/samd5x/cpu-conding-convention branch September 19, 2024 14:18
@maribu
Copy link
Member Author

maribu commented Sep 19, 2024

Thx :)

@benpicco benpicco added this to the Release 2024.10 milestone Nov 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mguetschow mguetschow mguetschow approved these changes

@benpicco benpicco Awaiting requested review from benpicco benpicco is a code owner

@dylad dylad Awaiting requested review from dylad dylad is a code owner

Assignees
No one assigned
Labels
Area: cpu Area: CPU/MCU ports CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer Platform: ARM Platform: This PR/issue effects ARM-based platforms Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation
Projects
None yet
Milestone
Release 2024.10
Development

Successfully merging this pull request may close these issues.
4 participants
@maribu @riot-ci @mguetschow @benpicco