On 14.10.21 14:14, Ariel D'Alessandro wrote:
+Cc Sebastien, Frieder, Baruch,

Adding you as you were involved in this file commit log. Any comments on
this simple patch?

On 9/30/21 10:31 AM, Ariel D'Alessandro wrote:
Some boards use ddr3, not ddr4, so we need to check ddr3 firmware.

Some boards are still using this script, instead of binman, to properly
pad the firmware files and generate the final image. Could we add this
simple fix to also support ddr3?

Yes, this does not break anything, I merge it.

Best regards,
Stefano Babic



Signed-off-by: Ariel D'Alessandro <[email protected]>
---
  tools/imx8m_image.sh | 7 +++++++
  1 file changed, 7 insertions(+)

diff --git a/tools/imx8m_image.sh b/tools/imx8m_image.sh
index ba601044435..4ab3116ee00 100755
--- a/tools/imx8m_image.sh
+++ b/tools/imx8m_image.sh
@@ -41,6 +41,13 @@ if [ $post_process = 1 ]; then
                cat spl/u-boot-spl-pad.bin ddr4_1d_fw.bin ddr4_2d_fw.bin > 
spl/u-boot-spl-ddr.bin
                rm -f ddr4_1d_fw.bin ddr4_2d_fw.bin ddr4_imem_1d_pad.bin 
ddr4_dmem_1d_pad.bin ddr4_imem_2d_pad.bin spl/u-boot-spl-pad.bin
        fi
+       if [ -f ddr3_imem_1d.bin ]; then
+               objcopy -I binary -O binary --pad-to 0x8000 --gap-fill=0x0 
ddr3_imem_1d.bin ddr3_imem_1d_pad.bin
+               cat ddr3_imem_1d_pad.bin ddr3_dmem_1d.bin > ddr3_1d_fw.bin
+               dd if=spl/u-boot-spl.bin of=spl/u-boot-spl-pad.bin bs=4 
conv=sync
+               cat spl/u-boot-spl-pad.bin ddr3_1d_fw.bin > 
spl/u-boot-spl-ddr.bin
+               rm -f ddr3_1d_fw.bin ddr3_imem_1d_pad.bin spl/u-boot-spl-pad.bin
+       fi
  fi
exit 0


Regards,
Ariel



--
=====================================================================
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: [email protected]
=====================================================================

Reply via email to