-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
consolidate the pins #8866
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
base: develop
Are you sure you want to change the base?
consolidate the pins #8866
Conversation
|
@DaneEvans @porkcube @ndoo Many thanks! |
I do have a bunch of Ikoka variants built so could test those as well:
|
|
Generated on c6bd578 tried to flash a nonsense - same symptoms as I used to get on the sense', at least that's what I recall the symptoms were. Yes - the last word is cut off. |
|
the nonsense may have just worked on a reflash. but now it's browning out. And the sense is giving: Rolling back to my known versions worked without issue, although the low voltage warning was still present |
Might be worth trying again now I've fixed the default setup pins, although there might need to be a further tweak. If you drop me a reference to your branch, I'll take another look. |
|
That was after that commit. |
|
The .bin and .uf2 also aren't getting produced by this branch in it's current state, just the .elf Just made fw off 2.7.16 successfully on my updated branches: and successfully used the same target on commit 61e41a8 (divergence between this branch and develop), so it's definitely this branch that broke things. But using it on this branch gives the same crash on reading I2C on both board varieties If I had to guess, the others didn't reboot vsc, and ran on cached targets, so it worked for them. Otherwise I can't explain the discrepancy |
|
Moving to draft, as I can't be sure it's building successfully. |
This PR is a tidying up and extending of the defaults for the Xiao NRF, to allow each of the variations in use to coexist harmoniously whilst also sharing a single variant.h. Pins are assigned as follows:
Basically, Default and I2C variants just swap whether GPS or I2C gets to sit on "real" pins, vs the NFC pads at the back.
The Board-to-board connector version of the WIO radio board is assumed to follow the default pinout apart from that, and BLE legacy just shoves them on the back, because it can.
I've tested it on a sample of 1, and it works on that.
🤝 Attestations