Changeset 207 in xtideuniversalbios

Timestamp:

Jan 7, 2012, 7:57:57 PM (12 years ago)

Author:

gregli@…

google:author:

gregli@hotmail.com

Message:

Added buffer pointer denormalization check within the serial port code and reused the last portion of the NormalizeESSI and NormalizeESDI routines to save memory.

Location:

trunk

Files:

• Assembly_Library/Src/Util/Registers.asm (modified) (1 diff)
• XTIDE_Universal_BIOS/Src/Device/Serial/SerialCommand.asm (modified) (5 diffs)

trunk/Assembly_Library/Src/Util/Registers.asm

@@ -69 +69 @@
 ;   Corrupts registers:
 ;       AX, CX
+; 
+; Inline of NORMALIZE_FAR_POINTER so that we can share the last 2/3 of the 
+; routine with Registers_NormalizeFinish.
+; 
 ;--------------------------------------------------------------------
 ALIGN JUMP_ALIGN
 Registers_NormalizeESSI:
-    NORMALIZE_FAR_POINTER es, si, ax, cx
-    ret
+    mov         cx, si
+    and         si, byte 0fh
+    jmp         Registers_NormalizeFinish
 
 ALIGN JUMP_ALIGN
 Registers_NormalizeESDI:
-    NORMALIZE_FAR_POINTER es, di, ax, cx
+    mov         cx, di
+    and         di, byte 0fh
+;;; fall-through
+
+ALIGN JUMP_ALIGN        
+Registers_NormalizeFinish:      
+    eSHR_IM     cx, 4
+    mov         ax, es
+    add         ax, cx
+    mov         es, ax
     ret
-
+        
 
 ;--------------------------------------------------------------------

trunk/XTIDE_Universal_BIOS/Src/Device/Serial/SerialCommand.asm

@@ -177 +177 @@
         dec     dx
 
+;
+; Start off with a normalized buffer pointer
+; 
+        call    Registers_NormalizeESDI
+
 ;----------------------------------------------------------------------
 ;
@@ -183 +188 @@
 ; Sends first six bytes of IDEREGS_AND_INTPACK as the command
 ;
-        call    Registers_NormalizeESDI
-
         push    es              ; save off real buffer location
         push    di
@@ -323 +326 @@
         jnz     SerialCommand_OutputWithParameters_Error
 
+;
+; Normalize buffer pointer for next go round, if needed
+; 
+        test    di,di
+        jns     .clearBuffer
+        call    Registers_NormalizeESDI
 
 ;----------------------------------------------------------------------
@@ -332 +341 @@
 ; taken care of this, but I have seen cases where this is not true.
 ;
+        
 .clearBuffer:
         mov     dl,bh
@@ -344 +354 @@
 
 .clearBufferComplete:
+
         pop     ax              ; sector count and command byte
         dec     al              ; decrememnt sector count