Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

ESP32: Dual-SPI broken #3153

Open
bjoernQ opened this issue Feb 19, 2025 · 2 comments
Open

ESP32: Dual-SPI broken #3153

bjoernQ opened this issue Feb 19, 2025 · 2 comments
Labels
chip:esp32 Issue related to ESP32 chip peripheral:spi SPI peripheral

Comments

@bjoernQ
Copy link
Contributor

bjoernQ commented Feb 19, 2025

Running the spi_halfduplex_read_manufacturer_id example gives wrong results for Dual

Single [c8, 16]
Dual [20, 5b]
Quad [c8, 16]

While I'm sure I tested this before I get the same results with older esp-hal version. (But note: I am using a different flash chip now since I fried the one I used for testing, but it works fine for e.g. ESP32-C3, ESP32-S2, ESP32-C6 so it's not generally the flash chip's fault)

@bjoernQ bjoernQ added chip:esp32 Issue related to ESP32 chip peripheral:spi SPI peripheral status:needs-attention This should be prioritized labels Feb 19, 2025
@bjoernQ bjoernQ changed the title ESP32: Dual-SPI broken in multiple ways ESP32: Dual-SPI broken Feb 19, 2025
@Dominaezzz
Copy link
Collaborator

Might be time to improve the HIL tests, maybe we can use the slave driver to properly test the master driver.
I suppose this won't work for chips with just one SPI peripheral

@bugadani
Copy link
Contributor

To clarify the title, ESP32 is broken, not dual spi

@MabezDev MabezDev removed the status:needs-attention This should be prioritized label Feb 19, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
chip:esp32 Issue related to ESP32 chip peripheral:spi SPI peripheral
Projects
Status: Todo
Development

No branches or pull requests

4 participants