Changeset 181 in xtideuniversalbios for trunk/XTIDE_Universal_BIOS/Src/Device/Device.asm

Timestamp:

Nov 13, 2011, 3:38:40 PM (12 years ago)

Author:

krille_n_@…

google:author:

krille_n_@hotmail.com

Message:

Changes to all parts of the project:

• Size optimizations.
• Added a define (EXCLUDE_FROM_XTIDECFG) to exclude unused library code from XTIDECFG.
• Tried to minimize time spent with interrupts disabled.
• Some minor attempts to improve speed (reordering instructions etc).
• Tried to improve readability, did some cleanup and fixed some errors in comments.

File:

• trunk/XTIDE_Universal_BIOS/Src/Device/Device.asm (modified) (5 diffs)

trunk/XTIDE_Universal_BIOS/Src/Device/Device.asm

@@ -16 +16 @@
 ;       AX, BX, CX, DX
 ;--------------------------------------------------------------------
+%ifdef MODULE_SERIAL
 Device_FinalizeDPT:
-%ifdef MODULE_SERIAL
     test    BYTE [di+DPT.bFlagsHigh], FLGH_DPT_SERIAL_DEVICE
     jnz     SHORT .FinalizeDptForSerialPortDevice
+    jmp     IdeDPT_Finalize
+
+.FinalizeDptForSerialPortDevice:
+    jmp     SerialDPT_Finalize
+%else
+    Device_FinalizeDPT EQU IdeDPT_Finalize
 %endif
-    jmp     IdeDPT_Finalize
-%ifdef MODULE_SERIAL
-.FinalizeDptForSerialPortDevice:
-    jmp     SerialDPT_Finalize      
-%endif
+
 
 ;--------------------------------------------------------------------
@@ -37 +39 @@
 ;       AL, BX, CX, DX
 ;--------------------------------------------------------------------
+%ifdef MODULE_SERIAL
 Device_ResetMasterAndSlaveController:
-%ifdef MODULE_SERIAL
     test    BYTE [di+DPT.bFlagsHigh], FLGH_DPT_SERIAL_DEVICE
     jnz     SHORT ReturnSuccessForSerialPort
+    jmp     IdeCommand_ResetMasterAndSlaveController
+%else
+    Device_ResetMasterAndSlaveController EQU IdeCommand_ResetMasterAndSlaveController
 %endif
-    jmp     IdeCommand_ResetMasterAndSlaveController
 
 
@@ -58 +62 @@
 ;       AL, BL, CX, DX, SI, DI, ES
 ;--------------------------------------------------------------------
+%ifdef MODULE_SERIAL
 Device_IdentifyToBufferInESSIwithDriveSelectByteInBH:
-%ifdef MODULE_SERIAL
     cmp     BYTE [cs:bp+IDEVARS.bDevice], DEVICE_SERIAL_PORT
     je      SHORT .IdentifyDriveFromSerialPort
-%endif
     jmp     IdeCommand_IdentifyDeviceToBufferInESSIwithDriveSelectByteInBH
-%ifdef MODULE_SERIAL
+
 .IdentifyDriveFromSerialPort:
     jmp     SerialCommand_IdentifyDeviceToBufferInESSIwithDriveSelectByteInBH
+%else
+    Device_IdentifyToBufferInESSIwithDriveSelectByteInBH EQU IdeCommand_IdentifyDeviceToBufferInESSIwithDriveSelectByteInBH
 %endif
+
 
 ;--------------------------------------------------------------------
@@ -83 +89 @@
 ;       AL, BX, CX, DX, (ES:SI for data transfer commands)
 ;--------------------------------------------------------------------
+%ifdef MODULE_SERIAL
 ALIGN JUMP_ALIGN
 Device_OutputCommandWithParameters:
-%ifdef MODULE_SERIAL
     test    BYTE [di+DPT.bFlagsHigh], FLGH_DPT_SERIAL_DEVICE
     jnz     SHORT .OutputCommandToSerialPort
-%endif
     jmp     IdeCommand_OutputWithParameters
-%ifdef MODULE_SERIAL
+
 ALIGN JUMP_ALIGN
 .OutputCommandToSerialPort:
     jmp     SerialCommand_OutputWithParameters
+%else
+    Device_OutputCommandWithParameters EQU IdeCommand_OutputWithParameters
 %endif
+
 
 ;--------------------------------------------------------------------
@@ -107 +115 @@
 ;       AL, BX, CX, DX
 ;--------------------------------------------------------------------
+%ifdef MODULE_SERIAL
 ALIGN JUMP_ALIGN
 Device_SelectDrive:
-%ifdef MODULE_SERIAL
     test    BYTE [di+DPT.bFlagsHigh], FLGH_DPT_SERIAL_DEVICE
     jnz     SHORT ReturnSuccessForSerialPort
-%endif
     jmp     IdeCommand_SelectDrive
-%ifdef MODULE_SERIAL
+
 ReturnSuccessForSerialPort:
     xor     ax, ax
     ret
+%else
+    Device_SelectDrive EQU IdeCommand_SelectDrive
 %endif
+