Re: [edk2-devel] [PATCH] MdePkg: Add MmAccess and MmControl definition.

2019-08-17 Thread Ni, Ray
m > Cc: Kinney, Michael D > Subject: RE: [edk2-devel] [PATCH] MdePkg: Add MmAccess and MmControl > definition. > > I don't have strong preference on removing SmmAccess.h from MdeModulePkg. > If people except me all agreed to keep that header file for compatibility, I >

Re: [edk2-devel] [PATCH] MdePkg: Add MmAccess and MmControl definition.

2019-08-17 Thread Ni, Ray
el@edk2.groups.io; > ler...@redhat.com; Ni, Ray > Cc: Kinney, Michael D > Subject: RE: [edk2-devel] [PATCH] MdePkg: Add MmAccess and MmControl > definition. > > Thanks Liming for pushing the code. > > Can we have a conclusion for the SmmAccess.h removal from MdeModulePkg? Wh

Re: [edk2-devel] [PATCH] MdePkg: Add MmAccess and MmControl definition.

2019-08-14 Thread Marc W Chen
: devel@edk2.groups.io; Gao, Liming ; Chen, Marc > W ; ler...@redhat.com; Ni, Ray > > Cc: Kinney, Michael D > Subject: RE: [edk2-devel] [PATCH] MdePkg: Add MmAccess and MmControl > definition. > > Push @6f33f7a262314af35e2b99c849e08928ea49aa55 > > >-Original

Re: [edk2-devel] [PATCH] MdePkg: Add MmAccess and MmControl definition.

2019-08-13 Thread Liming Gao
Cc: Kinney, Michael D >Subject: Re: [edk2-devel] [PATCH] MdePkg: Add MmAccess and MmControl >definition. > >I agree. The patch to add them in MdePkg is clear. I will push it next Monday. > >Thanks >Liming >> -Original Message- >> From: Chen, Marc W >>

Re: [edk2-devel] [PATCH] MdePkg: Add MmAccess and MmControl definition.

2019-08-09 Thread Liming Gao
chael D > Subject: RE: [edk2-devel] [PATCH] MdePkg: Add MmAccess and MmControl > definition. > > Hi Liming > > Since the SmmAccess.h of MdeModulePkg removal is still under discussion, can > we push the MmAccess.h and MmControl.h to MdePkg > first? I have other tasks pen

Re: [edk2-devel] [PATCH] MdePkg: Add MmAccess and MmControl definition.

2019-08-09 Thread Marc W Chen
devel@edk2.groups.io On Behalf Of Laszlo > Ersek > Sent: Thursday, August 8, 2019 12:11 AM > To: Ni, Ray ; Chen, Marc W ; > devel@edk2.groups.io; Gao, Liming > Cc: Kinney, Michael D > Subject: Re: [edk2-devel] [PATCH] MdePkg: Add MmAccess and MmControl > definition. >

Re: [edk2-devel] [PATCH] MdePkg: Add MmAccess and MmControl definition.

2019-08-07 Thread Laszlo Ersek
019 4:57 PM >> To: devel@edk2.groups.io; ler...@redhat.com; Ni, Ray ; >> Gao, Liming >> Cc: Kinney, Michael D >> Subject: RE: [edk2-devel] [PATCH] MdePkg: Add MmAccess and MmControl >> definition. >> >> Hi Liming and Ray >> >> Do you also agre

Re: [edk2-devel] [PATCH] MdePkg: Add MmAccess and MmControl definition.

2019-08-06 Thread Liming Gao
i, Ray >Sent: Tuesday, August 06, 2019 6:09 PM >To: Chen, Marc W ; devel@edk2.groups.io; >ler...@redhat.com; Gao, Liming >Cc: Kinney, Michael D >Subject: RE: [edk2-devel] [PATCH] MdePkg: Add MmAccess and MmControl >definition. > >Laszlo, >Do you want to avoid touching th

Re: [edk2-devel] [PATCH] MdePkg: Add MmAccess and MmControl definition.

