Skip to content

Commit

Permalink
The SERIAL_RX_PULLUP_PIN fix/hack is no longer needed
Browse files Browse the repository at this point in the history
We no longer need to explicitly pull up the serial read pin on boards that are based on the Huzzah ESP32 breakout board as the core now sets things up correctly.

Also override the upload command for the OpenEVSE WiFi board to use curl to upload via the HTTP upload, this is a lot more reliable for that board
  • Loading branch information
jeremypoulter committed Jun 15, 2022
1 parent c5eae59 commit 917734b
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 9 deletions.
4 changes: 1 addition & 3 deletions platformio.ini
Original file line number Diff line number Diff line change
Expand Up @@ -177,7 +177,6 @@ build_flags =
-D WIFI_BUTTON_PRESSED_STATE=LOW
-D RAPI_PORT=Serial
-D DEBUG_PORT=Serial2
-D SERIAL_RX_PULLUP_PIN=3

[env:openevse_huzzah32]
board = featheresp32
Expand All @@ -191,7 +190,6 @@ build_flags =
-D WIFI_BUTTON_PRESSED_STATE=LOW
-D RAPI_PORT=Serial
-D DEBUG_PORT=Serial2
-D SERIAL_RX_PULLUP_PIN=3

[env:openevse_featheresp32]
board = featheresp32
Expand Down Expand Up @@ -223,12 +221,12 @@ build_flags =
-D WIFI_BUTTON_PRESSED_STATE=LOW
-D RAPI_PORT=Serial
-D DEBUG_PORT=Serial1
-D SERIAL_RX_PULLUP_PIN=3
-D I2C_SDA=21
-D I2C_SCL=22
-D ENABLE_MCP9808
-D ENABLE_PN532
-D TX1=16
upload_command = curl -F firmware=@$SOURCE http://$UPLOAD_PORT/update --progress-bar | cat

[env:openevse_esp32-gateway]
#For hardware older than RevE
Expand Down
6 changes: 0 additions & 6 deletions src/main.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -265,12 +265,6 @@ void event_send(JsonDocument &event)
void hardware_setup()
{
debug_setup();

#ifdef SERIAL_RX_PULLUP_PIN
// https://forums.adafruit.com/viewtopic.php?f=57&t=153553&p=759890&hilit=esp32+serial+pullup#p769168
pinMode(SERIAL_RX_PULLUP_PIN, INPUT_PULLUP);
#endif

enableLoopWDT();
}

Expand Down

0 comments on commit 917734b

Please sign in to comment.