From 48de631e04920364a27a7341b9a79141b2e038d0 Mon Sep 17 00:00:00 2001 From: geeksville Date: Sun, 24 May 2020 16:34:18 -0700 Subject: [PATCH] disable activelyReceiving for sx1262 for now - it doesn't yet work --- docs/software/nrf52-TODO.md | 3 ++- src/mesh/SX1262Interface.cpp | 4 +++- src/nrf52/main-nrf52.cpp | 1 + 3 files changed, 6 insertions(+), 2 deletions(-) diff --git a/docs/software/nrf52-TODO.md b/docs/software/nrf52-TODO.md index e48320ae..f29248c5 100644 --- a/docs/software/nrf52-TODO.md +++ b/docs/software/nrf52-TODO.md @@ -6,7 +6,8 @@ Minimum items needed to make sure hardware is good. -- find out why we reboot while debugging +- fix activelyReceiving for sx1262 +- find out why we reboot while debugging - seems to be power? try using external supply - DONE install a hardfault handler for null ptrs (if one isn't already installed) - test my hackedup bootloader on the real hardware - Use the PMU driver on real hardware diff --git a/src/mesh/SX1262Interface.cpp b/src/mesh/SX1262Interface.cpp index ad305a4f..bab1bf99 100644 --- a/src/mesh/SX1262Interface.cpp +++ b/src/mesh/SX1262Interface.cpp @@ -104,7 +104,9 @@ void SX1262Interface::startReceive() /** Could we send right now (i.e. either not actively receving or transmitting)? */ bool SX1262Interface::isActivelyReceiving() { - return lora.getPacketLength() > 0; + return false; // FIXME + // FIXME this is not correct - often always true - need to add an extra conditional + // return lora.getPacketLength() > 0; } bool SX1262Interface::sleep() diff --git a/src/nrf52/main-nrf52.cpp b/src/nrf52/main-nrf52.cpp index 5c38f0f0..e0d1ac5e 100644 --- a/src/nrf52/main-nrf52.cpp +++ b/src/nrf52/main-nrf52.cpp @@ -69,6 +69,7 @@ void nrf52Setup() { auto why = NRF_POWER->RESETREAS; + // per https://infocenter.nordicsemi.com/index.jsp?topic=%2Fcom.nordic.infocenter.nrf52832.ps.v1.1%2Fpower.html DEBUG_MSG("Reset reason: 0x%x\n", why); // Not yet on board