Re: [edk2-platforms][PATCH v2 7/7] PhytiumCommonPkg: Pipeline: Resolving newly introduced dependency


Sami Mujawar
 

Hi Kun,

Thank you for this patch.

This change fixes the build break for Phytium platforms.

Reviewed-by: Sami Mujawar <sami.mujawar@...>

Regards,

Sami Mujawar

On 21/07/2022 09:07 pm, Kun Qin via groups.io wrote:
The new changes in SecureBootVariableLib brought in a new dependency
of PlatformPKProtectionLib.

This change added the new library instance from SecurityPkg to resolve
PhytiumCommonPkg platforms build.

Cc: Leif Lindholm <quic_llindhol@...>
Cc: Peng Xie <xiepeng@...>
Cc: Ling Jia <jialing@...>
Cc: Yiqi Shu <shuyiqi@...>

Signed-off-by: Kun Qin <kuqin12@...>
---

Notes:
v2:
- No review, no change.

Silicon/Phytium/PhytiumCommonPkg/PhytiumCommonPkg.dsc.inc | 1 +
1 file changed, 1 insertion(+)

diff --git a/Silicon/Phytium/PhytiumCommonPkg/PhytiumCommonPkg.dsc.inc b/Silicon/Phytium/PhytiumCommonPkg/PhytiumCommonPkg.dsc.inc
index 2565b8ac626a..42fe2a5c2246 100644
--- a/Silicon/Phytium/PhytiumCommonPkg/PhytiumCommonPkg.dsc.inc
+++ b/Silicon/Phytium/PhytiumCommonPkg/PhytiumCommonPkg.dsc.inc
@@ -83,6 +83,7 @@ [LibraryClasses.common]
SafeIntLib|MdePkg/Library/BaseSafeIntLib/BaseSafeIntLib.inf

SecureBootVariableLib|SecurityPkg/Library/SecureBootVariableLib/SecureBootVariableLib.inf

SecureBootVariableProvisionLib|SecurityPkg/Library/SecureBootVariableProvisionLib/SecureBootVariableProvisionLib.inf

+ PlatformPKProtectionLib|SecurityPkg/Library/PlatformPKProtectionLibVarPolicy/PlatformPKProtectionLibVarPolicy.inf


TimerLib|ArmPkg/Library/ArmArchTimerLib/ArmArchTimerLib.inf

TpmMeasurementLib|MdeModulePkg/Library/TpmMeasurementLibNull/TpmMeasurementLibNull.inf

Join devel@edk2.groups.io to automatically receive all group messages.