esp32-weather-epd icon indicating copy to clipboard operation
esp32-weather-epd copied to clipboard

Always do SPI remapping

Open blind-oracle opened this issue 9 months ago • 1 comments

First, thanks for the awesome project!

I had a small problem when configuring the firmware - when the board is set to the recommended Despi C02 then the SPI bus is not remapped to the pins that are specified in the config.cpp. When using different ESP32 boards (I've used ESP32-S3) it causes frustration since you can't get the display working.

I'd recommend to just move this block out of #ifdef so that it executes for all board types (shouldn't hurt I guess)

  SPI.end();
  SPI.begin(PIN_EPD_SCK,
            PIN_EPD_MISO,
            PIN_EPD_MOSI,
            PIN_EPD_CS);

blind-oracle avatar May 10 '24 17:05 blind-oracle

`Thanks for troubleshooting this and presenting a solution. I am in the middle of moving, but I will apply this fix in the coming weeks.

lmarzen avatar May 13 '24 16:05 lmarzen