[SeaBIOS] [PATCH 3/6] paravirt: read QEMU_CFG_NOGRAPHIC, store in etc/sercon-enable romfile

[Gerd Hoffmann]

  Hi,

> Separately, how will sercon get the serial port that it is to use?
> 
> Looking closer at patch 4 I see:
> 
> +    u16 addr = PORT_SERIAL1;
> +    SET_LOW(sercon_port, addr);
> 
> Is the plan to pull this from a romfile so that it can be set on
> coreboot?  If so, perhaps that functionality could be merged with
> "etc/sercon-enable" so that coreboot users need only set one cbfs
> file.

No concrete plan yet, other than that I've added a variable for the
port intentionally so runtime configuration can be added easily.

One option would be to have a etc/sercon-ioport file, which will both
enable sercon and specify the io port address.

We could also depend on detect_serial() and lookup the detected io port
address via GET_BDA(port_com[count]).

There is also the question if and how we'll go support non-isa serial
ports.  qemu also has pci-serial for example.  serial-over-lan virtual
ports typically show up as pci device too.

cheers,
  Gerd