diff options
| author | Lukasz Hajec <lha@semihalf.com> | 2021-08-06 12:25:47 +0200 |
|---|---|---|
| committer | Commit Bot <commit-bot@chromium.org> | 2021-08-13 10:12:02 +0000 |
| commit | ff052aedbd21a41b52921c8c33b22eef03c03352 (patch) | |
| tree | e5a7a04f56a1daff54607f3cf9af7ea70f737fc4 /util | |
| parent | c425ace909b134095698dd604eb685cd2f620256 (diff) | |
| download | chrome-ec-ff052aedbd21a41b52921c8c33b22eef03c03352.tar.gz | |
console: Fix logs printed for some commands
E.g. in servod console there is always 'Disabling uart timestamps.'
printed, no matter what we enter to dut-control - on/off.
This patch fixes that issue.
BRANCH=none
BUG=b:173654272
TEST=Perform manual testing:
"sudo servod --board=nocturne"
"dut-control cr50_uart_timestamp:on"
"dut-control cr50_uart_timestamp:off"
observe proper logs in servod console.
Signed-off-by: Lukasz Hajec <lha@semihalf.com>
Change-Id: I52cde5e4a9cd70fdc3e9d9513f8aa4126c5f09f5
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3077540
Reviewed-by: Michał Barnaś <mb@semihalf.com>
Reviewed-by: Matthew Blecker <matthewb@chromium.org>
Diffstat (limited to 'util')
| -rwxr-xr-x | util/ec3po/console.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/util/ec3po/console.py b/util/ec3po/console.py index 185ad22b40..f26da30070 100755 --- a/util/ec3po/console.py +++ b/util/ec3po/console.py @@ -819,13 +819,13 @@ class Console(object): elif cmd[0] == b'timestamp': mode = cmd[1].lower() - self.timestamp_enabled = mode == 'on' + self.timestamp_enabled = (mode == b'on') self.logger.info('%sabling uart timestamps.', 'En' if self.timestamp_enabled else 'Dis') elif cmd[0] == b'rawdebug': mode = cmd[1].lower() - self.raw_debug = mode == 'on' + self.raw_debug = (mode == b'on') self.logger.info('%sabling per interrupt debug logs.', 'En' if self.raw_debug else 'Dis') |