From 6415589a5c9dccc1f23dcd8469ff0743a7037824 Mon Sep 17 00:00:00 2001 From: vasyl Date: Fri, 18 Jun 2021 11:18:33 +0300 Subject: arm-trusted-firmware: Fix BL2 fail on Renesas BSP 4.7.0. Backport [1] to fix boot error: NOTICE: BL2: Cannot set compatible string, board unsupported [1]: https://github.com/renesas-rcar/arm-trusted-firmware/commit/3ebd371fa1b1137ddee85b1825d1c1ce31c719d1 Bug-AGL: SPEC-3916 Change-Id: I5289437e71140d16a9e3505733b6423ec79ffe81 Signed-off-by: Vasyl Vavrychuk (cherry picked from commit c41ff5d9242d4c9e32237fcfeca18b727f641517) --- .../arm-trusted-firmware_%.bbappend | 2 + ...-rcar_gen3-plat-Delete-FDT-function-calls.patch | 145 +++++++++++++++++++++ 2 files changed, 147 insertions(+) create mode 100644 meta-agl-refhw-gen3/recipes-bsp/arm-trusted-firmware/files/0001-rcar_gen3-plat-Delete-FDT-function-calls.patch diff --git a/meta-agl-refhw-gen3/recipes-bsp/arm-trusted-firmware/arm-trusted-firmware_%.bbappend b/meta-agl-refhw-gen3/recipes-bsp/arm-trusted-firmware/arm-trusted-firmware_%.bbappend index 962f84e..d36946c 100644 --- a/meta-agl-refhw-gen3/recipes-bsp/arm-trusted-firmware/arm-trusted-firmware_%.bbappend +++ b/meta-agl-refhw-gen3/recipes-bsp/arm-trusted-firmware/arm-trusted-firmware_%.bbappend @@ -1,5 +1,7 @@ FILESEXTRAPATHS_prepend := "${THISDIR}/files:" +SRC_URI_append_agl-refhw-h3 = " file://0001-rcar_gen3-plat-Delete-FDT-function-calls.patch" + # Apply the patch AGL applies for all rcar3 in meta-agl-bsp when # building the standalone agl-refhw-h3 machine. SRC_URI_append_agl-refhw-h3 = " file://0001-Boot-Normal-World-in-EL2.patch" diff --git a/meta-agl-refhw-gen3/recipes-bsp/arm-trusted-firmware/files/0001-rcar_gen3-plat-Delete-FDT-function-calls.patch b/meta-agl-refhw-gen3/recipes-bsp/arm-trusted-firmware/files/0001-rcar_gen3-plat-Delete-FDT-function-calls.patch new file mode 100644 index 0000000..067bbb2 --- /dev/null +++ b/meta-agl-refhw-gen3/recipes-bsp/arm-trusted-firmware/files/0001-rcar_gen3-plat-Delete-FDT-function-calls.patch @@ -0,0 +1,145 @@ +From a8e6139af9a307cc30d2e804819da963e419f017 Mon Sep 17 00:00:00 2001 +From: Toshiyuki Ogasahara +Date: Tue, 15 Dec 2020 18:23:32 +0900 +Subject: [PATCH] rcar_gen3: plat: Delete FDT function calls + +Since U-boot configures the device tree, the FDT function call by +BL31 is removed. + +Signed-off-by: Toshiyuki Ogasahara +Signed-off-by: Yoshifumi Hosoya +--- + plat/renesas/rcar/bl2_plat_setup.c | 27 +++++++++++++++++++-------- + 1 file changed, 19 insertions(+), 8 deletions(-) + +diff --git a/plat/renesas/rcar/bl2_plat_setup.c b/plat/renesas/rcar/bl2_plat_setup.c +index 24a13c7f7..59a72b5bb 100644 +--- a/plat/renesas/rcar/bl2_plat_setup.c ++++ b/plat/renesas/rcar/bl2_plat_setup.c +@@ -110,6 +110,7 @@ static meminfo_t bl2_tzram_layout __aligned(CACHE_WRITEBACK_GRANULE); + + /* FDT with DRAM configuration */ + uint64_t fdt_blob[PAGE_SIZE_4KB / sizeof(uint64_t)]; ++#if 0 + static void *fdt = (void *)fdt_blob; + + static void unsigned_num_print(unsigned long long int unum, unsigned int radix, +@@ -133,7 +134,7 @@ static void unsigned_num_print(unsigned long long int unum, unsigned int radix, + while (--i >= 0) + *string++ = num_buf[i]; + } +- ++#endif + #if (RCAR_LOSSY_ENABLE == 1) + typedef struct bl2_lossy_info { + uint32_t magic; +@@ -145,6 +146,7 @@ static void bl2_lossy_gen_fdt(uint32_t no, uint64_t start_addr, + uint64_t end_addr, uint32_t format, + uint32_t enable, int fcnlnode) + { ++#if 0 + const uint64_t fcnlsize = cpu_to_fdt64(end_addr - start_addr); + char nodename[40] = { 0 }; + int ret, node; +@@ -199,6 +201,7 @@ static void bl2_lossy_gen_fdt(uint32_t no, uint64_t start_addr, + NOTICE("BL2: Cannot add FCNL formats prop (ret=%i)\n", ret); + panic(); + } ++#endif + } + + static void bl2_lossy_setting(uint32_t no, uint64_t start_addr, +@@ -449,6 +452,7 @@ struct meminfo *bl2_plat_sec_mem_layout(void) + return &bl2_tzram_layout; + } + ++#if 0 + static void bl2_populate_compatible_string(void *dt) + { + uint32_t board_type; +@@ -537,13 +541,17 @@ static void bl2_populate_compatible_string(void *dt) + panic(); + } + } ++#endif + + static void bl2_advertise_dram_entries(uint64_t dram_config[8]) + { ++#if 0 + char nodename[32] = { 0 }; +- uint64_t start, size; + uint64_t fdtsize; +- int ret, node, chan; ++ int ret, node; ++#endif ++ uint64_t start, size; ++ int chan; + + for (chan = 0; chan < 4; chan++) { + start = dram_config[2 * chan]; +@@ -556,7 +564,7 @@ static void bl2_advertise_dram_entries(uint64_t dram_config[8]) + (size >> 30) ? : size >> 20, + (size >> 30) ? "G" : "M"); + } +- ++#if 0 + /* + * We add the DT nodes in reverse order here. The fdt_add_subnode() + * adds the DT node before the first existing DT node, so we have +@@ -604,6 +612,7 @@ static void bl2_advertise_dram_entries(uint64_t dram_config[8]) + err: + NOTICE("BL2: Cannot add memory node to FDT (ret=%i)\n", ret); + panic(); ++#endif + } + + static void bl2_advertise_dram_size(uint32_t product) +@@ -927,7 +936,7 @@ lcm_state: + } + rcar_qos_init(); + } +- ++#if 0 + /* Set up FDT */ + ret = fdt_create_empty_tree(fdt, sizeof(fdt_blob)); + if (ret) { +@@ -937,7 +946,7 @@ lcm_state: + + /* Add platform compatible string */ + bl2_populate_compatible_string(fdt); +- ++#endif + /* Print DRAM layout */ + bl2_advertise_dram_size(product); + +@@ -989,14 +998,14 @@ lcm_state: + } + #if (RCAR_LOSSY_ENABLE == 1) + NOTICE("BL2: Lossy Decomp areas\n"); +- ++#if 0 + fcnlnode = fdt_add_subnode(fdt, 0, "reserved-memory"); + if (fcnlnode < 0) { + NOTICE("BL2: Cannot create reserved mem node (ret=%i)\n", + fcnlnode); + panic(); + } +- ++#endif + bl2_lossy_setting(0, LOSSY_ST_ADDR0, LOSSY_END_ADDR0, + LOSSY_FMT0, LOSSY_ENA_DIS0, fcnlnode); + bl2_lossy_setting(1, LOSSY_ST_ADDR1, LOSSY_END_ADDR1, +@@ -1005,8 +1014,10 @@ lcm_state: + LOSSY_FMT2, LOSSY_ENA_DIS2, fcnlnode); + #endif + ++#if 0 + fdt_pack(fdt); + NOTICE("BL2: FDT at %p\n", fdt); ++#endif + + if (boot_dev == MODEMR_BOOT_DEV_EMMC_25X1 || + boot_dev == MODEMR_BOOT_DEV_EMMC_50X8) +-- +2.30.2 + -- cgit 1.2.3-korg