Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bluetooth: controller: df: Fixes truncated transmission of CTE #36958

Commits on Jul 14, 2021

  1. Bluetooth: controller: df: Fixes truncated transmission of CTE

    There was an issue with wrong length of CTE send in connectionless
    mode, with periodic advertising PDUs. Radio peripheral was not
    configured to send CTE with correct length while PDU had CTEInfo
    field informing receiver that CTE is attached to the PDU.
    
    Source of the problem was in ll_df_set_cl_cte_tx_enable function.
    
    Order of parameters in ull_adv_sync_pdu_alloc was wrong.
    ULL_ADV_PDU_HDR_FIELD_CTE_INFO was speficed as hdr_rem_fields.
    Because of that extra_data, memory used to provide CTE configuration
    to LLL, was not allocated. PDU content is prepared in ULL, so CTEInfo
    field included correct information, while Radio was never configured
    by LLL to send CTE.
    
    ull_adv_sync_extra_data_set_clear received a pointer to hdr_data,
    instead of a direct pointer to df_cfg structure. When extra_data
    was allocated correclty, wrong CTE configuration was provided
    copied there and LLL received invalid CTE length. It was different
    than the length in PDUs CTEInfo field.
    
    Signed-off-by: Piotr Pryga <piotr.pryga@nordicsemi.no>
    ppryga-nordic committed Jul 14, 2021
    Configuration menu
    Copy the full SHA
    d2bb909 View commit details
    Browse the repository at this point in the history