[PATCH V4 7/8] arm: dts: imx8m: shrink ddr firmware size to actual file size
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
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
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