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

pkg/openthread: remove deprecated functions #19685

Merged
merged 1 commit into from
Jul 11, 2023

Conversation

maribu
Copy link
Member

@maribu maribu commented May 30, 2023

Contribution description

As the title says.

Testing procedure

Green CI

Issues/PRs references

#19638

@maribu maribu requested a review from jia200x as a code owner May 30, 2023 14:52
@github-actions github-actions bot added Area: network Area: Networking Area: pkg Area: External package ports labels May 30, 2023
@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 May 30, 2023
@riot-ci
Copy link

riot-ci commented May 30, 2023

Murdock results

✔️ PASSED

70e72fc pkg/openthread: remove deprecated functions

Success Failures Total Runtime
6933 0 6933 17m:07s

Artifacts

@benpicco
Copy link
Contributor

bors merge

bors bot added a commit that referenced this pull request Jul 11, 2023
19685: pkg/openthread: remove deprecated functions r=benpicco a=maribu



19798: cpu/nrf53: add I2C and SPI support r=benpicco a=dylad

### Contribution description

This PR provides support for nRF53 SPI and I2C.
It also moves common structs from each nRF CPU folder to `cpu/nrf5x_common` to avoid duplication.
Moreover, since nRF9160 and nRF5340 have shared IRQ for UART/SPI/I2C. Both this families now use a common file to register and manage these interrupts. Note that nRF9160 have different name for its interrupts than nRF5340 but they have the same purpose.

### Testing procedure

Since some structs were moved around, I think this PR should be carefully tested against nRF52, nRF53 and nRF9160 to avoid any issues.
On nRF5340DK-APP, SPI can be tested with its onboard SPI flash.

### Issues/PRs references



Co-authored-by: Marian Buschsieweke <[email protected]>
Co-authored-by: Dylan Laduranty <[email protected]>
@bors
Copy link
Contributor

bors bot commented Jul 11, 2023

Build failed (retrying...):

@bors
Copy link
Contributor

bors bot commented Jul 11, 2023

Build succeeded!

The publicly hosted instance of bors-ng is deprecated and will go away soon.

If you want to self-host your own instance, instructions are here.
For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.

@bors bors bot merged commit 9a46bcd into RIOT-OS:master Jul 11, 2023
@maribu maribu deleted the ot-remove-deprecated branch July 11, 2023 18:06
@maribu
Copy link
Member Author

maribu commented Jul 11, 2023

Hooray :) Thx :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: network Area: Networking Area: pkg Area: External package ports CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants