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

arch/nrf52: nvmc and flash should depends on ALLOW_BSD_COMPONENTS=y #8827

Merged
merged 2 commits into from Apr 21, 2023
Merged

arch/nrf52: nvmc and flash should depends on ALLOW_BSD_COMPONENTS=y #8827

merged 2 commits into from Apr 21, 2023

Conversation

raiden00pl
Copy link
Contributor

@raiden00pl raiden00pl commented Mar 15, 2023 •
edited

Summary

  • arch/nrf52: nvmc and flash should depends on ALLOW_BSD_COMPONENTS=y
  • arch/nrf52: reimplement I-Cache control operations in nrf52_start.c

Impact

Users using nvmc and flash must allow BSD components (CONFIG_ALLOW_BSD_COMPONENTS=y).

Testing

CI

@raiden00pl raiden00pl added the breaking change This change requires a mitigation entry in the release notes. label Mar 15, 2023
@xiaoxiang781216 xiaoxiang781216 merged commit d416ead into apache:master Apr 21, 2023
26 checks passed
@raiden00pl raiden00pl deleted the nrf52_flash branch May 3, 2023 12:19
@jerpelea jerpelea added this to To-Add in Release Notes - 12.2.0 Jun 13, 2023
@jerpelea jerpelea moved this from To-Add to breaking in Release Notes - 12.2.0 Jun 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hartmannathan hartmannathan hartmannathan left review comments

@pkarashchenko pkarashchenko pkarashchenko approved these changes

@xiaoxiang781216 xiaoxiang781216 xiaoxiang781216 approved these changes

Assignees
No one assigned
Labels
breaking change This change requires a mitigation entry in the release notes.
Projects
No open projects
Release Notes - 12.2.0
breaking
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@raiden00pl @pkarashchenko @xiaoxiang781216 @hartmannathan