From 7c46f4a34395cf4c3e659eb2360405acd06780dd Mon Sep 17 00:00:00 2001 From: liquidraver <504870+liquidraver@users.noreply.github.com> Date: Sun, 7 Jun 2026 13:02:07 +0200 Subject: [PATCH] legacy code cleanup --- zephcore/Kconfig | 2 +- zephcore/adapters/board/ZephyrBoard.cpp | 6 +++--- zephcore/app/CompanionMesh.cpp | 3 +-- zephcore/tools/formatter/src/main.c | 2 +- zephcore/tools/lr1110_updater/src/main.c | 4 ++-- 5 files changed, 8 insertions(+), 9 deletions(-) diff --git a/zephcore/Kconfig b/zephcore/Kconfig index c6d6cef..d21c3d2 100644 --- a/zephcore/Kconfig +++ b/zephcore/Kconfig @@ -282,7 +282,7 @@ config ZEPHCORE_BLE_ADV_SLOW_INTERVAL config ZEPHCORE_BLE_DFU bool "Buttonless BLE DFU (legacy Nordic/Adafruit OTA jump)" - depends on SOC_SERIES_NRF52X || SOC_SERIES_NRF52 + depends on SOC_SERIES_NRF52 default y help Register the legacy Nordic DFU service (00001530/00001531) in the diff --git a/zephcore/adapters/board/ZephyrBoard.cpp b/zephcore/adapters/board/ZephyrBoard.cpp index 5b90663..bc7a914 100644 --- a/zephcore/adapters/board/ZephyrBoard.cpp +++ b/zephcore/adapters/board/ZephyrBoard.cpp @@ -11,7 +11,7 @@ #include #include -#if defined(CONFIG_SOC_SERIES_NRF52X) || defined(CONFIG_SOC_SERIES_NRF52) +#if defined(CONFIG_SOC_SERIES_NRF52) #include /* Adafruit bootloader GPREGRET magic values */ #define BOOTLOADER_DFU_SERIAL_MAGIC 0x4e /* Enter serial DFU mode (CDC only) */ @@ -246,7 +246,7 @@ bool ZephyrBoard::startOTAUpdate(const char *id, char reply[]) bool ZephyrBoard::getBootloaderVersion(char *out, size_t max_len) { -#if defined(CONFIG_SOC_SERIES_NRF52X) || defined(CONFIG_SOC_SERIES_NRF52) +#if defined(CONFIG_SOC_SERIES_NRF52) /* Scan flash for UF2 bootloader version string. * info.txt lives somewhere in the 0xFB000-0xFE000 range depending * on SoftDevice version and bootloader build. */ @@ -291,7 +291,7 @@ uint8_t ZephyrBoard::getStartupReason() const bool ZephyrBoard::isExternalPowered() { -#if defined(CONFIG_SOC_SERIES_NRF52X) || defined(CONFIG_SOC_SERIES_NRF52) +#if defined(CONFIG_SOC_SERIES_NRF52) /* VBUS detect from the POWER peripheral — true when USB/charger is * attached. Read-only status register; safe alongside the BLE controller * (we already poke NRF_POWER->GPREGRET directly elsewhere). */ diff --git a/zephcore/app/CompanionMesh.cpp b/zephcore/app/CompanionMesh.cpp index 84ca8c2..cb7b238 100644 --- a/zephcore/app/CompanionMesh.cpp +++ b/zephcore/app/CompanionMesh.cpp @@ -2962,8 +2962,7 @@ bool CompanionMesh::handleProtocolFrame(const uint8_t *data, size_t len) /* FIRMWARE_VER_CODE 13+: allow requests to a non-contact pubkey by * creating a transient "anon" contact (ADV_TYPE_NONE). These are never * persisted or synced to the app; re-lookup to get the stable slot. */ - ContactInfo anon; - memset(&anon, 0, sizeof(anon)); + ContactInfo anon{}; memcpy(anon.id.pub_key, &data[1], PUB_KEY_SIZE); anon.out_path_len = 0; // zero-hop direct by default anon.type = ADV_TYPE_NONE; // transient/unknown diff --git a/zephcore/tools/formatter/src/main.c b/zephcore/tools/formatter/src/main.c index b462431..1cff1d7 100644 --- a/zephcore/tools/formatter/src/main.c +++ b/zephcore/tools/formatter/src/main.c @@ -33,7 +33,7 @@ #include #include "qspi_probe.h" -#if defined(CONFIG_SOC_SERIES_NRF52X) || defined(CONFIG_SOC_SERIES_NRF52) +#if defined(CONFIG_SOC_SERIES_NRF52) #include #define IS_NRF52 1 #else diff --git a/zephcore/tools/lr1110_updater/src/main.c b/zephcore/tools/lr1110_updater/src/main.c index 603e474..66ca3fe 100644 --- a/zephcore/tools/lr1110_updater/src/main.c +++ b/zephcore/tools/lr1110_updater/src/main.c @@ -31,7 +31,7 @@ #include "lr1110_bootloader.h" #include "lr1110_transceiver_0401.h" -#if defined(CONFIG_SOC_SERIES_NRF52X) || defined(CONFIG_SOC_SERIES_NRF52) +#if defined(CONFIG_SOC_SERIES_NRF52) #include #endif @@ -70,7 +70,7 @@ static void reboot_to_uf2(void) printk("You can now drag-drop ZephCore firmware UF2.\n"); k_msleep(500); -#if defined(CONFIG_SOC_SERIES_NRF52X) || defined(CONFIG_SOC_SERIES_NRF52) +#if defined(CONFIG_SOC_SERIES_NRF52) nrf_power_gpregret_set(NRF_POWER, 0, BOOTLOADER_DFU_UF2_MAGIC); #endif sys_reboot(SYS_REBOOT_COLD);