vl: remove serial_max_hds
serial_hd(i) is NULL if and only if i >= serial_max_hds(). Test serial_hd(i) instead of bounding the loop at serial_max_hds(), thus removing one more function that vl.c is expected to export. Reviewed-by: Igor Mammedov <imammedo@redhat.com> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
This commit is contained in:
parent
bf4d4056fb
commit
46ee119fb6
@ -2878,10 +2878,8 @@ static void spapr_machine_init(MachineState *machine)
|
||||
/* Set up VIO bus */
|
||||
spapr->vio_bus = spapr_vio_bus_init();
|
||||
|
||||
for (i = 0; i < serial_max_hds(); i++) {
|
||||
if (serial_hd(i)) {
|
||||
spapr_vty_create(spapr->vio_bus, serial_hd(i));
|
||||
}
|
||||
for (i = 0; serial_hd(i); i++) {
|
||||
spapr_vty_create(spapr->vio_bus, serial_hd(i));
|
||||
}
|
||||
|
||||
/* We always have at least the nvram device on VIO */
|
||||
|
@ -71,10 +71,6 @@ void hmp_pcie_aer_inject_error(Monitor *mon, const QDict *qdict);
|
||||
|
||||
/* Return the Chardev for serial port i, or NULL if none */
|
||||
Chardev *serial_hd(int i);
|
||||
/* return the number of serial ports defined by the user. serial_hd(i)
|
||||
* will always return NULL for any i which is greater than or equal to this.
|
||||
*/
|
||||
int serial_max_hds(void);
|
||||
|
||||
/* parallel ports */
|
||||
|
||||
|
@ -2159,11 +2159,6 @@ Chardev *serial_hd(int i)
|
||||
return NULL;
|
||||
}
|
||||
|
||||
int serial_max_hds(void)
|
||||
{
|
||||
return num_serial_hds;
|
||||
}
|
||||
|
||||
static int parallel_parse(const char *devname)
|
||||
{
|
||||
static int index = 0;
|
||||
|
Loading…
Reference in New Issue
Block a user