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.

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

cpu/sam0_common: ADC keep muxpos as legacy define #18688

Merged
merged 1 commit into from
Oct 5, 2022

Conversation

benpicco
Copy link
Contributor

@benpicco benpicco commented Oct 4, 2022

Contribution description

The .muxpos member was renamed to .inputctrl to make clearer that it can contain more than just the positive mux.

This however makes working with external boards much more painful, especially when bisecting something.

The semantics of the field did not change, so also provide the old name in a union to make the transition less painful.

Testing procedure

Issues/PRs references

follow-up to #18658

@benpicco benpicco requested review from dylad and keestux as code owners October 4, 2022 13:47
@github-actions github-actions bot added Area: cpu Area: CPU/MCU ports Platform: ARM Platform: This PR/issue effects ARM-based platforms labels Oct 4, 2022
@benpicco benpicco added the Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer label Oct 4, 2022
@benpicco benpicco added this to the Release 2022.10 milestone Oct 4, 2022
@benpicco benpicco added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Oct 4, 2022
@riot-ci
Copy link

riot-ci commented Oct 4, 2022

Murdock results

✔️ PASSED

5843ff4 cpu/sam0_common: ADC keep muxpos as legacy define

Success Failures Total Runtime
1979 0 1979 06m:55s

Artifacts

This only reflects a subset of all builds from https://ci-prod.riot-os.org. Please refer to https://ci.riot-os.org for a complete build for now.

Copy link
Member

@maribu maribu left a comment

Choose a reason for hiding this comment

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

Ahh, yes. The transition to a newer version of C allows anonymous unions as struct members :)

@dylad
Copy link
Member

dylad commented Oct 5, 2022

Don't forget to remove this in a few months :)

@dylad dylad merged commit 1a1b41b into RIOT-OS:master Oct 5, 2022
@benpicco benpicco deleted the cpu/sam0_common-muxpos branch October 5, 2022 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants