public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Gao, Liming" <liming.gao@intel.com>
To: "Ni, Ruiyu" <ruiyu.ni@intel.com>,
	"edk2-devel@lists.01.org" <edk2-devel@lists.01.org>
Subject: Re: [PATCH v2 0/5] Add multiple PCI segments configuration access support
Date: Mon, 28 Aug 2017 07:39:16 +0000	[thread overview]
Message-ID: <4A89E2EF3DFEDB4C8BFDE51014F606A14D781633@shsmsx102.ccr.corp.intel.com> (raw)
In-Reply-To: <20170825085723.396044-1-ruiyu.ni@intel.com>

Reviewed-by: Liming Gao <liming.gao@intel.com>

>-----Original Message-----
>From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of
>Ruiyu Ni
>Sent: Friday, August 25, 2017 4:57 PM
>To: edk2-devel@lists.01.org
>Subject: [edk2] [PATCH v2 0/5] Add multiple PCI segments configuration
>access support
>
>v2: Add missing function header and file header. Correct all incorrect file
>headers.
>
>Ruiyu Ni (5):
>  MdePkg/PciSegmentLib: Fix typo in function header comments
>  MdePkg/PciExpress: Add macro PCI_ECAM_ADDRESS
>  MdePkg/PciSegmentInfoLib: Add PciSegmentInfoLib class and instance.
>  MdePkg/PciSegmentLib: Add instances that consumes PciSegmentInfoLib
>  MdePkg/S3PciSegmentLib: Add S3PciSegmentLib class and instance.
>
> MdePkg/Include/IndustryStandard/PciExpress21.h     |  19 +-
> MdePkg/Include/Library/PciExpressLib.h             |   5 +-
> MdePkg/Include/Library/PciSegmentInfoLib.h         |  41 ++
> MdePkg/Include/Library/PciSegmentLib.h             | 102 ++--
> .../Library/{PciSegmentLib.h => S3PciSegmentLib.h} | 364 ++++++-----
> .../BasePciSegmentInfoLibNull.inf                  |  41 ++
> .../BasePciSegmentInfoLibNull.uni                  |  20 +
> .../BasePciSegmentInfoLibNull/PciSegmentInfoLib.c  |  38 ++
> .../Library/BasePciSegmentLibPci/PciSegmentLib.c   | 136 +++--
> .../BaseS3PciSegmentLib/BaseS3PciSegmentLib.inf    |  45 ++
> .../BaseS3PciSegmentLib/BaseS3PciSegmentLib.uni    |  23 +
> .../BaseS3PciSegmentLib/S3PciSegmentLib.c}         | 674 ++++++++++++++-----
>--
> .../PciSegmentLibSegmentInfo/BasePciSegmentLib.c   |  71 +++
> .../BasePciSegmentLibSegmentInfo.inf               |  46 ++
> .../BasePciSegmentLibSegmentInfo.uni               |  21 +
> .../DxeRuntimePciSegmentLib.c                      | 321 ++++++++++
> .../DxeRuntimePciSegmentLibSegmentInfo.inf         |  55 ++
> .../DxeRuntimePciSegmentLibSegmentInfo.uni         |  21 +
> .../PciSegmentLibCommon.c}                         | 656 +++++++++++++++-----
> .../PciSegmentLibSegmentInfo/PciSegmentLibCommon.h |  57 ++
> .../PeiPciSegmentLibPciCfg2/PciSegmentLib.c        | 187 +++---
> .../PciSegmentLib.c                                | 180 +++---
> MdePkg/MdePkg.dec                                  |   8 +
> MdePkg/MdePkg.dsc                                  |   4 +
> 24 files changed, 2250 insertions(+), 885 deletions(-)
> create mode 100644 MdePkg/Include/Library/PciSegmentInfoLib.h
> copy MdePkg/Include/Library/{PciSegmentLib.h => S3PciSegmentLib.h} (80%)
> create mode 100644
>MdePkg/Library/BasePciSegmentInfoLibNull/BasePciSegmentInfoLibNull.inf
> create mode 100644
>MdePkg/Library/BasePciSegmentInfoLibNull/BasePciSegmentInfoLibNull.uni
> create mode 100644
>MdePkg/Library/BasePciSegmentInfoLibNull/PciSegmentInfoLib.c
> create mode 100644
>MdePkg/Library/BaseS3PciSegmentLib/BaseS3PciSegmentLib.inf
> create mode 100644
>MdePkg/Library/BaseS3PciSegmentLib/BaseS3PciSegmentLib.uni
> copy MdePkg/{Include/Library/PciSegmentLib.h =>
>Library/BaseS3PciSegmentLib/S3PciSegmentLib.c} (66%)
> create mode 100644
>MdePkg/Library/PciSegmentLibSegmentInfo/BasePciSegmentLib.c
> create mode 100644
>MdePkg/Library/PciSegmentLibSegmentInfo/BasePciSegmentLibSegmentInf
>o.inf
> create mode 100644
>MdePkg/Library/PciSegmentLibSegmentInfo/BasePciSegmentLibSegmentInf
>o.uni
> create mode 100644
>MdePkg/Library/PciSegmentLibSegmentInfo/DxeRuntimePciSegmentLib.c
> create mode 100644
>MdePkg/Library/PciSegmentLibSegmentInfo/DxeRuntimePciSegmentLibSeg
>mentInfo.inf
> create mode 100644
>MdePkg/Library/PciSegmentLibSegmentInfo/DxeRuntimePciSegmentLibSeg
>mentInfo.uni
> copy MdePkg/{Include/Library/PciSegmentLib.h =>
>Library/PciSegmentLibSegmentInfo/PciSegmentLibCommon.c} (69%)
> create mode 100644
>MdePkg/Library/PciSegmentLibSegmentInfo/PciSegmentLibCommon.h
>
>--
>2.12.2.windows.2
>
>_______________________________________________
>edk2-devel mailing list
>edk2-devel@lists.01.org
>https://lists.01.org/mailman/listinfo/edk2-devel


      parent reply	other threads:[~2017-08-28  7:36 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-08-25  8:57 [PATCH v2 0/5] Add multiple PCI segments configuration access support Ruiyu Ni
2017-08-25  8:57 ` [PATCH v2 1/5] MdePkg/PciSegmentLib: Fix typo in function header comments Ruiyu Ni
2017-08-25  8:57 ` [PATCH v2 2/5] MdePkg/PciExpress: Add macro PCI_ECAM_ADDRESS Ruiyu Ni
2017-08-25  8:57 ` [PATCH v2 3/5] MdePkg/PciSegmentInfoLib: Add PciSegmentInfoLib class and instance Ruiyu Ni
2017-08-25  8:57 ` [PATCH v2 4/5] MdePkg/PciSegmentLib: Add instances that consumes PciSegmentInfoLib Ruiyu Ni
2017-08-29 18:51   ` Ard Biesheuvel
2017-08-29 20:39     ` Laszlo Ersek
2017-08-29 20:47       ` Andrew Fish
2017-08-25  8:57 ` [PATCH v2 5/5] MdePkg/S3PciSegmentLib: Add S3PciSegmentLib class and instance Ruiyu Ni
2017-08-28  7:39 ` Gao, Liming [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=4A89E2EF3DFEDB4C8BFDE51014F606A14D781633@shsmsx102.ccr.corp.intel.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