Update EBC sub-dir of BaseLib according to code review comments.

git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@6198 6f19259b-4bc3-4df7-8a09-765794883524

diff --git a/MdePkg/Library/BaseLib/Ebc/CpuBreakpoint.c b/MdePkg/Library/BaseLib/Ebc/CpuBreakpoint.c
index 77f4b2a7d2c66da0d25e3b98f1836970ccaf08eb..837c204cf31cb1c96131a527106475a49cd00c37 100644 (file)
--- a/MdePkg/Library/BaseLib/Ebc/CpuBreakpoint.c
+++ b/MdePkg/Library/BaseLib/Ebc/CpuBreakpoint.c
@@ -1,7 +1,7 @@
 /** @file\r
   Base Library CPU Functions for EBC\r
 \r
-  Copyright (c) 2006, Intel Corporation<BR>\r
+  Copyright (c) 2006 - 2008, Intel Corporation<BR>\r
   All rights reserved. This program and the accompanying materials\r
   are licensed and made available under the terms and conditions of the BSD License\r
   which accompanies this distribution.  The full text of the license may be found at\r
@@ -54,8 +54,6 @@ MemoryFence (
 /**\r
   Disables CPU interrupts.\r
 \r
-  Disables CPU interrupts.\r
-\r
 **/\r
 VOID\r
 EFIAPI\r
@@ -69,8 +67,6 @@ DisableInterrupts (
 /**\r
   Enables CPU interrupts.\r
 \r
-  Enables CPU interrupts.\r
-\r
 **/\r
 VOID\r
 EFIAPI\r
@@ -105,9 +101,6 @@ GetInterruptState (
   Enables CPU interrupts for the smallest window required to capture any\r
   pending interrupts.\r
 \r
-  Enables CPU interrupts for the smallest window required to capture any\r
-  pending interrupts.\r
-\r
 **/\r
 VOID\r
 EFIAPI\r

diff --git a/MdePkg/Library/BaseLib/Ebc/SetJumpLongJump.c b/MdePkg/Library/BaseLib/Ebc/SetJumpLongJump.c
index a57d534ae13f3dc51294b7e371ea3b2d0b5876f5..f2001393f0c82834cbd2a91cb01b129424fb2347 100644 (file)
--- a/MdePkg/Library/BaseLib/Ebc/SetJumpLongJump.c
+++ b/MdePkg/Library/BaseLib/Ebc/SetJumpLongJump.c
@@ -1,7 +1,10 @@
 /** @file\r
-  Switch Stack functions.\r
+  Implementation of SetJump() and LongJump() on EBC.\r
+  \r
+  SetJump() and LongJump() are not currently supported for the EBC processor type.\r
+  Implementation for EBC just returns 0 for SetJump(), and ASSERT() for LongJump().\r
 \r
-  Copyright (c) 2006, Intel Corporation\r
+  Copyright (c) 2006 - 2008, Intel Corporation\r
   All rights reserved. This program and the accompanying materials\r
   are licensed and made available under the terms and conditions of the BSD License\r
   which accompanies this distribution.  The full text of the license may be found at\r
@@ -14,23 +17,6 @@
 \r
 #include <BaseLibInternals.h>\r
 \r
-/**\r
-  Worker function that checks ASSERT condition for JumpBuffer\r
-\r
-  Checks ASSERT condition for JumpBuffer.\r
-\r
-  If JumpBuffer is NULL, then ASSERT().\r
-  For IPF CPUs, if JumpBuffer is not aligned on a 16-byte boundary, then ASSERT().\r
-\r
-  @param  JumpBuffer    A pointer to CPU context buffer.\r
-\r
-**/\r
-VOID\r
-EFIAPI\r
-InternalAssertJumpBuffer (\r
-  IN      BASE_LIBRARY_JUMP_BUFFER  *JumpBuffer\r
-  );\r
-\r
 /**\r
   Saves the current CPU context that can be restored with a call to LongJump() and returns 0.\r
 \r
@@ -59,7 +45,7 @@ SetJump (
 \r
   Restores the CPU context from the buffer specified by JumpBuffer.\r
   This function never returns to the caller.\r
-  Instead is resumes execution based on the state of JumpBuffer.\r
+  Instead it resumes execution based on the state of JumpBuffer.\r
 \r
   @param  JumpBuffer    A pointer to CPU context buffer.\r
   @param  Value         The value to return when the SetJump() context is restored.\r