FDC save and restore commands

Bug #424453 reported by Benjamin David Lunt
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
QEMU
Fix Released
Undecided
Unassigned

Bug Description

I believe there is an error in fdc.c, with the command codes for SAVE and RESTORE. In the Intel 82078 (44pin) and the
Intel 82078SL (64pin), each have the command codes as 2Eh and 4Eh respectively. The current fdc.c file has
them as 2Ch and 4Ch.

fdc.c Line 368
    FD_CMD_SCAN_HIGH_OR_EQUAL = 0x1d,
- FD_CMD_SAVE = 0x2c,
+ FD_CMD_SAVE = 0x2e,
    FD_CMD_OPTION = 0x33,
- FD_CMD_RESTORE = 0x4c,
+ FD_CMD_RESTORE = 0x4e,

Please correct me if I am wrong.

Revision history for this message
Jes Sorensen (jes-sorensen) wrote :

Hi,

I pushed the patch upstream, commit bb350a5e9b961a8c3940b3451c329ff07c027aa1

Closing

Jes

Changed in qemu:
status: New → Fix Committed
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.