Skip to content

Commit

Permalink
Revert "Fix ESP32 slow response when using UDP as in emulation (#21470)"
Browse files Browse the repository at this point in the history
This reverts commit f132663.
  • Loading branch information
arendst committed May 29, 2024
1 parent 7c98e0d commit 2347f37
Show file tree
Hide file tree
Showing 4 changed files with 2 additions and 12 deletions.
2 changes: 0 additions & 2 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,6 @@ All notable changes to this project will be documented in this file.

### Fixed
- Telegram TLS fingerprint, remove CA validation (#21514)
- ESP32 I2S multiple fixes (#21511)
- ESP32 slow response when using UDP as in emulation (#21470)

### Removed

Expand Down
1 change: 0 additions & 1 deletion RELEASENOTES.md
Original file line number Diff line number Diff line change
Expand Up @@ -149,7 +149,6 @@ The latter links can be used for OTA upgrades too like ``OtaUrl https://ota.tasm
- Telegram TLS fingerprint, remove CA validation [#21514](https://github.com/arendst/Tasmota/issues/21514)
- Zigbee crash when removing `ZbName` [#21449](https://github.com/arendst/Tasmota/issues/21449)
- Avoid connection errors when switching to safeboot to upload OTA firmware [#21428](https://github.com/arendst/Tasmota/issues/21428)
- ESP32 slow response when using UDP as in emulation [#21470](https://github.com/arendst/Tasmota/issues/21470)
- ESP32 BLE fix scanning [#21451](https://github.com/arendst/Tasmota/issues/21451)
- I2S APLL not supported on all SOCs [#21483](https://github.com/arendst/Tasmota/issues/21483)
- Webradio crash with invalid url [#21446](https://github.com/arendst/Tasmota/issues/21446)
Expand Down
8 changes: 2 additions & 6 deletions tasmota/tasmota_support/support_udp.ino
Original file line number Diff line number Diff line change
Expand Up @@ -60,10 +60,7 @@ bool UdpDisconnect(void)
{
if (udp_connected) {
// flush any outgoing packet
PortUdp.flush(); // Does nothing in core3
#ifdef ESP32
PortUdp.clear(); // New with core3. Does what flush() did in core2;
#endif
PortUdp.flush();
#ifdef ESP8266
UdpCtx.disconnect();
#endif
Expand Down Expand Up @@ -133,8 +130,7 @@ void PollUdp(void)

int32_t len = PortUdp.read(packet_buffer, UDP_BUFFER_SIZE -1);
packet_buffer[len] = 0;
// PortUdp.flush();
PortUdp.clear(); // New with core3. Does what flush() did in core2;
PortUdp.flush();
AddLog(LOG_LEVEL_DEBUG_MORE, PSTR("UDP: Packet (%d/%d)"), len, pack_len);
#endif // ESP32

Expand Down
3 changes: 0 additions & 3 deletions tasmota/tasmota_xdrv_driver/xdrv_10_scripter.ino
Original file line number Diff line number Diff line change
Expand Up @@ -1406,9 +1406,6 @@ void Script_Stop_UDP(void) {
if (!glob_script_mem.udp_flags.udp_used) return;
if (glob_script_mem.udp_flags.udp_connected) {
glob_script_mem.Script_PortUdp.flush();
#ifdef ESP32
glob_script_mem.Script_PortUdp.clear(); // New with core3. Does what flush() did in core2;
#endif
glob_script_mem.Script_PortUdp.stop();
glob_script_mem.udp_flags.udp_connected = 0;
}
Expand Down

0 comments on commit 2347f37

Please sign in to comment.