aboutsummaryrefslogtreecommitdiff
path: root/cmd/cache.c
diff options
context:
space:
mode:
authorPatrice Chotard2020-04-28 11:38:03 +0200
committerTom Rini2020-05-07 09:01:42 -0400
commitc2a2123e33371b2dc3406789764996d4fa73aac3 (patch)
tree7e1f4d1c9288724bb9bf015dcaa835f745cb93e6 /cmd/cache.c
parentaf6d4c0567f71fd24879fdb26b712b59b6a25160 (diff)
cmd: cache: Fix non-cached memory cachability
If dcache is switched OFF to ON state and if non-cached memory is used, this non-cached memory must be re-declared as uncached to mmu each time dcache is set ON. Introduce noncached_set_region() to set this non-cached region's mmu settings. Let architecture override it by defining it as a weak function. For ARM architecture, noncached_set_region() defines all noncached region as non-cacheable. Issue found on STM32MP1 platform using dwc_eth_qos ethernet driver, when going from dcache OFF to dcache ON state, ethernet driver issued TX timeout errors when performing dhcp or ping. It can be reproduced with the following sequence: dhcp while true ; do ping 192.168.1.300 ; dcache off ; ping 192.168.1.300 ; dcache on ; done Signed-off-by: Patrice Chotard <patrice.chotard@st.com> Cc: Marek Vasut <marex@denx.de> Cc: Joe Hershberger <joe.hershberger@ni.com> Cc: Ramon Fried <rfried.dev@gmail.com> Cc: Stephen Warren <swarren@nvidia.com> Reviewed-by: Marek Vasut <marex@denx.de>
Diffstat (limited to 'cmd/cache.c')
-rw-r--r--cmd/cache.c5
1 files changed, 5 insertions, 0 deletions
diff --git a/cmd/cache.c b/cmd/cache.c
index 27dcec09316..7678615dd83 100644
--- a/cmd/cache.c
+++ b/cmd/cache.c
@@ -20,6 +20,10 @@ void __weak invalidate_icache_all(void)
puts("No arch specific invalidate_icache_all available!\n");
}
+__weak void noncached_set_region(void)
+{
+}
+
static int do_icache(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
{
switch (argc) {
@@ -64,6 +68,7 @@ static int do_dcache(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
break;
case 1:
dcache_enable();
+ noncached_set_region();
break;
case 2:
flush_dcache_all();