From: Ye Li <ye...@nxp.com>

When SPL loading image to secure region, for example, ATF and tee to
DDR secure region. Because the USDHC controller is non-secure master,
it can't access this region and will cause loading issue.

So override h_spl_load_read to use a trampoline buffer in nonsecure
region, then use CPU to copy the image from trampoline buffer to
destination secure region.

Signed-off-by: Ye Li <ye...@nxp.com>
Signed-off-by: Alice Guo <alice....@nxp.com>
Reviewed-by: Peng Fan <peng....@nxp.com>
---
 arch/arm/mach-imx/imx9/scmi/soc.c | 40 +++++++++++++++++++++++++++++++++++++++
 common/spl/spl_mmc.c              |  2 +-
 2 files changed, 41 insertions(+), 1 deletion(-)

diff --git a/arch/arm/mach-imx/imx9/scmi/soc.c 
b/arch/arm/mach-imx/imx9/scmi/soc.c
index 
fefb1a6f4ca58722a5d93c39e4f6a7aaf85aa177..8a7df24afecc1d6d77b7162be639b56af95fd975
 100644
--- a/arch/arm/mach-imx/imx9/scmi/soc.c
+++ b/arch/arm/mach-imx/imx9/scmi/soc.c
@@ -41,6 +41,8 @@
 #include <scmi_agent.h>
 #include <scmi_protocols.h>
 #endif
+#include <spl.h>
+#include <mmc.h>
 
 DECLARE_GLOBAL_DATA_PTR;
 
@@ -830,3 +832,41 @@ enum boot_device get_boot_device(void)
        return boot_dev;
 }
 #endif
+
+ulong h_spl_load_read(struct spl_load_info *load, ulong off,
+                     ulong size, void *buf)
+{
+       struct blk_desc *bd = load->priv;
+       lbaint_t sector = off >> bd->log2blksz;
+       lbaint_t count = size >> bd->log2blksz;
+       ulong trampoline_sz = SZ_16M;
+       void *trampoline = (void *)((ulong)CFG_SYS_SDRAM_BASE + PHYS_SDRAM_SIZE 
- trampoline_sz);
+       ulong ns_ddr_end = CFG_SYS_SDRAM_BASE + PHYS_SDRAM_SIZE;
+       ulong read_count, trampoline_cnt = trampoline_sz >> bd->log2blksz, 
actual, total;
+
+#ifdef PHYS_SDRAM_2_SIZE
+       ns_ddr_end += PHYS_SDRAM_2_SIZE;
+#endif
+
+       /* Check if the buf is in non-secure world, otherwise copy from 
trampoline */
+       if ((ulong)buf < CFG_SYS_SDRAM_BASE || (ulong)buf + (count * sector) > 
ns_ddr_end) {
+               total = 0;
+               while (count) {
+                       read_count = trampoline_cnt > count ? count : 
trampoline_cnt;
+                       actual = blk_dread(bd, sector, read_count, trampoline);
+                       if (actual != read_count) {
+                               printf("Error in blk_dread, %lu, %lu\n", 
read_count, actual);
+                               return 0;
+                       }
+                       memcpy(buf, trampoline, actual * 512);
+                       buf += actual * 512;
+                       sector += actual;
+                       total += actual;
+                       count -= actual;
+               }
+
+               return total << bd->log2blksz;
+       }
+
+       return blk_dread(bd, sector, count, buf) << bd->log2blksz;
+}
diff --git a/common/spl/spl_mmc.c b/common/spl/spl_mmc.c
index 
1f69659321620739d066bd806acd1d282b66bad3..d15a6945f9c24ee427b847c036413c52475c44a8
 100644
--- a/common/spl/spl_mmc.c
+++ b/common/spl/spl_mmc.c
@@ -17,7 +17,7 @@
 #include <image.h>
 #include <imx_container.h>
 
-static ulong h_spl_load_read(struct spl_load_info *load, ulong off,
+ulong __weak h_spl_load_read(struct spl_load_info *load, ulong off,
                             ulong size, void *buf)
 {
        struct blk_desc *bd = load->priv;

-- 
2.34.1

Reply via email to