On 04/20/2018 11:52 AM, Peter Maydell wrote: > Currently the serial.c realize code has an explicit check that it is not > connected to a disconnected backend (ie one with a NULL chardev). > This isn't what we want -- you should be able to create a serial device > even if it isn't attached to anything. Remove the check. > > Signed-off-by: Peter Maydell <peter.mayd...@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <f4...@amsat.org> Tested-by: Philippe Mathieu-Daudé <f4...@amsat.org> > --- > hw/char/serial.c | 5 ----- > 1 file changed, 5 deletions(-) > > diff --git a/hw/char/serial.c b/hw/char/serial.c > index eb72191ee7..2c080c9862 100644 > --- a/hw/char/serial.c > +++ b/hw/char/serial.c > @@ -923,11 +923,6 @@ static int serial_be_change(void *opaque) > > void serial_realize_core(SerialState *s, Error **errp) > { > - if (!qemu_chr_fe_backend_connected(&s->chr)) { > - error_setg(errp, "Can't create serial device, empty char device"); > - return; > - } > - > s->modem_status_poll = timer_new_ns(QEMU_CLOCK_VIRTUAL, (QEMUTimerCB *) > serial_update_msl, s); > > s->fifo_timeout_timer = timer_new_ns(QEMU_CLOCK_VIRTUAL, (QEMUTimerCB *) > fifo_timeout_int, s); >