[edk2-devel] [edk2-platforms][PATCH v1 1/1] IntelSiliconPkg: Add BaseSmmAccessLibNull

[Michael Kubacki]

Hi Ray,

It's been a while.

This change is really not that complicated. Can it be merged if there 
are no substantial opens?

Regards,
Michael

On 8/12/2021 10:16 PM, Michael Kubacki wrote:
> Sure.
> 
> Scenario #1:
> 
> MinPlatformPkg/PlatformInit/PlatformInitPei/PlatformInitPreMem.inf and 
> MinPlatormPkg/PlatformInit/PlatformInitPei/PlatformInitPostMem.inf both 
> link against an instance of BoardInitLib.
> 
> Many boards link against a single BoardInitLib instance. See example - 
> https://github.com/tianocore/edk2-platforms/blob/cd4e6b716c7d1bcde94035e7dce14b53a553e103/Platform/Intel/KabylakeOpenBoardPkg/KabylakeRvp3/OpenBoardPkg.dsc#L203
> 
> That BoardInitLib instance may link against SmmAccessLib. 
> PlatformInitPreMem may wish to library class override the SmmAccessLib 
> to the NULL instance while keeping it to non-NULL instance in 
> PlatformInitPostMem.
> 
> Scenario #2:
> 
> A PEIM is built that checks whether the boot mode is S3. If so, it calls 
> PeiInstallSmmAccessPpi(). A particular platform does not support S3, 
> therefore, it links BaseSmmAccessLibNull as its library instance for 
> SmmAccessLib.
> 


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#79498): https://edk2.groups.io/g/devel/message/79498
Mute This Topic: https://groups.io/mt/84769134/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-