From 4cb0344efeb4441034274b90aa90f424a2dd3216 Mon Sep 17 00:00:00 2001 From: xli24 Date: Wed, 16 Jul 2008 14:36:38 +0000 Subject: [PATCH] Refine code for BasePalCallLibNull & PeiPalCallLib. git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@5506 6f19259b-4bc3-4df7-8a09-765794883524 --- MdePkg/Library/BasePalCallLibNull/PalCall.c | 28 +++++---------- MdePkg/Library/PeiPalCallLib/PalCallLib.c | 40 +++++++++++---------- 2 files changed, 30 insertions(+), 38 deletions(-) diff --git a/MdePkg/Library/BasePalCallLibNull/PalCall.c b/MdePkg/Library/BasePalCallLibNull/PalCall.c index 74c33067ee..3772f70972 100644 --- a/MdePkg/Library/BasePalCallLibNull/PalCall.c +++ b/MdePkg/Library/BasePalCallLibNull/PalCall.c @@ -2,7 +2,7 @@ Template and Sample instance of PalCallLib. - Copyright (c) 2006 - 2007, Intel Corporation + Copyright (c) 2006 - 2008, Intel Corporation All rights reserved. This program and the accompanying materials are licensed and made available under the terms and conditions of the BSD License which accompanies this distribution. The full text of the license may be found at @@ -19,26 +19,16 @@ /** Makes a PAL procedure call. - This is a wrapper function to make a PAL procedure call. Based on the Index value, - this API will make static or stacked PAL call. Architected procedures may be designated - as required or optional. If a PAL procedure is specified as optional, a unique return - code of 0xFFFFFFFFFFFFFFFF is returned in the Status field of the PAL_CALL_RETURN structure. - This indicates that the procedure is not present in this PAL implementation. It is the - caller's responsibility to check for this return code after calling any optional PAL - procedure. No parameter checking is performed on the 4 input parameters, but there are - some common rules that the caller should follow when making a PAL call. Any address - passed to PAL as buffers for return parameters must be 8-byte aligned. Unaligned addresses - may cause undefined results. For those parameters defined as reserved or some fields - defined as reserved must be zero filled or the invalid argument return value may be - returned or undefined result may occur during the execution of the procedure. - This function is only available on IPF. + This is a wrapper function to make a PAL procedure call. + This is just a template as simple instance. It does not + make real PAL call. It directly reports error if called. - @param Index - The PAL procedure Index number. - @param Arg2 - The 2nd parameter for PAL procedure calls. - @param Arg3 - The 3rd parameter for PAL procedure calls. - @param Arg4 - The 4th parameter for PAL procedure calls. + @param Index The PAL procedure Index number. + @param Arg2 The 2nd parameter for PAL procedure calls. + @param Arg3 The 3rd parameter for PAL procedure calls. + @param Arg4 The 4th parameter for PAL procedure calls. - @return structure returned from the PAL Call procedure, including the status and return value. + @return Structure returned from the PAL Call procedure, including the status and return value. **/ PAL_CALL_RETURN diff --git a/MdePkg/Library/PeiPalCallLib/PalCallLib.c b/MdePkg/Library/PeiPalCallLib/PalCallLib.c index b1a9b10cb7..aa423c94bd 100644 --- a/MdePkg/Library/PeiPalCallLib/PalCallLib.c +++ b/MdePkg/Library/PeiPalCallLib/PalCallLib.c @@ -1,7 +1,7 @@ /** @file PAL Call Services Function. - Copyright (c) 2006 - 2007 Intel Corporation. All rights reserved + Copyright (c) 2006 - 2008 Intel Corporation. All rights reserved This software and associated documentation (if any) is furnished under a license and may only be used or copied in accordance with the terms of the license. Except as permitted by such @@ -26,8 +26,8 @@ #include /** - Makes a PAL procedure call. + This is a wrapper function to make a PAL procedure call. Based on the Index value, this API will make static or stacked PAL call. Architected procedures may be designated as required or optional. If a PAL procedure is specified as optional, a unique return @@ -42,12 +42,12 @@ returned or undefined result may occur during the execution of the procedure. This function is only available on IPF. - @param Index - The PAL procedure Index number. - @param Arg2 - The 2nd parameter for PAL procedure calls. - @param Arg3 - The 3rd parameter for PAL procedure calls. - @param Arg4 - The 4th parameter for PAL procedure calls. + @param Index The PAL procedure Index number. + @param Arg2 The 2nd parameter for PAL procedure calls. + @param Arg3 The 3rd parameter for PAL procedure calls. + @param Arg4 The 4th parameter for PAL procedure calls. - @return structure returned from the PAL Call procedure, including the status and return value. + @return Structure returned from the PAL Call procedure, including the status and return value. **/ PAL_CALL_RETURN @@ -68,28 +68,30 @@ PalCall ( UINT64 RecordSize; // - // Get Pei Service Table Pointer + // Get PEI Service Table Pointer // PeiServices = (CONST EFI_PEI_SERVICES **) GetPeiServicesTablePointer (); // - // Locate SEC Ppi + // Locate SEC Platform Information PPI // - Status = PeiServicesLocatePpi ( - &gEfiSecPlatformInformationPpiGuid, - 0, - NULL, - (VOID **)&SecPlatformPpi - ); + &gEfiSecPlatformInformationPpiGuid, + 0, + NULL, + (VOID **)&SecPlatformPpi + ); ASSERT_EFI_ERROR (Status); + // + // Retrieve PAL call address from platform information reported by the PPI + // RecordSize = sizeof (IpfStatus); SecPlatformPpi->PlatformInformation ( - PeiServices, - &RecordSize, - (EFI_SEC_PLATFORM_INFORMATION_RECORD *) &IpfStatus - ); + PeiServices, + &RecordSize, + (EFI_SEC_PLATFORM_INFORMATION_RECORD *) &IpfStatus + ); PalCallAddress = IpfStatus.PalCallAddress; ReturnVal = AsmPalCall (PalCallAddress, Index, Arg2, Arg3, Arg4);