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

Commit 5ba43fdc authored by Henri Chataing's avatar Henri Chataing
Browse files

RootCanal: return the correct mode in Read Scan Enable 

Fixes the PTS test GAP/DM/LEP/BV-05-C. Supposedly the PTS is
reading back the scan_enable value to apply bit changes
to it

Test: PTS test GAP/DM/LEP/BV-05-C
Change-Id: I4bed5a6e47cb16001eb51c8ee51d365f2d85ed03
parent d2c1ddf7

tools/rootcanal/model/controller/dual_mode_controller.cc

+12 −1
Original line number Diff line number Diff line
@@ -1808,8 +1808,19 @@ void DualModeController::ReadScanEnable(CommandView command) { 
  auto command_view = gd_hci::ReadScanEnableView::Create(

      gd_hci::DiscoveryCommandView::Create(command));

  ASSERT(command_view.IsValid());



  bool inquiry_scan = link_layer_controller_.GetInquiryScanEnable();

  bool page_scan = link_layer_controller_.GetPageScanEnable();



  bluetooth::hci::ScanEnable scan_enable =

      inquiry_scan && page_scan

          ? bluetooth::hci::ScanEnable::INQUIRY_AND_PAGE_SCAN

      : inquiry_scan ? bluetooth::hci::ScanEnable::INQUIRY_SCAN_ONLY

      : page_scan    ? bluetooth::hci::ScanEnable::PAGE_SCAN_ONLY

                     : bluetooth::hci::ScanEnable::NO_SCANS;



  send_event_(bluetooth::hci::ReadScanEnableCompleteBuilder::Create(

      kNumCommandPackets, ErrorCode::SUCCESS, gd_hci::ScanEnable::NO_SCANS));

      kNumCommandPackets, ErrorCode::SUCCESS, scan_enable));

}



void DualModeController::WriteScanEnable(CommandView command) {

tools/rootcanal/model/controller/link_layer_controller.h

+4 −0
Original line number Diff line number Diff line
@@ -349,8 +349,12 @@ class LinkLayerController { 
  void SetInquiryMaxResponses(uint8_t max);

  void Inquiry();



  bool GetInquiryScanEnable() { return inquiry_scan_enable_; }

  void SetInquiryScanEnable(bool enable);



  bool GetPageScanEnable() { return page_scan_enable_; }

  void SetPageScanEnable(bool enable);



  uint16_t GetPageTimeout();

  void SetPageTimeout(uint16_t page_timeout);