Skip to content

fix spi names #151

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

Merged
merged 1 commit into from
Dec 16, 2021
Merged

fix spi names #151

merged 1 commit into from
Dec 16, 2021

Conversation

tore-espressif
Copy link
Collaborator

No description provided.

@arktrin
Copy link
Contributor

arktrin commented Dec 17, 2021

This commit breaks my hardware configuration with ESP32-C3 + ST7789 based display. At least SPI2_HOST won't work at all.

@tore-espressif
Copy link
Collaborator Author

Reverted. I'll double check next week. Sorry for the troubles :/

@C47D
Copy link
Collaborator

C47D commented Dec 17, 2021

@arktrin did the initialization of the SPI master failed? I don't have any C3 module to test.

@arktrin
Copy link
Contributor

arktrin commented Dec 17, 2021

I'm not sure what is happening, it compiles without any errors but SPI doesn't work at all.

@C47D
Copy link
Collaborator

C47D commented Dec 17, 2021

It's been a while since I touch the SPI code, but maybe we should add some logging and use idf monitor to try found the issue.

@arktrin
Copy link
Contributor

arktrin commented Dec 18, 2021

Sorry for delay, I'm getting this error for my ESP32-C3:
spi: spi_bus_initialize(762): invalid dma channel, chip only support spi dma channel auto-alloc

@arktrin
Copy link
Contributor

arktrin commented Dec 18, 2021

Just for testing I've replaced spi_bus_initialize(host, &buscfg, (spi_dma_chan_t)dma_channel); with spi_bus_initialize(host, &buscfg, 0); . Now spi configured and dma is not used at all but I'm getting another error:
E (1584) spi_master: check_trans_valid(689): txdata transfer > host maximum

@arktrin
Copy link
Contributor

arktrin commented Dec 18, 2021

Not looking in the right place, spi_bus_initialize(host, &buscfg, 3); is the proper initialization for ESP32-C3 with auto-dma.

@arktrin
Copy link
Contributor

arktrin commented Dec 18, 2021

I've done PR #153 with a slight change to workaround the problem.

@C47D C47D deleted the fix/spi-names branch February 2, 2022 22:53
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

Successfully merging this pull request may close these issues.

3 participants