[PR #414] [MERGED] C6 improvements / fixes (ver 0x001f). #2179

Closed
opened 2026-03-20 20:09:10 +01:00 by sascha_hemi · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/OpenEPaperLink/OpenEPaperLink/pull/414
Author: @skiphansen
Created: 12/14/2024
Status: Merged
Merged: 12/17/2024
Merged by: @skiphansen

Base: masterHead: subghz_c6_debug


📝 Commits (2)

  • c45d120 C6 improvements / fixes (ver 0x001f).
  • 0c82353 Added \r to log messages missing it for EOL consistency,

📊 Changes

5 files changed (+150 additions, -26 deletions)

View changed files

📝 ARM_Tag_FW/OpenEPaperLink_esp32_C6_AP/main/SubGigRadio.c (+105 -9)
📝 ARM_Tag_FW/OpenEPaperLink_esp32_C6_AP/main/cc1101_radio.c (+25 -10)
📝 ARM_Tag_FW/OpenEPaperLink_esp32_C6_AP/main/cc1101_radio.h (+9 -0)
📝 ARM_Tag_FW/OpenEPaperLink_esp32_C6_AP/main/main.c (+6 -6)
📝 ARM_Tag_FW/OpenEPaperLink_esp32_C6_AP/main/radio.c (+5 -1)

📄 Description

  1. Add 2 millisecond timeout for all SubGhz MISO wait loops to prevent watchdog timeouts when/if bad things happen (none observed during testing)
  2. Make CC1101 detection more robust and less intrusive by testing MISO and CSn before trying to read chip version number.
  3. Remove RxFifo overflow "error" reporting since it's confusing to users.
  4. Only call esp_ieee802154_receive_handle_done() when ack != NULL. This is the "fix" for 802.15.4 problems with C6 version 0x001e.

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/OpenEPaperLink/OpenEPaperLink/pull/414 **Author:** [@skiphansen](https://github.com/skiphansen) **Created:** 12/14/2024 **Status:** ✅ Merged **Merged:** 12/17/2024 **Merged by:** [@skiphansen](https://github.com/skiphansen) **Base:** `master` ← **Head:** `subghz_c6_debug` --- ### 📝 Commits (2) - [`c45d120`](https://github.com/OpenEPaperLink/OpenEPaperLink/commit/c45d1206df35a1a0011e16882e8331e44092b0b1) C6 improvements / fixes (ver 0x001f). - [`0c82353`](https://github.com/OpenEPaperLink/OpenEPaperLink/commit/0c82353f3f6b32244295e17c1ce6f443dc872ba2) Added \r to log messages missing it for EOL consistency, ### 📊 Changes **5 files changed** (+150 additions, -26 deletions) <details> <summary>View changed files</summary> 📝 `ARM_Tag_FW/OpenEPaperLink_esp32_C6_AP/main/SubGigRadio.c` (+105 -9) 📝 `ARM_Tag_FW/OpenEPaperLink_esp32_C6_AP/main/cc1101_radio.c` (+25 -10) 📝 `ARM_Tag_FW/OpenEPaperLink_esp32_C6_AP/main/cc1101_radio.h` (+9 -0) 📝 `ARM_Tag_FW/OpenEPaperLink_esp32_C6_AP/main/main.c` (+6 -6) 📝 `ARM_Tag_FW/OpenEPaperLink_esp32_C6_AP/main/radio.c` (+5 -1) </details> ### 📄 Description 1. Add 2 millisecond timeout for all SubGhz MISO wait loops to prevent watchdog timeouts when/if bad things happen (none observed during testing) 2. Make CC1101 detection more robust and less intrusive by testing MISO and CSn before trying to read chip version number. 3. Remove RxFifo overflow "error" reporting since it's confusing to users. 4. Only call esp_ieee802154_receive_handle_done() when ack != NULL. This is the "fix" for 802.15.4 problems with C6 version 0x001e. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
sascha_hemi added the pull-request label 2026-03-20 20:09:10 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/OpenEPaperLink#2179