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

Hi

Il lun 23 mag 2022, 23:13 Alper Nebi Yasak alpernebiyasak@gmail.com ha scritto:

On 20/05/2022 17:10, Peng Fan (OSS) wrote:

...

From: Peng Fan peng.fan@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@gateworks.com #imx8m[m,n,p]-venice Signed-off-by: Peng Fan peng.fan@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.

Let us test.

I did not find time

Michael

...

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";
          };
  };