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

add waveshare pico variants #4129

Open
wants to merge 4 commits into
base: dev
Choose a base branch
from

Conversation

crypto-smoke
Copy link

Adds board and target files for Waveshare 2040 LoRa and Plus boards.

@crypto-smoke
Copy link
Author

Not sure why test-all failed.

@aykevl
Copy link
Member

aykevl commented Feb 25, 2024

Not sure why test-all failed.

It's a flaky test that we need to investigate. I've restarted the build.

Copy link
Member

@aykevl aykevl left a comment

Choose a reason for hiding this comment

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

Can you also add these new targets to the smoke tests in GNUmakefile? That way we ensure these targets will at least continue to compile.

src/machine/board_waveshare-rp2040-lora.go Outdated Show resolved Hide resolved
@crypto-smoke
Copy link
Author

Looks like another flaky test caused this to fail? I implemented your requested changes. Bummed this didnt make the cut for 0.31.0

@deadprogram
Copy link
Member

Hello @crypto-smoke thanks for the PR and likewise sorry it did not get in the last release.

Furthermore, now that #4215 has been merged into dev there are some changes needed to this PR in order to get it in. Apologies for that. 😿

Would it be too much trouble to ask for you to rebase against the latest dev and make suitable changes to this PR? Thank you very much.

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.

None yet

3 participants