[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] hw/q35: fix floppy controller definition in ich
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH] hw/q35: fix floppy controller definition in ich9 |
Date: |
Mon, 01 Jun 2015 16:45:18 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 |
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.
Paolo