Release candidate for mbed-os-5.9.5 #7753
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
/morph build |
|
/morph mbed2-release |
Build : FAILUREBuild number : 2776 |
Result: FAILUREYour command has finished executing! Here's what you wrote!
OutputBuild Prep failed! |
…xist in RTOS-less build
Decreased stack size from 24kB to 1kB (stack is used on boot-up/interrupt handler). Increased heap size from 65kB to 89kB. Change is related to issue #7137 where UBLOX_EVK_ODIN_W2 runs out of heap on WLAN.
The issue rose up when using ARMC6. A test case didn't initialize NetID parameter for ABP while using connect(params) API. NetID is the first 7 bits of the Device Address. It makes sense to actually remove the net-id parameter from ABP settings as the stack can deduce it from device address. However, the ABP structure is exposed in public APIs, so we can't really do that at the moment. Simpler fix is to move the mask that helps us to extract first 7 bits of the device address is exposed in lorawan_types.h and the user can use it to deduce correct net-id.
Fixes error: IOTCLT-2883 - Blockwise observations not completing
DEVICE_EMAC is defined only for boards with default Ethernet emac. It is not defined for example for Realtek RTL8195AM board that does not have Ethernet. Removed the check for device emac from emac greentea tests.
This commit fixes some bugs from cancel_sending() method: - System crashed if method was called before initialization. Now LORAWAN_STATUS_NOT_INITIALIZED will be returned. - Method returned LORAWAN_STATUS_BUSY error when no send request was pending. LORAWAN_STATUS_OK should be returned in this case. - LORAWAN_STATUS_BUSY is now returned if backoff timer is just about to be dispatched (time_left returns 0).
- stm32f1xx hal V1.1.2
Although other venders implement this feature by using RTC, in my H/W(RZ_A1), I cannot use RTC because it does not satisfy the spec of LP Ticker (ms order and low frequency between 8 KHz and 64 KHz). Therefore I implemented this feature by creating 1024 division by MTU2(Multi function Timer pulse Unit 2) in order to satisfy this spec. As a result of investigating, the most unaffected channel among MTU2 placed on GR-PEACH and GR-LYCHEE was channel 3, so I use channel 3 for this feature. - mbed_drv_cfg.h I added a macro of MTU2 channel to this file for commonalizing code for GR-PEACH and GR-LYCHEE, and referenced it's macro at us_ticker.c. - targets.json I added a macro for enabling LP Ticker. - mtu2.c mtu2.h I defined fuction of MTU2's clock supply and stop. Because MTU2 is utilized by pwm driver too, those function were referenced at lp_ticker driver and pwm driver. - lp_ticker.c lp_ticker_init() In order to satisfy the LP Ticker spec, I implemented by creating 1024 division by MTU2. When an interrupt is required, it will be set with ticker_set_interrupt(). - lp_ticker.c lp_ticker_free() This function stops the counting and powerdown the lp_ticker. - lp_ticker.c lp_read() This function returns the timer counter of MTU2. - lp_ticker.c lp_ticker_set_interrupt() In order to satisfy specifications, I implemented lp_ticker_set_interrupt() function. - lp_ticker.c lp_ticker_fire_interrupt() In order to satisfy spec, I implemented lp_ticker_fire_interrupt() function. Also I added GIC_EnableIRQ for allowing the interrupt at end of function. - lp_ticker.c lp_ticker_get_info() To satisfy the spec, I implemented lp_ticker_get_info() function. The value of freq includes rounding off.
I added "lp_ticker_inited = 0" in lp_ticker_free() of lp_ticker.c, so re-initialization will work.
Ignore TEST_GROUP by AStyle. As this uses macro, is not visible to formatter.
By default, it assumes it is a function and treats methods as blocks inside a
function (inlined {}). We ignore it.
We might just not format UNITTESTS in the future if we face similar issues
in the future.
Based on the https://www.keil.com/boards2/nxp/frdm_k22f/ - it contains xxx12 MCU. However IAR 7.80 has only xxx5 version, thus leaving NXP added version as xxx5.
This fixes the failure for IAR regarding floating point.
As suggested by Russ Butler in mbed-os issue #7328, and until there is an implementation of mbed-os issue #4408, we are implementing a workaround at HAL level to check if there is any ongoing serial transfer (which happens if HW FIFO is not yet empty). In case a transfer is ongoing, we're not entering deep sleep and return immediately.
Missing from common flags: * `-D__MICROLIB` * `--library_type=microlib` This patch adds them
adbridge
force-pushed
the
release-candidate
branch
from
b6a8b39 to
5dd2904
Compare
|
/morph build |
|
Note one commit has been taken from #7646 (Although full PR is scheduled for 5.10) |
Build : SUCCESSBuild number : 2781 Triggering tests/morph test |
Exporter Build : SUCCESSBuild number : 2412 |
Test : FAILUREBuild number : 2516 |
|
Going to retry this. /morph test |
Test : SUCCESSBuild number : 2518 |
cmonr
approved these changes
Aug 13, 2018
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.