soc:esp32: Add config allow MbedTLS heap and LVGL Frambuffer use SPI RAM #85979
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
zephyrbot
requested review from
LucasTambor,
marekmatej,
raffarost,
sylvioalves,
uLipe and
wmrsouza
|
Hello @phb98, and thank you very much for your first pull request to the Zephyr project! |
phb98
changed the title
phb98
changed the title
wmrsouza
reviewed
Feb 19, 2025
soc/espressif/common/Kconfig.spiram
Outdated
| @@ -227,6 +245,7 @@ config SPIRAM_SPIWP_SD3_PIN | |||
| If burning a customized set of SPI flash pins in eFuse and using DIO or DOUT mode for flash, set this | |||
| value to the GPIO number of the SPIRAM WP pin. | |||
|
|
|||
|
|
|||
There was a problem hiding this comment.
Please, remove the extra line
|
Nice work @phb98! |
uLipe
requested changes
Feb 19, 2025
|
@wmrsouza I've tested http_get TLS sample with psram and this new Mbed config enable. for LVGL currently i'm waiting for the hardware so i will test later. |
Refactors the code in adin2111_offload_thread to improve its readability. Adds code to read RX FIFO in OA mode until it is empty. Signed-off-by: Maciej Panek <[email protected]>
Calculate the EBO properly, so the total frame written to TX FIFO does not exceed maximum Ethernet frame length. Makes the TCP stack to work properly, otherwise packets > 1472 are dopped, as they would be padded by 64 more bytes and hence exceed maximum Ethernet frame size. Signed-off-by: Maciej Panek <[email protected]>
This revert the idea of 3fa7d78 from zephyrproject-rtos#78845. The 3rd level IRQ APIs won't compile when CONFIG_3RD_LEVEL_INTERRUPT_BITS=0. Updated testcases accordingly. Signed-off-by: Yong Cong Sin <[email protected]> Signed-off-by: Yong Cong Sin <[email protected]>
Errors were logged for some cases in zephyr_img_mgmt.c, but not all. This commit adds error logs for all error cases. Signed-off-by: Nick Brook <[email protected]>
Add empty file to support building with CONFIG_BUILD_ONLY_NO_BLOBS option. Signed-off-by: Jiawei Yang <[email protected]>
Added reference to nPM1300 anomaly 38 replacing generic "LDO bug" Signed-off-by: Sergei Ovchinnikov <[email protected]>
Add the -imacros property for assembler language to compensate for changes introduced in PR zephyrproject-rtos#84800. Signed-off-by: Björn Bergman <[email protected]>
Sphinx documents should not appear in multiple toctrees. Consolidate SMP Groups under SMP Protocol Specification. Signed-off-by: Benjamin Cabé <[email protected]>
During the transition to STM32_CLOCK macro (in 57723cf), the `clocks` property of peripherals requiring more than one bit to be set were mistakenly modified. Commit 2c3294b partially fixed these errors, but some nodes for the U5 series are still wrong. Restore `clocks` on affected devices in corresponding STM32U5 DTSI. Fixes: 57723cf Signed-off-by: Mathieu Choplain <[email protected]>
Add workaround to HFCLK start and stop in nrf54l. In future workaround will be in nrfx driver. Signed-off-by: Krzysztof Chruściński <[email protected]>
I2C_SAM0_TRANSFER_TIMEOUT should depend on I2C_SAM0. Signed-off-by: Anas Nashif <[email protected]>
The current datasheet URL gives a 404. Update it to the correct location. Signed-off-by: Johan Hedberg <[email protected]>
Fixed a few occurrences of non-inclusive language. Signed-off-by: Benjamin Cabé <[email protected]>
As part of cleaning up usage of non-inclusive language in-tree, I spotted an obsolete mention to a udev rule that ModemManager doesn't use anymore. Drop it. See commit 9639e52726854782508e35a75c6abf874bd9e608 in ModemManager repo. Signed-off-by: Benjamin Cabé <[email protected]>
use allowlist terminology Signed-off-by: Benjamin Cabé <[email protected]>
s/avaialble/available/ Signed-off-by: Benjamin Cabé <[email protected]>
s/opperation/operation/ Signed-off-by: Benjamin Cabé <[email protected]>
s/condfiguration/configuration/ Signed-off-by: Benjamin Cabé <[email protected]>
s/condfiguration/configuration/ Signed-off-by: Benjamin Cabé <[email protected]>
s/advertisment/advertisement/ Signed-off-by: Benjamin Cabé <[email protected]>
s/programing/programming/ Signed-off-by: Benjamin Cabé <[email protected]>
s/programing/programming/ Signed-off-by: Benjamin Cabé <[email protected]>
s/excactly/exactly/ Signed-off-by: Benjamin Cabé <[email protected]>
This warning no longer seems to be triggering, so we shouldn't disable it. Signed-off-by: Tom Hughes <[email protected]>
While it doesn't cause any issues, it's more consistent and makes future commits which add handling for more exception types more readable. Based on: 0df8240 Signed-off-by: Michael Zimmermann <[email protected]>
NOTE: Even though previous commits indicate, that this can only happen on MacOS, that's actually not true. It happens on Linux as well. The constructor of `psutil.Process` can throw an exception as well, so we need to wrap the whole loop in another try, unfortunately. Signed-off-by: Michael Zimmermann <[email protected]>
…erty This commit removes the flexram,bank-spec property from dtsi files. The property causes flexram to be dynamically configured based on the configuration in flexram,bank-spec. This is a problem for 2 reasons: 1) The FlexRAM will always be dynamically reconfigured to default fuse configuration. This is unnecessary if using default fuses. 2) If a user decides to program fuses. The FlexRAM will still be reconfigured to the default fuse configuration. Modify description in the binding to show how to use the property at: dts/bindings/memory-controllers/nxp,flexram.yaml Added board overlay to mimxrt1170_evk_cm7 magic_addr: samples/boards/nxp/mimxrt1170_evk_cm7/magic_addr/boards Signed-off-by: Jacob Wienecke <[email protected]>
No files use MS_TO_US except tests/kernel/interrupt/src/nested_irq.c. While the macro is convenient, we should put it in a more central location and use it everywhere if we want to use it rather than just in interrupt_util.h. For now, remove it and replace the two usages to match how code is written in the rest of Zephyr. Signed-off-by: Tom Hughes <[email protected]>
This is a follow-up from a recent debate on Discord about whether splitting up related groups of properties is OK. Record the consensus outcome of the debate so this question is easier to answer next time. Signed-off-by: Martí Bolívar <[email protected]>
Signed-off-by: bac phan <[email protected]>
Signed-off-by: bac phan <[email protected]>
phb98
force-pushed
the
esp32_spiram_lvgl_mbed
branch
from
46556f0 to
14ab0d7
Compare
zephyrbot
requested review from
butok,
decsny,
DerekSnell,
dleach02,
EmilioCBen,
kartben,
mmahadevan108,
nashif and
soburi
|
I messed up git. Let me close this PR for now :( |
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.
Add 2 follow Config Symbol: ESP_SPIRAM_ALLOC_MBED_HEAP and ESP_SPIRAM_LVGL_FRAMEBUFFER to allow user place heavy-ram usage mbedtls heap and lvgl frame buffer to SPI ram.