2019-08-06 Thread Ni, Ray
oups.io; ler...@redhat.com; Ni, Ray ; > Gao, Liming > Cc: Kinney, Michael D > Subject: RE: [edk2-devel] [PATCH] MdePkg: Add MmAccess and MmControl > definition. > > Hi Liming and Ray > > Do you also agree? > > Thanks, > Marc > > > -Original Mes

Re: [edk2-devel] [PATCH] MdePkg: Add MmAccess and MmControl definition.

2019-08-06 Thread Marc W Chen
> Subject: Re: [edk2-devel] [PATCH] MdePkg: Add MmAccess and MmControl > definition. > > On 08/01/19 12:15, Marc W Chen wrote: > > Yes, my purpose is to avoid platform code update if the package is allowed > to use MdeModulePkg like OvmfPkg. > > For those packages that cannot

Re: [edk2-devel] [PATCH] MdePkg: Add MmAccess and MmControl definition.

2019-08-01 Thread Laszlo Ersek
--Original Message- >> From: Ni, Ray >> Sent: Thursday, August 1, 2019 6:04 PM >> To: Chen, Marc W ; Gao, Liming >> ; devel@edk2.groups.io >> Cc: Kinney, Michael D >> Subject: RE: [edk2-devel] [PATCH] MdePkg: Add MmAccess and MmControl >> definition. >

Re: [edk2-devel] [PATCH] MdePkg: Add MmAccess and MmControl definition.

2019-08-01 Thread Laszlo Ersek
ed as macros / typedefs on top of the new "MM" ones in MdePkg. Thanks Laszlo >> -Original Message- >> From: Gao, Liming >> Sent: Thursday, August 1, 2019 4:42 PM >> To: devel@edk2.groups.io; Chen, Marc W >> Cc: Kinney, Michael D ; Ni, Ray >> >&g

Re: [edk2-devel] [PATCH] MdePkg: Add MmAccess and MmControl definition.

2019-08-01 Thread Marc W Chen
2019 6:04 PM > To: Chen, Marc W ; Gao, Liming > ; devel@edk2.groups.io > Cc: Kinney, Michael D > Subject: RE: [edk2-devel] [PATCH] MdePkg: Add MmAccess and MmControl > definition. > > Marc, > Is the purpose to avoid platform code update with the wrapper header file in &g

Re: [edk2-devel] [PATCH] MdePkg: Add MmAccess and MmControl definition.

2019-08-01 Thread Ni, Ray
Original Message- > From: Chen, Marc W > Sent: Thursday, August 1, 2019 4:53 PM > To: Gao, Liming ; devel@edk2.groups.io > Cc: Kinney, Michael D ; Ni, Ray > > Subject: RE: [edk2-devel] [PATCH] MdePkg: Add MmAccess and MmControl > definition. > > Hi Liming > >

Re: [edk2-devel] [PATCH] MdePkg: Add MmAccess and MmControl definition.

2019-08-01 Thread Marc W Chen
August 1, 2019 4:48 PM > To: Gao, Liming ; devel@edk2.groups.io > Cc: Kinney, Michael D ; Ni, Ray > > Subject: RE: [edk2-devel] [PATCH] MdePkg: Add MmAccess and MmControl > definition. > > Hi Liming > > Since there are multiple packages are still using SmmAccess.h, we

Re: [edk2-devel] [PATCH] MdePkg: Add MmAccess and MmControl definition.

2019-08-01 Thread Marc W Chen
> Cc: Kinney, Michael D ; Ni, Ray > > Subject: RE: [edk2-devel] [PATCH] MdePkg: Add MmAccess and MmControl > definition. > > Marc: > The change is good. Reviewed-by: Liming Gao > > Besides, I see BZ also mention to remove the one in MdeModulePkg. Have > you

Re: [edk2-devel] [PATCH] MdePkg: Add MmAccess and MmControl definition.

2019-08-01 Thread Liming Gao
Marc: The change is good. Reviewed-by: Liming Gao Besides, I see BZ also mention to remove the one in MdeModulePkg. Have you the following patches for the change in MdeModulePkg? Thanks Liming >-Original Message- >From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of