* [PATCH] android/gatt: Don't response with error for NOTIFICATION and INDICATION
@ 2014-05-20 5:58 Marcin Kraglak
2014-05-20 18:24 ` Szymon Janc
0 siblings, 1 reply; 2+ messages in thread
From: Marcin Kraglak @ 2014-05-20 5:58 UTC (permalink / raw)
To: linux-bluetooth
We shouldn't response for these commands, as they will be serviced
in client implementation.
---
android/gatt.c | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/android/gatt.c b/android/gatt.c
index 8e0d72a..5caa50f 100644
--- a/android/gatt.c
+++ b/android/gatt.c
@@ -4762,11 +4762,13 @@ static void att_handler(const uint8_t *ipdu, uint16_t len, gpointer user_data)
case ATT_OP_FIND_BY_TYPE_REQ:
status = find_by_type_request(ipdu, len, dev);
break;
+ case ATT_OP_HANDLE_IND:
+ case ATT_OP_HANDLE_NOTIFY:
+ /* Client will handle this */
+ return;
case ATT_OP_EXEC_WRITE_REQ:
/* TODO */
case ATT_OP_HANDLE_CNF:
- case ATT_OP_HANDLE_IND:
- case ATT_OP_HANDLE_NOTIFY:
case ATT_OP_READ_MULTI_REQ:
default:
DBG("Unsupported request 0x%02x", ipdu[0]);
--
1.9.0
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] android/gatt: Don't response with error for NOTIFICATION and INDICATION
2014-05-20 5:58 [PATCH] android/gatt: Don't response with error for NOTIFICATION and INDICATION Marcin Kraglak
@ 2014-05-20 18:24 ` Szymon Janc
0 siblings, 0 replies; 2+ messages in thread
From: Szymon Janc @ 2014-05-20 18:24 UTC (permalink / raw)
To: Marcin Kraglak; +Cc: linux-bluetooth
Hi Marcin,
On Tuesday 20 May 2014 07:58:31 Marcin Kraglak wrote:
> We shouldn't response for these commands, as they will be serviced
> in client implementation.
> ---
> android/gatt.c | 6 ++++--
> 1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/android/gatt.c b/android/gatt.c
> index 8e0d72a..5caa50f 100644
> --- a/android/gatt.c
> +++ b/android/gatt.c
> @@ -4762,11 +4762,13 @@ static void att_handler(const uint8_t *ipdu,
> uint16_t len, gpointer user_data) case ATT_OP_FIND_BY_TYPE_REQ:
> status = find_by_type_request(ipdu, len, dev);
> break;
> + case ATT_OP_HANDLE_IND:
> + case ATT_OP_HANDLE_NOTIFY:
> + /* Client will handle this */
> + return;
> case ATT_OP_EXEC_WRITE_REQ:
> /* TODO */
> case ATT_OP_HANDLE_CNF:
> - case ATT_OP_HANDLE_IND:
> - case ATT_OP_HANDLE_NOTIFY:
> case ATT_OP_READ_MULTI_REQ:
> default:
> DBG("Unsupported request 0x%02x", ipdu[0]);
Applied, thanks.
--
Szymon K. Janc
szymon.janc@gmail.com
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2014-05-20 18:24 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-05-20 5:58 [PATCH] android/gatt: Don't response with error for NOTIFICATION and INDICATION Marcin Kraglak
2014-05-20 18:24 ` Szymon Janc
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.