[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 5/7] hw/ide/piix: Use generic ide_init_ioport()
From: |
Philippe Mathieu-Daudé |
Subject: |
[PATCH 5/7] hw/ide/piix: Use generic ide_init_ioport() |
Date: |
Wed, 8 Feb 2023 01:07:41 +0100 |
TYPE_PIIX3_IDE is a PCI function inheriting from QOM
TYPE_PCI_DEVICE. To be able to call the ISA specific
ide_init_ioport_isa(), we call this function passing
a NULL ISADevice argument. Remove this hack by calling
the recently added generic ide_init_ioport(), which
doesn't expect any ISADevice.
Inspired-by: Bernhard Beschow <shentey@gmail.com>
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
---
hw/ide/piix.c | 10 ++++------
1 file changed, 4 insertions(+), 6 deletions(-)
diff --git a/hw/ide/piix.c b/hw/ide/piix.c
index a587541bb2..1cd4389611 100644
--- a/hw/ide/piix.c
+++ b/hw/ide/piix.c
@@ -136,15 +136,13 @@ static int pci_piix_init_ports(PCIIDEState *d)
{0x1f0, 0x3f6, 14},
{0x170, 0x376, 15},
};
- int i, ret;
+ int i;
for (i = 0; i < 2; i++) {
ide_bus_init(&d->bus[i], sizeof(d->bus[i]), DEVICE(d), i, 2);
- ret = ide_init_ioport_isa(&d->bus[i], NULL,
- port_info[i].iobase, port_info[i].iobase2);
- if (ret) {
- return ret;
- }
+ ide_init_ioport(&d->bus[i], OBJECT(d),
+ pci_address_space_io(PCI_DEVICE(d)),
+ port_info[i].iobase, port_info[i].iobase2);
ide_init2(&d->bus[i], isa_get_irq(NULL, port_info[i].isairq));
bmdma_init(&d->bus[i], &d->bmdma[i], d);
--
2.38.1
- Re: [PATCH 1/7] hw/isa: Un-inline isa_bus_from_device(), (continued)
[PATCH 5/7] hw/ide/piix: Use generic ide_init_ioport(),
Philippe Mathieu-Daudé <=
[PATCH 4/7] hw/ide: Introduce generic ide_init_ioport(), Philippe Mathieu-Daudé, 2023/02/07
[PATCH 7/7] hw/ide/piix: Remove dead code in pci_piix_init_ports(), Philippe Mathieu-Daudé, 2023/02/07
Re: [PATCH 0/7] hw/ide: Untangle ISA/PCI abuses of ide_init_ioport(), Philippe Mathieu-Daudé, 2023/02/07