Re: [edk2-devel] [RFC PATCH 1/1] ArmPkg,MdePkg: move ArmLib.h to MdePkg

["Leif Lindholm" <quic_llindhol@quicinc.com>]

Jiewen, can you clarify what you said back in
https://edk2.groups.io/g/devel/message/111551
?

On 2024-02-12 17:24, Pierre Gondois wrote:
>>> A ArmLibNull.inf library might also need to be created. If the
>>> OpensslLibFullAccel.inf module will depend on the ArmLib library,
>>> a Null implementation will be necessary for non-Arm architectures.
>>
>> Can ArmLib be declared under a [LibraryClasses.AArch64,
>> LibraryClasses.ARM]? Have I forgotten something that we discussed back
>> in ... November?
> 
>  From [1], it seems the MdePkg/CryptoPkg should build without a dependency
> on the ArmPkg. This is currently not really the case. cf. [2].
> 
> However, having a ArmLibNull implementation in the MdePkg would allow to
> avoid going in this direction when providing libraries to CryptoPkg.dsc.
>
> (Just in case, I think this ArmLibNull is a minor point.)

Well, sure, it is now.
Until we need a RiscV64LibNull, LoongarchLibNull, ...

> [1] https://edk2.groups.io/g/devel/message/111545
> [2] 
> https://github.com/tianocore/edk2/blob/8801c75b4d77c2e6e06b3ddc8560e0db590f6342/CryptoPkg/CryptoPkg.dsc#L117
> 
>>
>>> Otherwise I could apply and run the CryptoPkg/Arm native instructions
>>> patchset on top of this patch.
>>>
>>> ---
>>>
>>> As a side note, it also seems that:
>>> - ArmPkg/Include/Chipset/ArmCortexA5x.h
>>>     isn't used anymore in edk2/edk2-platorms
>>> - ArmPkg/Include/Chipset/ArmCortexA9.h
>>>     is barely used in edk2-platforms.
>>> Maybe the files should have been removed/simplified as part of
>>> - cffa7925a293 ("ArmPkg: remove ArmCpuLib header and implementations")
>>> - a913ad02479d ("ArmPlatformPkg: remove ArmVExpressPkg")
>>
>> I think you're right.
>> Well, ArmCortexA9.h is still *used*, but I can't imagine the Arm branch
>> of ArmVExpressLib has been build by anyone for some time. And surely the
>> inclusion of ArmVExpressLibSec in ArmVExpress-FVP-AArch64.dsc is
>> superfluous (such that that .inf can be deleted)?
> 
> The file could just be moved in the Library. I assume you/Sami/Ard
> will know more on the usage of the library itself,

Sami?

/
     Leif




-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#115368): https://edk2.groups.io/g/devel/message/115368
Mute This Topic: https://groups.io/mt/102731845/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-