This thread has been locked.

If you have a related question, please click the "Ask a related question" button in the top right corner. The newly created question will be automatically linked to this question.

CC2340R2: Disabling 2Mbps PHY

Expert 3890 points
Part Number: CC2340R2


Tool/software:

Hi team,

My customer added code to disable the 2Mb PHY referencing the related thread as a guide.

They run CSTAT on their code and CSTAT is flagging the memcpy above as an issue. CSTAT is complaining that &pMsg->pReturnParam[1] only points to 1 byte. The code works as expected so I assume pMsg->pReturnParam[1] points to at least 8 bytes. They will probably ignore this CSTAT issue, but I was wondering if there is something they can reference in our documentation when/if someone questions if there is an issue with this line. It could be something as simple as ‘the data returned in the HCI_LE_READ_LOCAL_SUPPORTED_FEATURES event is 8 bytes’.

I know the BLE5-Stack API Documentation contains some information on the return values but when digging through it, the remote features shows the feature array as 8 bytes:

But we are not able to find the equivalent for the read local features. Given what we see in the read remote features event structure, can they assume for reading the local features, that pReturnParams points to an 8 byte array?

Thanks,

Luke

  • Hello Luke,

    Thanks for reaching out. I believe it must be the same array size. Inside LL_EXT_SetLocalSupportedFeatures (in ll.c) the feature set is limited to LL_MAX_FEATURE_SET_SIZE (which is set to 8 bytes inside the ll.h). This could be better documented, so thanks for providing the feedback.

    Please let me know if it is clear enough.

    BR.

    David.