Hi Andrei,
On Mon, Nov 04, 2013, Andrei Emeltchenko wrote:
> ---
> android/adapter.c | 13 ++++++++-----
> 1 file changed, 8 insertions(+), 5 deletions(-)
The patch doesn't compile:
android/adapter.c: In function ‘send_adapter_name’:
android/adapter.c:1024:2: error: pointer targets in passing argument 1 of ‘adapter_name_changed’ differ in signedness [-Werror=pointer-sign]
adapter_name_changed(adapter->name);
^
android/adapter.c:145:13: note: expected ‘const uint8_t *’ but argument is of type ‘char *’
static void adapter_name_changed(const uint8_t *name)
^
cc1: all warnings being treated as errors
make[1]: *** [android/adapter.o] Error 1
> @@ -71,6 +71,8 @@ static void mgmt_local_name_changed_event(uint16_t index, uint16_t length,
> {
> const struct mgmt_cp_set_local_name *rp = param;
>
> + DBG("");
> +
> if (length < sizeof(*rp)) {
> error("Wrong size of local name changed parameters");
> return;
This change seems unrelated. Just drop it or move it to another patch.
Johan
--
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
