On 09/18/17 12:22, Zeng, Star wrote:
> Thanks for your good comments. :)
> Since there is no clear description for the behavior of Reset() :(, I prone 
> to align the behavior with MdeModulePkg/Bus/Pci/PciSioSerialDxe/SerialIo.c 
> and IntelFrameworkModulePkg/Bus/Isa/IsaSerialDxe/Serial.c, that means I agree 
> the fix.
> 
> 
> Laszlo and Gary, if you can help do some simple regression test with the 
> patch, that will be better.

Never used the SERMODE command before, so I didn't try it now either.

I tested the patch as follows: built ArmVirtQemu with it, and first
checked normal use of the serial terminal, then issued a
RECONNECT/CONNECT from the graphics console, and then verified if the
serial terminal was working OK again (UEFI shell and Setup utility). I
found no regression with this.

Regression-tested-by: Laszlo Ersek <[email protected]>

Thanks
Laszlo

_______________________________________________
edk2-devel mailing list
[email protected]
https://lists.01.org/mailman/listinfo/edk2-devel

Reply via email to