[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v2 08/12] hw/block/fdc-isa: Implement relocation and toggling

From: BALATON Zoltan
Subject: Re: [PATCH v2 08/12] hw/block/fdc-isa: Implement relocation and toggling for TYPE_ISA_FDC
Date: Tue, 19 Dec 2023 01:09:23 +0100 (CET)

On Mon, 18 Dec 2023, Bernhard Beschow wrote:
Implement isa_fdc_set_{enabled,iobase} in order to implement relocation and
toggling of SuperI/O functions in the VIA south bridges without breaking

You may want to revise these commit messages. What toggling means is only defined in the last patch but I can't think of a better name for it other than spelling out enable/disable. It's probably also not relevant in this commit message to mention VIA south bridges as this is a generic function not specific to that usage only.


Signed-off-by: Bernhard Beschow <shentey@gmail.com>
include/hw/block/fdc.h |  3 +++
hw/block/fdc-isa.c     | 14 ++++++++++++++
2 files changed, 17 insertions(+)

diff --git a/include/hw/block/fdc.h b/include/hw/block/fdc.h
index 35248c0837..c367c5efea 100644
--- a/include/hw/block/fdc.h
+++ b/include/hw/block/fdc.h
@@ -14,6 +14,9 @@ void fdctrl_init_sysbus(qemu_irq irq, hwaddr mmio_base, 
DriveInfo **fds);
void sun4m_fdctrl_init(qemu_irq irq, hwaddr io_base,
                       DriveInfo **fds, qemu_irq *fdc_tc);

+void isa_fdc_set_iobase(ISADevice *fdc, hwaddr iobase);
+void isa_fdc_set_enabled(ISADevice *fdc, bool enabled);
FloppyDriveType isa_fdc_get_drive_type(ISADevice *fdc, int i);
int cmos_get_fd_drive_type(FloppyDriveType fd0);

diff --git a/hw/block/fdc-isa.c b/hw/block/fdc-isa.c
index b4c92b40b3..c989325de3 100644
--- a/hw/block/fdc-isa.c
+++ b/hw/block/fdc-isa.c
@@ -192,6 +192,20 @@ static Aml *build_fdinfo_aml(int idx, FloppyDriveType type)
    return dev;

+void isa_fdc_set_iobase(ISADevice *fdc, hwaddr iobase)
+    FDCtrlISABus *isa = ISA_FDC(fdc);
+    fdc->ioport_id = iobase;
+    isa->iobase = iobase;
+    portio_list_set_address(&isa->portio_list, isa->iobase);
+void isa_fdc_set_enabled(ISADevice *fdc, bool enabled)
+    portio_list_set_enabled(&ISA_FDC(fdc)->portio_list, enabled);
int cmos_get_fd_drive_type(FloppyDriveType fd0)
    int val;

reply via email to

[Prev in Thread] Current Thread [Next in Thread]