From 5e365a97ecd92b49d17bf77e0d67bf88fff34197 Mon Sep 17 00:00:00 2001 From: Laszlo Ersek Date: Wed, 2 Aug 2017 12:03:27 +0200 Subject: [PATCH] OvmfPkg/IoMmuDxe: don't initialize local variables The edk2 coding style requires separate assignments. Cc: Ard Biesheuvel Cc: Brijesh Singh Cc: Jordan Justen Cc: Tom Lendacky Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Laszlo Ersek Tested-by: Brijesh Singh Reviewed-by: Brijesh Singh --- OvmfPkg/IoMmuDxe/IoMmuDxe.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/OvmfPkg/IoMmuDxe/IoMmuDxe.c b/OvmfPkg/IoMmuDxe/IoMmuDxe.c index 5809afc441..27b1856e0a 100644 --- a/OvmfPkg/IoMmuDxe/IoMmuDxe.c +++ b/OvmfPkg/IoMmuDxe/IoMmuDxe.c @@ -33,8 +33,10 @@ IoMmuDxeEntryPoint ( IN EFI_SYSTEM_TABLE *SystemTable ) { - EFI_STATUS Status = EFI_SUCCESS; - EFI_HANDLE Handle = NULL; + EFI_STATUS Status; + EFI_HANDLE Handle; + + Status = EFI_SUCCESS; // // When SEV is enabled, install IoMmu protocol otherwise install the @@ -43,6 +45,8 @@ IoMmuDxeEntryPoint ( if (MemEncryptSevIsEnabled ()) { AmdSevInstallIoMmuProtocol (); } else { + Handle = NULL; + Status = gBS->InstallMultipleProtocolInterfaces ( &Handle, &gIoMmuAbsentProtocolGuid,