Hi Ard, Thank you for this patch.
The timeout value 5 appears to be standard across platforms, so it looks good to me. Reviewed-by: Sami Mujawar <[email protected]> Regards, Sami Mujawar On 02/02/2022, 09:42, "[email protected] on behalf of PierreGondois via groups.io" <[email protected] on behalf of [email protected]> wrote: Hi, The value also gets updated from 3 to 5, is it possible to note it in the commit message or to let it to 3 ? Otherwise the patch looks good to me, Reviewed-by: Pierre Gondois <[email protected]> On 2/1/22 7:03 PM, Ard Biesheuvel wrote: > Use the correct PCD type for PcdPlatformBootTimeOut so it gets wired up > to the Timeout EFI variable automatically, which is how the boot manager > stores the timeout preference. > > Signed-off-by: Ard Biesheuvel <[email protected]> > --- > ArmVirtPkg/ArmVirtKvmTool.dsc | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/ArmVirtPkg/ArmVirtKvmTool.dsc b/ArmVirtPkg/ArmVirtKvmTool.dsc > index a1bf4b9be87b..3bd3ebd6e0b3 100644 > --- a/ArmVirtPkg/ArmVirtKvmTool.dsc > +++ b/ArmVirtPkg/ArmVirtKvmTool.dsc > @@ -170,9 +170,10 @@ [PcdsPatchableInModule.common] > > gEfiMdeModulePkgTokenSpaceGuid.PcdSerialRegisterBase|0x0 > > +[PcdsDynamicHii] > + gEfiMdePkgTokenSpaceGuid.PcdPlatformBootTimeOut|L"Timeout"|gEfiGlobalVariableGuid|0x0|5 > + > [PcdsDynamicDefault.common] > - gEfiMdePkgTokenSpaceGuid.PcdPlatformBootTimeOut|3 > - > gArmTokenSpaceGuid.PcdArmArchTimerSecIntrNum|0x0 > gArmTokenSpaceGuid.PcdArmArchTimerIntrNum|0x0 > gArmTokenSpaceGuid.PcdArmArchTimerVirtIntrNum|0x0 > IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you. -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#86332): https://edk2.groups.io/g/devel/message/86332 Mute This Topic: https://groups.io/mt/88839170/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
