[Buildroot] [PATCH 1/4] firmware-imx: add support for iMX8M firmware

Thomas Petazzoni thomas.petazzoni at bootlin.com
Thu Apr 12 20:18:11 UTC 2018


Hello,

On Thu, 12 Apr 2018 14:35:22 +0200, Erik Larsson wrote:

> +ifeq ($(BR2_PACKAGE_FREESCALE_IMX_PLATFORM_IMX8M),y)
> +FIRMWARE_IMX_INSTALL_IMAGES = YES
> +define FIRMWARE_IMX_PREPARE_LPDDR4_FW
> +	(cd $(@D)/firmware/ddr/synopsys/ && \
> +		$(TARGET_OBJCOPY) -I binary -O binary --pad-to 0x8000 --gap-fill=0x0 \
> +			lpddr4_pmu_train_$(1)_imem.bin \
> +			lpddr4_pmu_train_$(1)_imem_pad.bin && \
> +		$(TARGET_OBJCOPY) -I binary -O binary --pad-to 0x4000 --gap-fill=0x0 \
> +			lpddr4_pmu_train_$(1)_dmem.bin \
> +			lpddr4_pmu_train_$(1)_dmem_pad.bin && \
> +		cat lpddr4_pmu_train_$(1)_imem_pad.bin lpddr4_pmu_train_$(1)_dmem_pad.bin > \
> +			lpddr4_pmu_train_$(1)_fw.bin \
> +	)

It was still a bit annoying that we had to have all those commands
inside a single command, so I've instead introduced a
FIRMWARE_IMX_DDRFW_DIR variable, set to $(@D)/firmware/ddr/synopsys/
and used that everywhere. To me it makes the whole thing more readable.

Applied to master with this change.

Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com


More information about the buildroot mailing list