On Thu, Mar 07, 2019 at 11:09:35AM +0100, Ard Biesheuvel wrote: > On Wed, 6 Mar 2019 at 16:37, Achin Gupta <achin.gu...@arm.com> wrote: > > > > On Wed, Mar 06, 2019 at 04:17:51PM +0100, Ard Biesheuvel wrote: > > > On Wed, 6 Mar 2019 at 16:16, Achin Gupta <achin.gu...@arm.com> wrote: > > > > > > > > Hi Ard, > > > > > > > > On Tue, Mar 05, 2019 at 02:32:40PM +0100, Ard Biesheuvel wrote: > > > > > The PCD PcdStandaloneMmEnable is unused, and shouldn't exist in the > > > > > first place since the value is implied by the context (it is never > > > > > valid to set it to FALSE for standalone MM or TRUE for traditional > > > > > MM). So drop it. > > > > > > > > This is being used to determine if the ArmVExpressPkg should include > > > > support for > > > > StMM comm. buffer or not [1] but it does look redundant now. > > > > > > > > > > If that is the case, the PCD should be defined in that package. > > > > The Arm FVP port for StMM needs a rewrite on the lines of other platforms. > > This > > change is fine. > > > > Yes, you are right. SynQuacer also needs some tweaks to align with > these changes, but I will post those separately. > > So with those changes merged, the only thing preventing us from > building the SynQuacer + MM platform from upstream sources is the > MmCommunicate VA vs PA issue. Is there any progress on that front?
I am looking at this now after my holiday. I have some questions that I will post separately. cheers, Achin > > Thanks, > Ard. _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel