Thanks! It looks like the existing changes covered Platform/Marvell/Cn913xDb/Cn913xDbA.dsc and Armada7k8kCapsule isn't something that would be broken by my changes. -- Rebecca Cran On 12/19/21 03:32, Marcin Wojtas wrote: > Hi, > > czw., 16 gru 2021 o 23:09 Rebecca Cran napisał(a): >> On 12/16/21 6:16 AM, Ard Biesheuvel wrote: >>> There were some changes a while ago to support standalone MM in 32-bit >>> mode, but I don't think any of the Platform/ARM platforms implement >>> that. So the best fix here is to simply stop lying about ARM support. >>> >>> As for RdkQemu - I think it is time we just remove that. >> During the work I found that the following list of platforms didn't build: >> >> >> o Drivers/OptionRomPkg/OptionRomPkg.dsc for AARCH64 and ARM: due to OptionRomPkg/Library/FrameBufferBltLib/FrameBufferBltLib.inf. >> >> o Features/Ext4Pkg/Ext4Pkg.dsc for ARM: edk2-platforms/Features/Ext4Pkg/Ext4Dxe/Inode.c:341: undefined reference to `__aeabi_uidivmod') >> >> o Platform/ARM/SgiPkg/RdN1Edge/RdN1Edge.dsc for ARM >> >> o Platform/Hisilicon/{D03,D05,D06} for AARCH64: Platform/Hisilicon/D03/Library/OemMiscLib2P/BoardFeature2PHi1610.c:50:18: error: ‘EmHilink0Hccs1X8Width16’ undeclared here (not in a function)) >> >> o Silicon/Marvell/Armada7k8k/Armada7k8kCapsule.dsc for AARCH64 and ARM: error C0DE: Unknown fatal error when processing [Silicon/Marvell/Armada7k8k/Armada7k8kCapsule.dsc] >> > It is expected behavior. In order to succeed, proper -D FIRMWARE_IMAGE > and -D PLATFORM_NAME must be passed. > >> o Platform/ARM/SgiPkg/RdE1Edge/RdE1Edge.dsc for ARM. >> >> o Platform/ARM/SgiPkg/RdN1EdgeX2/RdN1EdgeX2.dsc for ARM. >> >> o Platform/ARM/SgiPkg/RdN2/RdN2.dsc for ARM. >> >> o Platform/ARM/SgiPkg/RdV1/RdV1.dsc for ARM. >> >> o Platform/ARM/SgiPkg/RdV1Mc/RdV1Mc.dsc for ARM. >> >> o Platform/ARM/SgiPkg/Sgi575/Sgi575.dsc for ARM. >> >> o Platform/Comcast/RDKQemu/RDKQemu.dsc for AARCH64 and ARM due to ACPI >> issue. >> >> o Platform/Marvell/Cn913xDb/Cn913xDbA.dsc for AARCH64 and ARM: tries to >> use literal $(PLATFORM_NAME). >> > The build is expected to fail - the user must explicitly pick the SoC > variant with -D flag, see the .dsc excerpt: > [Defines] > !if $(CN9130) > PLATFORM_NAME = Cn9130DbA > !elseif $(CN9131) > PLATFORM_NAME = Cn9131DbA > !elseif $(CN9132) > PLATFORM_NAME = Cn9132DbA > !endif > > Thanks, > Marcin > >> o Platform/Socionext/SynQuacerEvalBoard/SynQuacerEvalBoard.dsc for AARCH64 >> and ARM: VariablePolicyHelperLib issue. >> >> o Platform/SolidRun/Cn913xCEx7Eval/Cn913xCEx7Eval.dsc for AARCH64 and ARM: >> T91 not found. >> >> -- >> Rebecca Cran >>