Re: [PATCH] android: Remove temporary BLUEZ_EXTENSION flag

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

 



Hi Andrei,

On Thursday 06 of November 2014 12:18:50 Andrei Emeltchenko wrote:
> From: Andrei Emeltchenko <andrei.emeltchenko@xxxxxxxxx>
> 
> BLUEZ_EXTENSION is replaced by ANDROID_VERSION >= PLATFORM_VER(5, 0, 0)
> check.
> ---
>  android/Android.mk             | 13 -------------
>  android/client/haltest.c       |  4 ++--
>  android/client/if-bt.c         |  4 ++--
>  android/client/if-hf-client.c  |  4 ++++
>  android/client/if-main.h       |  8 +++++---
>  android/client/if-mce.c        |  4 ++++
>  android/hal-audio.c            |  1 +
>  android/hal-bluetooth.c        |  2 +-
>  android/hal-handsfree-client.c |  4 ++++
>  android/hal-map-client.c       |  4 ++++
>  android/hal-sco.c              |  1 +
>  android/hal-utils.h            |  2 --
>  android/hal.h                  |  6 ++++--
>  13 files changed, 32 insertions(+), 25 deletions(-)
> 
> diff --git a/android/Android.mk b/android/Android.mk
> index 16f94b9..6694f1e 100644
> --- a/android/Android.mk
> +++ b/android/Android.mk
> @@ -14,10 +14,6 @@ BLUEZ_COMMON_CFLAGS := -DVERSION=\"$(BLUEZ_VERSION)\" \
>  			-DANDROID_VERSION=$(ANDROID_VERSION) \
>  			-DANDROID_STORAGEDIR=\"/data/misc/bluetooth\" \
>  
> -ifeq ($(BLUEZ_EXTENSIONS), true)
> -BLUEZ_COMMON_CFLAGS += -DBLUEZ_EXTENSIONS
> -endif
> -
>  # Enable warnings enabled in autotools build
>  BLUEZ_COMMON_CFLAGS += -Wall -Wextra \
>  			-Wdeclaration-after-statement \
> @@ -133,14 +129,9 @@ LOCAL_SRC_FILES := \
>  	bluez/android/hal-gatt.c \
>  	bluez/android/hal-utils.c \
>  	bluez/android/hal-health.c \
> -
> -ifeq ($(BLUEZ_EXTENSIONS), true)
> -LOCAL_SRC_FILES += \
>  	bluez/android/hal-handsfree-client.c \
>  	bluez/android/hal-map-client.c \
>  
> -endif
> -
>  LOCAL_C_INCLUDES += \
>  	$(call include-path-for, system-core) \
>  	$(call include-path-for, libhardware) \
> @@ -182,12 +173,8 @@ LOCAL_SRC_FILES := \
>  	bluez/android/client/if-sock.c \
>  	bluez/android/client/if-gatt.c \
>  	bluez/android/hal-utils.c \
> -
> -ifeq ($(BLUEZ_EXTENSIONS), true)
> -LOCAL_SRC_FILES += \
>  	bluez/android/client/if-hf-client.c \
>  	bluez/android/client/if-mce.c
> -endif
>  
>  LOCAL_C_INCLUDES += \
>  	$(call include-path-for, system-core) \
> diff --git a/android/client/haltest.c b/android/client/haltest.c
> index 781e45c..c8cfdc4 100644
> --- a/android/client/haltest.c
> +++ b/android/client/haltest.c
> @@ -48,7 +48,7 @@ const struct interface *interfaces[] = {
>  	&pan_if,
>  	&hl_if,
>  	&sock_if,
> -#ifdef BLUEZ_EXTENSIONS
> +#if ANDROID_VERSION >= PLATFORM_VER(5, 0, 0)
>  	&hf_client_if,
>  	&mce_if,
>  #endif
> @@ -396,7 +396,7 @@ static void init(void)
>  		BT_PROFILE_PAN_ID,
>  		BT_PROFILE_GATT_ID,
>  		BT_PROFILE_SOCKETS_ID,
> -#ifdef BLUEZ_EXTENSIONS
> +#if ANDROID_VERSION >= PLATFORM_VER(5, 0, 0)
>  		BT_PROFILE_HANDSFREE_CLIENT_ID,
>  		BT_PROFILE_MAP_CLIENT_ID,
>  #endif
> diff --git a/android/client/if-bt.c b/android/client/if-bt.c
> index eef4454..48aff1b 100644
> --- a/android/client/if-bt.c
> +++ b/android/client/if-bt.c
> @@ -757,7 +757,7 @@ static void get_profile_interface_c(int argc, const char **argv,
>  		BT_PROFILE_PAN_ID,
>  		BT_PROFILE_GATT_ID,
>  		BT_PROFILE_AV_RC_ID,
> -#ifdef BLUEZ_EXTENSIONS
> +#if ANDROID_VERSION >= PLATFORM_VER(5, 0, 0)
>  		BT_PROFILE_HANDSFREE_CLIENT_ID,
>  		BT_PROFILE_MAP_CLIENT_ID,
>  #endif
> @@ -799,7 +799,7 @@ static void get_profile_interface_p(int argc, const char **argv)
>  		pif = (const void **) &if_rc;
>  	else if (strcmp(BT_PROFILE_GATT_ID, id) == 0)
>  		pif = (const void **) &if_gatt;
> -#ifdef BLUEZ_EXTENSIONS
> +#if ANDROID_VERSION >= PLATFORM_VER(5, 0, 0)
>  	else if (strcmp(BT_PROFILE_HANDSFREE_CLIENT_ID, id) == 0)
>  		pif = (const void **) &if_hf_client;
>  	else if (strcmp(BT_PROFILE_MAP_CLIENT_ID, id) == 0)
> diff --git a/android/client/if-hf-client.c b/android/client/if-hf-client.c
> index 3f6f7c2..5783ffe 100644
> --- a/android/client/if-hf-client.c
> +++ b/android/client/if-hf-client.c
> @@ -18,6 +18,8 @@
>  #include "if-main.h"
>  #include "../hal-utils.h"
>  
> +#if ANDROID_VERSION >= PLATFORM_VER(5, 0, 0)
> +

I'd prefer if you handle this on Android.mk level.

>  const bthf_client_interface_t *if_hf_client = NULL;
>  
>  static char last_addr[MAX_ADDR_STR_LEN];
> @@ -666,3 +668,5 @@ const struct interface hf_client_if = {
>  	.name = "handsfree_client",
>  	.methods = methods
>  };
> +
> +#endif
> diff --git a/android/client/if-main.h b/android/client/if-main.h
> index cb5f558..93cc98d 100644
> --- a/android/client/if-main.h
> +++ b/android/client/if-main.h
> @@ -37,7 +37,9 @@
>  #include <hardware/bt_hf.h>
>  #include <hardware/bt_hl.h>
>  
> -#ifdef BLUEZ_EXTENSIONS
> +#include "hal.h"
> +
> +#if ANDROID_VERSION >= PLATFORM_VER(5, 0, 0)
>  #include <hardware/bt_hf_client.h>
>  #include <hardware/bt_mce.h>
>  #endif
> @@ -62,7 +64,7 @@ extern const btsock_interface_t *if_sock;
>  extern const btgatt_interface_t *if_gatt;
>  extern const btgatt_server_interface_t *if_gatt_server;
>  extern const btgatt_client_interface_t *if_gatt_client;
> -#ifdef BLUEZ_EXTENSIONS
> +#if ANDROID_VERSION >= PLATFORM_VER(5, 0, 0)
>  extern const bthf_client_interface_t *if_hf_client;
>  extern const btmce_interface_t *if_mce;
>  #endif
> @@ -89,7 +91,7 @@ extern const struct interface sock_if;
>  extern const struct interface hf_if;
>  extern const struct interface hh_if;
>  extern const struct interface hl_if;
> -#ifdef BLUEZ_EXTENSIONS
> +#if ANDROID_VERSION >= PLATFORM_VER(5, 0, 0)
>  extern const struct interface hf_client_if;
>  extern const struct interface mce_if;
>  #endif
> diff --git a/android/client/if-mce.c b/android/client/if-mce.c
> index 5f101e0..b656328 100644
> --- a/android/client/if-mce.c
> +++ b/android/client/if-mce.c
> @@ -18,6 +18,8 @@
>  #include "if-main.h"
>  #include "../hal-utils.h"
>  
> +#if ANDROID_VERSION >= PLATFORM_VER(5, 0, 0)
> +
>  const btmce_interface_t *if_mce = NULL;
>  
>  /*
> @@ -85,3 +87,5 @@ const struct interface mce_if = {
>  	.name = "mce",
>  	.methods = methods
>  };
> +
> +#endif
> diff --git a/android/hal-audio.c b/android/hal-audio.c
> index 1caa4eb..207101f 100644
> --- a/android/hal-audio.c
> +++ b/android/hal-audio.c
> @@ -36,6 +36,7 @@
>  #include "hal-msg.h"
>  #include "hal-audio.h"
>  #include "hal-utils.h"
> +#include "hal.h"
>  
>  #define FIXED_A2DP_PLAYBACK_LATENCY_MS 25
>  
> diff --git a/android/hal-bluetooth.c b/android/hal-bluetooth.c
> index 1599174..9b5ce16 100644
> --- a/android/hal-bluetooth.c
> +++ b/android/hal-bluetooth.c
> @@ -865,7 +865,7 @@ static const void *get_profile_interface(const char *profile_id)
>  	if (!strcmp(profile_id, BT_PROFILE_HEALTH_ID))
>  		return bt_get_health_interface();
>  
> -#if BLUEZ_EXTENSIONS
> +#if ANDROID_VERSION >= PLATFORM_VER(5, 0, 0)
>  	if (!strcmp(profile_id, BT_PROFILE_HANDSFREE_CLIENT_ID))
>  		return bt_get_hf_client_interface();
>  
> diff --git a/android/hal-handsfree-client.c b/android/hal-handsfree-client.c
> index a152cab..2bf9136 100644
> --- a/android/hal-handsfree-client.c
> +++ b/android/hal-handsfree-client.c
> @@ -28,6 +28,8 @@
>  #include "ipc-common.h"
>  #include "hal-ipc.h"
>  
> +#if ANDROID_VERSION >= PLATFORM_VER(5, 0, 0)
> +
>  static const bthf_client_callbacks_t *cbs = NULL;
>  
>  static bool interface_ready(void)
> @@ -650,3 +652,5 @@ bthf_client_interface_t *bt_get_hf_client_interface(void)
>  {
>  	return &iface;
>  }
> +
> +#endif
> diff --git a/android/hal-map-client.c b/android/hal-map-client.c
> index 8255d2b..c08c87b 100644
> --- a/android/hal-map-client.c
> +++ b/android/hal-map-client.c
> @@ -24,6 +24,8 @@
>  #include "hal-msg.h"
>  #include "hal-ipc.h"
>  
> +#if ANDROID_VERSION >= PLATFORM_VER(5, 0, 0)
> +
>  static const btmce_callbacks_t *cbs = NULL;
>  
>  static bool interface_ready(void)
> @@ -150,3 +152,5 @@ btmce_interface_t *bt_get_map_client_interface(void)
>  {
>  	return &iface;
>  }
> +
> +#endif
> diff --git a/android/hal-sco.c b/android/hal-sco.c
> index 6c4e975..380b2a8 100644
> --- a/android/hal-sco.c
> +++ b/android/hal-sco.c
> @@ -33,6 +33,7 @@
>  #include "sco-msg.h"
>  #include "ipc-common.h"
>  #include "hal-log.h"
> +#include "hal.h"
>  
>  #define AUDIO_STREAM_DEFAULT_RATE	44100
>  #define AUDIO_STREAM_SCO_RATE		8000
> diff --git a/android/hal-utils.h b/android/hal-utils.h
> index ee0320c..9c59948 100644
> --- a/android/hal-utils.h
> +++ b/android/hal-utils.h
> @@ -19,8 +19,6 @@
>  
>  #include <hardware/bluetooth.h>
>  
> -#define PLATFORM_VER(a,b,c) ((a << 16) | ( b << 8) | (c))
> -
>  #define MAX_UUID_STR_LEN	37
>  #define HAL_UUID_LEN		16
>  #define MAX_ADDR_STR_LEN	18
> diff --git a/android/hal.h b/android/hal.h
> index 9b41952..a2888f0 100644
> --- a/android/hal.h
> +++ b/android/hal.h
> @@ -27,7 +27,9 @@
>  #include <hardware/bt_gatt_server.h>
>  #include <hardware/bt_hl.h>
>  
> -#ifdef BLUEZ_EXTENSIONS
> +#define PLATFORM_VER(a,b,c) ((a << 16) | ( b << 8) | (c))
> +
> +#if ANDROID_VERSION >= PLATFORM_VER(5, 0, 0)
>  #include <hardware/bt_hf_client.h>
>  #include <hardware/bt_mce.h>
>  #endif
> @@ -41,7 +43,7 @@ bthf_interface_t *bt_get_handsfree_interface(void);
>  btgatt_interface_t *bt_get_gatt_interface(void);
>  bthl_interface_t *bt_get_health_interface(void);
>  
> -#ifdef BLUEZ_EXTENSIONS
> +#if ANDROID_VERSION >= PLATFORM_VER(5, 0, 0)
>  bthf_client_interface_t *bt_get_hf_client_interface(void);
>  btmce_interface_t *bt_get_map_client_interface(void);
>  #endif
> 

-- 
Best regards, 
Szymon Janc
--
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