Hi Ling, Apologies for not responding sooner. As you can imagine, the news last week that NUVIA was being acquired by Qualcomm has caused some distractions.
I will get on with reviewing the individual patches, but I will start by proposing some name changes and structure changes to better fit with tianocore design principles. First of all - the concept of "packages" is fairly central to edk2. Any directory that contains a .dec file is a package, and this can be more strongly indicated by giving the directory a name ending in Pkg. Personally, I also tend to see directories containing .dsc/.fdf as packages, even if they don't currently hold a .dec. Platform Phytium DurianPkg DurianPkg.dsc DurianPkg.fdf Silicon Phytium Phytium2000-4Pkg Drivers Include Library Protocol Library PhytiumCommonPkg PhytiumCommon.dec PhytiumCommon.dsc.inc Drivers Include Library Protocol Library Logo You could then have a new Phytium2000-4Pkg.dec, to provide the [Include] statement for that package. Does this make sense? On Fri, Jan 15, 2021 at 08:47:52 +0000, Ling Jia wrote: > From: Ling <jial...@phytium.com.cn> > > The modules could be runed at the silicon of Phytium2000-4. > They supported Acpi parameter configuration, Pci bus scaning, > flash read-write and erase abd operating system boot function. > Maintainers.txt: Added maintainers and reviewers for the DurianPkg. > > The public git repository is : > https://github.com/jialing2020/edk2-platforms/tree/phytium_opensource_for_2004_v1 > > *** BLURB HERE *** You can delete this line - it's just there to remind you to write a message :) Best Regards, Leif > > Ling (10): > Silicon/Phytium/: added PhytiumPlatformLib to Phytium2000-4 > Silicon/Phytium: Added Acpi support to Phytium2000-4 > Silicon/Phytium: Added SMBIOS support to Phytium2000-4 > Silicon/Phytium/Phytium2000-4/Library: Added PciSegmentLib to > Phytium2000-4 > Silicon/Phytium: Added PciHostBridgeLib to Phytium2000-4 > Silicon/Phytium: Added Logo support to Phytium Silicon > Silicon/Phytium: Added Spi driver support to Phytium2000-4 > Silicon/Phytium: Added flash driver support to Phytium Silicon > Silicon/Phytium: Added fvb driver for norflash > Silicon/Phytium: Added Rtc driver to Phytium2000-4 > > Silicon/Phytium/Phytium.dec > | 60 + > Silicon/Phytium/Phytium.dsc.inc > | 388 ++++++ > Platform/Phytium/Durian/DurianPkg.dsc > | 340 +++++ > Platform/Phytium/Durian/DurianPkg.fdf > | 241 ++++ > Silicon/Phytium/CommonDrivers/PhytiumFlashFvbDxe/PhytiumFlashFvbDxe.inf > | 72 + > > Silicon/Phytium/CommonDrivers/PhytiumSpiNorFlashDxe/PhytiumSpiNorFlashDxe.inf > | 54 + > Silicon/Phytium/Library/LogoLib/LogoLib.inf > | 58 + > Silicon/Phytium/Phytium2000-4/Drivers/AcpiPlatformDxe/AcpiPlatformDxe.inf > | 58 + > Silicon/Phytium/Phytium2000-4/Drivers/AcpiTables/AcpiTables.inf > | 61 + > Silicon/Phytium/Phytium2000-4/Drivers/PhytiumSpiDxe/PhytiumSpiDxe.inf > | 52 + > > Silicon/Phytium/Phytium2000-4/Drivers/SmbiosPlatformDxe/SmbiosPlatformDxe.inf > | 55 + > Silicon/Phytium/Phytium2000-4/Library/PciHostBridgeLib/PciHostBridgeLib.inf > | 55 + > Silicon/Phytium/Phytium2000-4/Library/PciSegmentLib/PciSegmentLib.inf > | 28 + > > Silicon/Phytium/Phytium2000-4/Library/PhytiumPlatformLib/PhytiumPlatformLib.inf > | 66 + > > Silicon/Phytium/Phytium2000-4/Library/PhytiumRealTimeClockLib/PhytiumRealTimeClockLib.inf > | 44 + > Silicon/Phytium/CommonDrivers/PhytiumFlashFvbDxe/PhytiumFlashFvbDxe.h > | 106 ++ > Silicon/Phytium/CommonDrivers/PhytiumSpiNorFlashDxe/PhytiumSpiNorFlashDxe.h > | 106 ++ > Silicon/Phytium/Include/PhytiumPlatform.h > | 93 ++ > Silicon/Phytium/Include/PhytiumSystemServiceInterface.h > | 112 ++ > Silicon/Phytium/Include/Protocol/PhytiumSpi.h > | 51 + > Silicon/Phytium/Include/Protocol/PhytiumSpiNorFlash.h > | 74 + > Silicon/Phytium/Phytium2000-4/Drivers/PhytiumSpiDxe/PhytiumSpiDxe.h > | 73 + > > Silicon/Phytium/Phytium2000-4/Library/PhytiumRealTimeClockLib/PhytiumRealTimeClockLib.h > | 24 + > Silicon/Phytium/CommonDrivers/PhytiumFlashFvbDxe/PhytiumFlashFvbDxe.c > | 1235 +++++++++++++++++ > Silicon/Phytium/CommonDrivers/PhytiumSpiNorFlashDxe/PhytiumSpiNorFlashDxe.c > | 435 ++++++ > Silicon/Phytium/Library/LogoLib/Logo.c > | 133 ++ > Silicon/Phytium/Phytium2000-4/Drivers/AcpiPlatformDxe/AcpiPlatform.c > | 254 ++++ > Silicon/Phytium/Phytium2000-4/Drivers/PhytiumSpiDxe/PhytiumSpiDxe.c > | 189 +++ > Silicon/Phytium/Phytium2000-4/Drivers/SmbiosPlatformDxe/SmbiosPlatformDxe.c > | 959 +++++++++++++ > Silicon/Phytium/Phytium2000-4/Library/PciHostBridgeLib/PciHostBridgeLib.c > | 182 +++ > Silicon/Phytium/Phytium2000-4/Library/PciSegmentLib/PciSegmentLib.c > | 1440 ++++++++++++++++++++ > > Silicon/Phytium/Phytium2000-4/Library/PhytiumPlatformLib/PhytiumPlatformLib.c > | 135 ++ > > Silicon/Phytium/Phytium2000-4/Library/PhytiumPlatformLib/PhytiumPlatformLibMem.c > | 148 ++ > > Silicon/Phytium/Phytium2000-4/Library/PhytiumRealTimeClockLib/PhytiumRealTimeClockLib.c > | 468 +++++++ > Maintainers.txt > | 7 + > Silicon/Phytium/Logo/PhytiumLogo.bmp > | Bin 0 -> 32454 bytes > Silicon/Phytium/Phytium.fdf.inc > | 119 ++ > Silicon/Phytium/Phytium2000-4/Drivers/AcpiTables/AcpiSsdtRootPci.asl > | 234 ++++ > Silicon/Phytium/Phytium2000-4/Drivers/AcpiTables/Dbg2.aslc > | 85 ++ > Silicon/Phytium/Phytium2000-4/Drivers/AcpiTables/Dsdt/Cpu.asl > | 85 ++ > Silicon/Phytium/Phytium2000-4/Drivers/AcpiTables/Dsdt/Dsdt.asl > | 15 + > Silicon/Phytium/Phytium2000-4/Drivers/AcpiTables/Dsdt/Uart.asl > | 65 + > Silicon/Phytium/Phytium2000-4/Drivers/AcpiTables/Fadt.aslc > | 81 ++ > Silicon/Phytium/Phytium2000-4/Drivers/AcpiTables/Gtdt.aslc > | 87 ++ > Silicon/Phytium/Phytium2000-4/Drivers/AcpiTables/Iort.aslc > | 89 ++ > Silicon/Phytium/Phytium2000-4/Drivers/AcpiTables/Madt.aslc > | 66 + > Silicon/Phytium/Phytium2000-4/Drivers/AcpiTables/Mcfg.aslc > | 69 + > Silicon/Phytium/Phytium2000-4/Drivers/AcpiTables/Pptt.aslc > | 219 +++ > Silicon/Phytium/Phytium2000-4/Drivers/AcpiTables/Spcr.aslc > | 83 ++ > > Silicon/Phytium/Phytium2000-4/Library/PhytiumPlatformLib/AArch64/PhytiumPlatformHelper.S > | 76 ++ > 50 files changed, 9229 insertions(+) > create mode 100644 Silicon/Phytium/Phytium.dec > create mode 100644 Silicon/Phytium/Phytium.dsc.inc > create mode 100644 Platform/Phytium/Durian/DurianPkg.dsc > create mode 100644 Platform/Phytium/Durian/DurianPkg.fdf > create mode 100644 > Silicon/Phytium/CommonDrivers/PhytiumFlashFvbDxe/PhytiumFlashFvbDxe.inf > create mode 100644 > Silicon/Phytium/CommonDrivers/PhytiumSpiNorFlashDxe/PhytiumSpiNorFlashDxe.inf > create mode 100644 Silicon/Phytium/Library/LogoLib/LogoLib.inf > create mode 100644 > Silicon/Phytium/Phytium2000-4/Drivers/AcpiPlatformDxe/AcpiPlatformDxe.inf > create mode 100644 > Silicon/Phytium/Phytium2000-4/Drivers/AcpiTables/AcpiTables.inf > create mode 100644 > Silicon/Phytium/Phytium2000-4/Drivers/PhytiumSpiDxe/PhytiumSpiDxe.inf > create mode 100644 > Silicon/Phytium/Phytium2000-4/Drivers/SmbiosPlatformDxe/SmbiosPlatformDxe.inf > create mode 100644 > Silicon/Phytium/Phytium2000-4/Library/PciHostBridgeLib/PciHostBridgeLib.inf > create mode 100644 > Silicon/Phytium/Phytium2000-4/Library/PciSegmentLib/PciSegmentLib.inf > create mode 100644 > Silicon/Phytium/Phytium2000-4/Library/PhytiumPlatformLib/PhytiumPlatformLib.inf > create mode 100644 > Silicon/Phytium/Phytium2000-4/Library/PhytiumRealTimeClockLib/PhytiumRealTimeClockLib.inf > create mode 100644 > Silicon/Phytium/CommonDrivers/PhytiumFlashFvbDxe/PhytiumFlashFvbDxe.h > create mode 100644 > Silicon/Phytium/CommonDrivers/PhytiumSpiNorFlashDxe/PhytiumSpiNorFlashDxe.h > create mode 100644 Silicon/Phytium/Include/PhytiumPlatform.h > create mode 100644 Silicon/Phytium/Include/PhytiumSystemServiceInterface.h > create mode 100644 Silicon/Phytium/Include/Protocol/PhytiumSpi.h > create mode 100644 Silicon/Phytium/Include/Protocol/PhytiumSpiNorFlash.h > create mode 100644 > Silicon/Phytium/Phytium2000-4/Drivers/PhytiumSpiDxe/PhytiumSpiDxe.h > create mode 100644 > Silicon/Phytium/Phytium2000-4/Library/PhytiumRealTimeClockLib/PhytiumRealTimeClockLib.h > create mode 100644 > Silicon/Phytium/CommonDrivers/PhytiumFlashFvbDxe/PhytiumFlashFvbDxe.c > create mode 100644 > Silicon/Phytium/CommonDrivers/PhytiumSpiNorFlashDxe/PhytiumSpiNorFlashDxe.c > create mode 100644 Silicon/Phytium/Library/LogoLib/Logo.c > create mode 100644 > Silicon/Phytium/Phytium2000-4/Drivers/AcpiPlatformDxe/AcpiPlatform.c > create mode 100644 > Silicon/Phytium/Phytium2000-4/Drivers/PhytiumSpiDxe/PhytiumSpiDxe.c > create mode 100644 > Silicon/Phytium/Phytium2000-4/Drivers/SmbiosPlatformDxe/SmbiosPlatformDxe.c > create mode 100644 > Silicon/Phytium/Phytium2000-4/Library/PciHostBridgeLib/PciHostBridgeLib.c > create mode 100644 > Silicon/Phytium/Phytium2000-4/Library/PciSegmentLib/PciSegmentLib.c > create mode 100644 > Silicon/Phytium/Phytium2000-4/Library/PhytiumPlatformLib/PhytiumPlatformLib.c > create mode 100644 > Silicon/Phytium/Phytium2000-4/Library/PhytiumPlatformLib/PhytiumPlatformLibMem.c > create mode 100644 > Silicon/Phytium/Phytium2000-4/Library/PhytiumRealTimeClockLib/PhytiumRealTimeClockLib.c > create mode 100644 Silicon/Phytium/Logo/PhytiumLogo.bmp > create mode 100644 Silicon/Phytium/Phytium.fdf.inc > create mode 100644 > Silicon/Phytium/Phytium2000-4/Drivers/AcpiTables/AcpiSsdtRootPci.asl > create mode 100644 Silicon/Phytium/Phytium2000-4/Drivers/AcpiTables/Dbg2.aslc > create mode 100644 > Silicon/Phytium/Phytium2000-4/Drivers/AcpiTables/Dsdt/Cpu.asl > create mode 100644 > Silicon/Phytium/Phytium2000-4/Drivers/AcpiTables/Dsdt/Dsdt.asl > create mode 100644 > Silicon/Phytium/Phytium2000-4/Drivers/AcpiTables/Dsdt/Uart.asl > create mode 100644 Silicon/Phytium/Phytium2000-4/Drivers/AcpiTables/Fadt.aslc > create mode 100644 Silicon/Phytium/Phytium2000-4/Drivers/AcpiTables/Gtdt.aslc > create mode 100644 Silicon/Phytium/Phytium2000-4/Drivers/AcpiTables/Iort.aslc > create mode 100644 Silicon/Phytium/Phytium2000-4/Drivers/AcpiTables/Madt.aslc > create mode 100644 Silicon/Phytium/Phytium2000-4/Drivers/AcpiTables/Mcfg.aslc > create mode 100644 Silicon/Phytium/Phytium2000-4/Drivers/AcpiTables/Pptt.aslc > create mode 100644 Silicon/Phytium/Phytium2000-4/Drivers/AcpiTables/Spcr.aslc > create mode 100644 > Silicon/Phytium/Phytium2000-4/Library/PhytiumPlatformLib/AArch64/PhytiumPlatformHelper.S > > -- > 2.25.1 > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#70674): https://edk2.groups.io/g/devel/message/70674 Mute This Topic: https://groups.io/mt/79697753/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-