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

Wiseconnect 3.4 #68

Merged
merged 6 commits into from
Feb 3, 2025

Conversation

jerome-pouiller
Copy link
Contributor

@jerome-pouiller jerome-pouiller commented Dec 3, 2024

Migrate to WiseConnect 3.4. The integration issue has been fixed.

@jerome-pouiller jerome-pouiller force-pushed the wiseconnect-3.4 branch 5 times, most recently from 83f7d3f to e5bd024 Compare January 24, 2025 15:46
@jerome-pouiller jerome-pouiller marked this pull request as ready for review January 24, 2025 15:46
@jerome-pouiller jerome-pouiller requested a review from a team as a code owner January 24, 2025 15:46
@@ -142,3 +142,6 @@ zephyr_library_sources_ifdef(CONFIG_WISECONNECT_NETWORK_STACK
${PROTOCOL_DIR}/wifi/si91x/sl_wifi.c
${PROTOCOL_DIR}/wifi/src/sl_wifi_callback_framework.c
)

zephyr_linker_sources(ROM_SECTIONS linker/code_classification_text.ld)
zephyr_linker_sources(RAMFUNC_SECTION linker/code_classification_ramfunc.ld)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This code is duplicated with simplicity_sdk. I have to work on that.

(In fact, I get errors from sl_core_cortexm.c, but I have not yet understand how this file come into the compilation.)

@jerome-pouiller jerome-pouiller marked this pull request as draft January 24, 2025 15:59
@jerome-pouiller
Copy link
Contributor Author

Converting to a draft. I have an issue with SiSdk 2024.12.

@jerome-pouiller
Copy link
Contributor Author

My bad, my TA firmware was not up to date.

@jerome-pouiller jerome-pouiller marked this pull request as ready for review January 28, 2025 16:13
@jhedberg jhedberg added the DNM label Jan 28, 2025
@jhedberg
Copy link
Collaborator

I added the DNM label manually for now, until the west.yml reference is to an actual commit rather than a PR.

Wiseconnect relies on Simplicity SDK.

For now Gecko SDK still work with Wiseconnect because Simplicity SDK
didn't yet diverged from Gecko SDK.

Simplicity SDK 2024.12 introduce "Code Classification", so we need extra
linker scripts.

Signed-off-by: Jérôme Pouiller <jerome.pouiller@silabs.com>
Some files become orphans when socket offloaded is not used.

Signed-off-by: Jérôme Pouiller <jerome.pouiller@silabs.com>
Currently, IRQ074_Handler is declared with ISR_FLAG_DIRECT. So it
bypasses the Zephyr ISR wrapper.

There is not real reason to apply this flag. Worse, ISR_FLAG_DIRECT
skips tasks rescheduling and may lead to unwanted behaviors.

Signed-off-by: Jérôme Pouiller <jerome.pouiller@silabs.com>
Update to the last Wiseconnect version.

Signed-off-by: Jérôme Pouiller <jerome.pouiller@silabs.com>
Wiseconnect 3.4 rename the symbol NUMBER_OF_BSD_SOCKETS

Signed-off-by: Jérôme Pouiller <jerome.pouiller@silabs.com>
Wiseconnect 3.4 renamed the symbol ULP_UART_ULP_MHZ_RC_CLK

Signed-off-by: Jérôme Pouiller <jerome.pouiller@silabs.com>
@jhedberg jhedberg merged commit 0b2db8b into SiliconLabsSoftware:main Feb 3, 2025
3 of 4 checks passed
@jerome-pouiller jerome-pouiller deleted the wiseconnect-3.4 branch February 18, 2025 11:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants