mirror of
https://github.com/xemu-project/xemu.git
synced 2024-12-11 21:44:04 +00:00
d4c8fcd91a
4c3119a6e3
andcd9526ab7c
introduced an incorrect and inconsistent use of Chardev->be. Also, this CharBackend member is private and is not supposed to be accessible. Fix it by removing the inconsistent check. Cc: qemu-stable@nongnu.org Reported-by: Marc-André Lureau <marcandre.lureau@redhat.com> Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org> Message-Id: <20180515152500.19460-2-f4bug@amsat.org> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com> Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com>
214 lines
7.6 KiB
C
214 lines
7.6 KiB
C
/*
|
|
* Generic ISA Super I/O
|
|
*
|
|
* Copyright (c) 2010-2012 Herve Poussineau
|
|
* Copyright (c) 2011-2012 Andreas Färber
|
|
* Copyright (c) 2018 Philippe Mathieu-Daudé
|
|
*
|
|
* This code is licensed under the GNU GPLv2 and later.
|
|
* See the COPYING file in the top-level directory.
|
|
* SPDX-License-Identifier: GPL-2.0-or-later
|
|
*/
|
|
#include "qemu/osdep.h"
|
|
#include "qemu/error-report.h"
|
|
#include "qapi/error.h"
|
|
#include "sysemu/sysemu.h"
|
|
#include "sysemu/blockdev.h"
|
|
#include "chardev/char.h"
|
|
#include "hw/isa/superio.h"
|
|
#include "hw/input/i8042.h"
|
|
#include "hw/char/serial.h"
|
|
#include "trace.h"
|
|
|
|
static void isa_superio_realize(DeviceState *dev, Error **errp)
|
|
{
|
|
ISASuperIODevice *sio = ISA_SUPERIO(dev);
|
|
ISASuperIOClass *k = ISA_SUPERIO_GET_CLASS(sio);
|
|
ISABus *bus = isa_bus_from_device(ISA_DEVICE(dev));
|
|
ISADevice *isa;
|
|
DeviceState *d;
|
|
Chardev *chr;
|
|
DriveInfo *drive;
|
|
char *name;
|
|
int i;
|
|
|
|
/* Parallel port */
|
|
for (i = 0; i < k->parallel.count; i++) {
|
|
if (i >= ARRAY_SIZE(sio->parallel)) {
|
|
warn_report("superio: ignoring %td parallel controllers",
|
|
k->parallel.count - ARRAY_SIZE(sio->parallel));
|
|
break;
|
|
}
|
|
if (!k->parallel.is_enabled || k->parallel.is_enabled(sio, i)) {
|
|
/* FIXME use a qdev chardev prop instead of parallel_hds[] */
|
|
chr = parallel_hds[i];
|
|
if (chr == NULL) {
|
|
name = g_strdup_printf("discarding-parallel%d", i);
|
|
chr = qemu_chr_new(name, "null");
|
|
} else {
|
|
name = g_strdup_printf("parallel%d", i);
|
|
}
|
|
isa = isa_create(bus, "isa-parallel");
|
|
d = DEVICE(isa);
|
|
qdev_prop_set_uint32(d, "index", i);
|
|
if (k->parallel.get_iobase) {
|
|
qdev_prop_set_uint32(d, "iobase",
|
|
k->parallel.get_iobase(sio, i));
|
|
}
|
|
if (k->parallel.get_irq) {
|
|
qdev_prop_set_uint32(d, "irq", k->parallel.get_irq(sio, i));
|
|
}
|
|
qdev_prop_set_chr(d, "chardev", chr);
|
|
qdev_init_nofail(d);
|
|
sio->parallel[i] = isa;
|
|
trace_superio_create_parallel(i,
|
|
k->parallel.get_iobase ?
|
|
k->parallel.get_iobase(sio, i) : -1,
|
|
k->parallel.get_irq ?
|
|
k->parallel.get_irq(sio, i) : -1);
|
|
object_property_add_child(OBJECT(dev), name,
|
|
OBJECT(sio->parallel[i]), NULL);
|
|
g_free(name);
|
|
}
|
|
}
|
|
|
|
/* Serial */
|
|
for (i = 0; i < k->serial.count; i++) {
|
|
if (i >= ARRAY_SIZE(sio->serial)) {
|
|
warn_report("superio: ignoring %td serial controllers",
|
|
k->serial.count - ARRAY_SIZE(sio->serial));
|
|
break;
|
|
}
|
|
if (!k->serial.is_enabled || k->serial.is_enabled(sio, i)) {
|
|
/* FIXME use a qdev chardev prop instead of serial_hd() */
|
|
chr = serial_hd(i);
|
|
if (chr == NULL) {
|
|
name = g_strdup_printf("discarding-serial%d", i);
|
|
chr = qemu_chr_new(name, "null");
|
|
} else {
|
|
name = g_strdup_printf("serial%d", i);
|
|
}
|
|
isa = isa_create(bus, TYPE_ISA_SERIAL);
|
|
d = DEVICE(isa);
|
|
qdev_prop_set_uint32(d, "index", i);
|
|
if (k->serial.get_iobase) {
|
|
qdev_prop_set_uint32(d, "iobase",
|
|
k->serial.get_iobase(sio, i));
|
|
}
|
|
if (k->serial.get_irq) {
|
|
qdev_prop_set_uint32(d, "irq", k->serial.get_irq(sio, i));
|
|
}
|
|
qdev_prop_set_chr(d, "chardev", chr);
|
|
qdev_init_nofail(d);
|
|
sio->serial[i] = isa;
|
|
trace_superio_create_serial(i,
|
|
k->serial.get_iobase ?
|
|
k->serial.get_iobase(sio, i) : -1,
|
|
k->serial.get_irq ?
|
|
k->serial.get_irq(sio, i) : -1);
|
|
object_property_add_child(OBJECT(dev), name,
|
|
OBJECT(sio->serial[0]), NULL);
|
|
g_free(name);
|
|
}
|
|
}
|
|
|
|
/* Floppy disc */
|
|
if (!k->floppy.is_enabled || k->floppy.is_enabled(sio, 0)) {
|
|
isa = isa_create(bus, "isa-fdc");
|
|
d = DEVICE(isa);
|
|
if (k->floppy.get_iobase) {
|
|
qdev_prop_set_uint32(d, "iobase", k->floppy.get_iobase(sio, 0));
|
|
}
|
|
if (k->floppy.get_irq) {
|
|
qdev_prop_set_uint32(d, "irq", k->floppy.get_irq(sio, 0));
|
|
}
|
|
/* FIXME use a qdev drive property instead of drive_get() */
|
|
drive = drive_get(IF_FLOPPY, 0, 0);
|
|
if (drive != NULL) {
|
|
qdev_prop_set_drive(d, "driveA", blk_by_legacy_dinfo(drive),
|
|
&error_fatal);
|
|
}
|
|
/* FIXME use a qdev drive property instead of drive_get() */
|
|
drive = drive_get(IF_FLOPPY, 0, 1);
|
|
if (drive != NULL) {
|
|
qdev_prop_set_drive(d, "driveB", blk_by_legacy_dinfo(drive),
|
|
&error_fatal);
|
|
}
|
|
qdev_init_nofail(d);
|
|
sio->floppy = isa;
|
|
trace_superio_create_floppy(0,
|
|
k->floppy.get_iobase ?
|
|
k->floppy.get_iobase(sio, 0) : -1,
|
|
k->floppy.get_irq ?
|
|
k->floppy.get_irq(sio, 0) : -1);
|
|
}
|
|
|
|
/* Keyboard, mouse */
|
|
sio->kbc = isa_create_simple(bus, TYPE_I8042);
|
|
|
|
/* IDE */
|
|
if (k->ide.count && (!k->ide.is_enabled || k->ide.is_enabled(sio, 0))) {
|
|
isa = isa_create(bus, "isa-ide");
|
|
d = DEVICE(isa);
|
|
if (k->ide.get_iobase) {
|
|
qdev_prop_set_uint32(d, "iobase", k->ide.get_iobase(sio, 0));
|
|
}
|
|
if (k->ide.get_iobase) {
|
|
qdev_prop_set_uint32(d, "iobase2", k->ide.get_iobase(sio, 1));
|
|
}
|
|
if (k->ide.get_irq) {
|
|
qdev_prop_set_uint32(d, "irq", k->ide.get_irq(sio, 0));
|
|
}
|
|
qdev_init_nofail(d);
|
|
sio->ide = isa;
|
|
trace_superio_create_ide(0,
|
|
k->ide.get_iobase ?
|
|
k->ide.get_iobase(sio, 0) : -1,
|
|
k->ide.get_irq ?
|
|
k->ide.get_irq(sio, 0) : -1);
|
|
}
|
|
}
|
|
|
|
static void isa_superio_class_init(ObjectClass *oc, void *data)
|
|
{
|
|
DeviceClass *dc = DEVICE_CLASS(oc);
|
|
|
|
dc->realize = isa_superio_realize;
|
|
/* Reason: Uses parallel_hds[0] in realize(), so it can't be used twice */
|
|
dc->user_creatable = false;
|
|
}
|
|
|
|
static const TypeInfo isa_superio_type_info = {
|
|
.name = TYPE_ISA_SUPERIO,
|
|
.parent = TYPE_ISA_DEVICE,
|
|
.abstract = true,
|
|
.class_size = sizeof(ISASuperIOClass),
|
|
.class_init = isa_superio_class_init,
|
|
};
|
|
|
|
/* SMS FDC37M817 Super I/O */
|
|
static void fdc37m81x_class_init(ObjectClass *klass, void *data)
|
|
{
|
|
ISASuperIOClass *sc = ISA_SUPERIO_CLASS(klass);
|
|
|
|
sc->serial.count = 2; /* NS16C550A */
|
|
sc->parallel.count = 1;
|
|
sc->floppy.count = 1; /* SMSC 82077AA Compatible */
|
|
sc->ide.count = 0;
|
|
}
|
|
|
|
static const TypeInfo fdc37m81x_type_info = {
|
|
.name = TYPE_FDC37M81X_SUPERIO,
|
|
.parent = TYPE_ISA_SUPERIO,
|
|
.instance_size = sizeof(ISASuperIODevice),
|
|
.class_init = fdc37m81x_class_init,
|
|
};
|
|
|
|
static void isa_superio_register_types(void)
|
|
{
|
|
type_register_static(&isa_superio_type_info);
|
|
type_register_static(&fdc37m81x_type_info);
|
|
}
|
|
|
|
type_init(isa_superio_register_types)
|