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

2022-05-20 Thread Peng Fan (OSS) 
From: Peng Fan 

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  #imx8m[m,n,p]-venice
Signed-off-by: Peng Fan 
---
 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 
 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 {

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

2022-05-23 Thread Alper Nebi Yasak 
On 20/05/2022 17:10, Peng Fan (OSS) wrote:
> From: Peng Fan 
> 
> 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  #imx8m[m,n,p]-venice
> Signed-off-by: Peng Fan 
> ---
>  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:

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

2022-05-23 Thread Michael Nazzareno Trimarchi 
Hi

Il lun 23 mag 2022, 23:13 Alper Nebi Yasak  ha
scritto:

> On 20/05/2022 17:10, Peng Fan (OSS) wrote:
> > From: Peng Fan 
> >
> > 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  #imx8m[m,n,p]-venice
> > Signed-off-by: Peng Fan 
> > ---
> >  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