Since we've pushed a load of patches to move ARM Ltd platform support from EDK2 to OpenPlatformPkg, it's time for a cleanup of the configs we hold.
[PATCH 1/5] platforms.config: build from OpenPlatformPkg [PATCH 2/5] platforms.config: remove ARM_FOUNDATION_FVP [PATCH 3/5] platforms.config: add fvp_gicv2_legacy [PATCH 4/5] platforms.config: fvp_full: enable PL111 [PATCH 5/5] platforms.config: remove obsolete ARM Ltd configs
All the ARM Ltd platform support has moved to OpenPlatformPkg and has been removed from EDK2.
Signed-off-by: Ryan Harkin ryan.harkin@linaro.org --- platforms.config | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/platforms.config b/platforms.config index ca9dab2..6595772 100644 --- a/platforms.config +++ b/platforms.config @@ -64,7 +64,7 @@
[juno] LONGNAME=aarch64 Juno -DSC=ArmPlatformPkg/ArmJunoPkg/ArmJuno.dsc +DSC=OpenPlatformPkg/Platforms/ARM/Juno/ArmJuno.dsc BUILDFLAGS= ARCH=AARCH64 BUILD_ATF=yes @@ -75,7 +75,7 @@ SCP_BIN=ArmPlatformPkg/ArmJunoPkg/NonFree/bl30.bin # ARM FVP BASE AEMv8-A model [fvp_full] LONGNAME=aarch64 FVP RTSM with full perhiperhal set -DSC=ArmPlatformPkg/ArmVExpressPkg/ArmVExpress-FVP-AArch64.dsc +DSC=OpenPlatformPkg/Platforms/ARM/VExpress/ArmVExpress-FVP-AArch64.dsc BUILDFLAGS=-D EDK2_OUT_DIR=Build/ArmVExpress-FVP-AArch64-Full -D EDK2_ENABLE_SMSC_91X=1 ARCH=AARCH64 UEFI_BIN=FVP_AARCH64_EFI.fd @@ -83,7 +83,7 @@ UEFI_IMAGE_DIR=ArmVExpress-FVP-AArch64-Full
[fvp] LONGNAME=aarch64 FVP RTSM -DSC=ArmPlatformPkg/ArmVExpressPkg/ArmVExpress-FVP-AArch64.dsc +DSC=OpenPlatformPkg/Platforms/ARM/VExpress/ArmVExpress-FVP-AArch64.dsc BUILDFLAGS=-D ARM_FOUNDATION_FVP=1 -D EDK2_ENABLE_SMSC_91X=1 ARCH=AARCH64 BUILD_ATF=yes @@ -92,7 +92,7 @@ UEFI_IMAGE_DIR=ArmVExpress-FVP-AArch64
[fvp_minimal] LONGNAME=aarch64 FVP RTSM/Foundation with Ethernet -DSC=ArmPlatformPkg/ArmVExpressPkg/ArmVExpress-FVP-AArch64.dsc +DSC=OpenPlatformPkg/Platforms/ARM/VExpress/ArmVExpress-FVP-AArch64.dsc BUILDFLAGS=-D ARM_FOUNDATION_FVP=1 -D EDK2_ENABLE_SMSC_91X=1 -D EDK2_OUT_DIR=Build/ArmVExpress-FVP-AArch64-Minimal ARCH=AARCH64 BUILD_ATF=yes @@ -102,7 +102,7 @@ UEFI_IMAGE_DIR=ArmVExpress-FVP-AArch64-Minimal
[fvp_no_eth] LONGNAME=aarch64 FVP RTSM without Ethernet -DSC=ArmPlatformPkg/ArmVExpressPkg/ArmVExpress-FVP-AArch64.dsc +DSC=OpenPlatformPkg/Platforms/ARM/VExpress/ArmVExpress-FVP-AArch64.dsc BUILDFLAGS=-D ARM_FOUNDATION_FVP=1 -D EDK2_OUT_DIR=Build/ArmVExpress-FVP-AArch64-No-Eth ARCH=AARCH64 UEFI_BIN=FVP_AARCH64_EFI.fd @@ -157,7 +157,7 @@ UEFI_IMAGE_DIR=ArmVExpress-RTSM-A15_MPCore [tc2] LONGNAME=Versatile Express TC2 BUILDFLAGS=-D ARM_BIGLITTLE_TC2=1 -DSC=ArmPlatformPkg/ArmVExpressPkg/ArmVExpress-CTA15-A7.dsc +DSC=OpenPlatformPkg/Platforms/ARM/VExpress/ArmVExpress-CTA15-A7.dsc ARCH=ARM UEFI_BIN=ARM_VEXPRESS_CTA15A7_EFI.fd UEFI_IMAGE_DIR=ArmVExpress-CTA15-A7
The ARM_FOUNDATION_FVP is obsolete since Ard fixed the drivers so they fail gracefully if the device isn't present.
Signed-off-by: Ryan Harkin ryan.harkin@linaro.org --- platforms.config | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/platforms.config b/platforms.config index 6595772..48ccc05 100644 --- a/platforms.config +++ b/platforms.config @@ -84,7 +84,7 @@ UEFI_IMAGE_DIR=ArmVExpress-FVP-AArch64-Full [fvp] LONGNAME=aarch64 FVP RTSM DSC=OpenPlatformPkg/Platforms/ARM/VExpress/ArmVExpress-FVP-AArch64.dsc -BUILDFLAGS=-D ARM_FOUNDATION_FVP=1 -D EDK2_ENABLE_SMSC_91X=1 +BUILDFLAGS=-D EDK2_ENABLE_SMSC_91X=1 ARCH=AARCH64 BUILD_ATF=yes UEFI_BIN=FVP_AARCH64_EFI.fd @@ -93,7 +93,7 @@ UEFI_IMAGE_DIR=ArmVExpress-FVP-AArch64 [fvp_minimal] LONGNAME=aarch64 FVP RTSM/Foundation with Ethernet DSC=OpenPlatformPkg/Platforms/ARM/VExpress/ArmVExpress-FVP-AArch64.dsc -BUILDFLAGS=-D ARM_FOUNDATION_FVP=1 -D EDK2_ENABLE_SMSC_91X=1 -D EDK2_OUT_DIR=Build/ArmVExpress-FVP-AArch64-Minimal +BUILDFLAGS=-D EDK2_ENABLE_SMSC_91X=1 -D EDK2_OUT_DIR=Build/ArmVExpress-FVP-AArch64-Minimal ARCH=AARCH64 BUILD_ATF=yes ATF_PLATFORM=fvp @@ -103,7 +103,7 @@ UEFI_IMAGE_DIR=ArmVExpress-FVP-AArch64-Minimal [fvp_no_eth] LONGNAME=aarch64 FVP RTSM without Ethernet DSC=OpenPlatformPkg/Platforms/ARM/VExpress/ArmVExpress-FVP-AArch64.dsc -BUILDFLAGS=-D ARM_FOUNDATION_FVP=1 -D EDK2_OUT_DIR=Build/ArmVExpress-FVP-AArch64-No-Eth +BUILDFLAGS=-D EDK2_OUT_DIR=Build/ArmVExpress-FVP-AArch64-No-Eth ARCH=AARCH64 UEFI_BIN=FVP_AARCH64_EFI.fd UEFI_IMAGE_DIR=ArmVExpress-FVP-AArch64-No-Eth
Add a new config to build with the ARM_FVP_GICV3_WITH_V2_LEGACY flag.
Signed-off-by: Ryan Harkin ryan.harkin@linaro.org --- platforms.config | 10 ++++++++++ 1 file changed, 10 insertions(+)
diff --git a/platforms.config b/platforms.config index 48ccc05..93368f1 100644 --- a/platforms.config +++ b/platforms.config @@ -100,6 +100,16 @@ ATF_PLATFORM=fvp UEFI_BIN=FVP_AARCH64_EFI.fd UEFI_IMAGE_DIR=ArmVExpress-FVP-AArch64-Minimal
+[fvp_gicv2_legacy] +LONGNAME=aarch64 FVP RTSM/Foundation with Ethernet and GICv3 in GICv2 legacy mode +DSC=OpenPlatformPkg/Platforms/ARM/VExpress/ArmVExpress-FVP-AArch64.dsc +BUILDFLAGS=-D EDK2_ENABLE_SMSC_91X=1 -D EDK2_OUT_DIR=Build/ArmVExpress-FVP-AArch64-gicv2legacy -D ARM_FVP_GICV3_WITH_V2_LEGACY=1 +ARCH=AARCH64 +BUILD_ATF=yes +ATF_PLATFORM=fvp +UEFI_BIN=FVP_AARCH64_EFI.fd +UEFI_IMAGE_DIR=ArmVExpress-FVP-AArch64-Minimal + [fvp_no_eth] LONGNAME=aarch64 FVP RTSM without Ethernet DSC=OpenPlatformPkg/Platforms/ARM/VExpress/ArmVExpress-FVP-AArch64.dsc
I've realised the error of my ways and decided to drop this patch.
On 11 February 2016 at 17:57, Ryan Harkin ryan.harkin@linaro.org wrote:
Add a new config to build with the ARM_FVP_GICV3_WITH_V2_LEGACY flag.
Signed-off-by: Ryan Harkin ryan.harkin@linaro.org
platforms.config | 10 ++++++++++ 1 file changed, 10 insertions(+)
diff --git a/platforms.config b/platforms.config index 48ccc05..93368f1 100644 --- a/platforms.config +++ b/platforms.config @@ -100,6 +100,16 @@ ATF_PLATFORM=fvp UEFI_BIN=FVP_AARCH64_EFI.fd UEFI_IMAGE_DIR=ArmVExpress-FVP-AArch64-Minimal
+[fvp_gicv2_legacy] +LONGNAME=aarch64 FVP RTSM/Foundation with Ethernet and GICv3 in GICv2 legacy mode +DSC=OpenPlatformPkg/Platforms/ARM/VExpress/ArmVExpress-FVP-AArch64.dsc +BUILDFLAGS=-D EDK2_ENABLE_SMSC_91X=1 -D EDK2_OUT_DIR=Build/ArmVExpress-FVP-AArch64-gicv2legacy -D ARM_FVP_GICV3_WITH_V2_LEGACY=1 +ARCH=AARCH64 +BUILD_ATF=yes +ATF_PLATFORM=fvp +UEFI_BIN=FVP_AARCH64_EFI.fd +UEFI_IMAGE_DIR=ArmVExpress-FVP-AArch64-Minimal
[fvp_no_eth] LONGNAME=aarch64 FVP RTSM without Ethernet DSC=OpenPlatformPkg/Platforms/ARM/VExpress/ArmVExpress-FVP-AArch64.dsc -- 2.1.4
On 12 February 2016 at 11:59, Ryan Harkin ryan.harkin@linaro.org wrote:
I've realised the error of my ways
Does that include your position in the DT discussion we had yesterday? :-)
and decided to drop this patch.
But seriously, care to explain what made you change your mind?
On 11 February 2016 at 17:57, Ryan Harkin ryan.harkin@linaro.org wrote:
Add a new config to build with the ARM_FVP_GICV3_WITH_V2_LEGACY flag.
Signed-off-by: Ryan Harkin ryan.harkin@linaro.org
platforms.config | 10 ++++++++++ 1 file changed, 10 insertions(+)
diff --git a/platforms.config b/platforms.config index 48ccc05..93368f1 100644 --- a/platforms.config +++ b/platforms.config @@ -100,6 +100,16 @@ ATF_PLATFORM=fvp UEFI_BIN=FVP_AARCH64_EFI.fd UEFI_IMAGE_DIR=ArmVExpress-FVP-AArch64-Minimal
+[fvp_gicv2_legacy] +LONGNAME=aarch64 FVP RTSM/Foundation with Ethernet and GICv3 in GICv2 legacy mode +DSC=OpenPlatformPkg/Platforms/ARM/VExpress/ArmVExpress-FVP-AArch64.dsc +BUILDFLAGS=-D EDK2_ENABLE_SMSC_91X=1 -D EDK2_OUT_DIR=Build/ArmVExpress-FVP-AArch64-gicv2legacy -D ARM_FVP_GICV3_WITH_V2_LEGACY=1 +ARCH=AARCH64 +BUILD_ATF=yes +ATF_PLATFORM=fvp +UEFI_BIN=FVP_AARCH64_EFI.fd +UEFI_IMAGE_DIR=ArmVExpress-FVP-AArch64-Minimal
[fvp_no_eth] LONGNAME=aarch64 FVP RTSM without Ethernet DSC=OpenPlatformPkg/Platforms/ARM/VExpress/ArmVExpress-FVP-AArch64.dsc -- 2.1.4
On 12 February 2016 at 11:01, Ard Biesheuvel ard.biesheuvel@linaro.org wrote:
On 12 February 2016 at 11:59, Ryan Harkin ryan.harkin@linaro.org wrote:
I've realised the error of my ways
Does that include your position in the DT discussion we had yesterday? :-)
Oh, I'm still right about that :-)
and decided to drop this patch.
But seriously, care to explain what made you change your mind?
Two reasons really: I can pass the flag in to uefi-build.sh using the -D param, so I don't think we need the clutter on the config file.
I think having a "full" and "normal" config is fine, because most people probably don't want the performance hit of the PL111 CLCD output.
Another reason is that I'm not using EDK2 to support my kernel releases any more, so it doesn't matter that we're still clinging on to a gicv2 DTB over there.
On 11 February 2016 at 17:57, Ryan Harkin ryan.harkin@linaro.org wrote:
Add a new config to build with the ARM_FVP_GICV3_WITH_V2_LEGACY flag.
Signed-off-by: Ryan Harkin ryan.harkin@linaro.org
platforms.config | 10 ++++++++++ 1 file changed, 10 insertions(+)
diff --git a/platforms.config b/platforms.config index 48ccc05..93368f1 100644 --- a/platforms.config +++ b/platforms.config @@ -100,6 +100,16 @@ ATF_PLATFORM=fvp UEFI_BIN=FVP_AARCH64_EFI.fd UEFI_IMAGE_DIR=ArmVExpress-FVP-AArch64-Minimal
+[fvp_gicv2_legacy] +LONGNAME=aarch64 FVP RTSM/Foundation with Ethernet and GICv3 in GICv2 legacy mode +DSC=OpenPlatformPkg/Platforms/ARM/VExpress/ArmVExpress-FVP-AArch64.dsc +BUILDFLAGS=-D EDK2_ENABLE_SMSC_91X=1 -D EDK2_OUT_DIR=Build/ArmVExpress-FVP-AArch64-gicv2legacy -D ARM_FVP_GICV3_WITH_V2_LEGACY=1 +ARCH=AARCH64 +BUILD_ATF=yes +ATF_PLATFORM=fvp +UEFI_BIN=FVP_AARCH64_EFI.fd +UEFI_IMAGE_DIR=ArmVExpress-FVP-AArch64-Minimal
[fvp_no_eth] LONGNAME=aarch64 FVP RTSM without Ethernet DSC=OpenPlatformPkg/Platforms/ARM/VExpress/ArmVExpress-FVP-AArch64.dsc -- 2.1.4
PL111 is optional for FVP, so enable it on the "full" build.
Signed-off-by: Ryan Harkin ryan.harkin@linaro.org --- platforms.config | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/platforms.config b/platforms.config index 93368f1..37e20a9 100644 --- a/platforms.config +++ b/platforms.config @@ -76,7 +76,7 @@ SCP_BIN=ArmPlatformPkg/ArmJunoPkg/NonFree/bl30.bin [fvp_full] LONGNAME=aarch64 FVP RTSM with full perhiperhal set DSC=OpenPlatformPkg/Platforms/ARM/VExpress/ArmVExpress-FVP-AArch64.dsc -BUILDFLAGS=-D EDK2_OUT_DIR=Build/ArmVExpress-FVP-AArch64-Full -D EDK2_ENABLE_SMSC_91X=1 +BUILDFLAGS=-D EDK2_OUT_DIR=Build/ArmVExpress-FVP-AArch64-Full -D EDK2_ENABLE_SMSC_91X=1 -D EDK2_ENABLE_PL111=1 ARCH=AARCH64 UEFI_BIN=FVP_AARCH64_EFI.fd UEFI_IMAGE_DIR=ArmVExpress-FVP-AArch64-Full
Various ARM Ltd platforms have been removed from the code base, so remove their configs.
Also, the LAN91x drivers went upstream a long time ago, so there's no real reason to build without them any more.
The fvp_minimal and fvp configs ended up being the same, so we may as well remove the minimal config.
Signed-off-by: Ryan Harkin ryan.harkin@linaro.org --- platforms.config | 64 -------------------------------------------------------- 1 file changed, 64 deletions(-)
diff --git a/platforms.config b/platforms.config index 37e20a9..a10fbb2 100644 --- a/platforms.config +++ b/platforms.config @@ -90,16 +90,6 @@ BUILD_ATF=yes UEFI_BIN=FVP_AARCH64_EFI.fd UEFI_IMAGE_DIR=ArmVExpress-FVP-AArch64
-[fvp_minimal] -LONGNAME=aarch64 FVP RTSM/Foundation with Ethernet -DSC=OpenPlatformPkg/Platforms/ARM/VExpress/ArmVExpress-FVP-AArch64.dsc -BUILDFLAGS=-D EDK2_ENABLE_SMSC_91X=1 -D EDK2_OUT_DIR=Build/ArmVExpress-FVP-AArch64-Minimal -ARCH=AARCH64 -BUILD_ATF=yes -ATF_PLATFORM=fvp -UEFI_BIN=FVP_AARCH64_EFI.fd -UEFI_IMAGE_DIR=ArmVExpress-FVP-AArch64-Minimal - [fvp_gicv2_legacy] LONGNAME=aarch64 FVP RTSM/Foundation with Ethernet and GICv3 in GICv2 legacy mode DSC=OpenPlatformPkg/Platforms/ARM/VExpress/ArmVExpress-FVP-AArch64.dsc @@ -110,60 +100,6 @@ ATF_PLATFORM=fvp UEFI_BIN=FVP_AARCH64_EFI.fd UEFI_IMAGE_DIR=ArmVExpress-FVP-AArch64-Minimal
-[fvp_no_eth] -LONGNAME=aarch64 FVP RTSM without Ethernet -DSC=OpenPlatformPkg/Platforms/ARM/VExpress/ArmVExpress-FVP-AArch64.dsc -BUILDFLAGS=-D EDK2_OUT_DIR=Build/ArmVExpress-FVP-AArch64-No-Eth -ARCH=AARCH64 -UEFI_BIN=FVP_AARCH64_EFI.fd -UEFI_IMAGE_DIR=ArmVExpress-FVP-AArch64-No-Eth - -[rtsm_aarch64] -LONGNAME=aarch64 RTSM -DSC=ArmPlatformPkg/ArmVExpressPkg/ArmVExpress-RTSM-AEMv8Ax4.dsc -BUILDFLAGS= -ARCH=AARCH64 -UEFI_BIN=RTSM_VE_AEMV8_EFI.fd -UEFI_IMAGE_DIR=ArmVExpress-RTSM-AEMv8Ax4 - -[foundation_legacy] -LONGNAME=Legacy Foundation Model -DSC=ArmPlatformPkg/ArmVExpressPkg/ArmVExpress-RTSM-AEMv8Ax4-foundation.dsc -BUILDFLAGS=-D EDK2_ENABLE_SMSC_91X=1 -ARCH=AARCH64 -UEFI_BIN=RTSM_VE_FOUNDATIONV8_EFI.fd -UEFI_IMAGE_DIR=ArmVExpress-RTSM-AEMv8Ax4-foundation - -[a9] -LONGNAME=Versatile Express A9 -BUILDFLAGS=-D EDK2_ARMVE_STANDALONE=1 -D EDK2_ARMVE_SINGLE_BINARY=1 -DSC=ArmPlatformPkg/ArmVExpressPkg/ArmVExpress-CTA9x4.dsc -ARCH=ARM -UEFI_BIN=ARMVEXPRESS_EFI.fd -UEFI_IMAGE_DIR=ArmVExpress-CTA9x4 - -[rtsm_a9x4] -LONGNAME=Versatile Express RTSM A9x4 -BUILDFLAGS=-D EDK2_ARMVE_STANDALONE=1 -DSC=ArmPlatformPkg/ArmVExpressPkg/ArmVExpress-RTSM-A9x4.dsc -ARCH=ARM -UEFI_BIN=RTSM_VE_CORTEX-A9_EFI.fd -UEFI_IMAGE_DIR=ArmVExpress-RTSM-A9x4 - -[rtsm_a15x1] -LONGNAME=Versatile Express RTSM A15 single core -BUILDFLAGS=-D EDK2_ARMVE_STANDALONE=1 -DSC=ArmPlatformPkg/ArmVExpressPkg/ArmVExpress-RTSM-A15.dsc -ARCH=ARM - -[rtsm_a15mpcore] -LONGNAME=Versatile Express RTSM A15 MPCore -BUILDFLAGS=-D EDK2_ARMVE_STANDALONE=1 -DSC=ArmPlatformPkg/ArmVExpressPkg/ArmVExpress-RTSM-A15_MPCore.dsc -ARCH=ARM -UEFI_BIN=RTSM_VE_CORTEX-A15_MPCORE_EFI.fd -UEFI_IMAGE_DIR=ArmVExpress-RTSM-A15_MPCore - [tc2] LONGNAME=Versatile Express TC2 BUILDFLAGS=-D ARM_BIGLITTLE_TC2=1
I keep forgetting to include the link to my tree in these messages:
https://git.linaro.org/landing-teams/working/arm/uefi-tools.git/shortlog/ref...
It's all in the "master" and "16.02" branches and the "armlt-20160211" tag. There's nothing else in there at the moment.
On 11 February 2016 at 17:57, Ryan Harkin ryan.harkin@linaro.org wrote:
Since we've pushed a load of patches to move ARM Ltd platform support from EDK2 to OpenPlatformPkg, it's time for a cleanup of the configs we hold.
[PATCH 1/5] platforms.config: build from OpenPlatformPkg [PATCH 2/5] platforms.config: remove ARM_FOUNDATION_FVP [PATCH 3/5] platforms.config: add fvp_gicv2_legacy [PATCH 4/5] platforms.config: fvp_full: enable PL111 [PATCH 5/5] platforms.config: remove obsolete ARM Ltd configs
On 11 February 2016 at 18:57, Ryan Harkin ryan.harkin@linaro.org wrote:
Since we've pushed a load of patches to move ARM Ltd platform support from EDK2 to OpenPlatformPkg, it's time for a cleanup of the configs we hold.
[PATCH 1/5] platforms.config: build from OpenPlatformPkg [PATCH 2/5] platforms.config: remove ARM_FOUNDATION_FVP [PATCH 3/5] platforms.config: add fvp_gicv2_legacy [PATCH 4/5] platforms.config: fvp_full: enable PL111 [PATCH 5/5] platforms.config: remove obsolete ARM Ltd configs
All looks fine to me
Acked-by: Ard Biesheuvel ard.biesheuvel@linaro.org
On 12 February 2016 at 08:55, Ard Biesheuvel ard.biesheuvel@linaro.org wrote:
On 11 February 2016 at 18:57, Ryan Harkin ryan.harkin@linaro.org wrote:
Since we've pushed a load of patches to move ARM Ltd platform support from EDK2 to OpenPlatformPkg, it's time for a cleanup of the configs we hold.
[PATCH 1/5] platforms.config: build from OpenPlatformPkg [PATCH 2/5] platforms.config: remove ARM_FOUNDATION_FVP [PATCH 3/5] platforms.config: add fvp_gicv2_legacy [PATCH 4/5] platforms.config: fvp_full: enable PL111 [PATCH 5/5] platforms.config: remove obsolete ARM Ltd configs
All looks fine to me
Acked-by: Ard Biesheuvel ard.biesheuvel@linaro.org
Thanks Ard.
As it happens, I decided to drop PATCH 3/5 to add the legacy config. It's clutter that we don't need. If I want to build with the gicv2 legacy mode, I can pass a -D param to the uefi-build.sh script.
I pushed the updated series to my tree here:
https://git.linaro.org/landing-teams/working/arm/uefi-tools.git/shortlog/ref...
On Fri, Feb 12, 2016 at 11:02:08AM +0000, Ryan Harkin wrote:
On 12 February 2016 at 08:55, Ard Biesheuvel ard.biesheuvel@linaro.org wrote:
On 11 February 2016 at 18:57, Ryan Harkin ryan.harkin@linaro.org wrote:
Since we've pushed a load of patches to move ARM Ltd platform support from EDK2 to OpenPlatformPkg, it's time for a cleanup of the configs we hold.
[PATCH 1/5] platforms.config: build from OpenPlatformPkg [PATCH 2/5] platforms.config: remove ARM_FOUNDATION_FVP [PATCH 3/5] platforms.config: add fvp_gicv2_legacy [PATCH 4/5] platforms.config: fvp_full: enable PL111 [PATCH 5/5] platforms.config: remove obsolete ARM Ltd configs
All looks fine to me
Acked-by: Ard Biesheuvel ard.biesheuvel@linaro.org
Thanks Ard.
As it happens, I decided to drop PATCH 3/5 to add the legacy config. It's clutter that we don't need. If I want to build with the gicv2 legacy mode, I can pass a -D param to the uefi-build.sh script.
I pushed the updated series to my tree here:
https://git.linaro.org/landing-teams/working/arm/uefi-tools.git/shortlog/ref...
For the (shortened) series: Acked-by: Leif Lindholm leif.lindholm@linaro.org
Feel free to push it yourself :)
/ Leif
On 12 February 2016 at 15:04, Leif Lindholm leif.lindholm@linaro.org wrote:
On Fri, Feb 12, 2016 at 11:02:08AM +0000, Ryan Harkin wrote:
On 12 February 2016 at 08:55, Ard Biesheuvel ard.biesheuvel@linaro.org wrote:
On 11 February 2016 at 18:57, Ryan Harkin ryan.harkin@linaro.org wrote:
Since we've pushed a load of patches to move ARM Ltd platform support from EDK2 to OpenPlatformPkg, it's time for a cleanup of the configs we hold.
[PATCH 1/5] platforms.config: build from OpenPlatformPkg [PATCH 2/5] platforms.config: remove ARM_FOUNDATION_FVP [PATCH 3/5] platforms.config: add fvp_gicv2_legacy [PATCH 4/5] platforms.config: fvp_full: enable PL111 [PATCH 5/5] platforms.config: remove obsolete ARM Ltd configs
All looks fine to me
Acked-by: Ard Biesheuvel ard.biesheuvel@linaro.org
Thanks Ard.
As it happens, I decided to drop PATCH 3/5 to add the legacy config. It's clutter that we don't need. If I want to build with the gicv2 legacy mode, I can pass a -D param to the uefi-build.sh script.
I pushed the updated series to my tree here:
https://git.linaro.org/landing-teams/working/arm/uefi-tools.git/shortlog/ref...
For the (shortened) series: Acked-by: Leif Lindholm leif.lindholm@linaro.org
Feel free to push it yourself :)
Thanks, will do :)
/ Leif
-
Ard Biesheuvel -
Leif Lindholm -
Ryan Harkin