Skip to content
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

Some Mbed OS 6 targets don't compile with Baremetal profile #13099

Closed
MarceloSalazar opened this issue Jun 11, 2020 · 17 comments
Closed

Some Mbed OS 6 targets don't compile with Baremetal profile #13099

MarceloSalazar opened this issue Jun 11, 2020 · 17 comments

Comments

@MarceloSalazar
Copy link

MarceloSalazar commented Jun 11, 2020

Description of defect

Mbed OS 6 has enabled the bare-metal profile for a few targets, but not all of them (details in targets.json)

Developers are trying to use the mbed-os-example-blinky-baremetal application with some Mbed OS 6 targets that are not yet supported - see the issue raised in forums: https://forums.mbed.com/t/some-issues-with-the-mbedos6-and-mbed-tools

Eventually, all targets should support the bare-metal profile (not a requirement at the moment).
Until then, this issue with list of targets should help to keep track of the status.

Target(s) affected by this defect ?

Targets marked with ❌ fail to compile.

Updated table for Mbed OS master 2f87d59(29th Oct 2020)

Target ARM GCC_ARM Issue/comment
ADV_WISE_1510 ✔️ ✔️
ADV_WISE_1570 ✔️ ✔️
ARCH_MAX ✔️
ARCH_PRO ✔️ ✔️
ARDUINO_NANO33BLE ✔️ ✔️
ARM_CM3DS_MPS2 ✔️
ARM_MPS2_M0 ✔️
ARM_MPS2_M0P ✔️
ARM_MPS2_M3 ✔️
ARM_MPS2_M4 ✔️
ARM_MPS2_M7 ✔️
ARM_MUSCA_B1
ARM_MUSCA_B1_NS #13110 #13165
ARM_MUSCA_S1 #13850
ARM_MUSCA_S1_NS #13850
B_L4S5I_IOT01A ✔️ ✔️
CY8CKIT_062_BLE ✔️ ✔️ #13108
CY8CKIT_062_WIFI_BT ✔️ ✔️ #13108
CY8CKIT_062S2_43012 ✔️ ✔️ #13108
CY8CKIT064B0S2_4343W #13108
CY8CPROTO_062_4343W ✔️ ✔️ #13108
CY8CPROTO_062S3_4343W ✔️ ✔️ #13108
CYSBSYSKIT_01 #13108
CYTFM_064B0S2_4343W #13108
CYW9P62S1_43012EVB_01 ✔️ ✔️ #13108
CYW9P62S1_43438EVB_01 ✔️ ✔️ #13108
DISCO_F413ZH ✔️ ✔️
DISCO_F429ZI ✔️ ✔️
DISCO_F469NI ✔️ ✔️
DISCO_F746NG ✔️ ✔️
DISCO_F769NI ✔️ ✔️
DISCO_H747I ✔️ ✔️
DISCO_H747I_CM4 ✔️ ✔️
DISCO_H747I_CM7 ✔️ ✔️
DISCO_L072CZ_LRWAN1 ✔️ ✔️
DISCO_L475VG_IOT01A ✔️ ✔️
DISCO_L476VG ✔️ ✔️
DISCO_L496AG ✔️ ✔️
DISCO_L4R9I ✔️ ✔️
DISCO_L562QE ✔️ ✔️
EFM32GG_STK3700 ✔️ #13137
EFM32GG11_STK3701 ✔️ #13137
EP_AGORA ✔️ ✔️ tests fail #13185
EP_ATLAS ✔️ ✔️
EV_COG_AD3029LZ ✔️ ✔️
EV_COG_AD4050LZ ✔️ ✔️
FF_LPC546XX ✔️ #13140
FF1705_L151CC ✔️
FVP_MPS2_M0 ✔️ #13138
FVP_MPS2_M0P ✔️ #13138
FVP_MPS2_M3 ✔️ #13138
FVP_MPS2_M4 ✔️ #13138
FVP_MPS2_M7 ✔️ #13138
GD32_F307VG ✔️ #13136
GD32_F450ZI ✔️ #13136
GR_LYCHEE ✔️ ✔️
GR_MANGO ✔️ ✔️
HEXIWEAR ✔️ ✔️
K22F ✔️ ✔️
K64F ✔️ ✔️
K66F ✔️ ✔️
K82F ✔️ ✔️ tests fail #13186
KL25Z ✔️ ✔️
KL43Z ✔️ ✔️
KL46Z ✔️ ✔️
KW41Z ✔️ ✔️ #13109 , #13119
LPC1114 ✔️ ✔️
LPC1768 ✔️ ✔️
LPC54114 ✔️ #13140
LPC546XX ✔️ #13140
MAX32600MBED ✔️ #13139
MAX32620FTHR ✔️ #13139
MAX32625MBED ✔️ #13139
MAX32625PICO ✔️ #13139
MAX32630FTHR ✔️ #13139
MIMXRT1050_EVK ✔️ ✔️
MOTE_L152RC ✔️
MTS_DRAGONFLY_F411RE ✔️
MTS_DRAGONFLY_F413RH ✔️ ✔️
MTS_DRAGONFLY_L471QG ✔️
MTS_MDOT_F411RE ✔️
NRF52_DK ✔️ #13857
NRF52840_DK ✔️ ✔️
NUCLEO_F070RB ✔️ ✔️
NUCLEO_F072RB ✔️ ✔️
NUCLEO_F091RC ✔️ ✔️
NUCLEO_F103RB ✔️ ✔️
NUCLEO_F207ZG ✔️ ✔️
NUCLEO_F303K8 ✔️ ✔️
NUCLEO_F303RE ✔️ ✔️
NUCLEO_F303ZE ✔️ ✔️
NUCLEO_F401RE ✔️ ✔️
NUCLEO_F411RE ✔️ ✔️
NUCLEO_F412ZG ✔️ ✔️
NUCLEO_F413ZH ✔️ ✔️
NUCLEO_F429ZI ✔️ ✔️
NUCLEO_F439ZI ✔️ ✔️
NUCLEO_F446RE ✔️ ✔️
NUCLEO_F446ZE ✔️ ✔️
NUCLEO_F746ZG ✔️ ✔️
NUCLEO_F756ZG ✔️ ✔️
NUCLEO_F767ZI ✔️ ✔️
NUCLEO_G031K8 ✔️ ✔️
NUCLEO_G071RB ✔️ ✔️
NUCLEO_G474RE ✔️ ✔️
NUCLEO_H743ZI2 ✔️ ✔️
NUCLEO_H7A3ZI_Q ✔️ ✔️
NUCLEO_L073RZ ✔️ ✔️
NUCLEO_L152RE ✔️ ✔️
NUCLEO_L432KC ✔️ ✔️
NUCLEO_L433RC_P ✔️ ✔️
NUCLEO_L452RE_P ✔️ ✔️
NUCLEO_L476RG ✔️ ✔️
NUCLEO_L486RG ✔️ ✔️
NUCLEO_L496ZG ✔️ ✔️
NUCLEO_L496ZG_P ✔️ ✔️
NUCLEO_L4R5ZI ✔️ ✔️
NUCLEO_L4R5ZI_P ✔️ ✔️
NUCLEO_L552ZE_Q ✔️ ✔️
NUCLEO_WB55RG ✔️ ✔️
NUMAKER_IOT_M252 ✔️ ✔️
NUMAKER_IOT_M263A ✔️ ✔️
NUMAKER_IOT_M487 ✔️ ✔️
NUMAKER_PFM_M453 ✔️ ✔️
NUMAKER_PFM_M487 ✔️ ✔️
NUMAKER_PFM_NANO130 ✔️ ✔️
NUMAKER_PFM_NUC472 ✔️ ✔️
RHOMBIO_L476DMW1K ✔️ ✔️
RZ_A1H ✔️ ✔️
S1SBP6A #13851
S5JS100 #13111
SDP_K1 ✔️ ✔️
SDT32620B ✔️
SDT32625B ✔️
SDT52832B ✔️
SDT64B ✔️ ✔️
SFE_ARTEMIS #13852
SFE_ARTEMIS_ATP #13852
SFE_ARTEMIS_DK #13852
SFE_ARTEMIS_MODULE #13852
SFE_ARTEMIS_NANO #13852
SFE_ARTEMIS_THING_PLUS #13852
SFE_EDGE #13852
SFE_EDGE2 #13852
TB_SENSE_12 ✔️ #13109 , #13119
TMPM46B ✔️ #13141
TMPM4G9 ✔️ #13141
UHURU_RAVEN ✔️ ✔️ tests fail #13187
WIO_3G ✔️ ✔️
WIO_BG96 ✔️ ✔️
WIO_EMW3166 ✔️ ✔️ #13112
XDOT_L151CC ✔️

