Follow up EDKT247-253
authoryshang1 <yshang1@6f19259b-4bc3-4df7-8a09-765794883524>
Tue, 5 Sep 2006 05:41:32 +0000 (05:41 +0000)
committeryshang1 <yshang1@6f19259b-4bc3-4df7-8a09-765794883524>
Tue, 5 Sep 2006 05:41:32 +0000 (05:41 +0000)
git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@1449 6f19259b-4bc3-4df7-8a09-765794883524

EdkModulePkg/Library/EdkSerialPortLibNull/SerialPortLibNull.c
EdkModulePkg/Universal/StatusCode/Dxe/Common/StatusCode.dxs
EdkModulePkg/Universal/StatusCode/Dxe/DxeStatusCode.c
EdkModulePkg/Universal/StatusCode/Dxe/DxeStatusCode.h
EdkModulePkg/Universal/StatusCode/Dxe/Ipf/DxeStatusCodeIpf.c
EdkModulePkg/Universal/StatusCode/Dxe/Ipf/StatusCode.dxs

index 7832f37..1c42eb0 100644 (file)
@@ -50,6 +50,9 @@ SerialPortWrite (
   IN UINTN     NumberOfBytes\r
 )\r
 {\r
+  ASSERT (NULL != Buffer);\r
+  ASSERT (0 != NumberOfBytes);\r
+\r
   return 0;\r
 }\r
 \r
@@ -74,6 +77,9 @@ SerialPortRead (
   IN  UINTN     NumberOfBytes\r
 )\r
 {\r
+  ASSERT (NULL != Buffer);\r
+  ASSERT (0 != NumberOfBytes);\r
+\r
   return 0;\r
 }\r
 \r
index 6371258..9e0cc6b 100644 (file)
@@ -11,7 +11,7 @@
 # \r
 # Module Name:\r
 #\r
-#   Ia32StatusCode.dxs\r
+#   StatusCode.dxs\r
 #\r
 # Abstract:\r
 #\r
index c47d34b..27ea8dc 100644 (file)
@@ -137,7 +137,7 @@ InitializationDispatcherWorker (
       //\r
       // See whether there is gap of packet or not\r
       //\r
-      if (HobStart) {\r
+      if (NULL != HobStart) {\r
         HobStart  = NULL;\r
         Hob.Raw   = HobStart;\r
         continue;\r
index 72fe097..16fa3e8 100644 (file)
@@ -35,10 +35,10 @@ Abstract:
 // The point type always equal to PHYSICAL_MODE on IA32/X64/EBC architecture\r
 // Otherwise, VIRTUAL_MODE/PHYSICAL_MODE would be used on Ipf architecture, \r
 // \r
-enum {\r
+typedef enum {\r
   PHYSICAL_MODE,\r
   VIRTUAL_MODE\r
-};\r
+} PROCESSOR_MODE;\r
 \r
 typedef struct {\r
   UINTN                     Signature;\r
@@ -124,7 +124,8 @@ EfiSerialStatusCodeInitializeWorker (
 \r
   @param  Data          This optional parameter may be used to pass additional data\r
  \r
-  @return               The function always return EFI_SUCCESS.\r
+  @retval EFI_SUCCESS         Success to report status code to serial I/O.\r
+  @retval EFI_DEVICE_ERROR    EFI serial device can not work after ExitBootService() is called .\r
 \r
 **/\r
 EFI_STATUS\r
@@ -218,6 +219,7 @@ DataHubStatusCodeInitializeWorker (
   @param  Data          This optional parameter may be used to pass additional data\r
  \r
   @retval EFI_OUT_OF_RESOURCES   Can not acquire record buffer.\r
+  @retval EFI_DEVICE_ERROR       EFI serial device can not work after ExitBootService() is called .\r
   @retval EFI_SUCCESS            Success to cache status code and signal log data event.\r
 \r
 **/\r
index 47442a9..3838069 100644 (file)
@@ -1,5 +1,5 @@
 /** @file\r
-  Status code driver for IA32/X64/EBC architecture.\r
+* Status code driver for IPF architecture.\r
 \r
   Copyright (c) 2006, Intel Corporation                                                         \r
   All rights reserved. This program and the accompanying materials                          \r
@@ -76,8 +76,8 @@ ReportEsalServiceEntry (
     // Use atom operation to avoid the reentant of report.\r
     // If current status is not zero, then the function is reentrancy.\r
     //\r
-    if (InterlockedCompareExchange32 (&DxeStatusCode->StatusCodeNestStatus, 0, 1)) {\r
-      ReturnVal.Status = EFI_DEVICE_ERROR ;\r
+    if (1 == InterlockedCompareExchange32 (&DxeStatusCode->StatusCodeNestStatus, 0, 1)) {\r
+      ReturnVal.Status = EFI_DEVICE_ERROR;\r
       return ReturnVal;\r
     }\r
 \r
@@ -150,6 +150,8 @@ DxeStatusCodeDriverEntry (
   IN EFI_SYSTEM_TABLE  *SystemTable\r
   )\r
 {\r
+  EFI_STATUS Status;\r
+\r
   //\r
   // Dispatch initialization request to supported devices\r
   //\r
@@ -158,13 +160,14 @@ DxeStatusCodeDriverEntry (
   //\r
   // Initialize ESAL capabilities.\r
   //\r
-  RegisterEsalClass (\r
-    &gEfiExtendedSalStatusCodeServicesProtocolGuid,\r
-    &gDxeStatusCode,\r
-    ReportEsalServiceEntry,\r
-    StatusCode,\r
-    NULL\r
-    );\r
+  Status = RegisterEsalClass (\r
+             &gEfiExtendedSalStatusCodeServicesProtocolGuid,\r
+             &gDxeStatusCode,\r
+             ReportEsalServiceEntry,\r
+             StatusCode,\r
+             NULL\r
+             );\r
+  ASSERT_EFI_ERROR (Status);\r
 \r
   return EFI_SUCCESS;\r
 }\r
index aaa3efe..8aae3fc 100644 (file)
@@ -11,7 +11,7 @@
 # \r
 # Module Name:\r
 #\r
-#   IpfStatusCode.dxs\r
+#   StatusCode.dxs\r
 #\r
 # Abstract:\r
 #\r