Hi Leif, Ard,
Any comments for this series of patches?
Thanks,
Heyi
On Wed, Mar 21, 2018 at 09:03:06AM +0800, Heyi Guo wrote:
> For BAR address translation support was added to edk2 generic PciHostBridge by
> commit 74d0a33, now we can also use it for D03/D05 platforms.
> This series of patches include 3 parts of change:
> - Preparation for the switch, moving platform specific code out of
> PciHostBridge
> driver.
> - Add depending libraries and protocol implementations, like PciHostBridgeLib,
> PciSegmentLib and CpuIo2 Protocol.
> - Other enhancement and refinement.
>
> Cc: Ard Biesheuvel <ard.biesheu...@linaro.org>
> Cc: Leif Lindholm <leif.lindh...@linaro.org>
> Cc: Michael D Kinney <michael.d.kin...@intel.com>
> Cc: Haojian Zhuang <haojian.zhu...@linaro.org>
>
> Heyi Guo (12):
> Hisilicon: Enable WARN and INFO debug message
> Hisilicon/D05/PlatformPciLib: fix misuse of macro
> Hisilicon/Pci: move ATU configuration to PcieInitDxe
> Hisilicon/Pci: Merge PciPlatform into PcieInit Driver
> Hisilicon/Pci: Move EnlargeAtuConfig0() to PcieInitDxe
> Hisilicon/PlatformPciLib: add segment for each root bridge
> Hisilicon: add PciHostBridgeLib
> Hisilicon: add PciCpuIo2Dxe
> Hisilicon: add PciSegmentLib for Hi161x
> Hisilicon/D0x: Switch to generic PciHostBridge driver
> Hisilicon: remove platform specific PciHostBridge
> Hisilicon/PlatformPciLib: clear redundant felds in RESOURCE_APPETURE
>
> Silicon/Hisilicon/Hisilicon.dsc.inc
> | 8 +-
> Platform/Hisilicon/D03/D03.dsc
> | 7 +-
> Platform/Hisilicon/D05/D05.dsc
> | 7 +-
> Platform/Hisilicon/D03/D03.fdf
> | 4 +-
> Platform/Hisilicon/D05/D05.fdf
> | 4 +-
> Platform/Hisilicon/D03/Drivers/PciPlatform/PciPlatform.inf
> | 53 -
> Platform/Hisilicon/Library/PciHostBridgeLib/PciHostBridgeLib.inf
> | 51 +
> Silicon/Hisilicon/Drivers/ArmPciCpuIo2Dxe/ArmPciCpuIo2Dxe.inf
> | 48 +
> Silicon/Hisilicon/Drivers/PciHostBridgeDxe/PciHostBridgeDxe.inf
> | 74 -
> Silicon/Hisilicon/Hi1610/Drivers/PcieInit1610/PcieInitDxe.inf
> | 9 +-
> Silicon/Hisilicon/Hi1610/Library/Hi161xPciSegmentLib/Hi161xPciSegmentLib.inf
> | 36 +
> Silicon/Hisilicon/Drivers/PciHostBridgeDxe/PciHostBridge.h
> | 528 -----
> {Platform/Hisilicon/D03/Drivers/PciPlatform =>
> Silicon/Hisilicon/Hi1610/Drivers/PcieInit1610}/PciPlatform.h | 0
> Silicon/Hisilicon/Hi1610/Drivers/PcieInit1610/PcieInitLib.h
> | 13 +
> Silicon/Hisilicon/Include/Library/PlatformPciLib.h
> | 3 +-
> Platform/Hisilicon/D03/Library/PlatformPciLib/PlatformPciLib.c
> | 24 +-
> Platform/Hisilicon/D05/Library/PlatformPciLib/PlatformPciLib.c
> | 66 +-
> Platform/Hisilicon/Library/PciHostBridgeLib/PciHostBridgeLib.c
> | 304 +++
> Silicon/Hisilicon/Drivers/ArmPciCpuIo2Dxe/ArmPciCpuIo2Dxe.c
> | 557 +++++
> Silicon/Hisilicon/Drivers/PciHostBridgeDxe/PciHostBridge.c
> | 1659 --------------
> Silicon/Hisilicon/Drivers/PciHostBridgeDxe/PciRootBridgeIo.c
> | 2404 --------------------
> {Platform/Hisilicon/D03/Drivers/PciPlatform =>
> Silicon/Hisilicon/Hi1610/Drivers/PcieInit1610}/PciPlatform.c | 12 +
> Silicon/Hisilicon/Hi1610/Drivers/PcieInit1610/PcieInit.c
> | 7 +-
> Silicon/Hisilicon/Hi1610/Drivers/PcieInit1610/PcieInitAtu.c
> | 309 +++
> Silicon/Hisilicon/Hi1610/Library/Hi161xPciSegmentLib/PciSegmentLib.c
> | 1503 ++++++++++++
> 25 files changed, 2897 insertions(+), 4793 deletions(-)
> delete mode 100644 Platform/Hisilicon/D03/Drivers/PciPlatform/PciPlatform.inf
> create mode 100644
> Platform/Hisilicon/Library/PciHostBridgeLib/PciHostBridgeLib.inf
> create mode 100644
> Silicon/Hisilicon/Drivers/ArmPciCpuIo2Dxe/ArmPciCpuIo2Dxe.inf
> delete mode 100644
> Silicon/Hisilicon/Drivers/PciHostBridgeDxe/PciHostBridgeDxe.inf
> create mode 100644
> Silicon/Hisilicon/Hi1610/Library/Hi161xPciSegmentLib/Hi161xPciSegmentLib.inf
> delete mode 100644 Silicon/Hisilicon/Drivers/PciHostBridgeDxe/PciHostBridge.h
> rename {Platform/Hisilicon/D03/Drivers/PciPlatform =>
> Silicon/Hisilicon/Hi1610/Drivers/PcieInit1610}/PciPlatform.h (100%)
> create mode 100644
> Platform/Hisilicon/Library/PciHostBridgeLib/PciHostBridgeLib.c
> create mode 100644
> Silicon/Hisilicon/Drivers/ArmPciCpuIo2Dxe/ArmPciCpuIo2Dxe.c
> delete mode 100644 Silicon/Hisilicon/Drivers/PciHostBridgeDxe/PciHostBridge.c
> delete mode 100644
> Silicon/Hisilicon/Drivers/PciHostBridgeDxe/PciRootBridgeIo.c
> rename {Platform/Hisilicon/D03/Drivers/PciPlatform =>
> Silicon/Hisilicon/Hi1610/Drivers/PcieInit1610}/PciPlatform.c (93%)
> create mode 100644
> Silicon/Hisilicon/Hi1610/Drivers/PcieInit1610/PcieInitAtu.c
> create mode 100644
> Silicon/Hisilicon/Hi1610/Library/Hi161xPciSegmentLib/PciSegmentLib.c
>
> --
> 2.7.4
>
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
