Incorrect information for ZBHCI_CMD_DATA_CONFIRM

[wes58]

When I receive ZBHCI_CMD_DATA_CONFIRM message I get the following:
hci_uart: 0x3fc97bb0 55 82 00 00 09 2b 02 36 b5 01 0b 00 19 00 32 aa
zbhci: Type: 0x8200
zbhci: (Data Confirm)
zbhci: Status: 0x36
zbhci: Src Ep: 0x2
zbhci: Aps Cnt: 0xb5

This is incorrect.
Telink ZGC TOOL shows the following:
22-09-09 08:22:21.544841 recv<--:55 82 00 00 09 c7 02 36 b5 01 0b 00 19 00 a4 aa
hci commandid: 0x8200 (ZBHCI_CMD_DATA_CONFIRM)
payload len: 9
dstAddr_mode:0x02(APS_SHORT_DSTADDR_WITHEP)
dst_addr: 0x36b5
src_endpoint: 0x01
dst_endpoint: 0x0b
cluster_id: 0x0019(OTA)
status:0x0(SUCCESS)
aps_cnt:0xa4

This makes more sense. Can you please have a look at this?

[lbuque]

But telink sdk only sends Status, Src Ep and Aps Cnt.

[wes58]

They must have fixed it in the latest SDK v3.6.8. if(pApsDataCnf->dstAddrMode == APS_DSTADDR_EP_NOTPRESETNT){ *pBuf++ = pApsDataCnf->srcEndpoint; }else if(pApsDataCnf->dstAddrMode == APS_SHORT_GROUPADDR_NOEP){ COPY_U16TOBUFFER_BE(pBuf, pApsDataCnf->dstAddr.addr_short); pBuf += 2; *pBuf++ = pApsDataCnf->srcEndpoint; }else if(pApsDataCnf->dstAddrMode == APS_SHORT_DSTADDR_WITHEP){ COPY_U16TOBUFFER_BE(pBuf, pApsDataCnf->dstAddr.addr_short); pBuf += 2; *pBuf++ = pApsDataCnf->srcEndpoint; *pBuf++ = pApsDataCnf->dstEndpoint; }else if(pApsDataCnf->dstAddrMode == APS_LONG_DSTADDR_WITHEP){ ZB_IEEE_ADDR_REVERT(pBuf, pApsDataCnf->dstAddr.addr_long); pBuf += 8; *pBuf++ = pApsDataCnf->srcEndpoint; *pBuf++ = pApsDataCnf->dstEndpoint; } COPY_U16TOBUFFER_BE(pBuf, pApsDataCnf->clusterId); pBuf += 2; *pBuf++ = pApsDataCnf->status; *pBuf++ = pApsDataCnf->apsCnt; zbhciTx(ZBHCI_CMD_DATA_CONFIRM, (u8)(pBuf - conf), conf);

[lbuque]

Adapting SDK v3.6.8.2

https://github.com/Xinyuan-LilyGO/T-ZigBee/tree/adapt_3.6.8.2
https://github.com/Xinyuan-LilyGO/telink-zigbee-sdk/tree/3.6.8.2

This is the compiled tlsr8258 firmware:

bin.zip

Any feedback is welcome!