[PATCH V2] imx8mm_beacon/imx8mn_beacon: Update build instructions

[PATCH V2] imx8mm_beacon/imx8mn_beacon: Update build instructions

[Adam Ford]

With binman generating flash.bin, it's not longer necessary to
specify either the location of ATF nor is it necessary to
specify building flash.bin, so let's update the build instructions
to remove those.  While in here, update the revision of ATF and
DDR firmware so both Mini and Nano reference the same revision.

Signed-off-by: Adam Ford <aford173@gmail.com>
---
V2:  Use tag lf-5.10.72-2.2.0 for ATF instead of branch.
     Fix small typo in Nano readme

diff --git a/board/beacon/imx8mm/README b/board/beacon/imx8mm/README
index 03d9412f0d..c65acef22d 100644
--- a/board/beacon/imx8mm/README
+++ b/board/beacon/imx8mm/README
@@ -12,21 +12,21 @@ Get and Build the ARM Trusted firmware
 Note: $(srctree) is U-Boot source directory
 
 $ git clone https://source.codeaurora.org/external/imx/imx-atf
-$ git checkout imx_5.4.70_2.3.0
+$ git lf-5.10.72-2.2.0
 $ make PLAT=imx8mm bl31 CROSS_COMPILE=aarch64-linux-gnu-
 $ cp build/imx8mm/release/bl31.bin $(srctree)
 
 Get the DDR firmware
 ====================
-$ wget https://www.nxp.com/lgfiles/NMG/MAD/YOCTO/firmware-imx-8.5.bin
-$ chmod +x firmware-imx-8.5.bin
-$ ./firmware-imx-8.5
-$ cp firmware-imx-8.5/firmware/ddr/synopsys/lpddr4*.bin $(srctree)
+$ wget https://www.nxp.com/lgfiles/NMG/MAD/YOCTO/firmware-imx-8.9.bin
+$ chmod +x firmware-imx-8.9.bin
+$ ./firmware-imx-8.9
+$ cp firmware-imx-8.9/firmware/ddr/synopsys/lpddr4*.bin $(srctree)
 
 Build U-Boot
 ============
 $ make imx8mm_beacon_defconfig
-$ make flash.bin CROSS_COMPILE=aarch64-linux-gnu-
+$ make CROSS_COMPILE=aarch64-linux-gnu-
 
 Burn U-Boot to microSD Card
 ===========================
diff --git a/board/beacon/imx8mn/README b/board/beacon/imx8mn/README
index a9eddd4e64..788ab1093b 100644
--- a/board/beacon/imx8mn/README
+++ b/board/beacon/imx8mn/README
@@ -12,9 +12,9 @@ Get and Build the ARM Trusted firmware
 Note: $(srctree) is U-Boot source directory
 
 $ git clone https://source.codeaurora.org/external/imx/imx-atf
-$ git checkout imx_5.4.47_2.2.0
+$ git lf-5.10.72-2.2.0
 $ make PLAT=imx8mn bl31 CROSS_COMPILE=aarch64-linux-gnu-
-$ cp build/imx8mm/release/bl31.bin $(srctree)
+$ cp build/imx8mn/release/bl31.bin $(srctree)
 
 Get the DDR firmware
 ====================
@@ -26,7 +26,7 @@ $ cp firmware-imx-8.9/firmware/ddr/synopsys/lpddr4*.bin $(srctree)
 Build U-Boot
 ============
 $ make imx8mn_beacon_defconfig
-$ make flash.bin CROSS_COMPILE=aarch64-linux-gnu- ATF_LOAD_ADDR=0x960000
+$ make CROSS_COMPILE=aarch64-linux-gnu-
 
 Burn U-Boot to microSD Card
 ===========================
-- 
2.32.0

Re: [PATCH V2] imx8mm_beacon/imx8mn_beacon: Update build instructions

[Fabio Estevam]

On Sat, Jan 22, 2022 at 5:48 PM Adam Ford <aford173@gmail.com> wrote:
>
> With binman generating flash.bin, it's not longer necessary to

s/not/no

> diff --git a/board/beacon/imx8mm/README b/board/beacon/imx8mm/README
> index 03d9412f0d..c65acef22d 100644
> --- a/board/beacon/imx8mm/README
> +++ b/board/beacon/imx8mm/README
> @@ -12,21 +12,21 @@ Get and Build the ARM Trusted firmware
>  Note: $(srctree) is U-Boot source directory
>
>  $ git clone https://source.codeaurora.org/external/imx/imx-atf
> -$ git checkout imx_5.4.70_2.3.0
> +$ git lf-5.10.72-2.2.0

git checkout lf-5.10.72-2.2.0

>  $ git clone https://source.codeaurora.org/external/imx/imx-atf
> -$ git checkout imx_5.4.47_2.2.0
> +$ git lf-5.10.72-2.2.0

git checkout lf-5.10.72-2.2.0

Reviewed-by: Fabio Estevam <festevam@gmail.com>

[PATCH V2] imx8mm_beacon/imx8mn_beacon: Update build instructions

[sbabic]

> With binman generating flash.bin, it's not longer necessary to
> specify either the location of ATF nor is it necessary to
> specify building flash.bin, so let's update the build instructions
> to remove those.  While in here, update the revision of ATF and
> DDR firmware so both Mini and Nano reference the same revision.
> Signed-off-by: Adam Ford <aford173@gmail.com>
> Reviewed-by: Fabio Estevam <festevam@gmail.com>
Applied to u-boot-imx, master, thanks !

Best regards,
Stefano Babic

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