nodemcu / nodemcu-firmware

Lua based interactive firmware for ESP8266, ESP8285 and ESP32
https://nodemcu.readthedocs.io
MIT License
7.64k stars 3.12k forks source link

Give the SPI module a chance of working... #3496

Closed pjsg closed 2 years ago

pjsg commented 2 years ago

Fixes #3494.

There were a number of things broken about the spi support.

This PR fixes all of the above and it appears to work (at least it drives the one device that I have tried it on!)

marcelstoer commented 2 years ago

requested a review from marcelstoer

Sorry Philip, I wish I did understand that code. I guess @jmattsson should take a look instead.

jmattsson commented 2 years ago

Other than the documentation issue Marcel picked up on, this looks good as far as I can see. Feel free to merge once that command->cmd has been adressed!