manifest: Update hal_nxp to add Wi-Fi driver #79536
Open
+15
−133
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
The following west manifest projects have been modified in this Pull Request:
Note: This message is automatically posted and updated by the Manifest GitHub Action. |
zephyrbot
added
manifest
manifest-hal_nxp
DNM
bjarki-andreasen
previously approved these changes
Oct 8, 2024
|
looks good, ping when its a sha :) |
MaochenWang1
force-pushed
the
main_mc_update_hal_nxp_416
branch
from
e5787e5
to
8666a0b
Compare
dleach02
force-pushed
the
main_mc_update_hal_nxp_416
branch
2 times, most recently
from
f45f3e2
to
a72fb7f
Compare
dleach02
previously approved these changes
Oct 18, 2024
danieldegrasse
previously approved these changes
Oct 18, 2024
danieldegrasse
dismissed
their stale review
Too quick, still discussing this internally
danieldegrasse
force-pushed
the
main_mc_update_hal_nxp_416
branch
from
a72fb7f
to
beffd8e
Compare
zephyrbot
added
area: Samples
zephyrbot
requested review from
MarkWangChinese,
mmahadevan108,
nashif and
rlubos
|
Updated this PR with the contents of #80082, as the WiFi driver was previously not building in CI (and did not build without the changes now applied to this PR) |
krish2718
reviewed
Oct 19, 2024
| @@ -15,7 +15,7 @@ | |||
| !defined(CONFIG_EAP_PSK) && !defined(CONFIG_EAP_PAX) && \ | |||
| !defined(CONFIG_EAP_SAKE) && !defined(CONFIG_EAP_GPSK) && \ | |||
| !defined(CONFIG_EAP_PWD) && !defined(CONFIG_EAP_EKE) && \ | |||
| !defined(CONFIG_EAP_IKEV2 && !defined(CONFIG_EAP_GTC) | |||
| !defined(CONFIG_EAP_IKEV2) && !defined(CONFIG_EAP_GTC) | |||
There was a problem hiding this comment.
@jukkar how do you want deal with this? Merge this and abandon that PR or wait for it to be merged and rebase this?
There was a problem hiding this comment.
I think you should let the other PR still move forward. @jukkar PR is needed to add the test case going forward.
| @@ -75,33 +75,6 @@ CONFIG_NET_SOCKETS_SERVICE_THREAD_PRIO=3 | |||
| CONFIG_NET_TC_SKIP_FOR_HIGH_PRIO=y | |||
| CONFIG_NET_CONTEXT_PRIORITY=y | |||
|
|
|||
| CONFIG_WIFI_NM_WPA_SUPPLICANT=y | |||
There was a problem hiding this comment.
but without this will Wi-Fi work on RW612?
There was a problem hiding this comment.
remove this commit
krish2718
reviewed
Oct 19, 2024
MaochenWang1
force-pushed
the
main_mc_update_hal_nxp_416
branch
2 times, most recently
from
c0c206b
to
fb039ea
Compare
MaochenWang1
requested review from
krish2718,
danieldegrasse,
dleach02 and
bjarki-andreasen
krish2718
previously approved these changes
Oct 21, 2024
danieldegrasse
force-pushed
the
main_mc_update_hal_nxp_416
branch
from
fb039ea
to
f5cc0f0
Compare
MaochenWang1
force-pushed
the
main_mc_update_hal_nxp_416
branch
from
ab077b0
to
e913ee4
Compare
Update hal_nxp to add Wi-Fi driver code. Signed-off-by: Maochen Wang <[email protected]>
Fix a missing stray parenthesis in the conditional within the supplicant implementation, which was causing a build error for WiFi samples Signed-off-by: Daniel DeGrasse <[email protected]>
ap_bandwidth API implementation is not complete. Remove the structure definition and implementation from the NXP WiFi driver to fix a build error. Signed-off-by: Daniel DeGrasse <[email protected]>
set_btwt API implementation is not complete. Remove the structure definition and implementation from the NXP WiFi driver to fix a build error. Signed-off-by: Daniel DeGrasse <[email protected]>
Remove references to TWT fields that did not exist, to resolve a build error. Signed-off-by: Daniel DeGrasse <[email protected]>
Remove unused function `nxp_wifi_uap_status`. Signed-off-by: Daniel DeGrasse <[email protected]>
Add CONFIG_NXP_WIFI_BUILD_ONLY_MODE, which allows the NXP WiFi driver to be built without a binary blob. The built application will not be functional, but this enables continuous integration to run upstream. Signed-off-by: Daniel DeGrasse <[email protected]>
Add support for building NXP WiFi. Since the driver typically requires binary blobs, enable CONFIG_NXP_WIFI_BUILD_ONLY_MODE for this testcase. Signed-off-by: Daniel DeGrasse <[email protected]>
Fix build error of multiple definition of `dh5_init' in crypto_mbedtls_alt.c when WPS enabled. Signed-off-by: Maochen Wang <[email protected]>
Remove CONFIG_MBEDTLS_HAVE_TIME_DATE, which will cause that mbedtls include Zephyr time.h and leads to 'DIV_ROUND_UP' redefined warning. Signed-off-by: Maochen Wang <[email protected]>
MaochenWang1
force-pushed
the
main_mc_update_hal_nxp_416
branch
from
e913ee4
to
aa50575
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Update hal_nxp to add Wi-Fi driver code.