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 _______________________________________________ SeaBIOS mailing list SeaBIOS@seabios.org https://mail.coreboot.org/mailman/listinfo/seabios
