Changeset 592 in xtideuniversalbios for trunk/XTIDE_Universal_BIOS/Inc

Timestamp:

Jun 25, 2018, 10:29:27 PM (7 years ago)

Author:

Krister Nordvall

Message:

Changes:

• The problem with NASM in the previous revision (r591) has been fixed.
• The colors used by the boot menu and hotkey bar can now be customized by selecting one of a number of pre-defined color themes. Suggestions for additional themes are more than welcome!
• Large builds are now 10 KB. Small builds are still 8 KB with the exception of the Tiny build which is now 4 KB. In other words, builds are now as small as possible to make it easier to combine them with other BIOSes.
• Added code to the library to improve drive error handling. XTIDECFG can now handle "Drive Not Ready" errors.
• Fixed a couple of potential bugs in AtaID.asm (AtaID_GetMaxPioModeToAXandMinCycleTimeToCX); 1) ATA1.bPioMode was treated as a WORD variable. 2) ATA2.bPIOSupp was assumed to be non-zero which would result in PIO mode 3 being returned if the assumption was wrong.
• Made the same changes in the equivalent function used by BIOSDRVS (DisplayPioModeInformationUsingAtaInfoFromDSBX in AtaInfo.asm).
• Fixed a bug from r587 in PDC20x30.asm in PDC20x30_GetMaxPioModeToALandMinPioCycleTimeToBX.
• Fixed a bug from r523 in XTIDECFG where Auto Configure would only set the IRQ on one IDE interface on AT-builds.
• XTIDECFG will now restore the default settings for the "Serial port virtual device" when reselecting it in the list of device types. This makes it behave consistently for all device types.
• The eAAM macro is now used regardless if USE_UNDOC_INTEL is defined or not because it is apparently supported on all processors including the NEC V20/V30 CPUs.
• Renamed the EXCLUDE_FROM_XTIDE_UNIVERSAL_BIOS define to EXCLUDE_FROM_XUB.
• Added a define to exclude unused library code from BIOSDRVS (EXCLUDE_FROM_BIOSDRVS). This makes it a lot smaller than in previous revisions.
• All unnecessary CLD-instructions are now under a new define 'CLD_NEEDED' which is only enabled for the BIOS. It is disabled for XTIDECFG and BIOSDRVS but can be enabled if needed by adding this define to the respective makefile. This change was made because these unnecessary instructions are wasteful and should never be needed. In fact, they only serve to hide bugs (in other peoples code) which I strongly believe should be avoided. I recommend people making their own BIOSes from source to not use this define as it's extremely unlikely to be needed.
• Updated the copyright info in SerDrive and changed an URL to point to the new site.
• Updated the copyright info and version number in BIOSDRVS.
• Updated the copyright info in XTIDECFG.
• Optimizations in general.

Location:

trunk/XTIDE_Universal_BIOS/Inc

Files:

• BootVars.inc (modified) (3 diffs)
• IdeIO.inc (modified) (1 diff)
• RomVars.inc (modified) (2 diffs)
• Version.inc (modified) (2 diffs)

trunk/XTIDE_Universal_BIOS/Inc/BootVars.inc

@@ -28 +28 @@
 ; Number of times to retry booting before accepting error
 BOOT_READ_RETRY_TIMES       EQU     3
-
-
 
 ; Pre-boot variables. These do not exist after successful boot to OS.
@@ -69 +67 @@
 %endif ; MODULE_HOTKEYS
 
+; MAX_HARD_DISK_NAME_LENGTH must be defined ahead of the DRVDETECTINFO structure to avoid problems with NASM
+MAX_HARD_DISK_NAME_LENGTH   EQU     30      ; Bytes reserved for drive name
+
 struc DRVDETECTINFO
     .StartOfDrvDetectInfo:
@@ -79 +80 @@
 endstruc
 
-; Boot Menu Information Table. These are generated for all XTIDE Universal
-; BIOS drives. Available only until boot is successful.
-MAX_HARD_DISK_NAME_LENGTH           EQU     30      ; Bytes reserved for drive name
 DPT_DRVDETECTINFO_SIZE_MULTIPLIER   EQU     DRVDETECTINFO_size / LARGEST_DPT_SIZE
 
-
 %ifndef CHECK_FOR_UNUSED_ENTRYPOINTS
-
 %if MAX_HARD_DISK_NAME_LENGTH % 2 <> 0
     %error "MAX_HARD_DISK_NAME_LENGTH needs to be a multiple of 2, memory is moved with word operations."
 %endif
-
-%if DRVDETECTINFO_size % LARGEST_DPT_SIZE <> 0
-    %error "DRVDETECTINFO's size must be an even multiple of DPT's size.  Add or remove padding at the bottom of DRVDETECTINFO to bring the two sizes into alignment.  As DRVDETECTINFO is only used at boot time, with plenty of memory to consume, it is OK to waste some space here."
-%endif
-
-%if DRVDETECTINFO.szDrvName <> 0
-    %error "DRVDETECTINFO.szDrvName is assumed to be the first member of struc DRVDETECTINFO, in BootMenuPrint_RefreshItem"
-%endif
-
 %endif
 

trunk/XTIDE_Universal_BIOS/Inc/IdeIO.inc

@@ -57 +57 @@
 %ifndef MODULE_8BIT_IDE ; Standard IDE controllers only
 
+    eMOVZX  bx, [di+DPT.bIdevarsOffset]
     %ifnidni %1, dx
         mov     dx, %1
     %endif
-    eMOVZX  bx, BYTE [di+DPT.bIdevarsOffset]
     add     dx, [cs:bx+IDEVARS.wControlBlockPort]
     out     dx, al

trunk/XTIDE_Universal_BIOS/Inc/RomVars.inc

@@ -35 +35 @@
     .wDisplayMode       resb    2   ; Display mode for boot menu
     .wBootTimeout       resb    2   ; Boot Menu selection timeout in system timer ticks
+    .pColorTheme        resb    2   ; Ptr to the color attribute struc used by the boot menu and hotkey bar
     .bIdeCnt            resb    1   ; Number of available IDE controllers
     .bBootDrv           resb    1   ; Default drive to boot from
@@ -182 +183 @@
 endstruc
 
-; Bit defines for DRVPARAMS.wFlags
+; Bit defines for DRVPARAMS.wFlags - these flags are accessed as bytes so changes here might require changes elsewhere
 MASK_DRVPARAMS_WRITECACHE       EQU (3<<0)  ; Bits 0...1, Drive internal write cache settings (must start at bit 0)
     DEFAULT_WRITE_CACHE             EQU 0   ; Must be 0

trunk/XTIDE_Universal_BIOS/Inc/Version.inc

@@ -20 +20 @@
 
 ; Flash signature revisions:
+; XTIDE206  Added support for Color Themes
 ; XTIDE205  Added DEVICE_8BIT_XTCF_PIO16_WITH_BIU_OFFLOAD after other XT-CF
 ;           PIO modes (prevents empty indexes in PIO jump tables)
@@ -48 +49 @@
 %define TITLE_STRING            TITLE_STRING_START, TITLE_STRING_END
 %define ROM_VERSION_STRING      "v2.0.0",BETA,"3+ (",__DATE__,")",NULL
-%define FLASH_SIGNATURE         "XTIDE205"  ; Do not terminate with NULL
+%define FLASH_SIGNATURE         "XTIDE206"  ; Do not terminate with NULL