Hi Luiz,
> -----Original Message-----
> From: Vikrampal [mailto:[email protected]]
> Sent: Thursday, January 08, 2015 2:17 PM
> To: 'Luiz Augusto von Dentz'
> Cc: '[email protected]'; '[email protected]'
> Subject: RE: [PATCH 6/6] monitor: Add AVRCP SetBrowsedPlayer support
>
> Hi Luiz,
>
> > -----Original Message-----
> > From: Luiz Augusto von Dentz [mailto:[email protected]]
> > Sent: Wednesday, January 07, 2015 6:55 PM
> > To: Vikrampal Yadav
> > Cc: [email protected]; [email protected]
> > Subject: Re: [PATCH 6/6] monitor: Add AVRCP SetBrowsedPlayer support
> >
> > Hi Vikram,
> >
> > On Wed, Jan 7, 2015 at 6:23 AM, Vikrampal Yadav
> > <[email protected]> wrote:
> > > Support for decoding AVRCP SetBrowsedPlayer added in Bluetooth
> > monitor.
> > >
> > > Channel: 65 len 12 ctrl 0x0102 [PSM 27 mode 3] {chan 1}
> > > I-frame: Unsegmented TxSeq 1 ReqSeq 1
> > > AVCTP Browsing: Command: type 0x00 label 0 PID 0x110e
> > > AVRCP: SetBrowsedPlayer: len 0x0002
> > > PlayerID: 0x0001 (1)
> >
> > Can you please add the response frame as well.
>
> I couldn't find any response, not even in hcidump.
I'm using MecApp tool for validating it.
>
> >
> > > ---
> > > monitor/avctp.c | 86
> > > ++++++++++++++++++++++++++++++++++++++++++++++++++++++++-
> > > 1 file changed, 85 insertions(+), 1 deletion(-)
> > >
> > > diff --git a/monitor/avctp.c b/monitor/avctp.c index
> > > af91ecc..0a1e92d
> > > 100644
> > > --- a/monitor/avctp.c
> > > +++ b/monitor/avctp.c
> > > @@ -1637,11 +1637,95 @@ static bool avrcp_control_packet(struct
> > avctp_frame *avctp_frame)
> > > }
> > > }
> > >
> > > +static bool avrcp_set_browsed_player(struct avctp_frame
> > > +*avctp_frame) {
> > > + struct l2cap_frame *frame = &avctp_frame->l2cap_frame;
> > > + uint32_t items;
> > > + uint16_t id, uids, charset;
> > > + uint8_t status, folders, indent = 2;
> > > +
> > > + if (avctp_frame->hdr & 0x02)
> > > + goto response;
> > > +
> > > + if (!l2cap_frame_get_be16(frame, &id))
> > > + return false;
> > > +
> > > + print_field("%*cPlayerID: 0x%04x (%u)", indent, ' ', id, id);
> > > + return true;
> > > +
> > > +response:
> > > + if (!l2cap_frame_get_u8(frame, &status))
> > > + return false;
> > > +
> > > + print_field("%*cStatus: 0x%02x (%s)", indent, ' ', status,
> > > +
> > > + error2str(status));
> > > +
> > > + if (!l2cap_frame_get_be16(frame, &uids))
> > > + return false;
> > > +
> > > + print_field("%*cUIDCounter: 0x%04x (%u)", indent, ' ', uids,
> > > + uids);
> > > +
> > > + if (!l2cap_frame_get_be32(frame, &items))
> > > + return false;
> > > +
> > > + print_field("%*cNumber of Items: 0x%08x (%u)", indent, ' ',
> > > +
> > > + items, items);
> > > +
> > > + if (!l2cap_frame_get_be16(frame, &charset))
> > > + return false;
> > > +
> > > + print_field("%*cCharsetID: 0x%04x (%s)", indent, ' ',
> > > + charset,
> > > +
> > > + charset2str(charset));
> > > +
> > > + if (!l2cap_frame_get_u8(frame, &folders))
> > > + return false;
> > > +
> > > + print_field("%*cFolder Depth: 0x%02x (%u)", indent, ' ',
> > > + folders,
> > > +
> > > + folders);
> > > +
> > > + for (; folders > 0; folders--) {
> > > + uint8_t len;
> > > +
> > > + if (!l2cap_frame_get_u8(frame, &len))
> > > + return false;
> > > +
> > > + printf("Folder: ");
> > > + for (; len > 0; len--) {
> > > + uint8_t c;
> > > +
> > > + if (!l2cap_frame_get_u8(frame, &c))
> > > + return false;
> > > +
> > > + printf("%1c", isprint(c) ? c : '.');
> > > + }
> > > + printf("\n");
> > > + }
> > > +
> > > + return true;
> > > +}
> > > +
> > > static bool avrcp_browsing_packet(struct avctp_frame *avctp_frame) {
> > > struct l2cap_frame *frame = &avctp_frame->l2cap_frame;
> > > + uint16_t len;
> > > + uint8_t pduid;
> > > +
> > > + if (!l2cap_frame_get_u8(frame, &pduid))
> > > + return false;
> > > +
> > > + if (!l2cap_frame_get_be16(frame, &len))
> > > + return false;
> > > +
> > > + print_field("AVRCP: %s: len 0x%04x", pdu2str(pduid), len);
> > > +
> > > + switch (pduid) {
> > > + case AVRCP_SET_BROWSED_PLAYER:
> > > + avrcp_set_browsed_player(avctp_frame);
> > > + break;
> > > + default:
> > > + packet_hexdump(frame->data, frame->size);
> > > + }
> > >
> > > - packet_hexdump(frame->data, frame->size);
> > > return true;
> > > }
> > >
> > > --
> > > 1.9.1
> > >
> >
> >
> >
> > --
> > Luiz Augusto von Dentz
>
> Regards,
> Vikram
Regards,
Vikram