- INITIAL_STALL_COUNT makes more sense in decimal (100) than hex (0x64) so define...

[reactos.git] / reactos / hal / halx86 / generic / beep.c

diff --git a/reactos/hal/halx86/generic/beep.c b/reactos/hal/halx86/generic/beep.c
index 2bfe8a6..4b82f2b 100644 (file)
--- a/reactos/hal/halx86/generic/beep.c
+++ b/reactos/hal/halx86/generic/beep.c
@@ -31,9 +31,9 @@ HalMakeBeep(IN ULONG Frequency)
 {
     UCHAR Data;
     ULONG Divider;
-    BOOLEAN Result = TRUE;
 
-    /* FIXME: Acquire CMOS Lock */
+    /* Acquire CMOS Lock */
+    HalpAcquireSystemHardwareSpinLock();
 
     /* Turn the register off */
     Data = READ_PORT_UCHAR(PORT_B);
@@ -49,8 +49,8 @@ HalMakeBeep(IN ULONG Frequency)
         if (Divider > 0x10000)
         {
             /* Fail */
-            Result = FALSE;
-            goto Cleanup;
+            HalpReleaseCmosSpinLock();
+            return FALSE;
         }
 
         /* Set timer divider */
@@ -62,11 +62,9 @@ HalMakeBeep(IN ULONG Frequency)
         WRITE_PORT_UCHAR(PORT_B, READ_PORT_UCHAR(PORT_B) | 0x03);
     }
 
-Cleanup:
-    /* FIXME: Release hardware lock */
+    /* Release CMOS lock */
+    HalpReleaseCmosSpinLock();
 
-    /* Return result */
-    return Result;
+    /* Return success */
+    return TRUE;
 }
-
-