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

Commit 8aff065f authored by Myles Watson's avatar Myles Watson
Browse files

AVRCP: Route CType NOT_IMPLEMENTED as a response 

Bug: 79698590
Test: net_test_avrcp, Connect to the Anker MP140
Change-Id: Ie88dcea949ecb33bc3d76d86b07ab0396a21500d
parent 3aa80ee8

system/include/hardware/avrcp/avrcp_common.h

+1 −0
Original line number Diff line number Diff line
@@ -31,6 +31,7 @@ enum class CType : uint8_t { 
  CONTROL = 0x0,

  STATUS = 0x1,

  NOTIFY = 0x3,

  NOT_IMPLEMENTED = 0x8,

  ACCEPTED = 0x9,

  REJECTED = 0xa,

  STABLE = 0xc,

system/profile/avrcp/device.cc

+5 −1
Original line number Diff line number Diff line
@@ -60,7 +60,11 @@ void Device::VendorPacketHandler(uint8_t label, 
  CHECK(media_interface_);

  DEVICE_VLOG(3) << __func__ << ": pdu=" << pkt->GetCommandPdu();



  // All CTypes at and above ACCEPTED are all response types.

  // All CTypes at and above NOT_IMPLEMENTED are all response types.

  if (pkt->GetCType() == CType::NOT_IMPLEMENTED) {

    return;

  }



  if (pkt->GetCType() >= CType::ACCEPTED) {

    switch (pkt->GetCommandPdu()) {

      // VOLUME_CHANGED is the only notification we register for while target.