[ipxe-devel] [serial] Allow serial port configuration at runtime

[Geert Stappers]

On Sun, Jun 16, 2019 at 05:44:20AM +0200, Ján ONDREJ (SAL) wrote:
> On Sat, Jun 15, 2019 at 04:10:27PM +0100, Faidon Liambotis wrote:
> > Allow serial port configuration at runtime
> >
> > Add configuration variables for the serial port and its options,
> > allowing the configuration of a serial port at runtime via either "set",
> > or DHCP options.
> > 
> > The new "serial-port" option accepts values of "1" through "4" for COM1
> > to COM4 respectively. For the serial options, rather than adding a
> > configuration variable for each of them (baud, parity etc.) and
> > complicate both the code and the configuration unncessarily, add a
> > single, generic "serial-options" setting that accepts string values such
> > as "115200" or "38400n81". It also accepts the special value "preserve"
> > to preserve the previous configuration of the serial port.
> > 
> > Note that this commit does _not_ enable CONSOLE_SERIAL by default,
> > although that could potentially follow in a subsequent commit, given
> > this would be now a no-op unless a port is explicitly configured.
> > 
> > Signed-off-by: Faidon Liambotis <paravoid at debian.org>
> > ---
> Hello,
> 
>   I am waiting a long time for this functionality. Thank you for this patch.
> This functionality is needed for my boot.salstar.sk.
> 
>   I like this functionality. Works well, I was able to set and change
> serial port device and parameters.
> 
>   Just for backward compatibility it would be nice to predefine serial-port
> and serial-options variables by settings from C #defines.  My recompiled
> binaries are not working as before you patch.

What is the different behaviour?

> Need to set at least serial-port variable manually.
 
How?


>   Michael, is is possible to add this or something similar to master branch?
 
Please describe how a better iPXE should look like.



Regards
Geert Stappers