Hi Vikram, On Thu, Sep 4, 2014 at 11:40 AM, Vikrampal Yadav <vikram.pal@xxxxxxxxxxx> wrote: > Support for decoding AVRCP GetCurrentPlayerApplicationSettingValue > added in Bluetooth monitor. > --- > monitor/avctp.c | 49 +++++++++++++++++++++++++++++++++++++++++++++++++ > 1 file changed, 49 insertions(+) > > diff --git a/monitor/avctp.c b/monitor/avctp.c > index 64d4b58..c462381 100644 > --- a/monitor/avctp.c > +++ b/monitor/avctp.c > @@ -609,6 +609,54 @@ response: > return true; > } > > +static bool avrcp_get_current_player_value(struct l2cap_frame *frame, > + uint8_t ctype, uint8_t len, > + uint8_t indent) > +{ > + uint8_t num; > + > + if (!l2cap_frame_get_u8(frame, &num)) > + return false; > + > + if (ctype > AVC_CTYPE_GENERAL_INQUIRY) > + goto response; > + > + print_field("%*cAttributeCount: 0x%02x", (indent - 8), ' ', num); > + > + for (; num > 0; num--) { > + uint8_t attr; > + > + if (!l2cap_frame_get_u8(frame, &attr)) > + return false; > + > + print_field("%*cAttributeID: 0x%02x (%s)", (indent - 8), > + ' ', attr, attr2str(attr)); > + } > + > + return true; > + > +response: > + print_field("%*cValueCount: 0x%02x", (indent - 8), ' ', num); > + > + for (; num > 0; num--) { > + uint8_t attr, value; > + > + if (!l2cap_frame_get_u8(frame, &attr)) > + return false; > + > + print_field("%*cAttributeID: 0x%02x (%s)", (indent - 8), > + ' ', attr, attr2str(attr)); > + > + if (!l2cap_frame_get_u8(frame, &value)) > + return false; > + > + print_field("%*cValueID: 0x%02x (%s)", (indent - 8), > + ' ', value, value2str(attr, value)); > + } > + > + return true; > +} > + > struct avrcp_ctrl_pdu_data { > uint8_t pduid; > bool (*func) (struct l2cap_frame *frame, uint8_t ctype, uint8_t len, > @@ -619,6 +667,7 @@ static const struct avrcp_ctrl_pdu_data avrcp_ctrl_pdu_table[] = { > { 0x10, avrcp_get_capabilities }, > { 0x11, avrcp_list_player_attributes }, > { 0x12, avrcp_list_player_values }, > + { 0x13, avrcp_get_current_player_value }, > { } > }; > > -- > 1.9.1 This along with the other 4 patches in the original series have been applied, thanks. -- Luiz Augusto von Dentz -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html