On Sat, May 7, 2022 at 1:39 AM Peng Fan (OSS) <[email protected]> wrote: > > From: Peng Fan <[email protected]> > > binman symbol is a good feature, but only used on X86 for now. This patchset > is to use it for i.MX8M platform. > > The current imx8m ddr phy firmware consumes lots of space, because we pad > them to the largest 32KB and 16KB for IMEM and DMEM. > > With this patchset we use binman symbols to get firmware location and size, > we could save near 36KB with i.MX8MP-EVK. > > Please help check and test > > Note: with this patchset, nxp downstream imx-mkimage will not work. > Later patches will follow up to address. > > Peng Fan (7): > spl: guard u_boot_any with X86 > arm: dts: imx8m: update binman ddr firmware node name > armv8: u-boot-spl.lds: mark __image_copy_start as symbol > binman_sym: update symbol alignment to 8 bytes > tools: binman: section: replace @ with - > ddr: imx8m: helper: load ddr firmware according to binman symbols > arm: dts: imx8m: shrink ddr firmware size to actual file size > > arch/arm/cpu/armv8/u-boot-spl.lds | 2 +- > arch/arm/dts/imx8mm-u-boot.dtsi | 16 +++---- > arch/arm/dts/imx8mn-beacon-kit-u-boot.dtsi | 8 ++-- > .../dts/imx8mn-bsh-smm-s2-u-boot-common.dtsi | 4 +- > arch/arm/dts/imx8mn-ddr4-evk-u-boot.dtsi | 8 ++-- > arch/arm/dts/imx8mn-evk-u-boot.dtsi | 8 ++-- > .../dts/imx8mn-var-som-symphony-u-boot.dtsi | 8 ++-- > arch/arm/dts/imx8mn-venice-u-boot.dtsi | 16 +++---- > arch/arm/dts/imx8mp-u-boot.dtsi | 8 ++-- > arch/arm/dts/imx8mq-cm-u-boot.dtsi | 8 ++-- > arch/arm/dts/imx8mq-u-boot.dtsi | 16 +++---- > common/spl/spl.c | 8 +++- > drivers/ddr/imx/imx8m/helper.c | 45 ++++++++++++++++--- > include/binman_sym.h | 6 +-- > tools/binman/etype/section.py | 2 +- > 15 files changed, 101 insertions(+), 62 deletions(-) >
Peng, Thank you for doing this, its something I've wanted to get in for a long time now. I can confirm this saves 39K in the IMX8M SPL. Tested-by: Tim Harvey <[email protected]> #imx8mm-venice #imx8mn-venice #imx8mp-venice Best Regards, Tim
