On 4 February 2016 at 11:40, Ryan Harkin ryan.harkin@linaro.org wrote:
On 3 February 2016 at 17:42, Leif Lindholm leif.lindholm@linaro.org wrote:
Hi Ryan,
On Wed, Feb 03, 2016 at 05:09:28PM +0000, Ryan Harkin wrote:
This is an update to my original series titled:
[PATCH 0/2] ArmPlatformPkg: Remove FVP and JunoARM Ltd Platform support is migrating to use OpenPlatformPkg [1].
Currently, TC2, Juno and FVP exist both in EDK2's ArmPlatformPkg and in OpenPlatformPkg. And they are starting to diverge, with OpenPlatformPkg being the most up-to-date with current developments. To prevent this divergence, remove the .dsc and .fdf files from ArmPlatformPkg and leave OpenPlatformPkg as the master.
32-bit models are no longer tested or supported, so this series also removes the .dsc/fdf files for them.
And the Sec library has no clients either, so that can also be removed as it hasn't been tested for a long time.
We can't remove ArmJuno.dec yet because ACPI still uses it to set the include path to ArmPlatform.h.
[PATCH 1/6] ArmPlatformPkg: remove ArmVExpress-CTA15-A7.dsc/fdf [PATCH 2/6] ArmPlatformPkg: remove ArmVExpress-FVP-AArch64.dsc/fdf [PATCH 3/6] ArmPlatformPkg: remove ArmJuno.dsc/fdf [PATCH 4/6] ArmPlatformPkg: remove ArmVExpress-RTSM-A15_MPCore.dsc/fdf [PATCH 5/6] ArmPlatformPkg: remove unused Sec library [PATCH 6/6] Revert "ArmPlatformPkg: Create an ARM Platform DSC / FDF
I am entirely happy with 1-5. I know Ard objected to (what is now) 2. This is something to be discussed.
Ard's last comment on the idea of removing FVP was:
"I am happy to switch to OpenPlatformPkg for building those once it is synced up with EDK2"
... which is now the case.
6/6 ... I think we can leave ArmPlatformLinBull in for now. It is meant to be a template for use by new platform ports. Ard: what's your take on dropping the generic .dsc files?
I discussed this with Leif on IRC. We'll drop PATCH 6/6 and I'll submit a different patch for review to remove these files:
ArmPlatformPkg/ArmPlatformPkg-2ndstage.dsc ArmPlatformPkg/ArmPlatformPkg-2ndstage.fdf ArmPlatformPkg/ArmPlatformPkg.dsc ArmPlatformPkg/ArmPlatformPkg.fdf
I've sent the patch to remove these files a short while ago. Let me know if you wish me to re-send this as a v3 series.
You can find the series here as patches f5cd78c^..61f537a
https://git.linaro.org/landing-teams/working/arm/edk2.git/shortlog/refs/tags...
As before, ignore my hacks that sit on top, although you're welcome to inspect and provide comment, of course.