Changeset 148 in xtideuniversalbios for trunk/XTIDE_Universal_BIOS/Src/Handlers/Int13h/AH23h_HFeatures.asm

Timestamp:

Mar 19, 2011, 8:09:41 PM (14 years ago)

Author:

Tomi Tilli

google:author:

aitotat

Message:

Changes to XTIDE Universal BIOS:

• INT 13h optimizations to save almost 100 bytes.

File:

• trunk/XTIDE_Universal_BIOS/Src/Handlers/Int13h/AH23h_HFeatures.asm (modified) (4 diffs)

trunk/XTIDE_Universal_BIOS/Src/Handlers/Int13h/AH23h_HFeatures.asm

@@ -1 @@
-; File name     :   AH23h_HFeatures.asm
-; Project name  :   IDE BIOS
-; Created date  :   28.12.2009
-; Last update   :   12.4.2010
-; Author        :   Tomi Tilli
+; Project name  :   XTIDE Universal BIOS
 ; Description   :   Int 13h function AH=23h,
 ;                   Set Controller Features Register.
@@ -15 +11 @@
 ; AH23h_HandlerForSetControllerFeatures
 ;   Parameters:
-;       AH:     Bios function 23h
+;       AL, CX: Same as in INTPACK
+;       DL:     Translated Drive number
+;       DS:DI:  Ptr to DPT (in RAMVARS segment)
+;       SS:BP:  Ptr to INTPACK
+;   Parameters on INTPACK in SS:BP:
 ;       AL:     Feature Number (parameter to Features Register = subcommand)
-;       DL:     Drive number (8xh)
-;   Parameters loaded by Int13h_Jump:
-;       DS:     RAMVARS segment
-;   Parameter registers are undocumented, specific for this BIOS:
+;   (Parameter registers are undocumented, there are specific for this BIOS):
 ;       BH:     Parameter to Sector Count Register (subcommand specific)
 ;       BL:     Parameter to Sector Number Register (subcommand specific)
 ;       CL:     Parameter to Low Cylinder Register (subcommand specific)
 ;       CH:     Parameter to High Cylinder Register (subcommand specific)
+;   Returns with INTPACK in SS:BP:
+;       AH:     Int 13h return status
+;       CF:     0 if succesfull, 1 if error
+;--------------------------------------------------------------------
+ALIGN JUMP_ALIGN
+AH23h_HandlerForSetControllerFeatures:
+%ifndef USE_186
+    call    AH23h_SetControllerFeatures
+    jmp     Int13h_ReturnFromHandlerAfterStoringErrorCodeFromAH
+%else
+    push    Int13h_ReturnFromHandlerAfterStoringErrorCodeFromAH
+    ; Fall through to AH23h_SetControllerFeatures
+%endif
+
+
+;--------------------------------------------------------------------
+; AH23h_SetControllerFeatures
+;   Parameters:
+;       AL:     Feature Number (parameter to Features Register = subcommand)
+;       BH:     Parameter to Sector Count Register (subcommand specific)
+;       BL:     Parameter to Sector Number Register (subcommand specific)
+;       CL:     Parameter to Low Cylinder Register (subcommand specific)
+;       CH:     Parameter to High Cylinder Register (subcommand specific)
+;       DS:DI:  Ptr to DPT (in RAMVARS segment)
 ;   Returns:
 ;       AH:     Int 13h return status
 ;       CF:     0 if succesfull, 1 if error
-;       IF:     1
 ;   Corrupts registers:
-;       Flags
+;       AX, BX, CX, DX, SI
 ;--------------------------------------------------------------------
 ALIGN JUMP_ALIGN
-AH23h_HandlerForSetControllerFeatures:
-    push    dx
-    push    cx
-    push    bx
-    push    ax
-    push    si
-
+AH23h_SetControllerFeatures:
     ; Backup AL and BH to SI
     mov     ah, bh
@@ -45 +59 @@
 
     ; Select Master or Slave and wait until ready
-    call    FindDPT_ForDriveNumber      ; DS:DI now points to DPT
     call    HDrvSel_SelectDriveAndDisableIRQ
-    jc      SHORT .Return               ; Return if error
+    jc      SHORT .ReturnWithErrorCodeInAH
 
     ; Output Feature Number
@@ -66 +79 @@
     call    HCommand_OutputSectorCountAndCommand
 
-    call    HStatus_WaitBsyDefTime      ; Wait until drive ready
-.Return:
-    pop     si
-    jmp     Int13h_PopXRegsAndReturn
+    jmp     HStatus_WaitBsyDefTime      ; Wait until drive ready
+.ReturnWithErrorCodeInAH:
+    ret