Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit d3ac874d authored by Alice Kuo's avatar Alice Kuo
Browse files

Minor refine the broadcast ISO setup comment

1. Change the comment content after adding ADSP offload support
2. Change the comment code place to along with the related code

Bug: 210987580
Test: build
Change-Id: Ib38d7fa5a375a7982dc37a05fed0d8f0ed2a84e4
Merged-In: Ib38d7fa5a375a7982dc37a05fed0d8f0ed2a84e4
parent 192b1dc2
Loading
Loading
Loading
Loading
+7 −6
Original line number Diff line number Diff line
@@ -466,17 +466,18 @@ class BroadcastStateMachineImpl : public BroadcastStateMachine {
  void TriggerIsoDatapathSetup(uint16_t conn_handle) {
    LOG_INFO("conn_hdl=%d", conn_handle);
    LOG_ASSERT(active_config_ != std::nullopt);

    /* Note: For the LC3 software encoding on the Host side, the coding format
     * should be set to 'Transparent' and no codec configuration shall be sent
     * to the controller. 'codec_id_company' and 'codec_id_vendor' shall be
     * ignored if 'codec_id_format' is not set to 'Vendor'.
     */
    auto data_path_id = bluetooth::hci::iso_manager::kIsoDataPathHci;
    if (CodecManager::GetInstance()->GetCodecLocation() !=
        CodecLocation::HOST) {
      data_path_id = bluetooth::hci::iso_manager::kIsoDataPathPlatformDefault;
    }

    /* Note: If the LC3 encoding isn't in the controller side, the coding format
     * should be set to 'Transparent' and no codec configuration shall be sent
     * to the controller. 'codec_id_company' and 'codec_id_vendor' shall be
     * ignored if 'codec_id_format' is not set to 'Vendor'. We currently only
     * support the codecLocation in the Host or ADSP side.
     */
    auto codec_id = sm_config_.codec_wrapper.GetLeAudioCodecId();
    uint8_t hci_coding_format =
        (codec_id.coding_format == le_audio::types::kLeAudioCodingFormatLC3)