public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Zeng, Star" <star.zeng@intel.com>
To: "Yao, Jiewen" <jiewen.yao@intel.com>,
	"edk2-devel@lists.01.org" <edk2-devel@lists.01.org>
Cc: "Zeng, Star" <star.zeng@intel.com>
Subject: Re: [PATCH 0/4] IntelSiliconPkg: move VTd related modules to feature dir.
Date: Sat, 16 Sep 2017 12:19:11 +0000	[thread overview]
Message-ID: <0C09AFA07DD0434D9E2A0C6AEB0483103B969F8B@shsmsx102.ccr.corp.intel.com> (raw)
In-Reply-To: <1505525983-27188-1-git-send-email-jiewen.yao@intel.com>

Thanks for making this.

Reviewed-by: Star Zeng <star.zeng@intel.com>


Thanks,
Star
-----Original Message-----
From: Yao, Jiewen 
Sent: Saturday, September 16, 2017 9:40 AM
To: edk2-devel@lists.01.org
Cc: Zeng, Star <star.zeng@intel.com>
Subject: [PATCH 0/4] IntelSiliconPkg: move VTd related modules to feature dir.

This patch is to address the feedback from Star.Zeng during code review.

We group 4 VTd related modules to Feature/VTd dir to better maintance.

No code update, only location move.

Suggested-by: Star Zeng <star.zeng@intel.com>
Cc: Star Zeng <star.zeng@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Jiewen Yao <jiewen.yao@intel.com>

