On 20/05/2022 17:10, Peng Fan (OSS) wrote:
> From: Peng Fan <peng....@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 <thar...@gateworks.com> #imx8m[m,n,p]-venice
> Signed-off-by: Peng Fan <peng....@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@1 {
> filename = "lpddr4_pmu_train_1d_imem.bin";
> - size = <0x8000>;
> + align-end = <4>;
> type = "blob-ext";
> };
>
> dmem_1d: blob-ext@2 {
> filename = "lpddr4_pmu_train_1d_dmem.bin";
> - size = <0x4000>;
> + align-end = <4>;
> type = "blob-ext";
> };
>
> imem_2d: blob-ext@3 {
> filename = "lpddr4_pmu_train_2d_imem.bin";
> - size = <0x8000>;
> + align-end = <4>;
> type = "blob-ext";
> };
>
> dmem_2d: blob-ext@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@1 {
> filename = "lpddr4_pmu_train_1d_imem.bin";
> - size = <0x8000>;
> + align-end = <4>;
> };
>
> blob_2: blob-ext@2 {
> filename = "lpddr4_pmu_train_1d_dmem.bin";
> - size = <0x4000>;
> + align-end = <4>;
> };
>
> blob_3: blob-ext@3 {
> filename = "lpddr4_pmu_train_2d_imem.bin";
> - size = <0x8000>;
> + align-end = <4>;
> };
>
> blob_4: blob-ext@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@1 {
> filename = "ddr4_imem_1d_201810.bin";
> - size = <0x8000>;
> + align-end = <4>;
> };
>
> blob_2: blob-ext@2 {
> filename = "ddr4_dmem_1d_201810.bin";
> - size = <0x4000>;
> + align-end = <4>;
> };
>
> blob_3: blob-ext@3 {
> filename = "ddr4_imem_2d_201810.bin";
> - size = <0x8000>;
> + align-end = <4>;
> };
>
> blob_4: blob-ext@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@1 {
> filename = "lpddr4_pmu_train_1d_imem.bin";
> - size = <0x8000>;
> + align-end = <4>;
> };
>
> blob_2: blob-ext@2 {
> filename = "lpddr4_pmu_train_1d_dmem.bin";
> - size = <0x4000>;
> + align-end = <4>;
> };
>
> blob_3: blob-ext@3 {
> filename = "lpddr4_pmu_train_2d_imem.bin";
> - size = <0x8000>;
> + align-end = <4>;
> };
>
> blob_4: blob-ext@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@1 {
> filename = "ddr4_imem_1d.bin";
> - size = <0x8000>;
> + align-end = <4>;
> type = "blob-ext";
> };
>
> blob_2: blob-ext@2 {
> filename = "ddr4_dmem_1d.bin";
> - size = <0x4000>;
> + align-end = <4>;
> type = "blob-ext";
> };
>
> blob_3: blob-ext@3 {
> filename = "ddr4_imem_2d.bin";
> - size = <0x8000>;
> + align-end = <4>;
> type = "blob-ext";
> };
>
> blob_4: blob-ext@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@1 {
> filename = "lpddr4_pmu_train_1d_imem.bin";
> - size = <0x8000>;
> + align-end = <4>;
> type = "blob-ext";
> };
>
> dmem_1d: blob-ext@2 {
> filename = "lpddr4_pmu_train_1d_dmem.bin";
> - size = <0x4000>;
> + align-end = <4>;
> type = "blob-ext";
> };
>
> imem_2d: blob-ext@3 {
> filename = "lpddr4_pmu_train_2d_imem.bin";
> - size = <0x8000>;
> + align-end = <4>;
> type = "blob-ext";
> };
>
> dmem_2d: blob-ext@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@1 {
> filename = "lpddr4_pmu_train_1d_imem_202006.bin";
> - size = <0x8000>;
> + align-end = <4>;
> };
>
> blob_2: blob-ext@2 {
> filename = "lpddr4_pmu_train_1d_dmem_202006.bin";
> - size = <0x4000>;
> + align-end = <4>;
> };
>
> blob_3: blob-ext@3 {
> filename = "lpddr4_pmu_train_2d_imem_202006.bin";
> - size = <0x8000>;
> + align-end = <4>;
> };
>
> blob_4: blob-ext@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@1 {
> filename = "lpddr4_pmu_train_1d_imem.bin";
> - size = <0x8000>;
> + align-end = <4>;
> };
>
> blob_2: blob-ext@2 {
> filename = "lpddr4_pmu_train_1d_dmem.bin";
> - size = <0x4000>;
> + align-end = <4>;
> };
>
> blob_3: blob-ext@3 {
> filename = "lpddr4_pmu_train_2d_imem.bin";
> - size = <0x8000>;
> + align-end = <4>;
> };
>
> blob_4: blob-ext@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@1 {
> filename = "lpddr4_pmu_train_1d_imem.bin";
> - size = <0x8000>;
> + align-end = <4>;
> type = "blob-ext";
> };
>
> dmem_1d: blob-ext@2 {
> filename = "lpddr4_pmu_train_1d_dmem.bin";
> - size = <0x4000>;
> + align-end = <4>;
> type = "blob-ext";
> };
>
> imem_2d: blob-ext@3 {
> filename = "lpddr4_pmu_train_2d_imem.bin";
> - size = <0x8000>;
> + align-end = <4>;
> type = "blob-ext";
> };
>
> dmem_2d: blob-ext@4 {
> filename = "lpddr4_pmu_train_2d_dmem.bin";
> - size = <0x4000>;
> + align-end = <4>;
> type = "blob-ext";
> };
> };
