Em Wed, Jun 14, 2023 at 09:59:14AM +0800, Leo Yan escreveu:

On Tue, Jun 13, 2023 at 04:54:08PM -0300, Arnaldo Carvalho de Melo wrote:

quoted

Em Tue, Jun 13, 2023 at 10:11:29PM +0530, Athira Rajeev escreveu:

quoted

From: Sourabh Jain <redacted>

Script doesn't use sys library, so remove it.

Please Cc the persons working on that file, I added Leo to the CC list
of this message.

Thanks, Arnaldo & Athira.  The change looks good to me.

quoted

Thanks, applied.

Since have applied this patch, it's no need to give my review tag :)

No, I usually can  add a Reviewed-by tag even after having applied it to
my local tree, as I still need to run tests before making it available
via perf-tools-next, when I should not make any further changes.

- Arnaldo
 

Thanks,
Leo

quoted

- Arnaldo
 

quoted

Report by pylint:
W0611: Unused import sys (unused-import)

Signed-off-by: Athira Rajeev <redacted>
Signed-off-by: Kajol Jain <redacted>
Signed-off-by: Sourabh Jain <redacted>
---
 tools/perf/scripts/python/arm-cs-trace-disasm.py | 1 -
 1 file changed, 1 deletion(-)

diff --git a/tools/perf/scripts/python/arm-cs-trace-disasm.py b/tools/perf/scripts/python/arm-cs-trace-disasm.py
index 4339692a8d0b..d59ff53f1d94 100755
--- a/tools/perf/scripts/python/arm-cs-trace-disasm.py
+++ b/tools/perf/scripts/python/arm-cs-trace-disasm.py

@@ -9,7 +9,6 @@
 from __future__ import print_function
 import os
 from os import path
-import sys
 import re
 from subprocess import *
 from optparse import OptionParser, make_option

-- 
2.39.1

-- 

- Arnaldo

-- 

- Arnaldo