8000 Add nRF52840DK-based Thread tutorial board definition by lschuermann · Pull Request #3979 · tock/tock · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Add nRF52840DK-based Thread tutorial board definition #3979

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

Merged
merged 8 commits into from
May 12, 2024

Conversation

lschuermann
Copy link
Member
@lschuermann lschuermann commented May 8, 2024

Pull Request Overview

This pull request adds a board definition for the nRR52840DK board we'll be using for the Thread tutorial at CPS-IoT Week. It's still a draft as we might need to add more things / split things out into other PRs.

Testing Strategy

Running the tutorial apps.

TODO or Help Wanted

  • Documentation for the board.
  • Should be moved to boards/tutorials.
  • Split out the lib/bin crate refactor in the nrf52840dk crate into its own PR?

Documentation Updated

  • Updated the relevant files in /docs, or no updates are required.

Formatting

  • Ran make prepush.

@lschuermann lschuermann marked this pull request as draft May 8, 2024 13:46
@bradjc
Copy link
Contributor
bradjc commented May 8, 2024

While I wish we didn't have to move code to a new file, this seems like a necessary step forward. I think it is worth it to have a convention for supporting boards with shields/peripherals. Since we are pretty invested in the nrf52840dk, this seems like a good place to start.

Can we merge the bin/lib split first?

@lschuermann
Copy link
Member Author

Can we merge the bin/lib split first?

Yup, working on that right now!

@lschuermann
Copy link
Member Author

@bradjc Implemented in #3981, I'll rebase this PR on that.

@lschuermann lschuermann force-pushed the cps-iot-24-tutorial-board branch from 644b8db to 4a59e87 Compare May 8, 2024 20:19
@lschuermann lschuermann force-pushed the cps-iot-24-tutorial-board branch from dda8843 to 6eb9b54 Compare May 10, 2024 15:32
@lschuermann lschuermann marked this pull request as ready for review May 10, 2024 15:34
@lschuermann
Copy link
Member Author

This should be ready as soon as CI passes.

Comment on lines -421 to +425
8 => &nrf52840_peripherals.gpio_port[Pin::P1_10],
9 => &nrf52840_peripherals.gpio_port[Pin::P1_11],
// Avoid exposing the I2C pins to userspace, as these are used in
// some tutorials (e.g., `nrf52840dk-thread-tutorial`).
//
// In the future we might want to make this configurable.
//
// 8 => &nrf52840_peripherals.gpio_port[Pin::P1_10],
// 9 => &nrf52840_peripherals.gpio_port[Pin::P1_11],
Copy link
Contributor

Choose a reason for hiding this comment

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

Ah yes, the dark side of the moon.

Copy link
Member Author
@lschuermann lschuermann May 11, 2024

Choose a reason for hiding this comment

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

Yeah... This is a larger issue around composability right now though...

Comment on lines +12 to +15
[features]
default = ["screen_ssd1306"]
screen_ssd1306 = []
screen_sh1106 = []
Copy link
Contributor

Choose a reason for hiding this comment

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

Cargo features just do not work for us do they.

Copy link
Member Author

Choose a reason for hiding this comment

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

😕 I know, do you feel like we should remove this? Really this is only such that we can still easily support users with the wrong model / develop using the other screen that we have.

@lschuermann
Copy link
Member Author
lschuermann commented May 11, 2024

Can we merge this today? I want to rebuild the tutorial VM image with this PR landed some time tomorrow.

@lschuermann
Copy link
Member Author

Merging as we need this for the tutorial, if this breaks anything we should feel free to revert afterwards.

@lschuermann lschuermann added this pull request to the merge queue May 12, 2024
Merged via the queue into master with commit af8b6fc May 12, 2024
18 checks passed
@lschuermann lschuermann deleted the cps-iot-24-tutorial-board branch May 12, 2024 12:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0