From: suda-morris <362953310@qq.com>
Date: Thu, 10 Oct 2019 03:38:14 +0000 (+0800)
Subject: make psram workaround depend on chip revison
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=b3720f80c772a275f71327f04cc35ab1c2c918ec;p=esp-idf

make psram workaround depend on chip revison

Since ESP32 revision 3, the PSRAM workaround is not needed.
---

diff --git a/components/esp32/Kconfig b/components/esp32/Kconfig
index 8c60767c60..9245420f5a 100644
--- a/components/esp32/Kconfig
+++ b/components/esp32/Kconfig
@@ -155,7 +155,7 @@ menu "ESP32-specific"
 
         config SPIRAM_CACHE_WORKAROUND
             bool "Enable workaround for bug in SPI RAM cache for Rev1 ESP32s"
-            depends on SPIRAM_USE_MEMMAP || SPIRAM_USE_CAPS_ALLOC || SPIRAM_USE_MALLOC
+            depends on (SPIRAM_USE_MEMMAP || SPIRAM_USE_CAPS_ALLOC || SPIRAM_USE_MALLOC) && (ESP32_REV_MIN < 3)
             default "y"
             help
                 Revision 1 of the ESP32 has a bug that can cause a write to PSRAM not to take place in some situations
@@ -166,6 +166,8 @@ menu "ESP32-specific"
                 This will also not use any bits of newlib that are located in ROM, opting for a version that is
                 compiled with the workaround and located in flash instead.
 
+                The workaround is not required for ESP32 revision 3 and above.
+
         config SPIRAM_BANKSWITCH_ENABLE
             bool "Enable bank switching for >4MiB external RAM"
             default y