On 5 September 2016 at 15:45, Leif Lindholm leif.lindholm@linaro.org wrote:
On Mon, Sep 05, 2016 at 03:33:06PM +0100, Ard Biesheuvel wrote:
This moves all platforms in OpenPlatformPkg to ArmBaseLib, which replaces all previously existing variants of ArmLib.
I have build tested each of them, and boot tested the ones I can test (FVP and Overdrive).
I picked up some fixes for D02/D03 along the way. These are listed first (#1 - #4), #1 is required to fix the build, #5 is required since ArmBaseLib no longer pulls in MemoryAllocationLib. #2 and #3 are optional, but useful improvements nonetheless. #5 does the ArmBaseLib switch itself for D02/D03
Patches #6 to #9 replace all ArmLib references to ArmBaseLib in the remaining platforms.
NOTE: This series combines with my armlib-cleanup branch [0], which deviates from the v1 I sent out today. When testing these patches, please use this branch; it will be the basis of my v2 submission once we have confirmed that everything works as expected. This series can be found here [1]
I would prefer to see Tested-by:s for 2/9 and 3/9. For the rest: Reviewed-by: Leif Lindholm leif.lindholm@linaro.org
Heyi,
Do you have any comments regarding patches 2/9 and 3/9?
Thanks, Ard.