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

Commit 026fb3d8 authored by Myles Watson's avatar Myles Watson Committed by android-build-merger
Browse files

AVRCP: Route CType NOT_IMPLEMENTED as a response 

am: 8aff065f

Change-Id: I1e9c0c55cc7f510f04c6a6add316248c6ae6ca52
parents ce935163 8aff065f

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.