From a8247b1aabfd30bc22531ef964ceb9c7741077ab Mon Sep 17 00:00:00 2001 From: Martin Budden Date: Sat, 13 Sep 2025 15:34:15 +0100 Subject: [PATCH 1/2] Fix shadow warning - parameter `byte` shadows `typedef uint8_t byte;` --- cores/esp32/Esp.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/cores/esp32/Esp.cpp b/cores/esp32/Esp.cpp index f911b2d5f20..42c74c56ffc 100644 --- a/cores/esp32/Esp.cpp +++ b/cores/esp32/Esp.cpp @@ -378,7 +378,7 @@ FlashMode_t EspClass::getFlashChipMode(void) { } #endif // if !defined(CONFIG_IDF_TARGET_ESP32P4) -uint32_t EspClass::magicFlashChipSize(uint8_t byte) { +uint32_t EspClass::magicFlashChipSize(uint8_t octet) { /* FLASH_SIZES = { "1MB": 0x00, @@ -391,7 +391,7 @@ uint32_t EspClass::magicFlashChipSize(uint8_t byte) { "128MB": 0x70, } */ - switch (byte & 0x0F) { + switch (octet & 0x0F) { case 0x0: return (1_MB); // 8 MBit (1MB) case 0x1: return (2_MB); // 16 MBit (2MB) case 0x2: return (4_MB); // 32 MBit (4MB) From ee541d52c5dfd7ea2578a7875030ce4fcd97c437 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jan=20Proch=C3=A1zka?= <90197375+P-R-O-C-H-Y@users.noreply.github.com> Date: Tue, 16 Sep 2025 11:38:44 +0200 Subject: [PATCH 2/2] fix(): Use flashbyte intead of byte --- cores/esp32/Esp.cpp | 18 +++++++++--------- cores/esp32/Esp.h | 6 +++--- 2 files changed, 12 insertions(+), 12 deletions(-) diff --git a/cores/esp32/Esp.cpp b/cores/esp32/Esp.cpp index 42c74c56ffc..1ba5e9c76d1 100644 --- a/cores/esp32/Esp.cpp +++ b/cores/esp32/Esp.cpp @@ -378,7 +378,7 @@ FlashMode_t EspClass::getFlashChipMode(void) { } #endif // if !defined(CONFIG_IDF_TARGET_ESP32P4) -uint32_t EspClass::magicFlashChipSize(uint8_t octet) { +uint32_t EspClass::magicFlashChipSize(uint8_t flashByte) { /* FLASH_SIZES = { "1MB": 0x00, @@ -391,7 +391,7 @@ uint32_t EspClass::magicFlashChipSize(uint8_t octet) { "128MB": 0x70, } */ - switch (octet & 0x0F) { + switch (flashByte & 0x0F) { case 0x0: return (1_MB); // 8 MBit (1MB) case 0x1: return (2_MB); // 16 MBit (2MB) case 0x2: return (4_MB); // 32 MBit (4MB) @@ -405,7 +405,7 @@ uint32_t EspClass::magicFlashChipSize(uint8_t octet) { } } -uint32_t EspClass::magicFlashChipSpeed(uint8_t byte) { +uint32_t EspClass::magicFlashChipSpeed(uint8_t flashByte) { #if CONFIG_IDF_TARGET_ESP32C2 /* FLASH_FREQUENCY = { @@ -415,7 +415,7 @@ uint32_t EspClass::magicFlashChipSpeed(uint8_t byte) { "15m": 0x2, } */ - switch (byte & 0x0F) { + switch (flashByte & 0x0F) { case 0xF: return (60_MHz); case 0x0: return (30_MHz); case 0x1: return (20_MHz); @@ -432,7 +432,7 @@ uint32_t EspClass::magicFlashChipSpeed(uint8_t byte) { "20m": 0x2, } */ - switch (byte & 0x0F) { + switch (flashByte & 0x0F) { case 0x0: return (80_MHz); case 0x2: return (20_MHz); default: // fail? @@ -449,7 +449,7 @@ uint32_t EspClass::magicFlashChipSpeed(uint8_t byte) { "12m": 0x2, } */ - switch (byte & 0x0F) { + switch (flashByte & 0x0F) { case 0xF: return (48_MHz); case 0x0: return (24_MHz); case 0x1: return (16_MHz); @@ -467,7 +467,7 @@ uint32_t EspClass::magicFlashChipSpeed(uint8_t byte) { "20m": 0x2, } */ - switch (byte & 0x0F) { + switch (flashByte & 0x0F) { case 0xF: return (80_MHz); case 0x0: return (40_MHz); case 0x1: return (26_MHz); @@ -478,8 +478,8 @@ uint32_t EspClass::magicFlashChipSpeed(uint8_t byte) { #endif } -FlashMode_t EspClass::magicFlashChipMode(uint8_t byte) { - FlashMode_t mode = (FlashMode_t)byte; +FlashMode_t EspClass::magicFlashChipMode(uint8_t flashByte) { + FlashMode_t mode = (FlashMode_t)flashByte; if (mode > FM_SLOW_READ) { mode = FM_UNKNOWN; } diff --git a/cores/esp32/Esp.h b/cores/esp32/Esp.h index 41068fb9e65..0b496c91c1b 100644 --- a/cores/esp32/Esp.h +++ b/cores/esp32/Esp.h @@ -92,9 +92,9 @@ class EspClass { uint32_t getFlashChipSpeed(); FlashMode_t getFlashChipMode(); - uint32_t magicFlashChipSize(uint8_t byte); - uint32_t magicFlashChipSpeed(uint8_t byte); - FlashMode_t magicFlashChipMode(uint8_t byte); + uint32_t magicFlashChipSize(uint8_t flashByte); + uint32_t magicFlashChipSpeed(uint8_t flashByte); + FlashMode_t magicFlashChipMode(uint8_t flashByte); uint32_t getSketchSize(); String getSketchMD5();