yamt commented on code in PR #16756:
URL: https://github.com/apache/nuttx/pull/16756#discussion_r2227087654


##########
arch/xtensa/src/esp32s3/Kconfig:
##########
@@ -1090,9 +1090,12 @@ choice ESP32S3_SPIRAM_HEAP
 
 config ESP32S3_SPIRAM_COMMON_HEAP
        bool "Additional region to kernel heap"
+       depends on !ESP32S3_SPIFLASH && !ESPRESSIF_WIFI && !BUILD_PROTECTED

Review Comment:
   isn't CONFIG_XTENSA_IMEM_USE_SEPARATE_HEAP enough for wifi?
   
   i guess it also makes sense to allocate LPWORK stack from the internal 
memory. what do you think?



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to