From: "Zhiguang Liu" <zhiguang.liu@intel.com>
To: Patrick Rudolph <patrick.rudolph@9elements.com>,
"devel@edk2.groups.io" <devel@edk2.groups.io>
Cc: gaoliming <gaoliming@byosoft.com.cn>
Subject: 回复: [edk2-devel] [Patch V4 0/9] Create multiple Hobs for Universal Payload
Date: Wed, 16 Jun 2021 01:07:48 +0000 [thread overview]
Message-ID: <PH0PR11MB50487893138C8A55D7949D76900F9@PH0PR11MB5048.namprd11.prod.outlook.com> (raw)
In-Reply-To: <CALNFmy1T6zxEFdeZE8xUNp6=aZ6fUHkmtebiQ6B86RX7=cgjfQ@mail.gmail.com>
[-- Attachment #1: Type: text/plain, Size: 6877 bytes --]
Thank Patrick for verifying.
I will add your Tested-by tag for this patch series.
Thanks
Zhiguang
________________________________
发件人: Patrick Rudolph <patrick.rudolph@9elements.com>
发送时间: 2021年6月15日 20:00
收件人: devel@edk2.groups.io <devel@edk2.groups.io>; Liu, Zhiguang <zhiguang.liu@intel.com>
抄送: gaoliming <gaoliming@byosoft.com.cn>
主题: Re: [edk2-devel] [Patch V4 0/9] Create multiple Hobs for Universal Payload
Tested the patch series on Intel Coffee Lake Platform using latest
coreboot master.
Everything seems to work fine.
Thanks
Patrick
On Thu, Jun 10, 2021 at 11:48 AM Zhiguang Liu <zhiguang.liu@intel.com> wrote:
>
> Liming,
>
> Bugzilla is created at https://bugzilla.tianocore.org/show_bug.cgi?id=3447
>
> Thanks
> Zhiguang
>
> > -----Original Message-----
> > From: gaoliming <gaoliming@byosoft.com.cn>
> > Sent: Thursday, June 10, 2021 5:14 PM
> > To: devel@edk2.groups.io; Liu, Zhiguang <zhiguang.liu@intel.com>
> > Subject: 回复: [edk2-devel] [Patch V4 0/9] Create multiple Hobs for Universal
> > Payload
> >
> > Zhiguang:
> > Can you submit one BZ for this new feature? I will add it into edk2 202108
> > stable tag planning.
> >
> > Thanks
> > Liming
> > > -----邮件原件-----
> > > 发件人: devel@edk2.groups.io <devel@edk2.groups.io> 代表 Zhiguang Liu
> > > 发送时间: 2021年6月10日 9:33
> > > 收件人: devel@edk2.groups.io
> > > 主题: [edk2-devel] [Patch V4 0/9] Create multiple Hobs for Universal
> > > Payload
> > >
> > > V1:
> > > This patch set is based on Universal Payload on
> > > https://universalpayload.github.io/documentation/payload-
> > interfaces/index.
> > > html
> > > This patch set introduce one general header, three different hob types
> > > and how Universal Payload consume these hobs.
> > >
> > > V2:
> > > Move all the header files and Guid define to MdeModulePkg Fix code bug
> > > when parsing SmbiosDxe.
> > > Enhance error handling in AcpiTableProtocol.c.
> > > Add AcpiTableDxe.inf in UefiPayload.fdf
> > >
> > > V3:
> > > Avoid duplicated code in SmBiosDxe.c
> > >
> > > V4:
> > > Add link to spec in header files' file comments Avoid using PLD,
> > > because it may be confusing
> > >
> > > All changes can be seen at
> > >
> > https://github.com/LiuZhiguang001/edk2/tree/UniversalPayloadHeaders_v4
> > >
> > > Zhiguang Liu (9):
> > > MdeModulePkg: Add Universal Payload general definition header file
> > > MdeModulePkg: Add new structure for the PCI Root Bridge Info Hob
> > > UefiPayloadPkg: UefiPayload retrieve PCI root bridge from Guid Hob
> > > MdeModulePkg: Add new structure for the Universal Payload SMBios
> > Table
> > > Info Hob
> > > MdeModulePkg/Universal/SmbiosDxe: Scan for existing tables
> > > UefiPayloadPkg: Creat gPldSmbiosTableGuid Hob
> > > MdeModulePkg: Add new structure for the Universal Payload ACPI Table
> > > Info Hob
> > > MdeModulePkg/ACPI: Install ACPI table from HOB.
> > > UefiPayloadPkg: Creat gPldAcpiTableGuid Hob
> > >
> > > MdeModulePkg/Include/UniversalPayload/AcpiTable.h |
> > > 30 ++++++++++++++++++++++++++++++
> > > MdeModulePkg/Include/UniversalPayload/PciRootBridges.h |
> > > 91
> > >
> > ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
> > ++++
> > > +++++++++++++++++++++++++++++
> > > MdeModulePkg/Include/UniversalPayload/SmbiosTable.h |
> > > 30 ++++++++++++++++++++++++++++++
> > > MdeModulePkg/Include/UniversalPayload/UniversalPayload.h |
> > > 35 +++++++++++++++++++++++++++++++++++
> > > MdeModulePkg/MdeModulePkg.dec
> > > | 15 +++++++++++++++
> > > MdeModulePkg/Universal/Acpi/AcpiTableDxe/AcpiSdt.c |
> > > 92
> > >
> > ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
> > ++++
> > > +++---------------------------
> > > MdeModulePkg/Universal/Acpi/AcpiTableDxe/AcpiTable.h |
> > > 38 +++++++++++++++++++++++++++++++++++++-
> > > MdeModulePkg/Universal/Acpi/AcpiTableDxe/AcpiTableDxe.inf | 8
> > > +++++---
> > > MdeModulePkg/Universal/Acpi/AcpiTableDxe/AcpiTableProtocol.c | 171
> > >
> > ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
> > ++++
> > >
> > ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
> > ++++
> > > ++++++++++++++++++++++++++++++++++++++++-------
> > > MdeModulePkg/Universal/SmbiosDxe/SmbiosDxe.c |
> > > 293
> > >
> > ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
> > ++++
> > >
> > ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
> > ++++
> > >
> > ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
> > ++++
> > >
> > ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
> > ++++
> > > ++++++++++++++++++++++++++++++++++++++++++++-
> > > MdeModulePkg/Universal/SmbiosDxe/SmbiosDxe.h |
> > > 65
> > >
> > ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
> > ++++
> > > ++-
> > > MdeModulePkg/Universal/SmbiosDxe/SmbiosDxe.inf |
> > > 5 ++++-
> > > UefiPayloadPkg/BlSupportDxe/BlSupportDxe.c |
> > > 28 +---------------------------
> > > UefiPayloadPkg/BlSupportDxe/BlSupportDxe.h |
> > > 5 +----
> > > UefiPayloadPkg/BlSupportDxe/BlSupportDxe.inf |
> > > 4 +---
> > > UefiPayloadPkg/Library/PciHostBridgeLib/PciHostBridge.h | 40
> > > ++++++++++++++++++++++++++++++++++++++--
> > > UefiPayloadPkg/Library/PciHostBridgeLib/PciHostBridgeLib.c | 47
> > > ++++++++++++++++++++++++++++++++++++++++++++---
> > > UefiPayloadPkg/Library/PciHostBridgeLib/PciHostBridgeLib.inf | 8
> > > +++++++-
> > > UefiPayloadPkg/Library/PciHostBridgeLib/PciHostBridgeSupport.c | 73
> > >
> > ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
> > ++++
> > > ++++++++++-
> > > UefiPayloadPkg/UefiPayloadEntry/UefiPayloadEntry.c | 23
> > > ++++++++++++++++++++++-
> > > UefiPayloadPkg/UefiPayloadEntry/UefiPayloadEntry.h | 5
> > > +++--
> > > UefiPayloadPkg/UefiPayloadEntry/UefiPayloadEntry.inf | 4
> > > +++-
> > > UefiPayloadPkg/UefiPayloadPkg.dsc |
> > > 2 +-
> > > UefiPayloadPkg/UefiPayloadPkg.fdf |
> > > 4 ++++
> > > 24 files changed, 1029 insertions(+), 87 deletions(-) create mode
> > > 100644 MdeModulePkg/Include/UniversalPayload/AcpiTable.h
> > > create mode 100644
> > > MdeModulePkg/Include/UniversalPayload/PciRootBridges.h
> > > create mode 100644
> > > MdeModulePkg/Include/UniversalPayload/SmbiosTable.h
> > > create mode 100644
> > > MdeModulePkg/Include/UniversalPayload/UniversalPayload.h
> > >
> > > --
> > > 2.30.0.windows.2
> > >
> > >
> > >
> > >
> > >
> >
> >
>
>
>
>
>
>
[-- Attachment #2: Type: text/html, Size: 12443 bytes --]
prev parent reply other threads:[~2021-06-16 1:07 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-06-10 1:33 [Patch V4 0/9] Create multiple Hobs for Universal Payload Zhiguang Liu
2021-06-10 1:33 ` [Patch V4 1/9] MdeModulePkg: Add Universal Payload general definition header file Zhiguang Liu
2021-06-10 1:33 ` [Patch V4 2/9] MdeModulePkg: Add new structure for the PCI Root Bridge Info Hob Zhiguang Liu
2021-06-10 1:33 ` [Patch V4 3/9] UefiPayloadPkg: UefiPayload retrieve PCI root bridge from Guid Hob Zhiguang Liu
2021-06-10 1:33 ` [Patch V4 4/9] MdeModulePkg: Add new structure for the Universal Payload SMBios Table Info Hob Zhiguang Liu
2021-06-10 1:33 ` [Patch V4 5/9] MdeModulePkg/Universal/SmbiosDxe: Scan for existing tables Zhiguang Liu
2021-06-10 1:33 ` [Patch V4 6/9] UefiPayloadPkg: Creat gPldSmbiosTableGuid Hob Zhiguang Liu
2021-06-10 2:58 ` Guo Dong
2021-06-10 1:33 ` [Patch V4 7/9] MdeModulePkg: Add new structure for the Universal Payload ACPI Table Info Hob Zhiguang Liu
2021-06-10 1:33 ` [Patch V4 8/9] MdeModulePkg/ACPI: Install ACPI table from HOB Zhiguang Liu
2021-06-10 1:33 ` [Patch V4 9/9] UefiPayloadPkg: Creat gPldAcpiTableGuid Hob Zhiguang Liu
2021-06-10 3:00 ` Guo Dong
2021-06-10 9:13 ` 回复: [edk2-devel] [Patch V4 0/9] Create multiple Hobs for Universal Payload gaoliming
2021-06-10 9:48 ` Zhiguang Liu
2021-06-15 12:00 ` Patrick Rudolph
2021-06-16 1:07 ` Zhiguang Liu [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=PH0PR11MB50487893138C8A55D7949D76900F9@PH0PR11MB5048.namprd11.prod.outlook.com \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox