-
Notifications
You must be signed in to change notification settings - Fork 3k
Resolve Realtek device small code space issue #4487
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
Conversation
cc @JanneKiiskila . |
@JanneKiiskila Can you please test? @Archcady how much space is left for the bss , as this is moving only part of it? |
@0xc0170 When tested with the WiFi example after wifi is connected, there was around 30KB for free global memory, and around 80K for heap usage. |
Why restrict to lwIP mem and not allow RW and ZI here as well? |
@Archcady bump |
Yes, we could allow RW and ZI as well. However TCM space is quite limited, if the user code have many RW / ZI data, then there will be problem if we arrange RW and ZI to TCM. By the way TCM has better performance so we put lwip mem in it for better wlan performance. |
Could be in the commit msg
Ok, so therefore just lwip rw/zi there |
/morph test |
Result: FAILUREYour command has finished executing! Here's what you wrote!
OutputExample Build failed! |
@Archcady please look at the failure. seems like sockets example fails for arm toolchain |
@@ -39,6 +39,14 @@ LR_RAM 0x10006000 0x6FFFF { | |||
} | |||
} | |||
|
|||
TCM 0x1FFF0000 0x10000{ | |||
TCM_OVERLAY 0x1FFF0000 0x10000{ |
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.
This is all zero init RAM right? Why does it need the load region TCM? If you move the execution region TCM_OVERLAY inside LR_RAM and remove the load region TCM then this builds.
switch on TCM section usage, this should give user code more codespace.
/morph test |
Result: SUCCESSYour command has finished executing! Here's what you wrote!
OutputAll builds and test passed! |
Switch on TCM section usage of Realtek RTL8195AM device. This should give user code more codespace.