All the mail mirrored from lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH BlueZ 1/2] audio/avdtp: Fix suspending only the first SEID
@ 2015-08-10 10:24 Luiz Augusto von Dentz
  2015-08-10 10:24 ` [PATCH BlueZ 2/2] audio/avdtp: Fix resuming " Luiz Augusto von Dentz
  2015-08-11 10:34 ` [PATCH BlueZ 1/2] audio/avdtp: Fix suspending " Luiz Augusto von Dentz
  0 siblings, 2 replies; 3+ messages in thread
From: Luiz Augusto von Dentz @ 2015-08-10 10:24 UTC (permalink / raw
  To: linux-bluetooth

From: Luiz Augusto von Dentz <luiz.von.dentz@intel.com>

AVDTP_Suspend can contain several Stream Endpoing IDs to be suspended.
---
 android/avdtp.c        | 2 +-
 profiles/audio/avdtp.c | 3 +--
 2 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/android/avdtp.c b/android/avdtp.c
index 7e61280..1f79e74 100644
--- a/android/avdtp.c
+++ b/android/avdtp.c
@@ -1678,7 +1678,7 @@ static gboolean avdtp_suspend_cmd(struct avdtp *session, uint8_t transaction,
 	for (i = 0; i < seid_count; i++, seid++) {
 		failed_seid = seid->seid;
 
-		sep = find_local_sep_by_seid(session, req->first_seid.seid);
+		sep = find_local_sep_by_seid(session, seid->seid);
 		if (!sep || !sep->stream) {
 			err = AVDTP_BAD_ACP_SEID;
 			goto failed;
diff --git a/profiles/audio/avdtp.c b/profiles/audio/avdtp.c
index 37f7a59..9c33373 100644
--- a/profiles/audio/avdtp.c
+++ b/profiles/audio/avdtp.c
@@ -1822,8 +1822,7 @@ static gboolean avdtp_suspend_cmd(struct avdtp *session, uint8_t transaction,
 	for (i = 0; i < seid_count; i++, seid++) {
 		failed_seid = seid->seid;
 
-		sep = find_local_sep_by_seid(session,
-					req->first_seid.seid);
+		sep = find_local_sep_by_seid(session, seid->seid);
 		if (!sep || !sep->stream) {
 			err = AVDTP_BAD_ACP_SEID;
 			goto failed;
-- 
2.1.0


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [PATCH BlueZ 2/2] audio/avdtp: Fix resuming only the first SEID
  2015-08-10 10:24 [PATCH BlueZ 1/2] audio/avdtp: Fix suspending only the first SEID Luiz Augusto von Dentz
@ 2015-08-10 10:24 ` Luiz Augusto von Dentz
  2015-08-11 10:34 ` [PATCH BlueZ 1/2] audio/avdtp: Fix suspending " Luiz Augusto von Dentz
  1 sibling, 0 replies; 3+ messages in thread
From: Luiz Augusto von Dentz @ 2015-08-10 10:24 UTC (permalink / raw
  To: linux-bluetooth

From: Luiz Augusto von Dentz <luiz.von.dentz@intel.com>

AVDTP_Start can contain several Stream Endpoing IDs to be resumed.
---
 android/avdtp.c        | 2 +-
 profiles/audio/avdtp.c | 3 +--
 2 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/android/avdtp.c b/android/avdtp.c
index 1f79e74..908c0ca 100644
--- a/android/avdtp.c
+++ b/android/avdtp.c
@@ -1567,7 +1567,7 @@ static gboolean avdtp_start_cmd(struct avdtp *session, uint8_t transaction,
 	for (i = 0; i < seid_count; i++, seid++) {
 		failed_seid = seid->seid;
 
-		sep = find_local_sep_by_seid(session, req->first_seid.seid);
+		sep = find_local_sep_by_seid(session, seid->seid);
 		if (!sep || !sep->stream) {
 			err = AVDTP_BAD_ACP_SEID;
 			goto failed;
diff --git a/profiles/audio/avdtp.c b/profiles/audio/avdtp.c
index 9c33373..52ee767 100644
--- a/profiles/audio/avdtp.c
+++ b/profiles/audio/avdtp.c
@@ -1710,8 +1710,7 @@ static gboolean avdtp_start_cmd(struct avdtp *session, uint8_t transaction,
 	for (i = 0; i < seid_count; i++, seid++) {
 		failed_seid = seid->seid;
 
-		sep = find_local_sep_by_seid(session,
-					req->first_seid.seid);
+		sep = find_local_sep_by_seid(session, seid->seid);
 		if (!sep || !sep->stream) {
 			err = AVDTP_BAD_ACP_SEID;
 			goto failed;
-- 
2.1.0


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH BlueZ 1/2] audio/avdtp: Fix suspending only the first SEID
  2015-08-10 10:24 [PATCH BlueZ 1/2] audio/avdtp: Fix suspending only the first SEID Luiz Augusto von Dentz
  2015-08-10 10:24 ` [PATCH BlueZ 2/2] audio/avdtp: Fix resuming " Luiz Augusto von Dentz
@ 2015-08-11 10:34 ` Luiz Augusto von Dentz
  1 sibling, 0 replies; 3+ messages in thread
From: Luiz Augusto von Dentz @ 2015-08-11 10:34 UTC (permalink / raw
  To: linux-bluetooth@vger.kernel.org

Hi,

On Mon, Aug 10, 2015 at 1:24 PM, Luiz Augusto von Dentz
<luiz.dentz@gmail.com> wrote:
> From: Luiz Augusto von Dentz <luiz.von.dentz@intel.com>
>
> AVDTP_Suspend can contain several Stream Endpoing IDs to be suspended.
> ---
>  android/avdtp.c        | 2 +-
>  profiles/audio/avdtp.c | 3 +--
>  2 files changed, 2 insertions(+), 3 deletions(-)
>
> diff --git a/android/avdtp.c b/android/avdtp.c
> index 7e61280..1f79e74 100644
> --- a/android/avdtp.c
> +++ b/android/avdtp.c
> @@ -1678,7 +1678,7 @@ static gboolean avdtp_suspend_cmd(struct avdtp *session, uint8_t transaction,
>         for (i = 0; i < seid_count; i++, seid++) {
>                 failed_seid = seid->seid;
>
> -               sep = find_local_sep_by_seid(session, req->first_seid.seid);
> +               sep = find_local_sep_by_seid(session, seid->seid);
>                 if (!sep || !sep->stream) {
>                         err = AVDTP_BAD_ACP_SEID;
>                         goto failed;
> diff --git a/profiles/audio/avdtp.c b/profiles/audio/avdtp.c
> index 37f7a59..9c33373 100644
> --- a/profiles/audio/avdtp.c
> +++ b/profiles/audio/avdtp.c
> @@ -1822,8 +1822,7 @@ static gboolean avdtp_suspend_cmd(struct avdtp *session, uint8_t transaction,
>         for (i = 0; i < seid_count; i++, seid++) {
>                 failed_seid = seid->seid;
>
> -               sep = find_local_sep_by_seid(session,
> -                                       req->first_seid.seid);
> +               sep = find_local_sep_by_seid(session, seid->seid);
>                 if (!sep || !sep->stream) {
>                         err = AVDTP_BAD_ACP_SEID;
>                         goto failed;
> --
> 2.1.0

Applied.



-- 
Luiz Augusto von Dentz

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2015-08-11 10:34 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-08-10 10:24 [PATCH BlueZ 1/2] audio/avdtp: Fix suspending only the first SEID Luiz Augusto von Dentz
2015-08-10 10:24 ` [PATCH BlueZ 2/2] audio/avdtp: Fix resuming " Luiz Augusto von Dentz
2015-08-11 10:34 ` [PATCH BlueZ 1/2] audio/avdtp: Fix suspending " Luiz Augusto von Dentz

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.