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/stm32wl: Enable rtc support #20536

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

cpu/stm32wl: Enable rtc support #20536

wants to merge 1 commit into from

Conversation

FlapKap
Copy link
Contributor

@FlapKap FlapKap commented Apr 3, 2024 •
edited

Contribution description

This PR adds support for the real-time clock on the stm32wl series devices.
Based on information from https://www.st.com/resource/en/reference_manual/dm00451556-stm32wl5x-advanced-armbased-32bit-mcus-with-subghz-radio-solution-stmicroelectronics.pdf

Testing procedure

Issues/PRs references

@github-actions github-actions bot added Platform: ARM Platform: This PR/issue effects ARM-based platforms Area: boards Area: Board ports Area: cpu Area: CPU/MCU ports labels Apr 3, 2024
@krzysztof-cabaj
Copy link
Contributor

krzysztof-cabaj commented Apr 3, 2024 •
edited

Nice PR.

One remark look at commit conventions.

@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 Apr 3, 2024
@riot-ci
Copy link

riot-ci commented Apr 3, 2024

Murdock results

✔️ PASSED

54ad88c initial

Success Failures Total Runtime
10027 0 10027 11m:02s

Artifacts

@FlapKap
Copy link
Contributor Author

FlapKap commented Apr 4, 2024

Nice PR.

One remark look at commit conventions.

Thanks Krzystof. I will fix it before final review. Currently I haven't gotten alarms to work yet so this PR is mainly to gather advice on that

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aabadie aabadie Awaiting requested review from aabadie aabadie will be requested when the pull request is marked ready for review aabadie is a code owner

@vincent-d vincent-d Awaiting requested review from vincent-d vincent-d will be requested when the pull request is marked ready for review vincent-d is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
Area: boards Area: Board ports 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 Platform: ARM Platform: This PR/issue effects ARM-based platforms
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@FlapKap @krzysztof-cabaj @riot-ci @benpicco