Skip to content

Add mbed-mbedtls-cryptocell310, mbed-ble-cordio_ll library #342

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.

Sign up for GitHub

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

Closed
wants to merge 1 commit into from
Closed

Add mbed-mbedtls-cryptocell310, mbed-ble-cordio_ll library #342

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions BLE_Advertising/CMakeLists.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,8 @@ target_link_libraries(${APP_TARGET}
mbed-ble-cordio
mbed-ble-blue_nrg
mbed-os-ble-utils
mbed-mbedtls-cryptocell310
mbed-ble-cordio_ll
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what's the diff between mbed-ble-cordio and mbed-ble-cordio_ll ? I assume only one should be needed ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The build was failed but let me try one more time without mbed-ble-cordio_ll

Copy link
Contributor Author

@rajkan01 rajkan01 Nov 12, 2020 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It expects some header files from mbed-ble-cordio_ll When I tried to build for NRF52840_DK

./mbed-os/connectivity/drivers/ble/FEATURE_BLE/TARGET_NORDIC/TARGET_NORDIC_CORDIO/TARGET_NRF5x/stack/sources/pal_timer.c:59:10: fatal error: sch_api.h: No such file or directory
   59 | #include "sch_api.h"
      |          ^~~~~~~~~~~

It failed when I tried to build for K64F, If I only kept mbed-ble-cordio_ll and removed mbed-ble-cordio

gerImpl.cpp:24:
../mbed-os/connectivity/FEATURE_BLE/./source/pal/PalSecurityManager.h:29:10: fatal error: wsf_types.h: No such file or directory
   29 | #include "wsf_types.h"
      |          ^~~~~~~~~~~~~

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wonder if this dep should not be fixed in our Mbed tree rather than here

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cordio_ll is the link layer, it depends on the board, nrf uses it, others use the HCI driver because their link layer is provided separately

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be fixed in Mbed Os. @rajkan01 please remove these and fix it in ble component.

)

mbed_set_post_build(${APP_TARGET})
Expand Down