Unit tests: Remove unused arm_math.h

Merge pull request #14970 from MubeenHCLite/USBSerial_connect_blocking_resolution ...

USB connect problem in non blocking mode

Merge pull request #14941 from hazzlim/update-tests-to-use-chrono ...

Update references to time values to use chrono

Merge pull request #14911 from DDC-NDRS/fatfs_0.14b ...

FatFs: upgrade to R0.14b

Merge pull request #14900 from 0xc0170/feature_CMSIS_5_13b9f72f2 ...

Update CMSIS to 5.8.0

cmsis importer: add commits for Mbed OS changes ...

See specific SHA for details.

Added PCD Start so that connection is established when a host is ...

connected in non blocking mode (cinnect_blocking = false)

rtos: fix delay with 0 ...

Fixes error osErrorParameter that triggers our assert.
RTX delay ignored 0 previously, it now returns osErrorParameter instead.

cmsis: patch fpu in asm for armcc5 ...

For FPU, use armasm to select fpu selection. This will be fixed upstream in the next version of CMSIS. Meanwhile, we use our local patch.

Taken from --cpu selection for armasm:

--cpu           | {FPU}
Cortex-M4.fp.sp | VFPv4_SP_D16
Cortex-M7.fp.sp | FPv5-SP
Cortex-M7.fp.dp | FPv5_D16

Tracking issue: https://github.com/ARM-software/CMSIS_5/issues/1266

cypress: fix cy_syslib breakpoint undefined ...

CMSIS 5.8.0 requires for armcc to include compat header prior any other header.
See Known issues for the release.

cmsis: preprocess irq files ...

We use preprocessor for asm files even for Armcc. If symbol is defined it's replaced by preprocessor,
asembler would just see 1 or 0 in this case and errors:
TARGET_M33\\irq_armv8mml.S", line 31: Error: A1185E: Symbol missing

Use preprocessor instead.

musca targets: use IsException instead of removed isIrqMode

cmsis: fix armcc compat header redefition of enable/disable irq ...

The fix will be in the 5.8.1 CMSIS, we cherry-pick it to our imported version.
See https://github.com/ARM-software/CMSIS_5/commit/e797cca3f42bef19a2c33b3b929f6010ddf53580

cmsis: fix rtx_def inclusion in .S files for Gcc Arm ...

rtx_def includes two CMSIS headers that pull in C/C++ headers in our case. As I found out,
they should only define macros. We can fix it but it will require some refactoring as our targets
use mbed rtx headers to define heap using stdin header, plus some other offenders.

Workaround is to exclude the headers we do not need in irq assembly files.

Tracking issue https://github.com/ARMmbed/mbed-os/issues/14962

semihosting: add compat header where is required ...

ARMCC provides __semihost via compat header. As CMSIS 5.8.0 removed this compat header,
we need to explicitly include it to fix definition missing error.

retarget: move compat header for ARMClang prior any cmsis headers ...

Reference: CMSIS 5.8.0 known issues and https://github.com/ARM-software/CMSIS_5/issues/1211

This fixes the error about redefinition of enable/disable irq. we need compat header because of
semihosting (not yet provided in CMSIS).

cmsis: fix license header in rtx_def ...

Upstream fix was merged https://github.com/ARM-software/CMSIS_5/pull/1238. Included here as it will be overwritten once again updated

Greentea: Remove deprecated APIs and use chrono ...

References to time should do so using std::chrono. We reworked tests in
connectivity and drivers to use std::chrono and new APIs in order to
remove deprecation warnings resulting from deprecated API calls.
This required addition of  a macro for test assertions using std::chrono
values.

As host test "timing_drift_auto" expects time values represented as an
integral number of microseconds, we explicitly provide this in place
using "microseconds{TICKER_TIME}.count()" in the relevant ticker tests.
We recognise this is ugly, but thought it best to descriptively convert
from std::chrono to the host test's required representation.

Co-authored-by: Hari Limaye <hari.limaye@arm.com>

Merge pull request #14575 from LDong-Arm/cmake_armclang_fix ...

CMake: Enable improved armclang support in CMake 3.21

Merge pull request #14288 from boraozgen/bg96-socket-send-segmentation ...

BG96: Add segmentation to TCP socket send

Allow LoRaWAN STM32WL driver debug led to be inverted (#14910)

Merge pull request #14944 from rwalton-arm/fix_ctest_reporting ...

CMake: unittests: Show all tests in CTest report

unittests: Test_LoRaWANStack.acquire_rx_metadata: Fix SEGFAULT ...

We failed to set `LoRaMac_stub::mlme_ind_ptr` to a valid pointer. When
we tried to dereference it in `LoRaWANStack::process_reception` we hit a
SEGFAULT.

unittests: Fix Test_LoRaWANStack.acquire_tx_metadata ...

This test was relying on the state of `LoRaMac_stub::bool_true_counter`
to be set to 1 from a previous test. When the test was ran in isolation
the `LoRaWANStack::rx_timeout_interrupt_handler` callback would fail
because we weren't returning true from the stub implementation of
`LoRaMac::nwk_joined`. Due to this
`LoRaWANStack::post_process_tx_no_reception` was never called. This
caused the LoRaWANStack object to think the "tx_metadata" was "stale"
(i.e the data hadn't changed since any previous read). So, when we
attempted to call `LoRaWANStack::acquire_tx_metadata` it returned
`LORAWAN_STATUS_METADATA_NOT_AVAILABLE` as it thought we had no new
metadata to report, causing the test assertion to fail.

unittests: Test_LoRaMac.post_process_mcps_req: Fix segfault ...

We had commented out a line where we reset LoRaPHY_stub::uint16_value to
0. This was causing an invalid array access in
LoRaMac::handle_data_frame, when trying to extract the
_mps_indication.channel from list of channel_params_t returned by
lora_phy->get_phy_channels.

unittests: Reduce runtime of TDBStoreModuleTest.corrupted_set_deinit_init_get ...

Instead of performing 10,000 "set, deinit, get" operations, let's just
perform 100. This reduces test time from 4.8s to 0.02s.

unittests: Remove ATHandler test interdependencies ...

Some tests in athandlertest.cpp relied on the state of stubs set in
previous tests. This caused test failures if the tests cases were ran in
isolation. Remove the interdependencies between tests by setting stub
values in the tests that rely on them.

CMake: unittests: Show all tests in CTest report ...

Previously a test executable was recognised as a single test by CTest.
However, test executables usually contain multiple test cases, the
results of the test cases should be individually reported. With our
previous setup we could miss test case failures that didn't cause the
executable to return an error code.

This commit uses gtest_discover_test to discover all test cases in a
test executable. This enables CTest to match test passes and failures
from the googletest binary output.

CMake: unittests: Remove trailing whitespace

Merge pull request #14948 from Patater/lorawan-timer-unittest-fake-fix ...

Fix lorawantimer unit test