diff mbox series

[RFC,v1] doc/mgmt-api - Return connection direction in Device Connected Event

Message ID 20210323145555.RFC.v1.1.I530e1c3e621abd34c342d657df119e12e576d8a7@changeid (mailing list archive)
State New, archived
Headers show
Series [RFC,v1] doc/mgmt-api - Return connection direction in Device Connected Event | expand

Commit Message

Yu Liu March 23, 2021, 9:56 p.m. UTC
When an MGMT_EV_DEVICE_CONNECTED event is reported back to the user
space we will set the flags to tell if the established connection is
outbound or not. This is useful for the user space to log better metrics
and error messages.

Reviewed-by: Miao-chen Chou <mcchou@chromium.org>
Reviewed-by: Alain Michaud <alainm@chromium.org>
---

Changes in v1:
- Initial change

 doc/mgmt-api.txt | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

bluez.test.bot@gmail.com March 23, 2021, 10:40 p.m. UTC | #1
This is automated email and please do not reply to this email!

Dear submitter,

Thank you for submitting the patches to the linux bluetooth mailing list.
This is a CI test results with your patch series:
PW Link:https://patchwork.kernel.org/project/bluetooth/list/?series=454217

---Test result---

##############################
Test: CheckPatch - PASS

##############################
Test: CheckGitLint - PASS

##############################
Test: CheckBuild - PASS

##############################
Test: MakeCheck - PASS



---
Regards,
Linux Bluetooth
Marcel Holtmann March 24, 2021, 7:55 a.m. UTC | #2
Hi Yu,

> When an MGMT_EV_DEVICE_CONNECTED event is reported back to the user
> space we will set the flags to tell if the established connection is
> outbound or not. This is useful for the user space to log better metrics
> and error messages.
> 
> Reviewed-by: Miao-chen Chou <mcchou@chromium.org>
> Reviewed-by: Alain Michaud <alainm@chromium.org>
> ---
> 
> Changes in v1:
> - Initial change
> 
> doc/mgmt-api.txt | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/doc/mgmt-api.txt b/doc/mgmt-api.txt
> index cab1fffc5..534272045 100644
> --- a/doc/mgmt-api.txt
> +++ b/doc/mgmt-api.txt
> @@ -4085,7 +4085,7 @@ Device Connected Event
> 	start using its identity.
> 
> 	The following bits are defined for the Flags parameter:
> -		0	Reserved (not in use)
> +		0	Outbound Connection
> 		1	Legacy Pairing
> 		2	Reserved (not in use)

the reserved ones are there to keep in sync with Device Found. So please use the next free one (and reserve it for Device Found).

Also I am not fully in favor of “Outbound Connection”. I rather use the term “Central” in case we are the Central / Master of a connection. Now that however gets into trouble with BR/EDR since there you can switch roles. So maybe “Initiated Connection” is an option.

Regards

Marcel
diff mbox series

Patch

diff --git a/doc/mgmt-api.txt b/doc/mgmt-api.txt
index cab1fffc5..534272045 100644
--- a/doc/mgmt-api.txt
+++ b/doc/mgmt-api.txt
@@ -4085,7 +4085,7 @@  Device Connected Event
 	start using its identity.
 
 	The following bits are defined for the Flags parameter:
-		0	Reserved (not in use)
+		0	Outbound Connection
 		1	Legacy Pairing
 		2	Reserved (not in use)