Toolchain(s) (name and version) displaying this defect ?

Arm Compile 6.13 and GCC 9 2019-q4

What version of Mbed-os are you using (tag or sha) ?

6.0.0 - 165be79

What version(s) of tools are you using. List all that apply (E.g. mbed-cli)

Mbed CLI 1.10.0

How is this defect reproduced ?

mbed import https://github.com/ARMmbed/mbed-os-example-blinky-baremetal
cd mbed-os-example-blinky-baremetal
mbed compile -t <toolchain> -m <target>
@ciarmcom
Copy link
Member

@MarceloSalazar thank you for raising this issue.Please take a look at the following comments:

Could you add some more detail to the description? A good description should be at least 25 words.

NOTE: If there are fields which are not applicable then please just add 'n/a' or 'None'.This indicates to us that at least all the fields have been considered.
Please update the issue header with the missing information, the issue will not be mirroredto our internal defect tracking system or investigated until this has been fully resolved.

@evedon
Copy link
Contributor

evedon commented Jul 1, 2020

@MarceloSalazar Fix for KW41Z was merged. Can you confirm if this target is now passing (and update the table)?

@MarceloSalazar
Copy link
Author

MarceloSalazar commented Jul 1, 2020

I'm in the process of updating the table (edit: DONE)
I've just tested KW41W with both ARM and GCC and fails (sha ffeb926)

