Here is a copy of what I sent to Ryan earlier:
Unfortunately, I cannot remove the ArmPlatformLib from PL390Gic INF file even if it build in most cases. The reason is PL390GicSec now uses ArmPlatformIsPrimaryCore() that is implemented by ArmPlatformLib library. If one day one module uses PL390GicSec but does not depend itself on ArmPlatformLib then there is no reason why the engineer should add ArmPlatformLib to the INF file of this new module to solve PL390GicSec dependency.
A reason why it could fail would be you are using a different ArmPlatformLib implementation than the one you are thinking. A way to check that is to generate the build report: build -A ARM -p ... -t ... -y report.log And check which libraries are used by each module.
Actually, I am planning to rename 'PL390GicDxe' into 'ArmGicDxe' that will follow the 'ARM Generic Interrupt Controller Architecture Specification'.
Cheers, Olivier
-----Original Message----- From: boot-architecture-bounces@lists.linaro.org [mailto:boot- architecture-bounces@lists.linaro.org] On Behalf Of Ryan Harkin Sent: 12 June 2013 12:03 To: ryan.harkin@linaro.org; boot-architecture@lists.linaro.org Subject: [PATCH 2/2] ArmPkg/PL390Gic: remove ArmPlatformLib from inf file
The PL390 GIC driver was including ArmPlatform in PL390GicSecLib.inf.
This change is fine as it, however, when the Arndale platfom is fixed to remove its duplicated PL390 GIC driver, it hangs unless this library is removed.
Signed-off-by: Ryan Harkin ryan.harkin@linaro.org
ArmPkg/Drivers/PL390Gic/PL390GicSecLib.inf | 1 - 1 file changed, 1 deletion(-)
diff --git a/ArmPkg/Drivers/PL390Gic/PL390GicSecLib.inf b/ArmPkg/Drivers/PL390Gic/PL390GicSecLib.inf index 205e503..5851bfe 100644 --- a/ArmPkg/Drivers/PL390Gic/PL390GicSecLib.inf +++ b/ArmPkg/Drivers/PL390Gic/PL390GicSecLib.inf @@ -31,7 +31,6 @@
[LibraryClasses] ArmLib
- ArmPlatformLib DebugLib IoLib PcdLib
-- 1.7.9.5
boot-architecture mailing list boot-architecture@lists.linaro.org http://lists.linaro.org/mailman/listinfo/boot-architecture