All D0x patches were tested OK on D02/D03.
Tested-by: Heyi Guo heyi.guo@linaro.org
However, I actually found something else wrong with the latest EDK2 for D02/D03. It seems DXE dispatcher has been changed and our SFCDriver will not be run after CpuDxe driver. The issue can be fixed after explicitly adding dependency for SFCDriver, and I have made a patch in the attachment for this. Please help to review and let me know your comments.
Thanks and regards,
heyi
在 9/8/2016 6:36 PM, Heyi Guo 写道:
在 9/8/2016 4:31 PM, Ard Biesheuvel 写道:
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?
I think it is good for the code. And I will test it further on D02/D03 and let you know the result.
Thanks Ard.
Heyi
Thanks, Ard.