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<rebe...@nuviainc.com>  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



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#85496): https://edk2.groups.io/g/devel/message/85496
Mute This Topic: https://groups.io/mt/87734393/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to