[Buildroot] [PATCH v3 1/4] package/freescale-imx: Move DDR FW choice down to firmware-imx package
Thomas Petazzoni
thomas.petazzoni at bootlin.com
Sat Jul 11 20:48:33 UTC 2020
Hello,
On Fri, 10 Jul 2020 14:00:42 +0200
Stephane Viau <stephane.viau at oss.nxp.com> wrote:
> The DDR FW along with all other FW code that need to be used in
> bootloader or installed on target are related to the firmware-imx package.
>
> This patch does this job as well as fixing the conjugation of NEED*s* in
> the symbol name. Also take advantage of this patch to make the DDR FW
> dependant on BR2_PACKAGE_FIRMWARE_IMX.
>
> Note that BR2_PACKAGE_FIRMWARE_IMX_xxx is the preferred naming
> convention for the upcoming couple of patches.
>
> Suggested-by: Thomas Petazzoni <thomas.petazzoni at bootlin.com>
> Signed-off-by: Stephane Viau <stephane.viau at oss.nxp.com>
I have applied, with some changes. See below.
> choice
> bool "DDR training binaries"
> - default BR2_PACKAGE_FIRMWARE_DDRFW_LPDDR4
> + default BR2_PACKAGE_FIRMWARE_IMX_LPDDR4
> help
> Choose the DDR training binaries to be used depending on the
> kind of memory that is available on the target board (DDR4,
> LPDDR4, etc...).
>
> -config BR2_PACKAGE_FIRMWARE_DDRFW_LPDDR4
> +config BR2_PACKAGE_FIRMWARE_IMX_LPDDR4
When renaming options, we need to keep the old option name in
Config.in.legacy so that it doesn't get silently dropped when users
update to a newer Buildroot version.
> bool "lpddr4"
> help
> Use LPDDR4 binaries (i.e.: lpddr4_pmu_train_*.bin)
>
> -config BR2_PACKAGE_FIRMWARE_DDRFW_DDR4
> +config BR2_PACKAGE_FIRMWARE_IMX_DDR4
Same here.
I fixed that when applying.
Thanks!
Thomas
--
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
More information about the buildroot
mailing list