Re: [PATCH v4 00/22] Implement AVRCP 1.3 for TG role

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

 



Hi Johan,

On Fri, Aug 12, 2011 at 5:34 AM, Johan Hedberg <johan.hedberg@xxxxxxxxx> wrote:
>
> Hi Lucas,
>
> On Thu, Aug 11, 2011, Lucas De Marchi wrote:
> > Change from previous version:
> >
> > - MediaPlayer interface is created only if remote device is the controller (and
> >   thus we are TG) as suggested by Luiz.
> >
> > Lucas De Marchi (22):
> >   audio: move interface declarations to their headers
> >   avrcp: add skeleton of MediaPlayer interface
> >   avrcp: implement SetProperty() method of MediaPlayer
> >   avrcp: implement ChangePlayback() method
> >   avrcp: implement ChangeTrack() method
> >   avrcp: handle query for company ids
> >   avrcp: handle ListPlayerApplicationSettingAttributes pdu
> >   avrcp: handle ListPlayerApplicationSettingValues pdu
> >   avrcp: handle GetCurrentPlayerAplicationSettingValue pdu
> >   avrcp: handle SetPlayerApplicationSettingValue pdu
> >   avrcp: handle commands for future extension
> >   avrcp: handle InformDisplayableCharacterSet pdu
> >   avrcp: handle InformBatteryStatusOfCT pdu
> >   avrcp: handle GetPlayStatus pdu
> >   avrcp: handle RegisterNotification pdu
> >   avrcp: handle query for supported events
> >   avrcp: handle GetElementAttributes pdu
> >   avrcp: send response for registered events
> >   avrcp: change TG record to use version 1.3
> >   avrcp: update copyright
> >   Add script to test MediaPlayer interface
> >   Update Control documentation
> >
> >  audio/control.c        | 1417 +++++++++++++++++++++++++++++++++++++++++++++++-
> >  audio/control.h        |    7 +-
> >  audio/device.h         |    4 -
> >  audio/manager.c        |   14 +-
> >  audio/manager.h        |    1 +
> >  audio/unix.c           |    1 +
> >  doc/control-api.txt    |   94 ++--
> >  test/test-media-player |  108 ++++
> >  8 files changed, 1580 insertions(+), 66 deletions(-)
> >  create mode 100755 test/test-media-player
>
> All patches have been applied. Thanks.
>
> Additionally applied an extra cleanup patch on top of them for some
> coding style issues. Please take a look that e.g. the avctp_send_event
> change is correct.

Thanks for applying them and doing this cleanup. They are right.


Lucas De Marchi
--
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