Skip to content

BLE fixes (SM whitelist creation, Nordic scatter file fix, missing TLS initialisation) #7089

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 5 commits into from
Jun 4, 2018

Conversation

paul-szczepanek-arm
Copy link
Member

@paul-szczepanek-arm paul-szczepanek-arm commented Jun 1, 2018

Description

Whitelist creation did not update the size making the consumer think no items where created (original PR: paul-szczepanek-arm#54).

Place observers sequentially in flash in the scatter file for Nordic (original PR: paul-szczepanek-arm#57).

Missing TLS initialisation (original PR: paul-szczepanek-arm#56).

MBED_ASSERT had the wrong condition, creating a false assert when ran in debug.

Pull request type

[x] Fix
[ ] Refactor
[ ] New target
[ ] Feature
[ ] Breaking change

@paul-szczepanek-arm paul-szczepanek-arm changed the title Fix whitelist creation and fix erroneous assert BLE fixes (SM whitelist creation, Nordic scatter file fix) Jun 1, 2018
@paul-szczepanek-arm paul-szczepanek-arm changed the title BLE fixes (SM whitelist creation, Nordic scatter file fix) BLE fixes (SM whitelist creation, Nordic scatter file fix, missing TLS initialisation) Jun 1, 2018
@donatieng
Copy link
Contributor

This is targeting mbed OS 5.9 RC2.

One more fix on its way for #7006, please do not merge for now

@donatieng
Copy link
Contributor

Actually looks like #7006 does not need further fixing, so this PR is good to go!

@cmonr
Copy link
Contributor

cmonr commented Jun 2, 2018

/morph build

@mbed-ci
Copy link

mbed-ci commented Jun 2, 2018

Build : SUCCESS

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

Triggering tests

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

@mbed-ci
Copy link

mbed-ci commented Jun 2, 2018

Test : SUCCESS

Build number : 2014
Test logs :http://mbed-os-logs.s3-website-us-west-1.amazonaws.com/?prefix=logs/7089/2014

@mbed-ci
Copy link

mbed-ci commented Jun 2, 2018

@cmonr
Copy link
Contributor

cmonr commented Jun 3, 2018

/morph uvisor-test

1 similar comment
@0xc0170
Copy link
Contributor

0xc0170 commented Jun 4, 2018

/morph uvisor-test

@0xc0170
Copy link
Contributor

0xc0170 commented Jun 4, 2018

/morph uvisor-test

@0xc0170
Copy link
Contributor

0xc0170 commented Jun 4, 2018

@orenc17 Please check CI uvisor for this PR

@cmonr cmonr merged commit 20a4412 into ARMmbed:master Jun 4, 2018
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.

6 participants