[Fatal Error] NanostackRfPhyKw41z.cpp@17,10: common_functions.h: No such file or directory
[ERROR] ./mbed-os/features/nanostack/targets/TARGET_Freescale/TARGET_KW41Z/NanostackRfPhyKw41z.cpp:17:10: fatal error: common_functions.h: No such file or directory
   17 | #include "common_functions.h"

@evedon
Copy link
Contributor

evedon commented Jul 3, 2020

@LDong-Arm see previous comment from Marcelo

@LDong-Arm
Copy link
Contributor

LDong-Arm commented Jul 3, 2020

The cause is that #13115 (an update from NXP) wasn't rebased onto my fix for bare-metal support (#13119). Now we have duplicated copies of Nanostack drivers for TARGET_KW41Z...
I'll create a PR to fix it.

Update: PR created: #13230 (merged)

@0Grit
Copy link

0Grit commented Aug 12, 2020

I need this fixed for CY8CKIT_062_WIFI_BT

From what I gather wifi requires threading but if we can get baremetal at least working for the target that would be a start.

@bentcooke

@LDong-Arm
Copy link
Contributor

I need this fixed for CY8CKIT_062_WIFI_BT

From what I gather wifi requires threading but if we can get baremetal at least working for the target that would be a start.

@bentcooke

Created a quick fix #13425, feel free to have a try.

@evedon
Copy link
Contributor

evedon commented Aug 28, 2020

@MarceloSalazar Could you update the table?

@MarceloSalazar
Copy link
Author

I'm in the process of updating the table - please bear with me.
I've re-run the compile tests and see many targets failing, but issues are related to storage.
Can you have a look?

$ mbed compile -t GCC_ARM -m DISCO_F413ZH 
[mbed] Working path "/Users/.../Mbed Programs/mbed-os-example-blinky-baremetal" (program)
Building project mbed-os-example-blinky-baremetal (DISCO_F413ZH, GCC_ARM)
Scan: mbed-os-example-blinky-baremetal
Compile [ 98.3%]: qspi_api.c
[Fatal Error] qspi_api.c@28,10: mbed-trace/mbed_trace.h: No such file or directory

@LDong-Arm
Copy link
Contributor

LDong-Arm commented Sep 21, 2020

@MarceloSalazar Please see the discussion here: #13617 (comment)

The solution we agreed is to enable mbed-trace by default. But this requires ipv6 printing to be disabled (as it requires mbed-client-libservice), and this would break existing applications (network and Pelion ones mainly). Not sure how to handle it...

@ciarmcom
Copy link
Member

ciarmcom commented Oct 2, 2020

Thank you for raising this detailed GitHub issue. I am now notifying our internal issue triagers.
Internal Jira reference: https://jira.arm.com/browse/IOTOSM-2199

@evedon
Copy link
Contributor

evedon commented Oct 22, 2020

@MarceloSalazar mbed-trace is now enabled by default in bare metal, see #13649
Can you update the table again?

@MarceloSalazar
Copy link
Author

MarceloSalazar commented Nov 4, 2020

Table updated and new issues raised

@harmut01
Copy link
Contributor

harmut01 commented Mar 8, 2021

@MarceloSalazar with the exception of the targets listed below, nearly all the targets have fixes merged or pending review. Can I update the table once the changes are in?

Target ARM GCC_ARM
ARM_MUSCA_B1
ARM_MUSCA_B1_NS
ARM_MUSCA_S1
ARM_MUSCA_S1_NS
CY8CKIT_062_BLE ❌ *(m/nlib) ❌ *(m/nlib)
CY8CKIT_062_WIFI_BT ❌ *(m/nlib) ❌ *(m/nlib)
CY8CKIT_062S2_43012 ❌ *(m/nlib) ❌ *(m/nlib)
CY8CKIT064B0S2_4343W ❌ *(m/nlib) ❌ *(m/nlib)
CY8CPROTO_062_4343W ❌ *(m/nlib) ❌ *(m/nlib)
CY8CPROTO_062S3_4343W ❌ *(m/nlib) ❌ *(m/nlib)
CYSBSYSKIT_01 ❌ *(m/nlib) ❌ *(m/nlib)
CYTFM_064B0S2_4343W ❌ *(m/nlib) ❌ *(m/nlib)
CYW9P62S1_43012EVB_01 ❌ *(m/nlib) ❌ *(m/nlib)
CYW9P62S1_43438EVB_01 ❌ *(m/nlib) ❌ *(m/nlib)
KW41Z
S5JS100
SDT32620B

@MarceloSalazar
Copy link
Author

MarceloSalazar commented Mar 8, 2021

@harmut01 thanks! I'll re-start the testing once a few PRs get merged

@harmut01
Copy link
Contributor

harmut01 commented Mar 8, 2021

Awesome! Here are the PRs that have already been merged: #14291, #14300, #14345, #14348, #14349, and #14363. I'll mention this issue in the remaining ones and let you know once they're in.

@ciarmcom
Copy link
Member

We closed this issue because it has been inactive for quite some time and we believe it to be low priority. If you think that the priority should be higher, then please reopen with your justification for increasing the priority.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants