[PATCH V4 7/8] arm: dts: imx8m: shrink ddr firmware size to actual file size

Alper Nebi Yasak alpernebiyasak at gmail.com
Mon May 23 23:12:21 CEST 2022


On 20/05/2022 17:10, Peng Fan (OSS) wrote:
> From: Peng Fan <peng.fan at nxp.com>
> 
> After we switch to use BINMAN_SYMBOLS, there is no need to pad
> the file size to 0x8000 and 0x4000. After we use BINMAN_SYMBOLS,
> the u-boot-spl-ddr.bin shrink about 36KB with i.MX8MP-EVK.
> 
> Tested-by: Tim Harvey <tharvey at gateworks.com> #imx8m[m,n,p]-venice
> Signed-off-by: Peng Fan <peng.fan at nxp.com>
> ---
>  arch/arm/dts/imx8mm-u-boot.dtsi                  | 8 ++++----
>  arch/arm/dts/imx8mn-beacon-kit-u-boot.dtsi       | 8 ++++----
>  arch/arm/dts/imx8mn-ddr4-evk-u-boot.dtsi         | 8 ++++----
>  arch/arm/dts/imx8mn-evk-u-boot.dtsi              | 8 ++++----
>  arch/arm/dts/imx8mn-var-som-symphony-u-boot.dtsi | 8 ++++----
>  arch/arm/dts/imx8mn-venice-u-boot.dtsi           | 8 ++++----
>  arch/arm/dts/imx8mp-u-boot.dtsi                  | 8 ++++----
>  arch/arm/dts/imx8mq-cm-u-boot.dtsi               | 8 ++++----
>  arch/arm/dts/imx8mq-u-boot.dtsi                  | 8 ++++----

Probably can be done for 'imx8mn-bsh-smm-s2-u-boot-common.dtsi' as well.

>  9 files changed, 36 insertions(+), 36 deletions(-)
> 
> diff --git a/arch/arm/dts/imx8mm-u-boot.dtsi b/arch/arm/dts/imx8mm-u-boot.dtsi
> index 5de55a2d80b..19a2da30f51 100644
> --- a/arch/arm/dts/imx8mm-u-boot.dtsi
> +++ b/arch/arm/dts/imx8mm-u-boot.dtsi
> @@ -41,25 +41,25 @@
>  
>  		imem_1d: blob-ext at 1 {
>  			filename = "lpddr4_pmu_train_1d_imem.bin";
> -			size = <0x8000>;
> +			align-end = <4>;
>  			type = "blob-ext";
>  		};
>  
>  		dmem_1d: blob-ext at 2 {
>  			filename = "lpddr4_pmu_train_1d_dmem.bin";
> -			size = <0x4000>;
> +			align-end = <4>;
>  			type = "blob-ext";
>  		};
>  
>  		imem_2d: blob-ext at 3 {
>  			filename = "lpddr4_pmu_train_2d_imem.bin";
> -			size = <0x8000>;
> +			align-end = <4>;
>  			type = "blob-ext";
>  		};
>  
>  		dmem_2d: blob-ext at 4 {
>  			filename = "lpddr4_pmu_train_2d_dmem.bin";
> -			size = <0x4000>;
> +			align-end = <4>;
>  			type = "blob-ext";
>  		};
>  	};
> diff --git a/arch/arm/dts/imx8mn-beacon-kit-u-boot.dtsi b/arch/arm/dts/imx8mn-beacon-kit-u-boot.dtsi
> index eb1dd8debba..e1740fa31a6 100644
> --- a/arch/arm/dts/imx8mn-beacon-kit-u-boot.dtsi
> +++ b/arch/arm/dts/imx8mn-beacon-kit-u-boot.dtsi
> @@ -149,22 +149,22 @@
>  
>  		blob_1: blob-ext at 1 {
>  			filename = "lpddr4_pmu_train_1d_imem.bin";
> -			size = <0x8000>;
> +			align-end = <4>;
>  		};
>  
>  		blob_2: blob-ext at 2 {
>  			filename = "lpddr4_pmu_train_1d_dmem.bin";
> -			size = <0x4000>;
> +			align-end = <4>;
>  		};
>  
>  		blob_3: blob-ext at 3 {
>  			filename = "lpddr4_pmu_train_2d_imem.bin";
> -			size = <0x8000>;
> +			align-end = <4>;
>  		};
>  
>  		blob_4: blob-ext at 4 {
>  			filename = "lpddr4_pmu_train_2d_dmem.bin";
> -			size = <0x4000>;
> +			align-end = <4>;
>  		};
>  	};
>  
> diff --git a/arch/arm/dts/imx8mn-ddr4-evk-u-boot.dtsi b/arch/arm/dts/imx8mn-ddr4-evk-u-boot.dtsi
> index 4d0ecb07d4f..1fe2d0fd507 100644
> --- a/arch/arm/dts/imx8mn-ddr4-evk-u-boot.dtsi
> +++ b/arch/arm/dts/imx8mn-ddr4-evk-u-boot.dtsi
> @@ -157,22 +157,22 @@
>  
>  		blob_1: blob-ext at 1 {
>  			filename = "ddr4_imem_1d_201810.bin";
> -			size = <0x8000>;
> +			align-end = <4>;
>  		};
>  
>  		blob_2: blob-ext at 2 {
>  			filename = "ddr4_dmem_1d_201810.bin";
> -			size = <0x4000>;
> +			align-end = <4>;
>  		};
>  
>  		blob_3: blob-ext at 3 {
>  			filename = "ddr4_imem_2d_201810.bin";
> -			size = <0x8000>;
> +			align-end = <4>;
>  		};
>  
>  		blob_4: blob-ext at 4 {
>  			filename = "ddr4_dmem_2d_201810.bin";
> -			size = <0x4000>;
> +			align-end = <4>;
>  		};
>  	};
>  
> diff --git a/arch/arm/dts/imx8mn-evk-u-boot.dtsi b/arch/arm/dts/imx8mn-evk-u-boot.dtsi
> index 3db46d4cbcb..4f6dcf307b2 100644
> --- a/arch/arm/dts/imx8mn-evk-u-boot.dtsi
> +++ b/arch/arm/dts/imx8mn-evk-u-boot.dtsi
> @@ -38,22 +38,22 @@
>  
>  		blob_1: blob-ext at 1 {
>  			filename = "lpddr4_pmu_train_1d_imem.bin";
> -			size = <0x8000>;
> +			align-end = <4>;
>  		};
>  
>  		blob_2: blob-ext at 2 {
>  			filename = "lpddr4_pmu_train_1d_dmem.bin";
> -			size = <0x4000>;
> +			align-end = <4>;
>  		};
>  
>  		blob_3: blob-ext at 3 {
>  			filename = "lpddr4_pmu_train_2d_imem.bin";
> -			size = <0x8000>;
> +			align-end = <4>;
>  		};
>  
>  		blob_4: blob-ext at 4 {
>  			filename = "lpddr4_pmu_train_2d_dmem.bin";
> -			size = <0x4000>;
> +			align-end = <4>;
>  		};
>  	};
>  
> diff --git a/arch/arm/dts/imx8mn-var-som-symphony-u-boot.dtsi b/arch/arm/dts/imx8mn-var-som-symphony-u-boot.dtsi
> index 001e725f568..32ef7929288 100644
> --- a/arch/arm/dts/imx8mn-var-som-symphony-u-boot.dtsi
> +++ b/arch/arm/dts/imx8mn-var-som-symphony-u-boot.dtsi
> @@ -132,25 +132,25 @@
>  
>  		blob_1: blob-ext at 1 {
>  			filename = "ddr4_imem_1d.bin";
> -			size = <0x8000>;
> +			align-end = <4>;
>  			type = "blob-ext";
>  		};
>  
>  		blob_2: blob-ext at 2 {
>  			filename = "ddr4_dmem_1d.bin";
> -			size = <0x4000>;
> +			align-end = <4>;
>  			type = "blob-ext";
>  		};
>  
>  		blob_3: blob-ext at 3 {
>  			filename = "ddr4_imem_2d.bin";
> -			size = <0x8000>;
> +			align-end = <4>;
>  			type = "blob-ext";
>  		};
>  
>  		blob_4: blob-ext at 4 {
>  			filename = "ddr4_dmem_2d.bin";
> -			size = <0x4000>;
> +			align-end = <4>;
>  			type = "blob-ext";
>  		};
>  	};
> diff --git a/arch/arm/dts/imx8mn-venice-u-boot.dtsi b/arch/arm/dts/imx8mn-venice-u-boot.dtsi
> index 67922146963..e3a0b170347 100644
> --- a/arch/arm/dts/imx8mn-venice-u-boot.dtsi
> +++ b/arch/arm/dts/imx8mn-venice-u-boot.dtsi
> @@ -128,25 +128,25 @@
>  
>  		imem_1d: blob-ext at 1 {
>  			filename = "lpddr4_pmu_train_1d_imem.bin";
> -			size = <0x8000>;
> +			align-end = <4>;
>  			type = "blob-ext";
>  		};
>  
>  		dmem_1d: blob-ext at 2 {
>  			filename = "lpddr4_pmu_train_1d_dmem.bin";
> -			size = <0x4000>;
> +			align-end = <4>;
>  			type = "blob-ext";
>  		};
>  
>  		imem_2d: blob-ext at 3 {
>  			filename = "lpddr4_pmu_train_2d_imem.bin";
> -			size = <0x8000>;
> +			align-end = <4>;
>  			type = "blob-ext";
>  		};
>  
>  		dmem_2d: blob-ext at 4 {
>  			filename = "lpddr4_pmu_train_2d_dmem.bin";
> -			size = <0x4000>;
> +			align-end = <4>;
>  			type = "blob-ext";
>  		};
>  	};
> diff --git a/arch/arm/dts/imx8mp-u-boot.dtsi b/arch/arm/dts/imx8mp-u-boot.dtsi
> index 20edd90cfad..274515a010e 100644
> --- a/arch/arm/dts/imx8mp-u-boot.dtsi
> +++ b/arch/arm/dts/imx8mp-u-boot.dtsi
> @@ -63,22 +63,22 @@
>  
>  		blob_1: blob-ext at 1 {
>  			filename = "lpddr4_pmu_train_1d_imem_202006.bin";
> -			size = <0x8000>;
> +			align-end = <4>;
>  		};
>  
>  		blob_2: blob-ext at 2 {
>  			filename = "lpddr4_pmu_train_1d_dmem_202006.bin";
> -			size = <0x4000>;
> +			align-end = <4>;
>  		};
>  
>  		blob_3: blob-ext at 3 {
>  			filename = "lpddr4_pmu_train_2d_imem_202006.bin";
> -			size = <0x8000>;
> +			align-end = <4>;
>  		};
>  
>  		blob_4: blob-ext at 4 {
>  			filename = "lpddr4_pmu_train_2d_dmem_202006.bin";
> -			size = <0x4000>;
> +			align-end = <4>;
>  		};
>  	};
>  
> diff --git a/arch/arm/dts/imx8mq-cm-u-boot.dtsi b/arch/arm/dts/imx8mq-cm-u-boot.dtsi
> index e2f4b0e740d..9538a04ed97 100644
> --- a/arch/arm/dts/imx8mq-cm-u-boot.dtsi
> +++ b/arch/arm/dts/imx8mq-cm-u-boot.dtsi
> @@ -30,22 +30,22 @@
>  
>  		blob_1: blob-ext at 1 {
>  			filename = "lpddr4_pmu_train_1d_imem.bin";
> -			size = <0x8000>;
> +			align-end = <4>;
>  		};
>  
>  		blob_2: blob-ext at 2 {
>  			filename = "lpddr4_pmu_train_1d_dmem.bin";
> -			size = <0x4000>;
> +			align-end = <4>;
>  		};
>  
>  		blob_3: blob-ext at 3 {
>  			filename = "lpddr4_pmu_train_2d_imem.bin";
> -			size = <0x8000>;
> +			align-end = <4>;
>  		};
>  
>  		blob_4: blob-ext at 4 {
>  			filename = "lpddr4_pmu_train_2d_dmem.bin";
> -			size = <0x4000>;
> +			align-end = <4>;
>  		};
>  	};
>  
> diff --git a/arch/arm/dts/imx8mq-u-boot.dtsi b/arch/arm/dts/imx8mq-u-boot.dtsi
> index 389414ad26f..1495869fcd2 100644
> --- a/arch/arm/dts/imx8mq-u-boot.dtsi
> +++ b/arch/arm/dts/imx8mq-u-boot.dtsi
> @@ -48,25 +48,25 @@
>  
>  		imem_1d: blob-ext at 1 {
>  			filename = "lpddr4_pmu_train_1d_imem.bin";
> -			size = <0x8000>;
> +			align-end = <4>;
>  			type = "blob-ext";
>  		};
>  
>  		dmem_1d: blob-ext at 2 {
>  			filename = "lpddr4_pmu_train_1d_dmem.bin";
> -			size = <0x4000>;
> +			align-end = <4>;
>  			type = "blob-ext";
>  		};
>  
>  		imem_2d: blob-ext at 3 {
>  			filename = "lpddr4_pmu_train_2d_imem.bin";
> -			size = <0x8000>;
> +			align-end = <4>;
>  			type = "blob-ext";
>  		};
>  
>  		dmem_2d: blob-ext at 4 {
>  			filename = "lpddr4_pmu_train_2d_dmem.bin";
> -			size = <0x4000>;
> +			align-end = <4>;
>  			type = "blob-ext";
>  		};
>  	};


More information about the U-Boot mailing list