Hi Andrei, On Fri, Mar 7, 2014 at 2:22 PM, Andrei Emeltchenko <andrei.emeltchenko.news@xxxxxxxxx> wrote: > Hi Luiz, > > On Fri, Mar 07, 2014 at 02:15:40PM +0200, Luiz Augusto von Dentz wrote: >> From: Luiz Augusto von Dentz <luiz.von.dentz@xxxxxxxxx> >> >> This adds avrcp_register_notification which can be used to send >> RegisterNotification PDUs. > > Didn't I sent the same patch already? Why are we redoing the same work? Im implementing the daemon side, this is the problem of sending big patch sets it takes more time and I want to finish the AVRCP quickly so we can have it tested. Btw, Im not sure why you are still defining the PDU length public, this should probably be keep private until we have proper struct defined, in fact I did mention that we might change the way the handlers works in avrcp-lib to incorporate more logic and do things like header checks that the unit test has been doing. -- 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