From d74f6ee237a0363a7097b12e4d62d97a9e9b1485 Mon Sep 17 00:00:00 2001 From: Michael Miller Date: Sat, 7 May 2022 14:35:47 -0700 Subject: [PATCH] Fix Warnings (#566) --- src/internal/NeoEsp8266UartMethod.cpp | 4 +++- src/internal/NeoEsp8266UartMethod.h | 2 +- src/internal/NeoEspBitBangMethod.h | 2 +- src/internal/TwoWireBitBangImple.h | 2 +- src/internal/TwoWireHspiImple.h | 2 +- src/internal/Ws2801GenericMethod.h | 2 +- 6 files changed, 8 insertions(+), 6 deletions(-) diff --git a/src/internal/NeoEsp8266UartMethod.cpp b/src/internal/NeoEsp8266UartMethod.cpp index 66324b9..1b3bbcc 100644 --- a/src/internal/NeoEsp8266UartMethod.cpp +++ b/src/internal/NeoEsp8266UartMethod.cpp @@ -139,7 +139,9 @@ void NeoEsp8266UartInterruptContext::Detach(uint8_t uartNum) ETS_UART_INTR_ENABLE(); } -void IRAM_ATTR NeoEsp8266UartInterruptContext::Isr(void* param) +// The xtos_1int handler calls with param1 as the arg, param2 as a pointer +// to an exception frame in memory. +void IRAM_ATTR NeoEsp8266UartInterruptContext::Isr(void* param, [[maybe_unused]] void* exceptionFrame) { // make sure this is for us if (param == s_uartInteruptContext) diff --git a/src/internal/NeoEsp8266UartMethod.h b/src/internal/NeoEsp8266UartMethod.h index da61e7f..1d303da 100644 --- a/src/internal/NeoEsp8266UartMethod.h +++ b/src/internal/NeoEsp8266UartMethod.h @@ -79,7 +79,7 @@ private: volatile const uint8_t* _asyncBuffEnd; volatile static NeoEsp8266UartInterruptContext* s_uartInteruptContext[2]; - static void IRAM_ATTR Isr(void* param); + static void IRAM_ATTR Isr(void* param, void* exceptionFrame); }; // this template feature class is used a base for all others and contains diff --git a/src/internal/NeoEspBitBangMethod.h b/src/internal/NeoEspBitBangMethod.h index 6667119..053e6eb 100644 --- a/src/internal/NeoEspBitBangMethod.h +++ b/src/internal/NeoEspBitBangMethod.h @@ -318,7 +318,7 @@ public: return _sizeData; }; - void applySettings(const SettingsObject& settings) + void applySettings([[maybe_unused]] const SettingsObject& settings) { } diff --git a/src/internal/TwoWireBitBangImple.h b/src/internal/TwoWireBitBangImple.h index eede5f9..92627a8 100644 --- a/src/internal/TwoWireBitBangImple.h +++ b/src/internal/TwoWireBitBangImple.h @@ -88,7 +88,7 @@ public: } } - void applySettings(const SettingsObject& settings) + void applySettings([[maybe_unused]] const SettingsObject& settings) { } diff --git a/src/internal/TwoWireHspiImple.h b/src/internal/TwoWireHspiImple.h index 5a96bd5..1c774be 100644 --- a/src/internal/TwoWireHspiImple.h +++ b/src/internal/TwoWireHspiImple.h @@ -81,7 +81,7 @@ public: _hspi->writeBytes(const_cast(data), dataSize); } - void applySettings(const SettingsObject& settings) + void applySettings([[maybe_unused]] const SettingsObject& settings) { _speed.applySettings(settings); } diff --git a/src/internal/Ws2801GenericMethod.h b/src/internal/Ws2801GenericMethod.h index 24521c6..fea18a1 100644 --- a/src/internal/Ws2801GenericMethod.h +++ b/src/internal/Ws2801GenericMethod.h @@ -112,7 +112,7 @@ public: return _sizeData; }; - void applySettings(const SettingsObject& settings) + void applySettings([[maybe_unused]] const SettingsObject& settings) { _wire.applySettings(settings); }