Re: [PATCH 0/7] L2CAP control field + AVRCP browsing PDU parsing

[Luiz Augusto von Dentz <luiz.dentz@gmail.com>]

Hi Vikram,

On Wed, Dec 3, 2014 at 9:08 AM, Vikrampal <vikram.pal@samsung.com> wrote:
> Hi Luiz,
>
>> -----Original Message-----
>> From: Luiz Augusto von Dentz [mailto:luiz.dentz@gmail.com]
>> Sent: Friday, November 28, 2014 2:07 PM
>> To: Vikrampal Yadav
>> Cc: linux-bluetooth@vger.kernel.org; Dmitry Kasatkin; cpgs@samsung.com
>> Subject: Re: [PATCH 0/7] L2CAP control field + AVRCP browsing PDU parsing
>>
>> Hi Vikram,
>>
>> On Fri, Nov 21, 2014 at 3:47 PM, Vikrampal Yadav
>> <vikram.pal@samsung.com> wrote:
>> > Support for ERTM control bytes adjustment, parsing L2CAP extended
>> > control field and AVRCP SetBrowsedPlayer added in Bluetooth monitor.
>> >
>> > Vikrampal Yadav (7):
>> >   monitor: Adjust for ERTM control bytes
>> >   monitor: Make the parameter name generic
>> >   monitor: Add functionality to store extented control in DB
>> >   monitor: Extract extended L2CAP extended control field
>> >   monitor: Add support for parsing L2CAP extended control field
>> >   monitor: Add support for parsing L2CAP control field
>> >   monitor: Add AVRCP SetBrowsedPlayer support
>> >
>> >  monitor/avctp.c |  86 +++++++++++++++++++-  monitor/l2cap.c | 242
>> > ++++++++++++++++++++++++++++++++++++++++++++++++++++++--
>> >  2 files changed, 322 insertions(+), 6 deletions(-)
>> >
>> > --
>> > 1.9.1
>>
>> Sorry for the delay, this one got lost in my inbox, most patches seems to be
>> missing the output they generate, could you please add it.
>
> Most patches in this series are helper patches for the last patch in this series.
> The final patch actually prints the output which shows the work done by
> Other patches.

Actually from patch 4/7 onward they all print different output and I
remember commenting before that I want to see then in the patches so
we can avoid format errors, typos etc.

-- 
Luiz Augusto von Dentz