Skip to content

BLE: Generic access service abstraction layer #5296

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Nov 16, 2017

Conversation

pan-
Copy link
Member

@pan- pan- commented Oct 11, 2017

Description

The top level API, GAP, mixed up different concepts, the GAP layer and the exposition of GAP parameters through the Generic Access Service at the GATT layer.

At the pal level, those concepts are separated, therefore this PR add an abstraction which manage the state of the GAP service exposed by the GATT server.

Status

READY

Migrations

NO

Related PRs

#5295

Note

Diff can be found here: pan-/mbed@GAP-platform-abstraction-layer...pan-:generic-access-service-abstraction-layer

@pan-
Copy link
Member Author

pan- commented Oct 11, 2017

@marcbonnici @nvlsianpu @apalmieriGH Could you review this PR ?

@mbed-ci
Copy link

mbed-ci commented Oct 11, 2017

Build : SUCCESS

Build number : 115
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/5296/

Triggering tests

/test mbed-os

@mbed-ci
Copy link

mbed-ci commented Oct 11, 2017

@0xc0170
Copy link
Contributor

0xc0170 commented Oct 12, 2017

@tommikas jenkins CI restart (no url here) pls

@pan- pan- force-pushed the generic-access-service-abstraction-layer branch from c5a5b2e to fa61c01 Compare October 12, 2017 09:41
@tommikas
Copy link
Contributor

@0xc0170 It's being built now. (As are the other 3 jobs Jenkins hadn't noticed.)

The url still hasn't updated here though so I'll keep an eye on it.

@mbed-ci
Copy link

mbed-ci commented Oct 12, 2017

Build : SUCCESS

Build number : 142
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/5296/

Triggering tests

/test mbed-os

@pan- pan- force-pushed the generic-access-service-abstraction-layer branch from fa61c01 to 43f8af6 Compare October 15, 2017 15:45
pan- added 2 commits October 31, 2017 17:32
This interface expose the primitives needed to realize operations defined in
the GAP layer. Data types, event and function definitions follow closely HCI
commands and events defined in the Bluetooth specification.
Add an abstraction which manage the state of the GAP service exposed by the GATT server.
@pan- pan- force-pushed the generic-access-service-abstraction-layer branch from 43f8af6 to 0025b68 Compare October 31, 2017 17:42
@pan-
Copy link
Member Author

pan- commented Nov 13, 2017

@0xc0170 Any update ?

@0xc0170
Copy link
Contributor

0xc0170 commented Nov 13, 2017

/morph build

@mbed-ci
Copy link

mbed-ci commented Nov 13, 2017

Build : FAILURE

Build number : 500
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/5296/

@mbed-ci
Copy link

mbed-ci commented Nov 13, 2017

Build : SUCCESS

Build number : 509
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/5296/

Triggering tests

/morph test
/morph uvisor-test
/morph export-build

@mbed-ci
Copy link

mbed-ci commented Nov 13, 2017

@mbed-ci
Copy link

mbed-ci commented Nov 13, 2017

* Public identity address.
* @note remove once privacy mode is supported.
*/
PUBLIC_IDENTITY_ADDRESS = 0x02,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure what this mean - how this is related to addressed defined by BLE standard (this, and next item)?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The note should not be here.

GapDisconnectionCompleteEvent(
uint8_t status,
connection_handle_t connection_handle,
uint8_t reason
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why not type of disconnection_reason_t?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Because the reasons of the disconnection can be most of hci_error_code_t; it is not necessarily initiated by the user. disconnection_reason_t list the reasons that can be used by the user code to ends the connection.

* - Time out in case of high duty cycle directed advertising.
*
* If the random address has not been set and the advertising parameter
* own_address_type is set to 0x01 then the procedure shall fail.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

0x01 == RANDOM_ADDRES

@0xc0170 0xc0170 merged commit 2a7fcdf into ARMmbed:master Nov 16, 2017
@pan- pan- deleted the generic-access-service-abstraction-layer branch July 3, 2018 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants