-
Notifications
You must be signed in to change notification settings - Fork 49
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
G0B1 support #68
base: main
Are you sure you want to change the base?
G0B1 support #68
Conversation
…ose are named differently
Btw, I made some pretty big DMA code changes over the past week (Just published in new version), so this PR may need some adjustments after the PAC issues are sorted. (Hopefully no special code will be needed depending on how the PAC changes resolve? Ie ideally they have the same API as other G0 variants?) |
…erent for them compared to G0
…Seems like FIFO for USART still needs some attention
Alright, the HAL compiles now, but unfortunately the G0B1 chip is also not yet supported on the probe-run project so I couldn't try the quickstart out yet :/ Could we keep this PR open until this is resolved? Not sure you want to merge something which hasn't been tested on hardware yet. |
Yes; this stays open. Are you able to make it work by selecting a diff target in probe-run? I've been able to do that in the past. We can still merge without hardware testing. Great work. Would you prefer a merge now, or after testing? |
Hey again, sorry for the delay, lot of stuff going on in real-life. If merging now without hardware testing is fine with you I would prefer it. I have tried running probe-run with a different target, but unfortunately I can't get that to work 😄 the probe-rs project already has g0b1 support on their dev-branch but havent released yet, and compiling probe-run with the dev-branch of probe-rs opens up different issues as well. As soon as probe-run releases, I will try testing again and see if anything breaks. |
Sweet. I think I broke it with significant DMA changes. Will figure out how to resolve conflicts. |
I tried to target a WB55RG this past weekend and was faced with a bunch of DMA-related errors, ostensibly due to some microcontrollers having two DMA peripherals. I'm just dipping my toe into Rust waters, so I'm not ready with a fix, but I'm looking at it myself. Below is the compiler output from my compilation attempt:
I plan to dig around a bit, and if I can figure out how to compile, I'll happily contribute a PR. (Again, I'm just starting with Rust, so we'll see what success I have.) |
This is due to my DMA changes; the Github repo is currently broken on WB due to this; I need to fix this before WB55 compiles again. |
OK, should now work. |
That clinches it: You're about 62 or 63 kinds of awesome! Thank you for the quick turn on that, I look forward to continuing my exploration of Rust courtesy your help. |
Should only be compiled with this PAC, which I am currently trying to merge.