On 12 May 2016 at 00:00, Ricardo Salveti ricardo.salveti@linaro.org wrote:
On Wed, May 11, 2016 at 1:00 PM, Ard Biesheuvel ard.biesheuvel@linaro.org wrote:
The primary change is the BDS change, which i sent out separately, but which i included for completeness. The whole series is pushed to my dev-FDK-102-ard branch again.
Platforms/AMD/Styx/AmdStyxPciHostBridgeLib: set unused regions base to MAX_UINT64
This prepares our code for upcoming changes in the core PCI root bridge driver
Platforms/AMD/Styx/FdtDxe: remove explicit BEFORE xxx Depex
FdtDxe does not need to be dispatched after the PCI driver, since it does not do anything before the ready-to-boot event is signalled
Platforms/AMD/Styx/AmdStyx.dec: remove cross-package includes Platforms/AMD/Styx: remove references to gArmMpCoreInfoGuid
Code cleanup
Platforms/AMD/Styx/CelloBoard: remove BdsLib library class resolution Platforms/AMD/Styx/OverdriveBoard: make BdsLib reference local to FdtDxe
Get rid of deprecated ARM BDS BdsLib references as much as we can
Platforms/AMD/Styx: switch Cello and Overdrive to generic BDS
BDS switch
Platforms/AMD/Styx: remove StatusCode PEI module Platforms/AMD/Styx: DSC spring cleaning
Cleanup
Platforms/AMD/SataControllerDxe: use different segment# from actual PCI root
Small 'fix' but no real bug here
Platforms/AMD/Styx/OverdriveBoard: make MnpDxe less noisy
Debug patch
Platforms/AMD/Styx/AmdStyx.dec | 14 +-- Platforms/AMD/Styx/CelloBoard/CelloBoard.dsc | 104 ++++-------------- Platforms/AMD/Styx/CelloBoard/CelloBoard.fdf | 4 +- Platforms/AMD/Styx/Drivers/SataControllerDxe/PciEmulation.c | 2 +- Platforms/AMD/Styx/Library/AmdStyxHelperLib/AmdStyxHelperLib.inf | 1 - Platforms/AMD/Styx/Library/AmdStyxPciHostBridgeLib/AmdStyxPciHostBridgeLib.c | 4 +- Platforms/AMD/Styx/OverdriveBoard/FdtDxe/FdtDxe.inf | 5 +- Platforms/AMD/Styx/OverdriveBoard/OverdriveBoard.dsc | 115 ++++++-------------- Platforms/AMD/Styx/OverdriveBoard/OverdriveBoard.fdf | 4 +- 9 files changed, 63 insertions(+), 190 deletions(-)
-- 2.7.4
Tested the series and it worked fine on both Overdrive and Cello. The only issue is what I already reported in the IRC, which is the lack of a timeout message when booting EDK2.
Since this series depends on an extra patch in EDK2, will wait to see if that gets merged tomorrow, otherwise will push that to the internal dev branch.
I should be able to merge that tomorrow if Leif is ok with it? I need to respin it to remove some commented out code, but I can do that when committing