Skip to content

Commit 91db7a3

Browse files
authored
Merge pull request #484 from pennam/unor4-warnings
UNO R4 WiFi fix warnings
2 parents 4c0d20e + 605e16b commit 91db7a3

File tree

3 files changed

+3
-2
lines changed

3 files changed

+3
-2
lines changed

src/ota/OTA.h

+1-2
Original file line numberDiff line numberDiff line change
@@ -50,8 +50,7 @@ using ArduinoCloudOTA = ESP32OTACloudProcess;
5050
#include "implementation/OTAUnoR4.h"
5151
using ArduinoCloudOTA = UNOR4OTACloudProcess;
5252

53-
// TODO Check if a macro already exist
54-
constexpr uint32_t OtaMagicNumber = 0x234110020; // TODO check this value is correct
53+
constexpr uint32_t OtaMagicNumber = 0x23411002;
5554

5655
#else
5756
#error "This Board doesn't support OTA"

src/tls/utility/TLSClientMqtt.cpp

+1
Original file line numberDiff line numberDiff line change
@@ -58,6 +58,7 @@ void TLSClientMqtt::begin(ConnectionHandler & connection) {
5858
* also present in older firmware revisions
5959
* https://github.com/arduino/uno-r4-wifi-usb-bridge/blob/f09ca94fdcab845b8368d4435fdac9f6999d21d2/certificates/certificates.pem#L852
6060
*/
61+
(void)connection;
6162
#elif defined(ARDUINO_ARCH_ESP32)
6263
setCACertBundle(x509_crt_bundle);
6364
#elif defined(ARDUINO_ARCH_ESP8266)

src/tls/utility/TLSClientOta.cpp

+1
Original file line numberDiff line numberDiff line change
@@ -54,6 +54,7 @@ void TLSClientOta::begin(ConnectionHandler &connection) {
5454
/* AWS Root CAs are configured in uno-r4-wifi-usb-bridge/libraries/Arduino_ESP32_OTA
5555
* https://github.com/arduino-libraries/Arduino_ESP32_OTA/blob/fc755e7d1d3946232107e2590662ee08d6ccdec4/src/tls/amazon_root_ca.h
5656
*/
57+
(void)connection;
5758
#elif defined(ARDUINO_ARCH_ESP32)
5859
setCACertBundle(x509_crt_bundle);
5960
#elif defined(ARDUINO_ARCH_ESP8266)

0 commit comments

Comments
 (0)