Re: [edk2-platforms PATCH 0/8] Update Arm platforms following addition of EFI_MP_SERVICES_PROTOCOL support in edk2

["Rebecca Cran" <quic_rcran@quicinc.com>]

[-- Attachment #1: Type: text/plain, Size: 2701 bytes --]

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<rebecca@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
>>

[-- Attachment #2: Type: text/html, Size: 4159 bytes --]