Hi Andrei, > bt_sock HAL handles Bluetooth sockets for Android. > --- > android/Android.mk | 1 + > android/hal_bt_sock.c | 84 +++++++++++++++++++++++++++++++++++++++++++++++++ > 2 files changed, 85 insertions(+) > create mode 100644 android/hal_bt_sock.c > > diff --git a/android/Android.mk b/android/Android.mk > index 7adbbcd..2cabff4 100644 > --- a/android/Android.mk > +++ b/android/Android.mk > @@ -41,6 +41,7 @@ include $(CLEAR_VARS) > > LOCAL_SRC_FILES := \ > hal_bluetooth.c \ > + hal_bt_sock.c \ > > LOCAL_SHARED_LIBRARIES := \ > libcutils \ > diff --git a/android/hal_bt_sock.c b/android/hal_bt_sock.c > new file mode 100644 > index 0000000..cbb42d1 > --- /dev/null > +++ b/android/hal_bt_sock.c > @@ -0,0 +1,84 @@ > +/* > + * Copyright (C) 2013 Intel Corporation > + * > + * Licensed under the Apache License, Version 2.0 (the "License"); > + * you may not use this file except in compliance with the License. > + * You may obtain a copy of the License at > + * > + * http://www.apache.org/licenses/LICENSE-2.0 > + * > + * Unless required by applicable law or agreed to in writing, software > + * distributed under the License is distributed on an "AS IS" BASIS, > + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. > + * See the License for the specific language governing permissions and > + * limitations under the License. > + * > + */ > + > +#include <stdlib.h> > + > +#include <hardware/bluetooth.h> > +#include <hardware/bt_sock.h> > + > +#define LOG_TAG "BlueZ" > +#include <cutils/log.h> > + > +static bt_status_t btsock_listen_rfcomm(const char *service_name, > + const uint8_t *uuid, int chan, > + int *sock, int flags) > +{ > + ALOGD(__func__); > + > + return BT_STATUS_UNSUPPORTED; > +} > + > +static bt_status_t listen(btsock_type_t type, const char *service_name, > + const uint8_t *uuid, int chan, > + int *sock, int flags) > +{ > + if ((uuid == NULL && chan <= 0) || sock == NULL) { we are moving away from uuid == NULL checks. Use !uuid instead. > + ALOGE("%s: invalid params: uuid %p, chan %d, sock %p", > + __func__, uuid, chan, sock); > + return BT_STATUS_PARM_INVALID; > + } > + > + ALOGD("%s: uuid %p chan %d sock %p type %d service_name %s", > + __func__, uuid, chan, sock, type, service_name); > + > + switch (type) { > + case BTSOCK_RFCOMM: > + return btsock_listen_rfcomm(service_name, uuid, chan, > + sock, flags); > + default: > + ALOGE("%s: Socket type %d not supported", __func__, type); I normally expect that even the default statement has a break; at the end. > + } > + > + return BT_STATUS_UNSUPPORTED; > +} > + > +static bt_status_t connect(const bt_bdaddr_t *bdaddr, btsock_type_t type, > + const uint8_t *uuid, int chan, > + int *sock, int flags) > +{ > + if ((uuid == NULL && chan <= 0) || bdaddr == NULL || sock == NULL) { > + ALOGE("invalid params: bd_addr %p, uuid %p, chan %d, sock %p", > + bdaddr, uuid, chan, sock); > + return BT_STATUS_PARM_INVALID; > + } > + > + ALOGD("%s: uuid %p chan %d sock %p type %d", __func__, uuid, chan, > + sock, type); > + > + return BT_STATUS_UNSUPPORTED; > +} > + > +static btsock_interface_t btsock_if = { Is this an Android issue in how they defined the APIs or why are these not static const. > + sizeof(btsock_if), > + listen, > + connect > +}; > + > +btsock_interface_t *bt_get_sock_interface(void) > +{ > + return &btsock_if; > +} Regards Marcel -- 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
