* [PATCH] android/client: Print peer and CHLD features only on connected event
@ 2014-10-06 10:34 Lukasz Rymanowski
2014-10-06 13:45 ` Szymon Janc
0 siblings, 1 reply; 2+ messages in thread
From: Lukasz Rymanowski @ 2014-10-06 10:34 UTC (permalink / raw)
To: linux-bluetooth; +Cc: Lukasz Rymanowski
We want to print peer and CHLD features are only when client gets
connected event. Otherwise it might be confusing as those fields are not
set.
---
android/client/if-hf-client.c | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/android/client/if-hf-client.c b/android/client/if-hf-client.c
index 07c09cd..0f4f3e9 100644
--- a/android/client/if-hf-client.c
+++ b/android/client/if-hf-client.c
@@ -203,6 +203,10 @@ static void hf_client_connection_state_callback(
haltest_info("%s: state=%s bd_addr=%s\n", __func__,
bthf_client_connection_state_t2str(state),
bt_bdaddr_t2str(bd_addr, last_addr));
+
+ if ( state != BTHF_CLIENT_CONNECTION_STATE_CONNECTED)
+ return;
+
haltest_info("\tpeer_features%s\n", pear_features_t2str(peer_feat));
haltest_info("\tchld_feat=%s\n", chld_features_t2str(chld_feat));
}
--
1.8.4
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] android/client: Print peer and CHLD features only on connected event
2014-10-06 10:34 [PATCH] android/client: Print peer and CHLD features only on connected event Lukasz Rymanowski
@ 2014-10-06 13:45 ` Szymon Janc
0 siblings, 0 replies; 2+ messages in thread
From: Szymon Janc @ 2014-10-06 13:45 UTC (permalink / raw)
To: Lukasz Rymanowski; +Cc: linux-bluetooth
Hi Łukasz,
On Monday 06 of October 2014 12:34:14 Lukasz Rymanowski wrote:
> We want to print peer and CHLD features are only when client gets
> connected event. Otherwise it might be confusing as those fields are not
> set.
> ---
> android/client/if-hf-client.c | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/android/client/if-hf-client.c b/android/client/if-hf-client.c
> index 07c09cd..0f4f3e9 100644
> --- a/android/client/if-hf-client.c
> +++ b/android/client/if-hf-client.c
> @@ -203,6 +203,10 @@ static void hf_client_connection_state_callback(
> haltest_info("%s: state=%s bd_addr=%s\n", __func__,
> bthf_client_connection_state_t2str(state),
> bt_bdaddr_t2str(bd_addr, last_addr));
> +
> + if ( state != BTHF_CLIENT_CONNECTION_STATE_CONNECTED)
> + return;
> +
> haltest_info("\tpeer_features%s\n", pear_features_t2str(peer_feat));
> haltest_info("\tchld_feat=%s\n", chld_features_t2str(chld_feat));
> }
>
Applied (after fixing bogus space after '('), thanks.
--
Best regards,
Szymon Janc
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2014-10-06 13:45 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-10-06 10:34 [PATCH] android/client: Print peer and CHLD features only on connected event Lukasz Rymanowski
2014-10-06 13:45 ` Szymon Janc
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox