-
Notifications
You must be signed in to change notification settings - Fork 117
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
Add connectivity framework middleware + needed patches #375
Conversation
d15f1d2
to
443a1a3
Compare
Why is this needed? |
|
443a1a3
to
e62a631
Compare
mcux/middleware/mcux-sdk-middleware-connectivity-framework/PDUM/Library/libPDUM.a
Outdated
Show resolved
Hide resolved
mcux/middleware/mcux-sdk-middleware-connectivity-framework/FSCI/pics/FSCI_packet_structure.PNG
Outdated
Show resolved
Hide resolved
e62a631
to
0a7b911
Compare
mcux/middleware/mcux-sdk-middleware-connectivity-framework/platform/rw61x/fwk_platform_ble.c
Outdated
Show resolved
Hide resolved
@@ -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 |
There was a problem hiding this comment.
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
There was a problem hiding this comment.
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
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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
cf2edd7
to
7ec2a30
Compare
a76921f
to
e9149ee
Compare
…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
e9149ee
to
0577e57
Compare
…support Add OSA, conn_fwloader, rpmsg and update wifi_bt_module. Signed-off-by: Fengming Ye <frank.ye@nxp.com>
Adding connectivity framework middleware.
Adding needed patches to be used with Zephyr.
Update Readme.