Changeset 617 in xtideuniversalbios for trunk/XTIDE_Universal_BIOS/Src/Device

Timestamp:

Aug 10, 2021, 7:27:30 AM (3 years ago)

Author:

Tomi Tilli

Message:

• Verify functions did not return correctly the number of succesfully verified sectors. That should be now fixed except for serial device. Serial device do not have error handling implemented yet.

Location:

trunk/XTIDE_Universal_BIOS/Src/Device

Files:

• Device.asm (modified) (1 diff)
• IDE/IdeCommand.asm (modified) (1 diff)

trunk/XTIDE_Universal_BIOS/Src/Device/Device.asm

@@ -164 +164 @@
 
 ;--------------------------------------------------------------------
+; Device_ReadLBAlowRegisterToAL
+; Returns LBA low register / Sector number register contents.
+; Note that this returns valid value only after transfer command (read/write/verify)
+; has stopped to an error. Do not call this otherwise.
+;   Parameters:
+;       DS:DI:  Ptr to DPT (in RAMVARS segment)
+;   Returns:
+;       AL:     Byte read from the device register
+;   Corrupts registers:
+;       BX, DX
+;--------------------------------------------------------------------
+;%ifdef MODULE_SERIAL   ; IDE + Serial
+;ALIGN JUMP_ALIGN
+;Device_ReadLBAlowRegisterToAL:
+;   test    BYTE [di+DPT.bFlagsHigh], FLGH_DPT_SERIAL_DEVICE
+;%ifdef USE_386
+;   jz      IdeCommand_ReadLBAlowRegisterToAL
+;   jmp     SerialCommand_ReadLBAlowRegisterToAL
+;%else
+;   jnz     SHORT .ReadFromSerialPort
+;   jmp     IdeCommand_ReadLBAlowRegisterToAL
+
+;ALIGN JUMP_ALIGN
+;.ReadFromSerialPort:
+;   jmp     SerialCommand_ReadLBAlowRegisterToAL
+;%endif
+
+;%else                  ; IDE only
+    Device_ReadLBAlowRegisterToAL       EQU     IdeCommand_ReadLBAlowRegisterToAL
+;%endif
+; TODO: For now we simply assume serial device do not produce verify errors
+
+
+;--------------------------------------------------------------------
 ; Device_SelectDrive
 ;   Parameters:

trunk/XTIDE_Universal_BIOS/Src/Device/IDE/IdeCommand.asm

@@ -288 +288 @@
     OUTPUT_AL_TO_IDE_REGISTER   LBA_HIGH_REGISTER
     ret
+
+
+;--------------------------------------------------------------------
+; IdeCommand_ReadLBAlowRegisterToAL
+; Returns LBA low register / Sector number register contents.
+; Note that this returns valid value only after transfer command (read/write/verify)
+; has stopped to an error. Do not call this otherwise.
+;   Parameters:
+;       DS:DI:  Ptr to DPT (in RAMVARS segment)
+;   Returns:
+;       AL:     Byte read from the register
+;   Corrupts registers:
+;       BX, DX
+;--------------------------------------------------------------------
+ALIGN JUMP_ALIGN
+IdeCommand_ReadLBAlowRegisterToAL:
+    ; HOB bit (defined in 48-bit address feature set) should be zero by default
+    ; so we get the correct value for CHS, LBA28 and LBA48 drives and commands
+    INPUT_TO_AL_FROM_IDE_REGISTER   LBA_LOW_REGISTER
+    ret