|
From: | Marcel Apfelbaum |
Subject: | Re: [Qemu-devel] [PATCH] hw/q35: fix floppy controller definition in ich9 |
Date: | Mon, 01 Jun 2015 17:54:44 +0300 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.5.0 |
On 06/01/2015 05:45 PM, Paolo Bonzini wrote:
On 01/06/2015 16:13, Marcel Apfelbaum wrote:In DSDT FDC0 declares the IO region as IO(Decode16, 0x03F2, 0x03F2, 0x00, 0x04). Use the same in lpc_ich9 initialization code. Now the floppy drive is detected correctly on Windows. Signed-off-by: Marcel Apfelbaum <address@hidden> --- hw/isa/lpc_ich9.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hw/isa/lpc_ich9.c b/hw/isa/lpc_ich9.c index dba7585..bc9afc6 100644 --- a/hw/isa/lpc_ich9.c +++ b/hw/isa/lpc_ich9.c @@ -494,7 +494,7 @@ static void ich9_lpc_machine_ready(Notifier *n, void *opaque) /* lpt */ pci_conf[0x82] |= 0x04; } - if (memory_region_present(io_as, 0x3f0)) { + if (memory_region_present(io_as, 0x3f2)) { /* floppy */ pci_conf[0x82] |= 0x08; }Hi, I cc-ed the developers involved in the mail thread: https://lists.gnu.org/archive/html/qemu-block/2015-05/msg01043.html and the FDC maintainer. I hope I didn't miss anybody. I also have no knowledge of FDC, but I think the above patch is correct and solves the problem of floppy disk missing in Windows for Q35 machines.The patch is okay, but the correct definition of the area is from 0x3F1 to 0x3F7.
Can we do other fixes on top of this? This patch solves the actual Windows problem and now we can use floppies for Q35. Can I take this okay as a "Reviewed-by"? ) Thanks, Marcel
Paolo
[Prev in Thread] | Current Thread | [Next in Thread] |