[edk2-devel] [PATCH v3 0/2] Add EDKII_PCI_DEVICE_PPI support to EDK2

[Wu, Hao A]

Thanks, the series look good to me. I have given my R-B tag for both of the patches.
Will wait some time to see if there are comments from other reviewers. If no further feedback received, I will merge the series before the end of this week.

Best Regards,
Hao Wu

> -----Original Message-----
> From: devel at edk2.groups.io <devel at edk2.groups.io> On Behalf Of Maciej
> Czajkowski
> Sent: Tuesday, August 2, 2022 1:00 AM
> To: devel at edk2.groups.io
> Subject: [edk2-devel] [PATCH v3 0/2] Add EDKII_PCI_DEVICE_PPI support to
> EDK2
> 
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3907
> 
> The purpose of those changes is to introduce the way to enumerate and
> assign resources in PEI for the systems with more than one PCI root.
> Here is a need to have an interface that will support such a mechanizm.
> For now, the part that performs the enumeration will be implemented in the
> silicon code.
> Sample code can be seen here: https://github.com/mczaj/edk2-
> platforms/commit/d443062e58f9fba228869b54f2546d9735b3b506
> 
> v1: https://edk2.groups.io/g/devel/topic/91575907
> v2: https://edk2.groups.io/g/devel/message/91893
> 
> v2 changes:
> - collected Acked-by tag for no.1 commit
> - followed-up with change suggestions in no.2 commit
> 
> v3 changes:
> - no.1 commit: picked up reviewed-by tag
> - no.2 commit: a few fixes based on v2 review
> 
> Maciej Czajkowski (2):
>   MdeModulePkg: Add EDKII_PCI_DEVICE_PPI definition
>   MdeModulePkg/AhciPei: Use PCI_DEVICE_PPI to manage AHCI device
> 
>  MdeModulePkg/Bus/Ata/AhciPei/AhciPei.c    | 505 +++++++++++++-------
>  MdeModulePkg/Bus/Ata/AhciPei/DevicePath.c |  44 --
>  MdeModulePkg/Bus/Ata/AhciPei/AhciPei.h    |  57 ++-
>  MdeModulePkg/Bus/Ata/AhciPei/AhciPei.inf  |   5 +-
>  MdeModulePkg/Include/Ppi/PciDevice.h      |  32 ++
>  MdeModulePkg/MdeModulePkg.dec             |   3 +
>  MdeModulePkg/MdeModulePkg.dsc             |   1 +
>  7 files changed, 425 insertions(+), 222 deletions(-)  create mode 100644
> MdeModulePkg/Include/Ppi/PciDevice.h
> 
> --
> 2.27.0.windows.1
> 
> ---------------------------------------------------------------------
> Intel Technology Poland sp. z o.o.
> ul. Slowackiego 173 | 80-298 Gdansk | Sad Rejonowy Gdansk Polnoc | VII
> Wydzial Gospodarczy Krajowego Rejestru Sadowego - KRS 101882 | NIP 957-
> 07-52-316 | Kapital zakladowy 200.000 PLN.
> Ta wiadomosc wraz z zalacznikami jest przeznaczona dla okreslonego
> adresata i moze zawierac informacje poufne. W razie przypadkowego
> otrzymania tej wiadomosci, prosimy o powiadomienie nadawcy oraz trwale
> jej usuniecie; jakiekolwiek przegladanie lub rozpowszechnianie jest
> zabronione.
> This e-mail and any attachments may contain confidential material for the
> sole use of the intended recipient(s). If you are not the intended recipient,
> please contact the sender and delete all copies; any review or distribution by
> others is strictly prohibited.
> 
> 
> 
> 
> 



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#92039): https://edk2.groups.io/g/devel/message/92039
Mute This Topic: https://groups.io/mt/92752847/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-