Comments checked

[mirror_edk2.git] / MdeModulePkg / Universal / PcatRealTimeClockRuntimeDxe / PcRtc.c

diff --git a/MdeModulePkg/Universal/PcatRealTimeClockRuntimeDxe/PcRtc.c b/MdeModulePkg/Universal/PcatRealTimeClockRuntimeDxe/PcRtc.c
index 88faa0e654cc6bef76ad9d289d445e0b706247a9..6f875744bad8b36bfc815d7779b8db1968507d56 100644 (file)
--- a/MdeModulePkg/Universal/PcatRealTimeClockRuntimeDxe/PcRtc.c
+++ b/MdeModulePkg/Universal/PcatRealTimeClockRuntimeDxe/PcRtc.c
@@ -108,8 +108,7 @@ PcRtcInit (
   //\r
   // Acquire RTC Lock to make access to RTC atomic\r
   //\r
-  //BugBug: the EfiAtRuntime should be encapsulated in EfiAcquireLock or\r
-  //        provide a new instance for EfiAcquireLock, say, RtEfiAcquireLock\r
+  //Code here doesn't consider the runtime environment.\r
   if (!EfiAtRuntime ()) {\r
     EfiAcquireLock (&Global->RtcLock);\r
   }\r
@@ -143,8 +142,7 @@ PcRtcInit (
   //\r
   Status = RtcWaitToUpdate (PcdGet32 (PcdRealTimeClockUpdateTimeout));\r
   if (EFI_ERROR (Status)) {\r
-    //BugBug: the EfiAtRuntime should be encapsulated in EfiAcquireLock or\r
-    //        provide a new instance for EfiAcquireLock, say, RtEfiAcquireLock\r
+    //Code here doesn't consider the runtime environment.\r
     if (!EfiAtRuntime ()) {\r
       EfiReleaseLock (&Global->RtcLock);\r
     }\r
@@ -175,9 +173,7 @@ PcRtcInit (
   //\r
   // Release RTC Lock.\r
   //\r
-  //BugBug: the EfiAtRuntime should be encapsulated in EfiAcquireLock or\r
-  //        provide a new instance for EfiAcquireLock, say, RtEfiAcquireLock\r
-  //\r
+  //Code here doesn't consider the runtime environment.\r
   if (!EfiAtRuntime ()) {\r
     EfiReleaseLock (&Global->RtcLock);\r
   }\r
@@ -259,8 +255,7 @@ PcRtcGetTime (
   //\r
   // Acquire RTC Lock to make access to RTC atomic\r
   //\r
-  //BugBug: the EfiAtRuntime should be encapsulated in EfiAcquireLock or\r
-  //        provide a new instance for EfiAcquireLock, say, RtEfiAcquireLock\r
+  //Code here doesn't consider the runtime environment.\r
   if (!EfiAtRuntime ()) {\r
     EfiAcquireLock (&Global->RtcLock);\r
   }\r
@@ -269,8 +264,7 @@ PcRtcGetTime (
   //\r
   Status = RtcWaitToUpdate (PcdGet32 (PcdRealTimeClockUpdateTimeout));\r
   if (EFI_ERROR (Status)) {\r
-      //BugBug: the EfiAtRuntime should be encapsulated in EfiReleaseLock or\r
-      //        provide a new instance for EfiReleaseLock, say, RtEfiReleaseLock\r
+      //Code here doesn't consider the runtime environment.\r
       if (!EfiAtRuntime ()) {\r
         EfiReleaseLock (&Global->RtcLock);\r
       }\r
@@ -300,8 +294,7 @@ PcRtcGetTime (
   //\r
   // Release RTC Lock.\r
   //\r
-  //BugBug: the EfiAtRuntime should be encapsulated in EfiReleaseLock or\r
-  //        provide a new instance for EfiReleaseLock, say, RtEfiReleaseLock\r
+  //Code here doesn't consider the runtime environment.\r
   if (!EfiAtRuntime ()) {\r
     EfiReleaseLock (&Global->RtcLock);\r
   }\r
@@ -378,8 +371,7 @@ PcRtcSetTime (
   //\r
   // Acquire RTC Lock to make access to RTC atomic\r
   //\r
-  //BugBug: the EfiAtRuntime should be encapsulated in EfiAcquireLock or\r
-  //        provide a new instance for EfiAcquireLock, say, RtEfiAcquireLock\r
+  //Code here doesn't consider the runtime environment.\r
   if (!EfiAtRuntime ()) {\r
     EfiAcquireLock (&Global->RtcLock);\r
   }\r
@@ -388,8 +380,7 @@ PcRtcSetTime (
   //\r
   Status = RtcWaitToUpdate (PcdGet32 (PcdRealTimeClockUpdateTimeout));\r
   if (EFI_ERROR (Status)) {\r
-     //BugBug: the EfiAtRuntime should be encapsulated in EfiReleaseLock or\r
-     //        provide a new instance for EfiReleaseLock, say, RtEfiReleaseLock\r
+     //Code here doesn't consider the runtime environment.\r
      if (!EfiAtRuntime ()) {\r
        EfiReleaseLock (&Global->RtcLock);\r
      }\r
@@ -425,8 +416,7 @@ PcRtcSetTime (
   //\r
   // Release RTC Lock.\r
   //\r
-  //BugBug: the EfiAtRuntime should be encapsulated in EfiReleaseLock or\r
-  //        provide a new instance for EfiReleaseLock, say, RtEfiReleaseLock\r
+  //Code here doesn't consider the runtime environment.\r
   if (!EfiAtRuntime ()) {\r
     EfiReleaseLock (&Global->RtcLock);\r
   }\r
@@ -489,8 +479,7 @@ PcRtcGetWakeupTime (
   //\r
   // Acquire RTC Lock to make access to RTC atomic\r
   //\r
-  //BugBug: the EfiAtRuntime should be encapsulated in EfiAcquireLock or\r
-  //        provide a new instance for EfiAcquireLock, say, RtEfiAcquireLock\r
+  //Code here doesn't consider the runtime environment.\r
   if (!EfiAtRuntime ()) {\r
     EfiAcquireLock (&Global->RtcLock);\r
   }\r
@@ -499,8 +488,7 @@ PcRtcGetWakeupTime (
   //\r
   Status = RtcWaitToUpdate (PcdGet32 (PcdRealTimeClockUpdateTimeout));\r
   if (EFI_ERROR (Status)) {\r
-    //BugBug: the EfiAtRuntime should be encapsulated in EfiReleaseLock or\r
-    //        provide a new instance for EfiReleaseLock, say, RtEfiReleaseLock\r
+    //Code here doesn't consider the runtime environment.\r
     if (!EfiAtRuntime ()) {\r
     EfiReleaseLock (&Global->RtcLock);\r
     }\r
@@ -541,8 +529,7 @@ PcRtcGetWakeupTime (
   //\r
   // Release RTC Lock.\r
   //\r
-  //BugBug: the EfiAtRuntime should be encapsulated in EfiReleaseLock or\r
-  //        provide a new instance for EfiReleaseLock, say, RtEfiReleaseLock\r
+  //Code here doesn't consider the runtime environment.\r
   if (!EfiAtRuntime ()) {\r
     EfiReleaseLock (&Global->RtcLock);\r
   }\r
@@ -618,8 +605,7 @@ PcRtcSetWakeupTime (
   //\r
   // Acquire RTC Lock to make access to RTC atomic\r
   //\r
-  //BugBug: the EfiAtRuntime should be encapsulated in EfiAcquireLock or\r
-  //        provide a new instance for EfiAcquireLock, say, RtEfiAcquireLock\r
+  //Code here doesn't consider the runtime environment.\r
   if (!EfiAtRuntime ()) {\r
     EfiAcquireLock (&Global->RtcLock);\r
   }\r
@@ -628,8 +614,7 @@ PcRtcSetWakeupTime (
   //\r
   Status = RtcWaitToUpdate (PcdGet32 (PcdRealTimeClockUpdateTimeout));\r
   if (EFI_ERROR (Status)) {\r
-    //BugBug: the EfiAtRuntime should be encapsulated in EfiReleaseLock or\r
-    //        provide a new instance for EfiReleaseLock, say, RtEfiReleaseLock\r
+    //Code here doesn't consider the runtime environment.\r
     if (!EfiAtRuntime ()) {\r
     EfiReleaseLock (&Global->RtcLock);\r
     }\r
@@ -667,8 +652,7 @@ PcRtcSetWakeupTime (
   //\r
   // Release RTC Lock.\r
   //\r
-  //BugBug: the EfiAtRuntime should be encapsulated in EfiReleaseLock or\r
-  //        provide a new instance for EfiReleaseLock, say, RtEfiReleaseLock\r
+  //Code here doesn't consider the runtime environment.\r
   if (!EfiAtRuntime ()) {\r
     EfiReleaseLock (&Global->RtcLock);\r
   }\r