Hi Ard,
pt., 24 maj 2019 o 18:57 Ard Biesheuvel <ard.biesheuvel@linaro.org> napisał(a):
>
> On Fri, 24 May 2019 at 17:59, Marcin Wojtas <mw@semihalf.com> wrote:
> >
> > Hi,
> >
> > The v3 of Armada PCIE support brings one change - the custom
> > PciExpressLib was replaced with PciSegmentLib, that will
> > allow to extend the support with more slots available
> > on some platforms.
> >
> > The patches are available in the github:
> > https://github.com/MarvellEmbeddedProcessors/edk2-open-platform/commits/pcie-upstream-r20190524
> >
> > I'm looking forward to your comments or remarks.
> >
> > Best regards,
> > Marcin
> >
>
> For the series
> Reviewed-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
>
> Pushed as 21e36d78ddae..50a9caf82b81
>
> Thanks a lot! I'm happy we finally have this stuff merged.
>
>
Me too! Thanks for the review and the last hint about the
PciSegmentLib. Only Smbios resend left for now.
Best regards,
Marcin
>
>
> > Changelog:
> > v2->v3:
> > * 6/14
> > - Replace PciExpressLib with PciSegmentLib
> >
> > * 8/14
> > - Adjust to new dependecies
> >
> > v1->v2:
> > *All
> > - s/PcieBaseAddress/PcieDbiAdress/
> >
> > *2/14
> > - fix alignment in comment
> >
> > * 3/14
> > - add CONST** to library callback
> >
> > * 4/14
> > - add missing reset GPIO to McBin description
> >
> > * 5/15
> > - add CONST** to protocol callback
> >
> > * 6/14
> > - cleanup all casting in file
> > - use MAX_UINTx macros
> > - add Linaro copyright
> > - use MmioWrite8 instead of volatile in PciExpressReadBuffer
> > - correct commient in IgnoreBusDeviceFunction ()
> > - fix typo in commit message
> >
> > * 7/10
> > - correct line endings
> > - use temporary variable for memory description in PciHostBridgeResourceConflict
> > - use MAX_UINTx macros
> > - add comments around stalls and MemoryFence in GPIO reset
> > - keep the reset active for 150ms
> > - assign translation values instead of asserting
> >
> > *8/14
> > - assign gArmTokenSpaceGuid.PcdPciIoTranslation value in .dsc
> >
> > * 9-11/14
> > - correct line endings
> > - remove unused methods
> > - extend commit messages with 32k shift description
> >
> >
> > Marcin Wojtas (14):
> > Marvell/Library: MvGpioLib: Extend GPIO pin description
> > Marvell/Library: ArmadaSoCDescLib: Add PCIE information
> > Marvell/Library: ArmadaBoardDescLib: Add PCIE information
> > Marvell/Armada7k8k: Extend board description libraries with PCIE
> > Marvell/Armada7k8k: MvBoardDesc: Extend protocol with PCIE support
> > Marvell/Armada7k8k: Add PciSegmentLib implementation
> > Marvell/Armada7k8k: Implement PciHostBridgeLib
> > Marvell/Armada7k8k: Enable PCIE support
> > Marvell/Armada80x0McBin: Enable ACPI PCIE support
> > Marvell/Armada80x0Db: Enable ACPI PCIE support
> > Marvell/Armada70x0Db: Enable ACPI PCIE support
> > Marvell/Armada80x0McBin: DeviceTree: Use pci-host-generic driver
> > Marvell/Armada7k8k: Remove duplication in .dsc files
> > Marvell/Armada7k8: Add 'acpiview' shell command to build
> >
> > Silicon/Marvell/Armada7k8k/Armada7k8k.dsc.inc | 15 +
> > Platform/Marvell/Armada70x0Db/Armada70x0Db.dsc | 4 +-
> > Platform/Marvell/Armada80x0Db/Armada80x0Db.dsc | 4 +-
> > Platform/SolidRun/Armada80x0McBin/Armada80x0McBin.dsc | 4 +-
> > Silicon/Marvell/Armada7k8k/Armada7k8k.fdf | 5 +
> > Silicon/Marvell/Armada7k8k/AcpiTables/Armada70x0Db.inf | 1 +
> > Silicon/Marvell/Armada7k8k/AcpiTables/Armada80x0Db.inf | 1 +
> > Silicon/Marvell/Armada7k8k/AcpiTables/Armada80x0McBin.inf | 1 +
> > Silicon/Marvell/Armada7k8k/Library/Armada7k8kPciHostBridgeLib/PciHostBridgeLib.inf | 52 +
> > Silicon/Marvell/Armada7k8k/Library/Armada7k8kPciSegmentLib/PciSegmentLib.inf | 33 +
> > Silicon/Marvell/Armada7k8k/AcpiTables/Armada70x0Db/Pcie.h | 26 +
> > Silicon/Marvell/Armada7k8k/AcpiTables/Armada80x0Db/Pcie.h | 26 +
> > Silicon/Marvell/Armada7k8k/AcpiTables/Armada80x0McBin/Pcie.h | 26 +
> > Silicon/Marvell/Armada7k8k/Library/Armada7k8kPciHostBridgeLib/PciHostBridgeLibConstructor.h | 95 ++
> > Silicon/Marvell/Armada7k8k/Library/Armada7k8kSoCDescLib/Armada7k8kSoCDescLib.h | 6 +
> > Silicon/Marvell/Include/Library/ArmadaBoardDescLib.h | 46 +
> > Silicon/Marvell/Include/Library/ArmadaSoCDescLib.h | 20 +
> > Silicon/Marvell/Include/Library/MvGpioLib.h | 1 +
> > Silicon/Marvell/Include/Protocol/BoardDesc.h | 22 +
> > Platform/Marvell/Armada70x0Db/Armada70x0DbBoardDescLib/Armada70x0DbBoardDescLib.c | 48 +
> > Platform/Marvell/Armada70x0Db/NonDiscoverableInitLib/NonDiscoverableInitLib.c | 4 +
> > Platform/Marvell/Armada80x0Db/Armada80x0DbBoardDescLib/Armada80x0DbBoardDescLib.c | 48 +
> > Platform/Marvell/Armada80x0Db/NonDiscoverableInitLib/NonDiscoverableInitLib.c | 6 +
> > Platform/SolidRun/Armada80x0McBin/Armada80x0McBinBoardDescLib/Armada80x0McBinBoardDescLib.c | 54 +
> > Platform/SolidRun/Armada80x0McBin/NonDiscoverableInitLib/NonDiscoverableInitLib.c | 1 +
> > Silicon/Marvell/Armada7k8k/Library/Armada7k8kPciHostBridgeLib/PciHostBridgeLib.c | 265 ++++
> > Silicon/Marvell/Armada7k8k/Library/Armada7k8kPciHostBridgeLib/PciHostBridgeLibConstructor.c | 345 +++++
> > Silicon/Marvell/Armada7k8k/Library/Armada7k8kPciSegmentLib/PciSegmentLib.c | 1390 ++++++++++++++++++++
> > Silicon/Marvell/Armada7k8k/Library/Armada7k8kSoCDescLib/Armada7k8kSoCDescLib.c | 44 +
> > Silicon/Marvell/Drivers/BoardDesc/MvBoardDescDxe.c | 86 ++
> > Silicon/Marvell/Armada7k8k/AcpiTables/Armada70x0Db/Dsdt.asl | 108 ++
> > Silicon/Marvell/Armada7k8k/AcpiTables/Armada70x0Db/Mcfg.aslc | 47 +
> > Silicon/Marvell/Armada7k8k/AcpiTables/Armada80x0Db/Dsdt.asl | 108 ++
> > Silicon/Marvell/Armada7k8k/AcpiTables/Armada80x0Db/Mcfg.aslc | 47 +
> > Silicon/Marvell/Armada7k8k/AcpiTables/Armada80x0McBin/Dsdt.asl | 108 ++
> > Silicon/Marvell/Armada7k8k/AcpiTables/Armada80x0McBin/Mcfg.aslc | 47 +
> > Silicon/Marvell/Armada7k8k/DeviceTree/armada-8040-mcbin.dts | 3 +
> > 37 files changed, 3138 insertions(+), 9 deletions(-)
> > create mode 100644 Silicon/Marvell/Armada7k8k/Library/Armada7k8kPciHostBridgeLib/PciHostBridgeLib.inf
> > create mode 100644 Silicon/Marvell/Armada7k8k/Library/Armada7k8kPciSegmentLib/PciSegmentLib.inf
> > create mode 100644 Silicon/Marvell/Armada7k8k/AcpiTables/Armada70x0Db/Pcie.h
> > create mode 100644 Silicon/Marvell/Armada7k8k/AcpiTables/Armada80x0Db/Pcie.h
> > create mode 100644 Silicon/Marvell/Armada7k8k/AcpiTables/Armada80x0McBin/Pcie.h
> > create mode 100644 Silicon/Marvell/Armada7k8k/Library/Armada7k8kPciHostBridgeLib/PciHostBridgeLibConstructor.h
> > create mode 100644 Silicon/Marvell/Armada7k8k/Library/Armada7k8kPciHostBridgeLib/PciHostBridgeLib.c
> > create mode 100644 Silicon/Marvell/Armada7k8k/Library/Armada7k8kPciHostBridgeLib/PciHostBridgeLibConstructor.c
> > create mode 100644 Silicon/Marvell/Armada7k8k/Library/Armada7k8kPciSegmentLib/PciSegmentLib.c
> > create mode 100644 Silicon/Marvell/Armada7k8k/AcpiTables/Armada70x0Db/Mcfg.aslc
> > create mode 100644 Silicon/Marvell/Armada7k8k/AcpiTables/Armada80x0Db/Mcfg.aslc
> > create mode 100644 Silicon/Marvell/Armada7k8k/AcpiTables/Armada80x0McBin/Mcfg.aslc
> >
> > --
> > 2.7.4
> >
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#41368): https://edk2.groups.io/g/devel/message/41368
Mute This Topic: https://groups.io/mt/31746165/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-