Re: [PATCH] tools: imx8m_image: Support ddr3 firmware

2021-10-21 Thread Stefano Babic

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 
---
  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: sba...@denx.de
=


Re: [PATCH] tools: imx8m_image: Support ddr3 firmware

2021-10-14 Thread Ariel D'Alessandro
+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?

> 
> Signed-off-by: Ariel D'Alessandro 
> ---
>  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