Changeset 618 in xtideuniversalbios for trunk/XTIDE_Universal_BIOS/Src/Initialization/AtaID.asm

Timestamp:

Aug 31, 2021, 2:17:47 PM (3 years ago)

Author:

krille_n_

Message:

Changes:

• Updated the BIOS makefile. Added the NO_ATAID_CORRECTION define back to the Tiny build as I've realized that the correction code should not be needed for builds without MODULE_EBIOS. Also added a new makefile target 'custom' to make it easier for people to make custom builds.
• Fixed a bug where calling INT 13h/AH=15h for drives not handled by XUB (floppy drives for example) would return an error due to the fact that any non-zero return value in AH from the other BIOS would cause the CF to be set in Int13h_SetErrorCodeToIntpackInSSBPfromAH. The return path is now via Int13h_ReturnFromHandlerWithoutStoringErrorCode which means that no status/error code will be returned in the BDA but that should not be a problem as the other BIOS should do that anyway. This change also fixed another potential problem where return values in DL from the other BIOS were assumed to be drive numbers when MODULE_SERIAL_FLOPPY is included in the build.
• Minor optimizations and fixes.

File:

• trunk/XTIDE_Universal_BIOS/Src/Initialization/AtaID.asm (modified) (1 diff)

trunk/XTIDE_Universal_BIOS/Src/Initialization/AtaID.asm

@@ -49 +49 @@
 
     ; Only correct cylinders since there are no reports that head or sectors could be wrong
-    MIN_U   WORD [es:si+ATA1.wCylCnt], MAX_PCHS_CYLINDERS       ; Limit to max allowed value
+    mov     cx, MAX_PCHS_CYLINDERS
+    MIN_U   [es:si+ATA1.wCylCnt], cx    ; Limit to max allowed value
 
     ; Note! There are ATA ID words 54-58 that also need to be modified! However,