Reviewed the code comments in the Include/Protocol directory for typos, grammar issue...

[mirror_edk2.git] / Nt32Pkg / CpuRuntimeDxe / Cpu.c

diff --git a/Nt32Pkg/CpuRuntimeDxe/Cpu.c b/Nt32Pkg/CpuRuntimeDxe/Cpu.c
index e538fc92d1c0cd17800f9fd51fae6dc19af6b6aa..d8d63d3da9408f7d813d7f16e96f2437e4f32a4f 100644 (file)
--- a/Nt32Pkg/CpuRuntimeDxe/Cpu.c
+++ b/Nt32Pkg/CpuRuntimeDxe/Cpu.c
@@ -1,4 +1,4 @@
-/*++\r
+/**@file\r
 \r
 Copyright (c) 2006 - 2007, Intel Corporation\r
 All rights reserved. This program and the accompanying materials\r
@@ -20,7 +20,7 @@ Abstract:
   the CPU-specific setjump/long pair.  Other services are not implemented\r
   in this driver.\r
 \r
---*/\r
+**/\r
 \r
 \r
 #include "CpuDriver.h"\r
@@ -254,9 +254,6 @@ Returns:
 // TODO:    This - add argument and description to function comment\r
 // TODO:    InitType - add argument and description to function comment\r
 {\r
-  CPU_ARCH_PROTOCOL_PRIVATE *Private;\r
-\r
-  Private = CPU_ARCH_PROTOCOL_PRIVATE_DATA_FROM_THIS (This);\r
   return EFI_UNSUPPORTED;\r
 }\r
 \r
@@ -291,7 +288,6 @@ Returns:
 // TODO:    InterruptType - add argument and description to function comment\r
 // TODO:    InterruptHandler - add argument and description to function comment\r
 {\r
-  CPU_ARCH_PROTOCOL_PRIVATE *Private;\r
 \r
   //\r
   // Do parameter checking for EFI spec conformance\r
@@ -302,7 +298,6 @@ Returns:
   //\r
   // Do nothing for Nt32 emulation\r
   //\r
-  Private = CPU_ARCH_PROTOCOL_PRIVATE_DATA_FROM_THIS (This);\r
   return EFI_UNSUPPORTED;\r
 }\r
 \r
@@ -381,8 +376,6 @@ Returns:
 // TODO:    Attributes - add argument and description to function comment\r
 // TODO:    EFI_INVALID_PARAMETER - add return value to function comment\r
 {\r
-  CPU_ARCH_PROTOCOL_PRIVATE *Private;\r
-\r
   //\r
   // Check for invalid parameter for Spec conformance\r
   //\r
@@ -393,7 +386,6 @@ Returns:
   //\r
   // Do nothing for Nt32 emulation\r
   //\r
-  Private = CPU_ARCH_PROTOCOL_PRIVATE_DATA_FROM_THIS (This);\r
   return EFI_UNSUPPORTED;\r
 }\r
 \r
@@ -426,7 +418,7 @@ Returns:
   //\r
   // Locate DataHub protocol.\r
   //\r
-  Status = gBS->LocateProtocol (&gEfiDataHubProtocolGuid, NULL, &DataHub);\r
+  Status = gBS->LocateProtocol (&gEfiDataHubProtocolGuid, NULL, (VOID**)&DataHub);\r
   if (EFI_ERROR (Status)) {\r
     return;\r
   }\r
@@ -445,8 +437,13 @@ Returns:
   //\r
   // Initialize strings to HII database\r
   //\r
-  HiiLibAddPackages (1, &gEfiProcessorProducerGuid, NULL, &HiiHandle, CpuStrings);\r
-  \r
+  HiiHandle = HiiAddPackages (\r
+                &gEfiProcessorProducerGuid,\r
+                NULL,\r
+                CpuStrings,\r
+                NULL\r
+                );\r
+  ASSERT (HiiHandle != NULL);\r
 \r
   CopyMem (RecordBuffer.Raw, &mCpuDataRecordHeader, HeaderSize);\r
 \r