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

Add connectivity framework middleware + needed patches #375

Merged
merged 5 commits into from
May 24, 2024

Conversation

yeaissa
Copy link
Contributor

@yeaissa yeaissa commented Apr 15, 2024

Adding connectivity framework middleware.
Adding needed patches to be used with Zephyr.
Update Readme.

@mmahadevan108
Copy link
Collaborator

Why is this needed?

@yeaissa
Copy link
Contributor Author

yeaissa commented Apr 17, 2024

Why is this needed?
@mmahadevan108 , this goes with zephyrproject-rtos/zephyr#71613
Connectivity Framework is used for BLE support, HCI driver is using its APIs

@@ -0,0 +1,12 @@
Release Name: MCUXpresso Software Development Kit (SDK) Framework Middleware
Release Version: 6.1.0
Package License: LA_OPT_NXP_Software_License.htm - Additional Distribution License, Section 2.3 applies
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is this framework licensed under LA_OPT, or BSD-3? All the files seem licensed under BSD-3, but the LA_OPT license is pulled in and referenced here. The LA_OPT license isn't compatible with Zephyr, so we need to clear this up

Copy link

Choose a reason for hiding this comment

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

I think we can remove it as all files are under BSD indeed, @yeaissa will address.
FYI, mcux-sdk folder seems to contain the LA_OPT license which is probably problematic according to your comment:
https://github.com/zephyrproject-rtos/hal_nxp/tree/master/mcux/mcux-sdk

Copy link
Collaborator

Choose a reason for hiding this comment

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

Thanks for the heads up- it is probably problematic, yes. I'll discuss this internally and see what we can do there.

Copy link
Collaborator

Choose a reason for hiding this comment

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

After checking with @yvanderv, I've confirmed we should drop this file from this PR as well as the MCUX SDK side

@yeaissa yeaissa force-pushed the add-connFWK-middleware branch 2 times, most recently from a76921f to e9149ee Compare May 22, 2024 08:28
…mework

-Add new Cmake file for Zephyr usage.
-Remove static from hci_cal_data buffers to be used inside Zephyr
-Fix assertion OSA:Mutex Alloc fail
@dleach02 dleach02 merged commit 5e84f1d into zephyrproject-rtos:master May 24, 2024
@yeaissa yeaissa deleted the add-connFWK-middleware branch May 27, 2024 07:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
7 participants