Re: [PATCH_v2 1/2] android/tester: Add AVRCP RegNotifPlayPositionChanged test case

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

 



Hi Ravi,

On Tue, Dec 2, 2014 at 11:53 AM, Ravi kumar Veeramally
<ravikumar.veeramally@xxxxxxxxxxxxxxx> wrote:
> ---
>  android/tester-avrcp.c | 69 +++++++++++++++++++++++++++++++++++++++++++++++++-
>  android/tester-main.h  |  5 ++++
>  2 files changed, 73 insertions(+), 1 deletion(-)
>
> diff --git a/android/tester-avrcp.c b/android/tester-avrcp.c
> index 7c02ec2..10cf1b6 100644
> --- a/android/tester-avrcp.c
> +++ b/android/tester-avrcp.c
> @@ -90,6 +90,15 @@ static struct emu_l2cap_cid_data sdp_data = {
>                         0x31, 0x00, 0x00, 0x09, 0x02, 0xFF, 0xFF, 0xFF, 0xFF, \
>                                                         0xFF, 0xFF, 0xFF, 0xFF
>
> +#define req_position_notif 0x00, 0x11, 0x0e, 0x03, 0x48, 0x00, 0x00, 0x19, \
> +                               0x58, 0x31, 0x00, 0x00, 0x05, 0x05, 0x00, \
> +                                                       0x00, 0x00, 0x00
> +
> +#define rsp_position_notif 0x00, 0x11, 0x0e, 0x0F, 0x48, 0x00, 0x00, 0x19, \
> +                               0x58, 0x31, 0x00, 0x00, 0x04, 0x05, 0xFF, \
> +                                                       0xFF, 0xFF, 0xFF
> +
> +
>  #define req_ele_attr 0x00, 0x11, 0x0e, 0x01, 0x48, 0x00, 0x00, 0x19, 0x58, \
>                         0x20, 0x00, 0x00, 0x0D, 0x00, 0x00, 0x00, 0x00, \
>                         0x02, 0x00, 0x00, 0x00, 0x01, 0x00, 0x00, 0x00, 0x07
> @@ -157,11 +166,21 @@ static void avrcp_cid_hook_cb(const void *data, uint16_t len, void *user_data)
>                 break;
>         case AVRCP_REGISTER_NOTIFICATION:
>                 event = ((uint8_t *) data)[13];
> -               if (event == 0x02) {
> +               switch (event) {
> +               case 0x02:
>                         step = g_new0(struct step, 1);
>                         step->callback = CB_AVRCP_REG_NOTIF_RSP;
>                         step->callback_result.rc_index = get_be64(data + 14);
>                         schedule_callback_verification(step);
> +                       break;
> +
> +               case 0x05:
> +                       step = g_new0(struct step, 1);
> +                       step->callback = CB_AVRCP_REG_NOTIF_RSP;
> +                       step->callback_result.song_position =
> +                                                       get_be32(data + 14);
> +                       schedule_callback_verification(step);
> +                       break;
>                 }
>                 break;
>         case AVRCP_GET_ELEMENT_ATTRIBUTES:
> @@ -314,6 +333,32 @@ static void avrcp_reg_notif_track_changed_rsp(void)
>         schedule_action_verification(step);
>  }
>
> +static void avrcp_reg_notif_play_position_changed_req(void)
> +{
> +       struct test_data *data = tester_get_data();
> +       struct bthost *bthost = hciemu_client_get_host(data->hciemu);
> +       const struct iovec pdu = raw_pdu(req_position_notif);
> +       struct step *step = g_new0(struct step, 1);
> +
> +       bthost_send_cid_v(bthost, avrcp_data.handle, avrcp_data.cid, &pdu, 1);
> +       step->action_status = BT_STATUS_SUCCESS;
> +       schedule_action_verification(step);
> +}
> +
> +static void avrcp_reg_notif_play_position_changed_rsp(void)
> +{
> +       struct test_data *data = tester_get_data();
> +       struct step *step = g_new0(struct step, 1);
> +       btrc_register_notification_t reg;
> +
> +       reg.song_pos = 0xffffffff;
> +       step->action_status = data->if_avrcp->register_notification_rsp(
> +                                               BTRC_EVT_PLAY_POS_CHANGED,
> +                                       BTRC_NOTIFICATION_TYPE_INTERIM, &reg);
> +
> +       schedule_action_verification(step);
> +}
> +
>  static void avrcp_get_element_attributes_req(void)
>  {
>         struct test_data *data = tester_get_data();
> @@ -424,6 +469,28 @@ static struct test_case test_cases[] = {
>                 ACTION_SUCCESS(bluetooth_disable_action, NULL),
>                 CALLBACK_STATE(CB_BT_ADAPTER_STATE_CHANGED, BT_STATE_OFF),
>         ),
> +       TEST_CASE_BREDRLE("AVRCP RegNotifPlayPositionChanged - Success",
> +               ACTION_SUCCESS(bluetooth_enable_action, NULL),
> +               CALLBACK_STATE(CB_BT_ADAPTER_STATE_CHANGED, BT_STATE_ON),
> +               ACTION_SUCCESS(emu_setup_powered_remote_action, NULL),
> +               ACTION_SUCCESS(emu_set_ssp_mode_action, NULL),
> +               ACTION_SUCCESS(set_default_ssp_request_handler, NULL),
> +               ACTION_SUCCESS(emu_add_l2cap_server_action, &sdp_setup_data),
> +               ACTION_SUCCESS(emu_add_l2cap_server_action, &a2dp_setup_data),
> +               ACTION_SUCCESS(emu_add_l2cap_server_action, &avrcp_setup_data),
> +               ACTION_SUCCESS(avrcp_connect_action, NULL),
> +               CALLBACK_AV_CONN_STATE(CB_A2DP_CONN_STATE,
> +                                       BTAV_CONNECTION_STATE_CONNECTING),
> +               CALLBACK_AV_CONN_STATE(CB_A2DP_CONN_STATE,
> +                                       BTAV_CONNECTION_STATE_CONNECTED),
> +               ACTION_SUCCESS(avrcp_reg_notif_play_position_changed_req, NULL),
> +               CALLBACK(CB_AVRCP_REG_NOTIF_REQ),
> +               ACTION_SUCCESS(avrcp_reg_notif_play_position_changed_rsp, NULL),
> +               CALLBACK_RC_REG_NOTIF_POSITION_CHANGED(CB_AVRCP_REG_NOTIF_RSP,
> +                                                               0xffffffff),
> +               ACTION_SUCCESS(bluetooth_disable_action, NULL),
> +               CALLBACK_STATE(CB_BT_ADAPTER_STATE_CHANGED, BT_STATE_OFF),
> +       ),
>         TEST_CASE_BREDRLE("AVRCP GetElementAttributes - Success",
>                 ACTION_SUCCESS(bluetooth_enable_action, NULL),
>                 CALLBACK_STATE(CB_BT_ADAPTER_STATE_CHANGED, BT_STATE_ON),
> diff --git a/android/tester-main.h b/android/tester-main.h
> index ac310a3..1a779a2 100644
> --- a/android/tester-main.h
> +++ b/android/tester-main.h
> @@ -409,6 +409,11 @@ struct pdu_set {
>                 .callback_result.rc_index = cb_index, \
>         }
>
> +#define CALLBACK_RC_REG_NOTIF_POSITION_CHANGED(cb, cb_position) { \
> +               .callback = cb, \
> +               .callback_result.song_position = cb_position, \
> +       }
> +
>  #define CALLBACK_RC_GET_ELEMENT_ATTRIBUTES(cb, cb_num_of_attrs, cb_attrs) { \
>                 .callback = cb, \
>                 .callback_result.num_of_attrs = cb_num_of_attrs, \
> --
> 2.1.0

Applied, thanks.


-- 
Luiz Augusto von Dentz
--
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