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

stm32/spi: new_blocking_txonly_nosck hangs on v2+ #2902

Closed
Dirbaio opened this issue May 1, 2024 · 2 comments · Fixed by #3035
Closed

stm32/spi: new_blocking_txonly_nosck hangs on v2+ #2902

Dirbaio opened this issue May 1, 2024 · 2 comments · Fixed by #3035
Labels
e-stm32 kind-bug Something isn't working

Comments

@Dirbaio
Copy link
Member

Dirbaio commented May 1, 2024

All SPI operations on an SPI created with new_blocking_txonly_nosck hang on chips with SPIv3+ SPIv2+.

It hangs in spin_until_rx_ready.

The only difference vs new_blocking_txonly is we don't configure any pin as SCK. Seems strange the SPI would care, intuitively I'd say SCK should be only an output so the SPI shouldn't care if it goes "nowhere". Perhaps it needs to be "looped back" somehow. Hopefully it's only for the RX part, so we can fix it by actually not doing any RX at all.

@Dirbaio Dirbaio added kind-bug Something isn't working e-stm32 labels May 1, 2024
@Dirbaio
Copy link
Member Author

Dirbaio commented Jun 2, 2024

fixed in #3028

@Dirbaio
Copy link
Member Author

Dirbaio commented Jun 2, 2024

Reopening since maybe we should try to get it working instead of just disabling it. There's two people that have confirmed it's working for them (in G0 the author of #1345, then another in G4). So i'm very confused why it's always hanging in tests for me...

@Dirbaio Dirbaio closed this as completed Jun 2, 2024
@Dirbaio Dirbaio reopened this Jun 2, 2024
@Dirbaio Dirbaio changed the title stm32/spi: new_blocking_txonly_nosck hangs on v3+ stm32/spi: new_blocking_txonly_nosck hangs on v2+ Jun 2, 2024
Dirbaio added a commit that referenced this issue Jun 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
e-stm32 kind-bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant