Collect superio functionality and its controlling config registers
handling in an abstract VIA_SUPERIO class that is a subclass of
ISA_SUPERIO and put vt82c686b specific parts in a subclass of this
abstract class.
Signed-off-by: BALATON Zoltan <balaton@eik.bme.hu>
---
hw/isa/vt82c686.c | 196 +++++++++++++++++++++++++-------------
include/hw/isa/vt82c686.h | 1 -
2 files changed, 132 insertions(+), 65 deletions(-)
diff --git a/hw/isa/vt82c686.c b/hw/isa/vt82c686.c
index 05d084f698..ede8f3b195 100644
--- a/hw/isa/vt82c686.c
+++ b/hw/isa/vt82c686.c
@@ -249,15 +249,80 @@ static const TypeInfo vt8231_pm_info = {
};
-typedef struct SuperIOConfig {
+#define TYPE_VIA_SUPERIO "via-superio"
+OBJECT_DECLARE_SIMPLE_TYPE(ViaSuperIOState, VIA_SUPERIO)
+
+struct ViaSuperIOState {
+ ISASuperIODevice superio;
uint8_t regs[0x100];
+ const MemoryRegionOps *io_ops;
MemoryRegion io;
-} SuperIOConfig;
+};
+
+static inline void via_superio_io_enable(ViaSuperIOState *s, bool enable)
+{
+ memory_region_set_enabled(&s->io, enable);
+}
+
+static void via_superio_realize(DeviceState *d, Error **errp)
+{
+ ViaSuperIOState *s = VIA_SUPERIO(d);
+ ISASuperIOClass *ic = ISA_SUPERIO_GET_CLASS(s);
+ Error *local_err = NULL;
+
+ assert(s->io_ops);
+ ic->parent_realize(d, &local_err);
+ if (local_err) {
+ error_propagate(errp, local_err);
+ return;
+ }
+ memory_region_init_io(&s->io, OBJECT(d), s->io_ops, s, "via-superio", 2);
+ memory_region_set_enabled(&s->io, false);
+ /* The floppy also uses 0x3f0 and 0x3f1 but this seems to work anyway */
+ memory_region_add_subregion(isa_address_space_io(ISA_DEVICE(s)), 0x3f0,
+ &s->io);
+}
+
+static uint64_t via_superio_cfg_read(void *opaque, hwaddr addr, unsigned size)
+{
+ ViaSuperIOState *sc = opaque;
+ uint8_t idx = sc->regs[0];
+ uint8_t val = sc->regs[idx];
+
+ if (addr == 0) {
+ return idx;
+ }
+ if (addr == 1 && idx == 0) {
+ val = 0; /* reading reg 0 where we store index value */
+ }
+ trace_via_superio_read(idx, val);
+ return val;
+}
+
+static void via_superio_class_init(ObjectClass *klass, void *data)
+{
+ DeviceClass *dc = DEVICE_CLASS(klass);
+ ISASuperIOClass *sc = ISA_SUPERIO_CLASS(klass);
+
+ sc->parent_realize = dc->realize;
+ dc->realize = via_superio_realize;
+}
+
+static const TypeInfo via_superio_info = {
+ .name = TYPE_VIA_SUPERIO,
+ .parent = TYPE_ISA_SUPERIO,
+ .instance_size = sizeof(ViaSuperIOState),
+ .class_size = sizeof(ISASuperIOClass),
+ .class_init = via_superio_class_init,
+ .abstract = true,
+};
+
+#define TYPE_VT82C686B_SUPERIO "vt82c686b-superio"
-static void superio_cfg_write(void *opaque, hwaddr addr, uint64_t data,
- unsigned size)
+static void vt82c686b_superio_cfg_write(void *opaque, hwaddr addr,
+ uint64_t data, unsigned size)
{
- SuperIOConfig *sc = opaque;
+ ViaSuperIOState *sc = opaque;
uint8_t idx = sc->regs[0];
if (addr == 0) { /* config index register */
@@ -288,25 +353,9 @@ static void superio_cfg_write(void *opaque, hwaddr addr, uint64_t data,
sc->regs[idx] = data;
}
-static uint64_t superio_cfg_read(void *opaque, hwaddr addr, unsigned size)
-{
- SuperIOConfig *sc = opaque;
- uint8_t idx = sc->regs[0];
- uint8_t val = sc->regs[idx];
-
- if (addr == 0) {
- return idx;
- }
- if (addr == 1 && idx == 0) {
- val = 0; /* reading reg 0 where we store index value */
- }
- trace_via_superio_read(idx, val);
- return val;
-}
-
-static const MemoryRegionOps superio_cfg_ops = {
- .read = superio_cfg_read,
- .write = superio_cfg_write,
+static const MemoryRegionOps vt82c686b_superio_cfg_ops = {
+ .read = via_superio_cfg_read,
+ .write = vt82c686b_superio_cfg_write,
.endianness = DEVICE_NATIVE_ENDIAN,
.impl = {
.min_access_size = 1,
@@ -314,13 +363,66 @@ static const MemoryRegionOps superio_cfg_ops = {
},
};
+static void vt82c686b_superio_reset(DeviceState *dev)
+{
+ ViaSuperIOState *s = VIA_SUPERIO(dev);
+
+ memset(s->regs, 0, sizeof(s->regs));
+ /* Device ID */
+ vt82c686b_superio_cfg_write(s, 0, 0xe0, 1);
+ vt82c686b_superio_cfg_write(s, 1, 0x3c, 1);
+ /* Function select - all disabled */
+ vt82c686b_superio_cfg_write(s, 0, 0xe2, 1);
+ vt82c686b_superio_cfg_write(s, 1, 0x03, 1);
+ /* Floppy ctrl base addr */
+ vt82c686b_superio_cfg_write(s, 0, 0xe3, 1);
+ vt82c686b_superio_cfg_write(s, 1, 0xfc, 1);
+ /* Parallel port base addr */
+ vt82c686b_superio_cfg_write(s, 0, 0xe6, 1);
+ vt82c686b_superio_cfg_write(s, 1, 0xde, 1);
+ /* Serial port 1 base addr */
+ vt82c686b_superio_cfg_write(s, 0, 0xe7, 1);
+ vt82c686b_superio_cfg_write(s, 1, 0xfe, 1);
+ /* Serial port 2 base addr */
+ vt82c686b_superio_cfg_write(s, 0, 0xe8, 1);
+ vt82c686b_superio_cfg_write(s, 1, 0xbe, 1);
+
+ vt82c686b_superio_cfg_write(s, 0, 0, 1);
+}
+
+static void vt82c686b_superio_init(Object *obj)
+{
+ VIA_SUPERIO(obj)->io_ops = &vt82c686b_superio_cfg_ops;
+}
+
+static void vt82c686b_superio_class_init(ObjectClass *klass, void *data)
+{
+ DeviceClass *dc = DEVICE_CLASS(klass);
+ ISASuperIOClass *sc = ISA_SUPERIO_CLASS(klass);
+
+ dc->reset = vt82c686b_superio_reset;
+ sc->serial.count = 2;
+ sc->parallel.count = 1;
+ sc->ide.count = 0; /* emulated by via-ide */
+ sc->floppy.count = 1;
+}
+
+static const TypeInfo vt82c686b_superio_info = {
+ .name = TYPE_VT82C686B_SUPERIO,
+ .parent = TYPE_VIA_SUPERIO,
+ .instance_size = sizeof(ViaSuperIOState),
+ .instance_init = vt82c686b_superio_init,
+ .class_size = sizeof(ISASuperIOClass),
+ .class_init = vt82c686b_superio_class_init,
+};
+
OBJECT_DECLARE_SIMPLE_TYPE(VT82C686BISAState, VT82C686B_ISA)
struct VT82C686BISAState {
PCIDevice dev;
qemu_irq cpu_intr;
- SuperIOConfig superio_cfg;
+ ViaSuperIOState *via_sio;
};
static void via_isa_request_i8259_irq(void *opaque, int irq, int level)
@@ -338,7 +440,7 @@ static void vt82c686b_write_config(PCIDevice *d, uint32_t addr,
pci_default_write_config(d, addr, val, len);
if (addr == 0x85) {
/* BIT(1): enable or disable superio config io ports */
- memory_region_set_enabled(&s->superio_cfg.io, val & BIT(1));
+ via_superio_io_enable(s->via_sio, val & BIT(1));
}
}
@@ -370,13 +472,6 @@ static void vt82c686b_isa_reset(DeviceState *dev)
pci_conf[0x5a] = 0x04; /* KBC/RTC Control*/
pci_conf[0x5f] = 0x04;
pci_conf[0x77] = 0x10; /* GPIO Control 1/2/3/4 */
-
- s->superio_cfg.regs[0xe0] = 0x3c; /* Device ID */
- s->superio_cfg.regs[0xe2] = 0x03; /* Function select */
- s->superio_cfg.regs[0xe3] = 0xfc; /* Floppy ctrl base addr */
- s->superio_cfg.regs[0xe6] = 0xde; /* Parallel port base addr */
- s->superio_cfg.regs[0xe7] = 0xfe; /* Serial port 1 base addr */
- s->superio_cfg.regs[0xe8] = 0xbe; /* Serial port 2 base addr */
}
static void vt82c686b_realize(PCIDevice *d, Error **errp)
@@ -394,7 +489,8 @@ static void vt82c686b_realize(PCIDevice *d, Error **errp)
isa_bus_irqs(isa_bus, i8259_init(isa_bus, *isa_irq));
i8254_pit_init(isa_bus, 0x40, 0, NULL);
i8257_dma_init(isa_bus, 0);
- isa_create_simple(isa_bus, TYPE_VT82C686B_SUPERIO);
+ s->via_sio = VIA_SUPERIO(isa_create_simple(isa_bus,
+ TYPE_VT82C686B_SUPERIO));
mc146818_rtc_init(isa_bus, 2000, NULL);
for (i = 0; i < PCI_CONFIG_HEADER_SIZE; i++) {
@@ -402,16 +498,6 @@ static void vt82c686b_realize(PCIDevice *d, Error **errp)
d->wmask[i] = 0;
}
}
-
- memory_region_init_io(&s->superio_cfg.io, OBJECT(d), &superio_cfg_ops,
- &s->superio_cfg, "superio_cfg", 2);
- memory_region_set_enabled(&s->superio_cfg.io, false);
- /*
- * The floppy also uses 0x3f0 and 0x3f1.
- * But we do not emulate a floppy, so just set it here.
- */
- memory_region_add_subregion(isa_bus->address_space_io, 0x3f0,
- &s->superio_cfg.io);
}
static void via_class_init(ObjectClass *klass, void *data)
@@ -447,32 +533,14 @@ static const TypeInfo via_info = {
};
-static void vt82c686b_superio_class_init(ObjectClass *klass, void *data)
-{
- ISASuperIOClass *sc = ISA_SUPERIO_CLASS(klass);
-
- sc->serial.count = 2;
- sc->parallel.count = 1;
- sc->ide.count = 0;
- sc->floppy.count = 1;
-}
-
-static const TypeInfo via_superio_info = {
- .name = TYPE_VT82C686B_SUPERIO,
- .parent = TYPE_ISA_SUPERIO,
- .instance_size = sizeof(ISASuperIODevice),
- .class_size = sizeof(ISASuperIOClass),
- .class_init = vt82c686b_superio_class_init,
-};
-
-
static void vt82c686b_register_types(void)
{
type_register_static(&via_pm_info);
type_register_static(&vt82c686b_pm_info);
type_register_static(&vt8231_pm_info);
- type_register_static(&via_info);
type_register_static(&via_superio_info);
+ type_register_static(&vt82c686b_superio_info);
+ type_register_static(&via_info);
}
type_init(vt82c686b_register_types)
diff --git a/include/hw/isa/vt82c686.h b/include/hw/isa/vt82c686.h
index 9b6d610e83..0692b9a527 100644
--- a/include/hw/isa/vt82c686.h
+++ b/include/hw/isa/vt82c686.h
@@ -2,7 +2,6 @@
#define HW_VT82C686_H
#define TYPE_VT82C686B_ISA "vt82c686b-isa"
-#define TYPE_VT82C686B_SUPERIO "vt82c686b-superio"
#define TYPE_VT82C686B_PM "vt82c686b-pm"
#define TYPE_VT8231_PM "vt8231-pm"
#define TYPE_VIA_AC97 "via-ac97"
--
2.21.4
On 16/03/2021 22:03, BALATON Zoltan wrote:
> Collect superio functionality and its controlling config registers
> handling in an abstract VIA_SUPERIO class that is a subclass of
> ISA_SUPERIO and put vt82c686b specific parts in a subclass of this
> abstract class.
>
> Signed-off-by: BALATON Zoltan <balaton@eik.bme.hu>
> ---
> hw/isa/vt82c686.c | 196 +++++++++++++++++++++++++-------------
> include/hw/isa/vt82c686.h | 1 -
> 2 files changed, 132 insertions(+), 65 deletions(-)
>
> diff --git a/hw/isa/vt82c686.c b/hw/isa/vt82c686.c
> index 05d084f698..ede8f3b195 100644
> --- a/hw/isa/vt82c686.c
> +++ b/hw/isa/vt82c686.c
> @@ -249,15 +249,80 @@ static const TypeInfo vt8231_pm_info = {
> };
>
>
> -typedef struct SuperIOConfig {
> +#define TYPE_VIA_SUPERIO "via-superio"
> +OBJECT_DECLARE_SIMPLE_TYPE(ViaSuperIOState, VIA_SUPERIO)
> +
> +struct ViaSuperIOState {
> + ISASuperIODevice superio;
> uint8_t regs[0x100];
> + const MemoryRegionOps *io_ops;
> MemoryRegion io;
> -} SuperIOConfig;
> +};
> +
> +static inline void via_superio_io_enable(ViaSuperIOState *s, bool enable)
> +{
> + memory_region_set_enabled(&s->io, enable);
> +}
> +
> +static void via_superio_realize(DeviceState *d, Error **errp)
> +{
> + ViaSuperIOState *s = VIA_SUPERIO(d);
> + ISASuperIOClass *ic = ISA_SUPERIO_GET_CLASS(s);
> + Error *local_err = NULL;
> +
> + assert(s->io_ops);
> + ic->parent_realize(d, &local_err);
> + if (local_err) {
> + error_propagate(errp, local_err);
> + return;
> + }
> + memory_region_init_io(&s->io, OBJECT(d), s->io_ops, s, "via-superio", 2);
> + memory_region_set_enabled(&s->io, false);
> + /* The floppy also uses 0x3f0 and 0x3f1 but this seems to work anyway */
> + memory_region_add_subregion(isa_address_space_io(ISA_DEVICE(s)), 0x3f0,
> + &s->io);
> +}
> +
> +static uint64_t via_superio_cfg_read(void *opaque, hwaddr addr, unsigned size)
> +{
> + ViaSuperIOState *sc = opaque;
> + uint8_t idx = sc->regs[0];
> + uint8_t val = sc->regs[idx];
> +
> + if (addr == 0) {
> + return idx;
> + }
> + if (addr == 1 && idx == 0) {
> + val = 0; /* reading reg 0 where we store index value */
> + }
> + trace_via_superio_read(idx, val);
> + return val;
> +}
> +
> +static void via_superio_class_init(ObjectClass *klass, void *data)
> +{
> + DeviceClass *dc = DEVICE_CLASS(klass);
> + ISASuperIOClass *sc = ISA_SUPERIO_CLASS(klass);
> +
> + sc->parent_realize = dc->realize;
> + dc->realize = via_superio_realize;
> +}
> +
> +static const TypeInfo via_superio_info = {
> + .name = TYPE_VIA_SUPERIO,
> + .parent = TYPE_ISA_SUPERIO,
> + .instance_size = sizeof(ViaSuperIOState),
> + .class_size = sizeof(ISASuperIOClass),
> + .class_init = via_superio_class_init,
> + .abstract = true,
> +};
> +
> +#define TYPE_VT82C686B_SUPERIO "vt82c686b-superio"
>
> -static void superio_cfg_write(void *opaque, hwaddr addr, uint64_t data,
> - unsigned size)
> +static void vt82c686b_superio_cfg_write(void *opaque, hwaddr addr,
> + uint64_t data, unsigned size)
> {
> - SuperIOConfig *sc = opaque;
> + ViaSuperIOState *sc = opaque;
> uint8_t idx = sc->regs[0];
>
> if (addr == 0) { /* config index register */
> @@ -288,25 +353,9 @@ static void superio_cfg_write(void *opaque, hwaddr addr, uint64_t data,
> sc->regs[idx] = data;
> }
>
> -static uint64_t superio_cfg_read(void *opaque, hwaddr addr, unsigned size)
> -{
> - SuperIOConfig *sc = opaque;
> - uint8_t idx = sc->regs[0];
> - uint8_t val = sc->regs[idx];
> -
> - if (addr == 0) {
> - return idx;
> - }
> - if (addr == 1 && idx == 0) {
> - val = 0; /* reading reg 0 where we store index value */
> - }
> - trace_via_superio_read(idx, val);
> - return val;
> -}
> -
> -static const MemoryRegionOps superio_cfg_ops = {
> - .read = superio_cfg_read,
> - .write = superio_cfg_write,
> +static const MemoryRegionOps vt82c686b_superio_cfg_ops = {
> + .read = via_superio_cfg_read,
> + .write = vt82c686b_superio_cfg_write,
> .endianness = DEVICE_NATIVE_ENDIAN,
> .impl = {
> .min_access_size = 1,
> @@ -314,13 +363,66 @@ static const MemoryRegionOps superio_cfg_ops = {
> },
> };
>
> +static void vt82c686b_superio_reset(DeviceState *dev)
> +{
> + ViaSuperIOState *s = VIA_SUPERIO(dev);
> +
> + memset(s->regs, 0, sizeof(s->regs));
> + /* Device ID */
> + vt82c686b_superio_cfg_write(s, 0, 0xe0, 1);
> + vt82c686b_superio_cfg_write(s, 1, 0x3c, 1);
> + /* Function select - all disabled */
> + vt82c686b_superio_cfg_write(s, 0, 0xe2, 1);
> + vt82c686b_superio_cfg_write(s, 1, 0x03, 1);
> + /* Floppy ctrl base addr */
> + vt82c686b_superio_cfg_write(s, 0, 0xe3, 1);
> + vt82c686b_superio_cfg_write(s, 1, 0xfc, 1);
> + /* Parallel port base addr */
> + vt82c686b_superio_cfg_write(s, 0, 0xe6, 1);
> + vt82c686b_superio_cfg_write(s, 1, 0xde, 1);
> + /* Serial port 1 base addr */
> + vt82c686b_superio_cfg_write(s, 0, 0xe7, 1);
> + vt82c686b_superio_cfg_write(s, 1, 0xfe, 1);
> + /* Serial port 2 base addr */
> + vt82c686b_superio_cfg_write(s, 0, 0xe8, 1);
> + vt82c686b_superio_cfg_write(s, 1, 0xbe, 1);
> +
> + vt82c686b_superio_cfg_write(s, 0, 0, 1);
> +}
I'd say that you need to add the base addresses of the floppy/parallel/serial devices
into the comment as otherwise it's impossible to validate that the configuration
you're writing here matches what is visible in "info mtree"/the SuperIO defaults
without having to dig through the datasheet.
> +static void vt82c686b_superio_init(Object *obj)
> +{
> + VIA_SUPERIO(obj)->io_ops = &vt82c686b_superio_cfg_ops;
> +}
> +
> +static void vt82c686b_superio_class_init(ObjectClass *klass, void *data)
> +{
> + DeviceClass *dc = DEVICE_CLASS(klass);
> + ISASuperIOClass *sc = ISA_SUPERIO_CLASS(klass);
> +
> + dc->reset = vt82c686b_superio_reset;
> + sc->serial.count = 2;
> + sc->parallel.count = 1;
> + sc->ide.count = 0; /* emulated by via-ide */
> + sc->floppy.count = 1;
> +}
> +
> +static const TypeInfo vt82c686b_superio_info = {
> + .name = TYPE_VT82C686B_SUPERIO,
> + .parent = TYPE_VIA_SUPERIO,
> + .instance_size = sizeof(ViaSuperIOState),
> + .instance_init = vt82c686b_superio_init,
> + .class_size = sizeof(ISASuperIOClass),
> + .class_init = vt82c686b_superio_class_init,
> +};
> +
>
> OBJECT_DECLARE_SIMPLE_TYPE(VT82C686BISAState, VT82C686B_ISA)
>
> struct VT82C686BISAState {
> PCIDevice dev;
> qemu_irq cpu_intr;
> - SuperIOConfig superio_cfg;
> + ViaSuperIOState *via_sio;
> };
>
> static void via_isa_request_i8259_irq(void *opaque, int irq, int level)
> @@ -338,7 +440,7 @@ static void vt82c686b_write_config(PCIDevice *d, uint32_t addr,
> pci_default_write_config(d, addr, val, len);
> if (addr == 0x85) {
> /* BIT(1): enable or disable superio config io ports */
> - memory_region_set_enabled(&s->superio_cfg.io, val & BIT(1));
> + via_superio_io_enable(s->via_sio, val & BIT(1));
> }
> }
>
> @@ -370,13 +472,6 @@ static void vt82c686b_isa_reset(DeviceState *dev)
> pci_conf[0x5a] = 0x04; /* KBC/RTC Control*/
> pci_conf[0x5f] = 0x04;
> pci_conf[0x77] = 0x10; /* GPIO Control 1/2/3/4 */
> -
> - s->superio_cfg.regs[0xe0] = 0x3c; /* Device ID */
> - s->superio_cfg.regs[0xe2] = 0x03; /* Function select */
> - s->superio_cfg.regs[0xe3] = 0xfc; /* Floppy ctrl base addr */
> - s->superio_cfg.regs[0xe6] = 0xde; /* Parallel port base addr */
> - s->superio_cfg.regs[0xe7] = 0xfe; /* Serial port 1 base addr */
> - s->superio_cfg.regs[0xe8] = 0xbe; /* Serial port 2 base addr */
> }
>
> static void vt82c686b_realize(PCIDevice *d, Error **errp)
> @@ -394,7 +489,8 @@ static void vt82c686b_realize(PCIDevice *d, Error **errp)
> isa_bus_irqs(isa_bus, i8259_init(isa_bus, *isa_irq));
> i8254_pit_init(isa_bus, 0x40, 0, NULL);
> i8257_dma_init(isa_bus, 0);
> - isa_create_simple(isa_bus, TYPE_VT82C686B_SUPERIO);
> + s->via_sio = VIA_SUPERIO(isa_create_simple(isa_bus,
> + TYPE_VT82C686B_SUPERIO));
> mc146818_rtc_init(isa_bus, 2000, NULL);
>
> for (i = 0; i < PCI_CONFIG_HEADER_SIZE; i++) {
> @@ -402,16 +498,6 @@ static void vt82c686b_realize(PCIDevice *d, Error **errp)
> d->wmask[i] = 0;
> }
> }
> -
> - memory_region_init_io(&s->superio_cfg.io, OBJECT(d), &superio_cfg_ops,
> - &s->superio_cfg, "superio_cfg", 2);
> - memory_region_set_enabled(&s->superio_cfg.io, false);
> - /*
> - * The floppy also uses 0x3f0 and 0x3f1.
> - * But we do not emulate a floppy, so just set it here.
> - */
> - memory_region_add_subregion(isa_bus->address_space_io, 0x3f0,
> - &s->superio_cfg.io);
> }
>
> static void via_class_init(ObjectClass *klass, void *data)
> @@ -447,32 +533,14 @@ static const TypeInfo via_info = {
> };
>
>
> -static void vt82c686b_superio_class_init(ObjectClass *klass, void *data)
> -{
> - ISASuperIOClass *sc = ISA_SUPERIO_CLASS(klass);
> -
> - sc->serial.count = 2;
> - sc->parallel.count = 1;
> - sc->ide.count = 0;
> - sc->floppy.count = 1;
> -}
> -
> -static const TypeInfo via_superio_info = {
> - .name = TYPE_VT82C686B_SUPERIO,
> - .parent = TYPE_ISA_SUPERIO,
> - .instance_size = sizeof(ISASuperIODevice),
> - .class_size = sizeof(ISASuperIOClass),
> - .class_init = vt82c686b_superio_class_init,
> -};
> -
> -
> static void vt82c686b_register_types(void)
> {
> type_register_static(&via_pm_info);
> type_register_static(&vt82c686b_pm_info);
> type_register_static(&vt8231_pm_info);
> - type_register_static(&via_info);
> type_register_static(&via_superio_info);
> + type_register_static(&vt82c686b_superio_info);
> + type_register_static(&via_info);
> }
>
> type_init(vt82c686b_register_types)
> diff --git a/include/hw/isa/vt82c686.h b/include/hw/isa/vt82c686.h
> index 9b6d610e83..0692b9a527 100644
> --- a/include/hw/isa/vt82c686.h
> +++ b/include/hw/isa/vt82c686.h
> @@ -2,7 +2,6 @@
> #define HW_VT82C686_H
>
> #define TYPE_VT82C686B_ISA "vt82c686b-isa"
> -#define TYPE_VT82C686B_SUPERIO "vt82c686b-superio"
> #define TYPE_VT82C686B_PM "vt82c686b-pm"
> #define TYPE_VT8231_PM "vt8231-pm"
> #define TYPE_VIA_AC97 "via-ac97"
With the updated comments:
Reviewed-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
ATB,
Mark.
© 2016 - 2026 Red Hat, Inc.