ArmVirtPkg/ArmVirtTimerFdtClientLib: eliminate unchecked PcdSetXX() calls

These are deprecated / disabled under the
DISABLE_NEW_DEPRECATED_INTERFACES feature test macro.

Introduce a variable called PcdStatus, and use it to assert the success of
these operations (there is no reason for them to fail here).

Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=165
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Laszlo Ersek <lersek@redhat.com>
Reviewed-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Tested-by: Ard Biesheuvel <ard.biesheuvel@linaro.org> # RVCT
Reviewed-by: Jordan Justen <jordan.l.justen@intel.com>

1 files changed, 9 insertions, 4 deletions

diff --git a/ArmVirtPkg/Library/ArmVirtTimerFdtClientLib/ArmVirtTimerFdtClientLib.c b/ArmVirtPkg/Library/ArmVirtTimerFdtClientLib/ArmVirtTimerFdtClientLib.c
index 6e7461c6b2..c408386323 100644
--- a/ArmVirtPkg/Library/ArmVirtTimerFdtClientLib/ArmVirtTimerFdtClientLib.c
+++ b/ArmVirtPkg/Library/ArmVirtTimerFdtClientLib/ArmVirtTimerFdtClientLib.c
@@ -41,6 +41,7 @@ ArmVirtTimerFdtClientLibConstructor (
   CONST INTERRUPT_PROPERTY      *InterruptProp;

   UINT32                        PropSize;

   INT32                         SecIntrNum, IntrNum, VirtIntrNum, HypIntrNum;

+  RETURN_STATUS                 PcdStatus;

 

   Status = gBS->LocateProtocol (&gFdtClientProtocolGuid, NULL,

                   (VOID **)&FdtClient);

@@ -78,10 +79,14 @@ ArmVirtTimerFdtClientLibConstructor (
   DEBUG ((EFI_D_INFO, "Found Timer interrupts %d, %d, %d, %d\n",

     SecIntrNum, IntrNum, VirtIntrNum, HypIntrNum));

 

-  PcdSet32 (PcdArmArchTimerSecIntrNum, SecIntrNum);

-  PcdSet32 (PcdArmArchTimerIntrNum, IntrNum);

-  PcdSet32 (PcdArmArchTimerVirtIntrNum, VirtIntrNum);

-  PcdSet32 (PcdArmArchTimerHypIntrNum, HypIntrNum);

+  PcdStatus = PcdSet32S (PcdArmArchTimerSecIntrNum, SecIntrNum);

+  ASSERT_RETURN_ERROR (PcdStatus);

+  PcdStatus = PcdSet32S (PcdArmArchTimerIntrNum, IntrNum);

+  ASSERT_RETURN_ERROR (PcdStatus);

+  PcdStatus = PcdSet32S (PcdArmArchTimerVirtIntrNum, VirtIntrNum);

+  ASSERT_RETURN_ERROR (PcdStatus);

+  PcdStatus = PcdSet32S (PcdArmArchTimerHypIntrNum, HypIntrNum);

+  ASSERT_RETURN_ERROR (PcdStatus);

 

   return EFI_SUCCESS;

 }