lora: Fix SNR value in SX126x received packets.

projectgus · dpgeorge · commit f8c8875e250e · 2025-04-24T16:20:47.000+10:00
Wasn't being treated as a signed value. Fixes issue #999. Signed-off-by: Angus Gratton <angus@redyak.com.au>
diff --git a/micropython/lora/lora-sx126x/lora/sx126x.py b/micropython/lora/lora-sx126x/lora/sx126x.py
@@ -596,8 +596,9 @@ def _read_packet(self, rx_packet, flags):
         pkt_status = self._cmd("B", _CMD_GET_PACKET_STATUS, n_read=4)
 
         rx_packet.ticks_ms = ticks_ms
-        rx_packet.snr = pkt_status[2]  # SNR, units: dB *4
-        rx_packet.rssi = 0 - pkt_status[1] // 2  # RSSI, units: dBm
+        # SNR units are dB * 4 (signed)
+        rx_packet.rssi, rx_packet.snr = struct.unpack("xBbx", pkt_status)
+        rx_packet.rssi //= -2  # RSSI, units: dBm
         rx_packet.crc_error = (flags & _IRQ_CRC_ERR) != 0
 
         return rx_packet
diff --git a/micropython/lora/lora-sx126x/manifest.py b/micropython/lora/lora-sx126x/manifest.py
@@ -1,3 +1,3 @@
-metadata(version="0.1.4")
+metadata(version="0.1.5")
 require("lora")
 package("lora")
