Re: [PATCH BlueZ] mcp: Implement Next Track and Previous Track commands

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

 



Hi Yauhen,

On Fri, Jun 7, 2024 at 8:15 AM Yauhen Kharuzhy
<yauhen.kharuzhy@xxxxxxxxxx> wrote:
>
> Add implementation of Next/Previous Track commands to audio/mcp profile
> and shared/mcp.{c,h} code.
> ---
>  profiles/audio/mcp.c | 18 ++++++++++++++++--
>  src/shared/mcp.c     | 20 ++++++++++++++++++++
>  src/shared/mcp.h     |  2 ++

Please have the changes to src/shared split from other changes, they
have different licenses and shared may actually be subject to unit
testing.

>  3 files changed, 38 insertions(+), 2 deletions(-)
>
> diff --git a/profiles/audio/mcp.c b/profiles/audio/mcp.c
> index b410b3d2a..8d1b7588e 100644
> --- a/profiles/audio/mcp.c
> +++ b/profiles/audio/mcp.c
> @@ -224,13 +224,27 @@ static int ct_stop(struct media_player *mp, void *user_data)
>         return bt_mcp_stop(mcp);
>  }
>
> +static int ct_next(struct media_player *mp, void *user_data)
> +{
> +       struct bt_mcp *mcp = user_data;
> +
> +       return bt_mcp_next_track(mcp);
> +}
> +
> +static int ct_previous(struct media_player *mp, void *user_data)
> +{
> +       struct bt_mcp *mcp = user_data;
> +
> +       return bt_mcp_previous_track(mcp);
> +}
> +
>  static const struct media_player_callback ct_cbs = {
>         .set_setting    = NULL,
>         .play           = &ct_play,
>         .pause          = &ct_pause,
>         .stop           = &ct_stop,
> -       .next           = NULL,
> -       .previous       = NULL,
> +       .next           = &ct_next,
> +       .previous       = &ct_previous,
>         .fast_forward   = NULL,
>         .rewind         = NULL,
>         .press          = NULL,
> diff --git a/src/shared/mcp.c b/src/shared/mcp.c
> index b3726ebae..71fc2d151 100644
> --- a/src/shared/mcp.c
> +++ b/src/shared/mcp.c
> @@ -628,6 +628,26 @@ unsigned int bt_mcp_stop(struct bt_mcp *mcp)
>         return mcp_send(mcp, BT_MCS_CMD_STOP);
>  }
>
> +unsigned int bt_mcp_next_track(struct bt_mcp *mcp)
> +{
> +       if (!(mcp->session.cp_op_supported & BT_MCS_CMD_NEXT_TRACK_SUPPORTED))
> +               return -ENOTSUP;
> +
> +       DBG(mcp, "mcp %p", mcp);
> +
> +       return mcp_send(mcp, BT_MCS_CMD_NEXT_TRACK);
> +}
> +
> +unsigned int bt_mcp_previous_track(struct bt_mcp *mcp)
> +{
> +       if (!(mcp->session.cp_op_supported & BT_MCS_CMD_PREV_TRACK_SUPPORTED))
> +               return -ENOTSUP;
> +
> +       DBG(mcp, "mcp %p", mcp);
> +
> +       return mcp_send(mcp, BT_MCS_CMD_PREV_TRACK);
> +}
> +
>  static void mcp_mp_set_player_name(struct bt_mcp *mcp, const uint8_t *value,
>                                         uint16_t length)
>  {
> diff --git a/src/shared/mcp.h b/src/shared/mcp.h
> index a2cd6fc45..ee57ed4bf 100644
> --- a/src/shared/mcp.h
> +++ b/src/shared/mcp.h
> @@ -59,3 +59,5 @@ void *bt_mcp_get_user_data(struct bt_mcp *mcp);
>  unsigned int bt_mcp_play(struct bt_mcp *mcp);
>  unsigned int bt_mcp_pause(struct bt_mcp *mcp);
>  unsigned int bt_mcp_stop(struct bt_mcp *mcp);
> +unsigned int bt_mcp_next_track(struct bt_mcp *mcp);
> +unsigned int bt_mcp_previous_track(struct bt_mcp *mcp);
> --
> 2.45.1
>
>


-- 
Luiz Augusto von Dentz





[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