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

ITM broken on Nucleo H743/H753 boards #346

Closed
bcantrill opened this issue Dec 25, 2021 · 1 comment
Closed

ITM broken on Nucleo H743/H753 boards #346

bcantrill opened this issue Dec 25, 2021 · 1 comment
Assignees

Comments

@bcantrill
Copy link
Collaborator

The tale of woe is in #345, but the synopsis is: ITM is (and has been) broken on the Nucleo H743/H753 boards due to TRACESWO being repurposed for SPI. We clearly need to pick another SPI block (or at the very least, pin for SPI3_SCK) to use on this board!

@bcantrill bcantrill self-assigned this Dec 28, 2021
bcantrill added a commit that referenced this issue Dec 28, 2021
ITM broken on Nucleo H743/H753 boards (#346)
SPI overruns are not handled, resulting in infinite loop (#349)
SPI can suffer from overruns due to TX/RX asymmetry (#350)

In addition to fixing these bugs, this work removes the Gimlet hostflash
task from the H743/H753 boards -- and replaces it with the venerable ping.
@cbiffle
Copy link
Collaborator

cbiffle commented Jan 5, 2022

I think this gets at a higher-level question, which is: permanently assigning a SPI block on the Nucleo board is weird, since we tend to use the Nucleo boards as general purpose I/O widgets. I'm not sure what to do about this.

bcantrill added a commit that referenced this issue Jan 10, 2022
This fixes the following bugs:

- Spi.exchange() hangs if RX bytes exceeds TX bytes (#341)
- ITM broken on Nucleo H743/H753 boards (#346)
- SPI overruns are not handled, resulting in infinite loop (#349)
- SPI can suffer from overruns due to TX/RX asymmetry (#350)
    
In addition to fixing these bugs, this work removes the Gimlet hostflash
task from the H743/H753 boards -- and replaces it with the venerable
"ping" task (which demonstrates a panicking task).
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

No branches or pull requests

2 participants