Skip to content

Release mbed-os-5.1.5 and mbed lib v127 #2850

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 138 commits into from
Sep 30, 2016
Merged

Conversation

adbridge
Copy link
Contributor

Release candidate.

jeromecoutant and others added 30 commits September 28, 2016 13:46
8K RAM target is too small for
"NUCLEO_L031K6"
"NUCLEO_L053R8"
"DISCO_L053C8"
"NUCLEO_F030R8"
- Add support of cortex-M7 for cthunk.
- Change the cthunk trampoline implementation to safer and quicker
solutions:
 * thumb2, the behaviour was undefined. new implementation use now 2
instructions
 * thumb, The new implementation use 3 instructions instead of 6.
bugfixing the can loop test which failed after PR #2504
This patch modifies:
* the Beetle GCC ARM linker script
* the Beetle startup code
in order to define the memory regions that enable uvisor.

Signed-off-by: Vincenzo Frascino <vincenzo.frascino@arm.com>
This patch backports the virtual NVIC mechanish from CMSIS 5 for the
Cortex M3 architecture in order to support uvisor in this MCU class.

Signed-off-by: Vincenzo Frascino <vincenzo.frascino@arm.com>
This patch adds CMSIS 5 virtual NVIC compliancy in order to support
uvisor.

Signed-off-by: Vincenzo Frascino <vincenzo.frascino@arm.com>
This patch disables the fcache stats into mbed_sdk_init if uvisor is
defined in order to prevent MEMMANAGEMENT faults during boot.

Signed-off-by: Vincenzo Frascino <vincenzo.frascino@arm.com>
This patch adds Beetle platform to uvisor importer.

Signed-off-by: Vincenzo Frascino <vincenzo.frascino@arm.com>
Build jobs are failing due build error "arm_hal_timer.cpp:50:5:
error: reference to 'callback' is ambiguous".

Fix the build error by renaming callback to arm_hal_callback to
avoid collision with callback defined in ./mbed-os/hal/api/Callback.h
- Check if disconnected in socket open
- Check if connected in interface connect
- Check if disconnected in interface disconnect
Removed code that checks if the path to the map file
matches with the selected toolchain.
The previous way of finding group names found the encompassing directory of each file. If the project is exported from the online compiler, this resulted in temporary folder names like tmpyKKWv_ showing up as group names.

I propose defaulting to the project name if the file is in the project root.
@adbridge
Copy link
Contributor Author

adbridge commented Sep 29, 2016

uvisor tests are failing due to some tools changes Azim made (which are only on Master). Mbed2 results look slightly better than the last release. So this looks good to me.
@0xc0170 or @sg- please merge to mbed-os-5.1

@adbridge
Copy link
Contributor Author

/morph test-nightly

@0xc0170
Copy link
Contributor

0xc0170 commented Sep 29, 2016

LGTM

@sg- please approve

@0xc0170
Copy link
Contributor

0xc0170 commented Sep 29, 2016

Thanks @adbridge .

@mbed-bot
Copy link

Result: FAILURE

Your command has finished executing! Here's what you wrote!

/morph test-nightly

@adbridge
Copy link
Contributor Author

WAIT on Merge!!

raise InvalidReleaseTargetException(reason)

tools.utils.InvalidReleaseTargetException: Target 'XDOT_L151CC' must set the 'default_build' to 'standard' to be included in the mbed OS 5.0 official release.
Currently it is set to 'std'

@bridadan
Copy link
Contributor

bridadan commented Sep 29, 2016

@adbridge it's missing a commit: https://github.com/ARMmbed/mbed-
os/pull/2827/commits/7b2942157a909b87a5709a57d7090ddd0dd8d191

Any idea why the script misses these? It makes me a bit worried other commits we're missing in these releases.

Nevermind, it looks like that commit is actually in, need to see what's actually happening...

@adbridge
Copy link
Contributor Author

It's there above:
@mfiore02 include xDot in mbed 5 releases c7045d0
@mfiore02 add IAR support fa22d4d

@adbridge
Copy link
Contributor Author

The xdot ones had to come across via their original commits rather than the PR sha commit due to the rebasing

@bridadan
Copy link
Contributor

Think I found the sucker: #2409

Any idea why this didn't come over?

@adbridge
Copy link
Contributor Author

I suspect the error is due to the targets.json file being slightly wrong. I had to manually fix that up so may have got it slightly wrong!

@adbridge
Copy link
Contributor Author

adbridge commented Sep 29, 2016

Nope that was pre-script territory ! ie when the process was still manual!

@adbridge
Copy link
Contributor Author

Is it just a case of changing "default_build": "std", to "default_build": "standard", in targets.json ?

@bridadan
Copy link
Contributor

@adbridge I think this is an error, as on master everything is named std, not standard. See the current targets.json: https://github.com/ARMmbed/mbed-os/blob/master/hal/targets.json

@0xc0170
Copy link
Contributor

0xc0170 commented Sep 29, 2016

@adbridge I think this is an error, as on master everything is named std, not standard. See the current targets.json: https://github.com/ARMmbed/mbed-os/blob/master/hal/targets.json

Don't forget that this is about backward compatibility, and it was standard up until this release. @adbridge please check with @screamerbg

@adbridge
Copy link
Contributor Author

/morph test-nightly

@mbed-bot
Copy link

Result: FAILURE

Your command has finished executing! Here's what you wrote!

/morph test-nightly

Output

mbed Build Number: 1002

Build failed!

@adbridge
Copy link
Contributor Author

/morph test-nightly

@mbed-bot
Copy link

Result: FAILURE

Your command has finished executing! Here's what you wrote!

/morph test-nightly

Output

mbed Build Number: 1004

Build failed!

@bridadan
Copy link
Contributor

@adbridge Unless I'm missing something, it looks like only one of the commits from this PR made it over: #2761

I think it's missing this commit: 2dfbf59

@adbridge
Copy link
Contributor Author

mmm looks like another rebasing issue!! Getting the commits directly from the PR just doesn't like it when a rebase has happened! This is fixed in the update to the script I've been working on in the background, but wasn't ready for this release! Argghh rebasing and backporting = nasty!

@adbridge
Copy link
Contributor Author

/morph test-nightly

@mbed-bot
Copy link

Result: FAILURE

Your command has finished executing! Here's what you wrote!

/morph test-nightly

Output

mbed Build Number: null

Examples Build failed!

@adbridge
Copy link
Contributor Author

adbridge commented Sep 30, 2016

For mbed-os 5 now looks like all builds are passing and only the mbed-os uvisor example is failing. Just waiting on mbed-os 2 results now.
Speaking to uvisor guys, uvisor-example on 5.1 branch is likely to fail due to API changes in the example ready for 5.2.
mbed-os 2 results look to have improved over the previous release.
@sg- @0xc0170 I think we are finally good to go. Please merge release-candidate to release branch

@adbridge adbridge merged commit a6f3fd1 into mbed-os-5.1 Sep 30, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.