Jiewen Yao (4):
  IntelSiliconPkg/IntelVTdDxe: Move to feature dir.
  IntelSiliconPkg/PlatformVTdSampleDxe: Move to feature dir.
  IntelSiliconPkg/IntelVTdPmrPei: Move to feature dir.
  IntelSiliconPkg/PlatformIntelVTdInfoSamplePei: Move to feature dir.

 IntelSiliconPkg/Feature/VTd/IntelVTdDxe/BmDma.c                                        |  441 +++++++++
 IntelSiliconPkg/Feature/VTd/IntelVTdDxe/DmaProtection.c                                |  503 ++++++++++
 IntelSiliconPkg/Feature/VTd/IntelVTdDxe/DmaProtection.h                                |  562 +++++++++++
 IntelSiliconPkg/Feature/VTd/IntelVTdDxe/DmarAcpiTable.c                                | 1016 +++++++++++++++++++
 IntelSiliconPkg/Feature/VTd/IntelVTdDxe/IntelVTdDxe.c                                  |  370 +++++++
 IntelSiliconPkg/Feature/VTd/IntelVTdDxe/IntelVTdDxe.inf                                |   85 ++
 IntelSiliconPkg/Feature/VTd/IntelVTdDxe/IntelVTdDxe.uni                                |   20 +
 IntelSiliconPkg/Feature/VTd/IntelVTdDxe/IntelVTdDxeExtra.uni                           |   20 +
 IntelSiliconPkg/Feature/VTd/IntelVTdDxe/PciInfo.c                                      |  369 +++++++
 IntelSiliconPkg/Feature/VTd/IntelVTdDxe/TranslationTable.c                             | 1018 ++++++++++++++++++++
 IntelSiliconPkg/Feature/VTd/IntelVTdDxe/TranslationTableEx.c                           |  157 +++
 IntelSiliconPkg/Feature/VTd/IntelVTdDxe/VtdReg.c                                       |  565 +++++++++++
 IntelSiliconPkg/Feature/VTd/IntelVTdPmrPei/IntelVTdPmr.c                               |  314 ++++++
 IntelSiliconPkg/Feature/VTd/IntelVTdPmrPei/IntelVTdPmrPei.c                            |  615 ++++++++++++
 IntelSiliconPkg/Feature/VTd/IntelVTdPmrPei/IntelVTdPmrPei.h                            |   68 ++
 IntelSiliconPkg/Feature/VTd/IntelVTdPmrPei/IntelVTdPmrPei.inf                          |   59 ++
 IntelSiliconPkg/Feature/VTd/IntelVTdPmrPei/IntelVTdPmrPei.uni                          |   20 +
 IntelSiliconPkg/Feature/VTd/IntelVTdPmrPei/IntelVTdPmrPeiExtra.uni                     |   20 +
 IntelSiliconPkg/Feature/VTd/PlatformVTdInfoSamplePei/PlatformVTdInfoSamplePei.c        |   65 ++
 IntelSiliconPkg/Feature/VTd/PlatformVTdInfoSamplePei/PlatformVTdInfoSamplePei.inf      |   51 +
 IntelSiliconPkg/Feature/VTd/PlatformVTdInfoSamplePei/PlatformVTdInfoSamplePei.uni      |   20 +
 IntelSiliconPkg/Feature/VTd/PlatformVTdInfoSamplePei/PlatformVTdInfoSamplePeiExtra.uni |   20 +
 IntelSiliconPkg/Feature/VTd/PlatformVTdSampleDxe/PlatformVTdSampleDxe.c                |  409 ++++++++
 IntelSiliconPkg/Feature/VTd/PlatformVTdSampleDxe/PlatformVTdSampleDxe.inf              |   59 ++
 IntelSiliconPkg/Feature/VTd/PlatformVTdSampleDxe/PlatformVTdSampleDxe.uni              |   20 +
 IntelSiliconPkg/Feature/VTd/PlatformVTdSampleDxe/PlatformVTdSampleDxeExtra.uni         |   20 +
 IntelSiliconPkg/IntelSiliconPkg.dsc                                                    |    8 +-
 IntelSiliconPkg/IntelVTdDxe/BmDma.c                                                    |  441 ---------
 IntelSiliconPkg/IntelVTdDxe/DmaProtection.c                                            |  503 ----------
 IntelSiliconPkg/IntelVTdDxe/DmaProtection.h                                            |  562 -----------
 IntelSiliconPkg/IntelVTdDxe/DmarAcpiTable.c                                            | 1016 -------------------
 IntelSiliconPkg/IntelVTdDxe/IntelVTdDxe.c                                              |  370 -------
 IntelSiliconPkg/IntelVTdDxe/IntelVTdDxe.inf                                            |   85 --
 IntelSiliconPkg/IntelVTdDxe/IntelVTdDxe.uni                                            |   20 -
 IntelSiliconPkg/IntelVTdDxe/IntelVTdDxeExtra.uni                                       |   20 -
 IntelSiliconPkg/IntelVTdDxe/PciInfo.c                                                  |  369 -------
 IntelSiliconPkg/IntelVTdDxe/TranslationTable.c                                         | 1018 --------------------
 IntelSiliconPkg/IntelVTdDxe/TranslationTableEx.c                                       |  157 ---
 IntelSiliconPkg/IntelVTdDxe/VtdReg.c                                                   |  565 -----------
 IntelSiliconPkg/IntelVTdPmrPei/IntelVTdPmr.c                                           |  314 ------
 IntelSiliconPkg/IntelVTdPmrPei/IntelVTdPmrPei.c                                        |  615 ------------
 IntelSiliconPkg/IntelVTdPmrPei/IntelVTdPmrPei.h                                        |   68 --
 IntelSiliconPkg/IntelVTdPmrPei/IntelVTdPmrPei.inf                                      |   59 --
 IntelSiliconPkg/IntelVTdPmrPei/IntelVTdPmrPei.uni                                      |   20 -
 IntelSiliconPkg/IntelVTdPmrPei/IntelVTdPmrPeiExtra.uni                                 |   20 -
 IntelSiliconPkg/PlatformVTdInfoSamplePei/PlatformVTdInfoSamplePei.c                    |   65 --
 IntelSiliconPkg/PlatformVTdInfoSamplePei/PlatformVTdInfoSamplePei.inf                  |   51 -
 IntelSiliconPkg/PlatformVTdInfoSamplePei/PlatformVTdInfoSamplePei.uni                  |   20 -
 IntelSiliconPkg/PlatformVTdInfoSamplePei/PlatformVTdInfoSamplePeiExtra.uni             |   20 -
 IntelSiliconPkg/PlatformVTdSampleDxe/PlatformVTdSampleDxe.c                            |  409 --------
 IntelSiliconPkg/PlatformVTdSampleDxe/PlatformVTdSampleDxe.inf                          |   59 --
 IntelSiliconPkg/PlatformVTdSampleDxe/PlatformVTdSampleDxe.uni                          |   20 -
 IntelSiliconPkg/PlatformVTdSampleDxe/PlatformVTdSampleDxeExtra.uni                     |   20 -
 53 files changed, 6890 insertions(+), 6890 deletions(-)  create mode 100644 IntelSiliconPkg/Feature/VTd/IntelVTdDxe/BmDma.c
 create mode 100644 IntelSiliconPkg/Feature/VTd/IntelVTdDxe/DmaProtection.c
 create mode 100644 IntelSiliconPkg/Feature/VTd/IntelVTdDxe/DmaProtection.h
 create mode 100644 IntelSiliconPkg/Feature/VTd/IntelVTdDxe/DmarAcpiTable.c
 create mode 100644 IntelSiliconPkg/Feature/VTd/IntelVTdDxe/IntelVTdDxe.c
 create mode 100644 IntelSiliconPkg/Feature/VTd/IntelVTdDxe/IntelVTdDxe.inf
 create mode 100644 IntelSiliconPkg/Feature/VTd/IntelVTdDxe/IntelVTdDxe.uni
 create mode 100644 IntelSiliconPkg/Feature/VTd/IntelVTdDxe/IntelVTdDxeExtra.uni
 create mode 100644 IntelSiliconPkg/Feature/VTd/IntelVTdDxe/PciInfo.c
 create mode 100644 IntelSiliconPkg/Feature/VTd/IntelVTdDxe/TranslationTable.c
 create mode 100644 IntelSiliconPkg/Feature/VTd/IntelVTdDxe/TranslationTableEx.c
 create mode 100644 IntelSiliconPkg/Feature/VTd/IntelVTdDxe/VtdReg.c
 create mode 100644 IntelSiliconPkg/Feature/VTd/IntelVTdPmrPei/IntelVTdPmr.c
 create mode 100644 IntelSiliconPkg/Feature/VTd/IntelVTdPmrPei/IntelVTdPmrPei.c
 create mode 100644 IntelSiliconPkg/Feature/VTd/IntelVTdPmrPei/IntelVTdPmrPei.h
 create mode 100644 IntelSiliconPkg/Feature/VTd/IntelVTdPmrPei/IntelVTdPmrPei.inf
 create mode 100644 IntelSiliconPkg/Feature/VTd/IntelVTdPmrPei/IntelVTdPmrPei.uni
 create mode 100644 IntelSiliconPkg/Feature/VTd/IntelVTdPmrPei/IntelVTdPmrPeiExtra.uni
 create mode 100644 IntelSiliconPkg/Feature/VTd/PlatformVTdInfoSamplePei/PlatformVTdInfoSamplePei.c
 create mode 100644 IntelSiliconPkg/Feature/VTd/PlatformVTdInfoSamplePei/PlatformVTdInfoSamplePei.inf
 create mode 100644 IntelSiliconPkg/Feature/VTd/PlatformVTdInfoSamplePei/PlatformVTdInfoSamplePei.uni
 create mode 100644 IntelSiliconPkg/Feature/VTd/PlatformVTdInfoSamplePei/PlatformVTdInfoSamplePeiExtra.uni
 create mode 100644 IntelSiliconPkg/Feature/VTd/PlatformVTdSampleDxe/PlatformVTdSampleDxe.c
 create mode 100644 IntelSiliconPkg/Feature/VTd/PlatformVTdSampleDxe/PlatformVTdSampleDxe.inf
 create mode 100644 IntelSiliconPkg/Feature/VTd/PlatformVTdSampleDxe/PlatformVTdSampleDxe.uni
 create mode 100644 IntelSiliconPkg/Feature/VTd/PlatformVTdSampleDxe/PlatformVTdSampleDxeExtra.uni
 delete mode 100644 IntelSiliconPkg/IntelVTdDxe/BmDma.c
 delete mode 100644 IntelSiliconPkg/IntelVTdDxe/DmaProtection.c
 delete mode 100644 IntelSiliconPkg/IntelVTdDxe/DmaProtection.h
 delete mode 100644 IntelSiliconPkg/IntelVTdDxe/DmarAcpiTable.c
 delete mode 100644 IntelSiliconPkg/IntelVTdDxe/IntelVTdDxe.c
 delete mode 100644 IntelSiliconPkg/IntelVTdDxe/IntelVTdDxe.inf
 delete mode 100644 IntelSiliconPkg/IntelVTdDxe/IntelVTdDxe.uni
 delete mode 100644 IntelSiliconPkg/IntelVTdDxe/IntelVTdDxeExtra.uni
 delete mode 100644 IntelSiliconPkg/IntelVTdDxe/PciInfo.c
 delete mode 100644 IntelSiliconPkg/IntelVTdDxe/TranslationTable.c
 delete mode 100644 IntelSiliconPkg/IntelVTdDxe/TranslationTableEx.c
 delete mode 100644 IntelSiliconPkg/IntelVTdDxe/VtdReg.c
 delete mode 100644 IntelSiliconPkg/IntelVTdPmrPei/IntelVTdPmr.c
 delete mode 100644 IntelSiliconPkg/IntelVTdPmrPei/IntelVTdPmrPei.c
 delete mode 100644 IntelSiliconPkg/IntelVTdPmrPei/IntelVTdPmrPei.h
 delete mode 100644 IntelSiliconPkg/IntelVTdPmrPei/IntelVTdPmrPei.inf
 delete mode 100644 IntelSiliconPkg/IntelVTdPmrPei/IntelVTdPmrPei.uni
 delete mode 100644 IntelSiliconPkg/IntelVTdPmrPei/IntelVTdPmrPeiExtra.uni
 delete mode 100644 IntelSiliconPkg/PlatformVTdInfoSamplePei/PlatformVTdInfoSamplePei.c
 delete mode 100644 IntelSiliconPkg/PlatformVTdInfoSamplePei/PlatformVTdInfoSamplePei.inf
 delete mode 100644 IntelSiliconPkg/PlatformVTdInfoSamplePei/PlatformVTdInfoSamplePei.uni
 delete mode 100644 IntelSiliconPkg/PlatformVTdInfoSamplePei/PlatformVTdInfoSamplePeiExtra.uni
 delete mode 100644 IntelSiliconPkg/PlatformVTdSampleDxe/PlatformVTdSampleDxe.c
 delete mode 100644 IntelSiliconPkg/PlatformVTdSampleDxe/PlatformVTdSampleDxe.inf
 delete mode 100644 IntelSiliconPkg/PlatformVTdSampleDxe/PlatformVTdSampleDxe.uni
 delete mode 100644 IntelSiliconPkg/PlatformVTdSampleDxe/PlatformVTdSampleDxeExtra.uni

--
2.7.4.windows.1



      parent reply	other threads:[~2017-09-16 12:16 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-09-16  1:39 [PATCH 0/4] IntelSiliconPkg: move VTd related modules to feature dir Jiewen Yao
2017-09-16  1:39 ` [PATCH 1/4] IntelSiliconPkg/IntelVTdDxe: Move " Jiewen Yao
2017-09-16  1:39 ` [PATCH 2/4] IntelSiliconPkg/PlatformVTdSampleDxe: " Jiewen Yao
2017-09-16  1:39 ` [PATCH 3/4] IntelSiliconPkg/IntelVTdPmrPei: " Jiewen Yao
2017-09-16  1:39 ` [PATCH 4/4] IntelSiliconPkg/PlatformIntelVTdInfoSamplePei: " Jiewen Yao
2017-09-16 12:19 ` Zeng, Star [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=0C09AFA07DD0434D9E2A0C6AEB0483103B969F8B@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