Return-Path: From: Ravi kumar Veeramally To: linux-bluetooth@vger.kernel.org Cc: Ravi kumar Veeramally Subject: [PATCH_v2] android/client: Add AVRCP register_notification_rsp support Date: Wed, 26 Mar 2014 13:32:05 +0200 Message-Id: <1395833525-16345-1-git-send-email-ravikumar.veeramally@linux.intel.com> Sender: linux-bluetooth-owner@vger.kernel.org List-ID: Input for this call doesn't require all parameters of structure to be filled at the same time. Input data depepnds on event_id and type. Implemented this api support based on android/hal_avrcp.c:register_notification_rsp. --- v2: Fixed Luiz's comments. --- android/client/if-rc.c | 78 ++++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 78 insertions(+) diff --git a/android/client/if-rc.c b/android/client/if-rc.c index 31efc29..cdf6311 100644 --- a/android/client/if-rc.c +++ b/android/client/if-rc.c @@ -54,6 +54,20 @@ SINTMAP(btrc_status_t, -1, "(unknown)") DELEMENT(BTRC_STS_NO_ERROR), ENDMAP +SINTMAP(btrc_event_id_t, -1, "(unknown)") + DELEMENT(BTRC_EVT_PLAY_STATUS_CHANGED), + DELEMENT(BTRC_EVT_TRACK_CHANGE), + DELEMENT(BTRC_EVT_TRACK_REACHED_END), + DELEMENT(BTRC_EVT_TRACK_REACHED_START), + DELEMENT(BTRC_EVT_PLAY_POS_CHANGED), + DELEMENT(BTRC_EVT_APP_SETTINGS_CHANGED), +ENDMAP + +SINTMAP(btrc_notification_type_t, -1, "(unknown)") + DELEMENT(BTRC_NOTIFICATION_TYPE_INTERIM), + DELEMENT(BTRC_NOTIFICATION_TYPE_CHANGED), +ENDMAP + static char last_addr[MAX_ADDR_STR_LEN]; static void remote_features_cb(bt_bdaddr_t *bd_addr, @@ -294,6 +308,63 @@ static void set_player_app_value_rsp_p(int argc, const char **argv) EXEC(if_rc->set_player_app_value_rsp, rsp_status); } +/* register_notification_rsp */ + +static void register_notification_rsp_c(int argc, const char **argv, + enum_func *enum_func, void **user) +{ + if (argc == 3) { + *user = TYPE_ENUM(btrc_event_id_t); + *enum_func = enum_defines; + } + + if (argc == 4) { + *user = TYPE_ENUM(btrc_notification_type_t); + *enum_func = enum_defines; + } +} + +static void register_notification_rsp_p(int argc, const char **argv) +{ + btrc_event_id_t event_id; + btrc_notification_type_t type; + btrc_register_notification_t reg; + uint32_t song_pos; + uint64_t track; + + RETURN_IF_NULL(if_rc); + + memset(®, 0, sizeof(reg)); + event_id = str2btrc_event_id_t(argv[2]); + type = str2btrc_notification_type_t(argv[3]); + + switch (event_id) { + case BTRC_EVT_PLAY_STATUS_CHANGED: + reg.play_status = str2btrc_play_status_t(argv[4]); + break; + + case BTRC_EVT_TRACK_CHANGE: + track = strtoull(argv[5], NULL, 10); + memcpy(reg.track, &track, sizeof(btrc_uid_t)); + break; + + case BTRC_EVT_TRACK_REACHED_END: + case BTRC_EVT_TRACK_REACHED_START: + break; + + case BTRC_EVT_PLAY_POS_CHANGED: + song_pos = strtoul(argv[4], NULL, 10); + memcpy(®.song_pos, &song_pos, sizeof(uint32_t)); + break; + + case BTRC_EVT_APP_SETTINGS_CHANGED: + haltest_error("not supported"); + return; + } + + EXEC(if_rc->register_notification_rsp, event_id, type, ®); +} + /* cleanup */ static void cleanup_p(int argc, const char **argv) @@ -311,6 +382,13 @@ static struct method methods[] = { STD_METHODCH(get_element_attr_rsp, " "), STD_METHODCH(set_player_app_value_rsp, ""), STD_METHODCH(set_volume, ""), + STD_METHODCH(register_notification_rsp, + " \n" + "BTRC_EVT_PLAY_STATUS_CHANGED \n" + "BTRC_EVT_TRACK_CHANGE \n" + "BTRC_EVT_TRACK_REACHED_END \n" + "BTRC_EVT_TRACK_REACHED_START \n" + "BTRC_EVT_PLAY_POS_CHANGED \n"), STD_METHOD(cleanup), END_METHOD }; -- 1.8.3.2