[PATCH v8 07/11] test: Avoid double echo when starting up
Simon Glass
sjg at chromium.org
Mon Nov 4 23:45:34 CET 2024
There is a very annoying bug at present where the terminal echos part
of the first command sent to the board. This happens because the
terminal is still set to echo for a period until Labgrid starts up and
can change this.
Fix this by disabling echo (and other terminal features) as soon as the
spawn happens.
Signed-off-by: Simon Glass <sjg at chromium.org>
---
(no changes since v6)
Changes in v6:
- Rebase without an earlier patch
Changes in v2:
- Only disable echo if a terminal is detected
test/py/u_boot_spawn.py | 14 ++++++++++++++
1 file changed, 14 insertions(+)
diff --git a/test/py/u_boot_spawn.py b/test/py/u_boot_spawn.py
index 24d369035e5..f2398098a00 100644
--- a/test/py/u_boot_spawn.py
+++ b/test/py/u_boot_spawn.py
@@ -11,6 +11,8 @@ import pty
import pytest
import signal
import select
+import sys
+import termios
import time
import traceback
@@ -115,11 +117,23 @@ class Spawn:
finally:
os._exit(255)
+ old = None
try:
+ if os.isatty(sys.stdout.fileno()):
+ new = termios.tcgetattr(self.fd)
+ old = new
+ new[3] = new[3] & ~(termios.ICANON | termios.ISIG)
+ new[3] = new[3] & ~termios.ECHO
+ new[6][termios.VMIN] = 0
+ new[6][termios.VTIME] = 0
+ termios.tcsetattr(self.fd, termios.TCSANOW, new)
+
self.poll = select.poll()
self.poll.register(self.fd, select.POLLIN | select.POLLPRI | select.POLLERR |
select.POLLHUP | select.POLLNVAL)
except:
+ if old:
+ termios.tcsetattr(self.fd, termios.TCSANOW, old)
self.close()
raise
--
2.43.0
More information about the U-Boot
mailing list