RE: AVCTP and AVRCP packet decoder file structure in btmon

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Sure Luiz!

However, I've one more small doubt. If I'm submitting a series of patches
Such as 1/4, 2/4, 3/4, 4/4... I guess in such scenarios no need to ensure that
the patches separately compiles. What we need to ensure is that all the patches together
gets compiled properly. Am I mistaken?

> -----Original Message-----
> From: Luiz Augusto von Dentz [mailto:luiz.dentz@xxxxxxxxx]
> Sent: Monday, August 04, 2014 4:15 PM
> To: Vikrampal
> Cc: linux-bluetooth@xxxxxxxxxxxxxxx; Dmitry Kasatkin
> Subject: Re: AVCTP and AVRCP packet decoder file structure in btmon
> 
> Hi Vikrampal,
> 
> We can start by reusing avctp.c to place AV/C and AVRCP decoders, later on
> we can review if it is the case of splitting. Also please note that
> android/Android.mk needs updating if we change anything on the build of
> btmon.
> 
> On Mon, Aug 4, 2014 at 11:48 AM, Vikrampal <vikram.pal@xxxxxxxxxxx>
> wrote:
> > Hi,
> >
> > I'm working on AVRCP packet decoder for Bluetooth monitor module.
> > There's a separate file avctp.c/h for AVCTP packet decoder.
> >
> > The file avctp.c is not a big file as such. So, I'm thinking whether
> > or not I should include AVRCP decoder in this file itself.
> >
> > Shall I create a separate file for AVRCP decoding?
> >
> > If both AVCTP & AVRCP go in the same file then what should be the
> > appropriate name of the file or shall I create a separate file for
> > AVRCP as well.
> >
> > Please advise me. Thanks!
> >
> > Regards,
> > Vikram
> >
> 
> 
> 
> --
> Luiz Augusto von Dentz

Regards,
Vikram

--
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




[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux