From: Sangki Park <sangki79.park@samsung.com>
Date: Mon, 19 Mar 2018 09:48:38 +0000 (+0900)
Subject: Fix : AVRCP common API error
X-Git-Tag: submit/tizen_4.0/20180328.084945~1
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=8c91cc1ff7e2cae2f26e6eccced8c4851afd6a94;p=platform%2Fcore%2Fapi%2Fbluetooth.git

Fix : AVRCP common API error

The API bt_avrcp_set_play_status_changed_cb() returns
"Not supported" error if the project supports AVRCP controller only.

This kinds of AVRCP APIs shuould be work if CONTROLLER feature is
defined only.

Change-Id: I9bbca080cce0da8f956dee7b541ccb82b7278a52
Signed-off-by: Sangki Park <sangki79.park@samsung.com>
---

diff --git a/src/bluetooth-avrcp.c b/src/bluetooth-avrcp.c
index fef8a3f..7122a82 100644
--- a/src/bluetooth-avrcp.c
+++ b/src/bluetooth-avrcp.c
@@ -26,6 +26,12 @@ static bool is_avrcp_target_initialized = false;
 static bool is_avrcp_control_initialized = false;
 
 #define BT_CHECK_AVRCP_SUPPORT() \
+{ \
+	BT_CHECK_SUPPORTED_FEATURE(BT_FEATURE_COMMON); \
+	BT_CHECK_SUPPORTED_FEATURES(BT_FEATURE_AUDIO_MEDIA, BT_FEATURE_AUDIO_CONTROLLER); \
+}
+
+#define BT_CHECK_AVRCP_TARGET_SUPPORT() \
 { \
 	BT_CHECK_SUPPORTED_FEATURE(BT_FEATURE_COMMON); \
 	BT_CHECK_SUPPORTED_FEATURE(BT_FEATURE_AUDIO_MEDIA); \
@@ -94,7 +100,7 @@ int bt_avrcp_target_initialize(bt_avrcp_target_connection_state_changed_cb callb
 {
 	int error;
 
-	BT_CHECK_AVRCP_SUPPORT();
+	BT_CHECK_AVRCP_TARGET_SUPPORT();
 	BT_CHECK_INIT_STATUS();
 	BT_CHECK_INPUT_PARAMETER(callback);
 	_bt_set_cb(BT_EVENT_AVRCP_CONNECTION_STATUS, callback, user_data);
@@ -116,7 +122,7 @@ int bt_avrcp_target_connect(const char *remote_address)
 	int error;
 	bluetooth_device_address_t addr_hex = { {0,} };
 
-	BT_CHECK_AVRCP_SUPPORT();
+	BT_CHECK_AVRCP_TARGET_SUPPORT();
 	BT_CHECK_INIT_STATUS();
 	BT_CHECK_AVRCP_TARGET_INIT_STATUS();
 	BT_CHECK_INPUT_PARAMETER(remote_address);
@@ -135,7 +141,7 @@ int bt_avrcp_target_disconnect(const char *remote_address)
 	int error;
 	bluetooth_device_address_t addr_hex = { {0,} };
 
-	BT_CHECK_AVRCP_SUPPORT();
+	BT_CHECK_AVRCP_TARGET_SUPPORT();
 	BT_CHECK_INIT_STATUS();
 	BT_CHECK_AVRCP_TARGET_INIT_STATUS();
 	BT_CHECK_INPUT_PARAMETER(remote_address);
@@ -153,7 +159,7 @@ int bt_avrcp_target_deinitialize(void)
 {
 	int error;
 
-	BT_CHECK_AVRCP_SUPPORT();
+	BT_CHECK_AVRCP_TARGET_SUPPORT();
 	BT_CHECK_INIT_STATUS();
 	BT_CHECK_AVRCP_TARGET_INIT_STATUS();
 	if (_bt_check_cb(BT_EVENT_AVRCP_CONNECTION_STATUS) == true)
@@ -176,7 +182,7 @@ int bt_avrcp_target_notify_equalizer_state(bt_avrcp_equalizer_state_e state)
 {
 	int error;
 
-	BT_CHECK_AVRCP_SUPPORT();
+	BT_CHECK_AVRCP_TARGET_SUPPORT();
 	BT_CHECK_INIT_STATUS();
 	BT_CHECK_AVRCP_TARGET_INIT_STATUS();
 	error = bluetooth_media_player_change_property(EQUALIZER, state);
@@ -191,7 +197,7 @@ int bt_avrcp_target_notify_repeat_mode(bt_avrcp_repeat_mode_e mode)
 {
 	int error;
 
-	BT_CHECK_AVRCP_SUPPORT();
+	BT_CHECK_AVRCP_TARGET_SUPPORT();
 	BT_CHECK_INIT_STATUS();
 	BT_CHECK_AVRCP_TARGET_INIT_STATUS();
 	error = bluetooth_media_player_change_property(REPEAT, mode);
@@ -207,7 +213,7 @@ int bt_avrcp_target_notify_shuffle_mode(bt_avrcp_shuffle_mode_e mode)
 {
 	int error;
 
-	BT_CHECK_AVRCP_SUPPORT();
+	BT_CHECK_AVRCP_TARGET_SUPPORT();
 	BT_CHECK_INIT_STATUS();
 	BT_CHECK_AVRCP_TARGET_INIT_STATUS();
 	error = bluetooth_media_player_change_property(SHUFFLE, mode);
@@ -222,7 +228,7 @@ int bt_avrcp_target_notify_scan_mode(bt_avrcp_scan_mode_e mode)
 {
 	int error;
 
-	BT_CHECK_AVRCP_SUPPORT();
+	BT_CHECK_AVRCP_TARGET_SUPPORT();
 	BT_CHECK_INIT_STATUS();
 	BT_CHECK_AVRCP_TARGET_INIT_STATUS();
 	error = bluetooth_media_player_change_property(SCAN, mode);
@@ -238,7 +244,7 @@ int bt_avrcp_target_notify_player_state(bt_avrcp_player_state_e state)
 {
 	int error;
 
-	BT_CHECK_AVRCP_SUPPORT();
+	BT_CHECK_AVRCP_TARGET_SUPPORT();
 	BT_CHECK_INIT_STATUS();
 	BT_CHECK_AVRCP_TARGET_INIT_STATUS();
 	error = bluetooth_media_player_change_property(STATUS, state);
@@ -254,7 +260,7 @@ int bt_avrcp_target_notify_position(unsigned int position)
 {
 	int error;
 
-	BT_CHECK_AVRCP_SUPPORT();
+	BT_CHECK_AVRCP_TARGET_SUPPORT();
 	BT_CHECK_INIT_STATUS();
 	BT_CHECK_AVRCP_TARGET_INIT_STATUS();
 	error = bluetooth_media_player_change_property(POSITION, position);
@@ -271,7 +277,7 @@ int bt_avrcp_target_notify_track(const char *title, const char *artist, const ch
 {
 	int error;
 
-	BT_CHECK_AVRCP_SUPPORT();
+	BT_CHECK_AVRCP_TARGET_SUPPORT();
 	BT_CHECK_INIT_STATUS();
 	BT_CHECK_AVRCP_TARGET_INIT_STATUS();
 	media_metadata_attributes_t metadata;