From 91546c0e88e72239478b8072eb55f6f82637ba0d Mon Sep 17 00:00:00 2001 From: Edward Poot Date: Wed, 6 Mar 2024 16:33:41 +0100 Subject: [PATCH] Do not broadcast link success Link success should not be a broadcast message. In my case, this prevented successful pairing. Your prior change to change `rx_id` to broadcast is correct when it concerns advertising the speed change, but it does not make sense (at least not to me) during the discovery/pairing phase. --- components/zehnder/zehnder.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/components/zehnder/zehnder.cpp b/components/zehnder/zehnder.cpp index d5efa28..2b2e3b5 100644 --- a/components/zehnder/zehnder.cpp +++ b/components/zehnder/zehnder.cpp @@ -306,7 +306,7 @@ void ZehnderRF::rfHandleReceived(const uint8_t *const pData, const uint8_t dataL (void) memset(this->_txFrame, 0, FAN_FRAMESIZE); // Clear frame data pTxFrame->rx_type = FAN_TYPE_MAIN_UNIT; // Set type to main unit - pTxFrame->rx_id = 0x00; // Broadcast + pTxFrame->rx_id = pResponse->tx_id; // Set ID to the ID of the main unit pTxFrame->tx_type = this->config_.fan_my_device_type; pTxFrame->tx_id = this->config_.fan_my_device_id; pTxFrame->ttl = FAN_TTL;