Re: [PATCH 01/17] perf: get rid of unused import

2023-06-13 Thread Leo Yan
On Tue, Jun 13, 2023 at 11:34:11PM -0300, Arnaldo Carvalho de Melo wrote: [...] > > 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

Re: [PATCH 01/17] perf: get rid of unused import

2023-06-13 Thread Arnaldo Carvalho de Melo
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: > > Em Tue, Jun 13, 2023 at 10:11:29PM +0530, Athira Rajeev escreveu: > > > From: Sourabh Jain > > > > > > Script doesn't use sys library, so remove it. > > >

Re: [PATCH 01/17] perf: get rid of unused import

2023-06-13 Thread Leo Yan
On Tue, Jun 13, 2023 at 04:54:08PM -0300, Arnaldo Carvalho de Melo wrote: > Em Tue, Jun 13, 2023 at 10:11:29PM +0530, Athira Rajeev escreveu: > > From: Sourabh Jain > > > > Script doesn't use sys library, so remove it. > > Please Cc the persons working on that file, I added Leo to the CC list >

Re: [PATCH 01/17] perf: get rid of unused import

2023-06-13 Thread Arnaldo Carvalho de Melo
Em Tue, Jun 13, 2023 at 10:11:29PM +0530, Athira Rajeev escreveu: > From: Sourabh Jain > > 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, applied. - Arnaldo > Report by pylint: > W0611: Unused

[PATCH 01/17] perf: get rid of unused import

2023-06-13 Thread Athira Rajeev
From: Sourabh Jain Script doesn't use sys library, so remove it. Report by pylint: W0611: Unused import sys (unused-import) Signed-off-by: Athira Rajeev Signed-off-by: Kajol Jain Signed-off-by: Sourabh Jain --- tools/perf/scripts/python/arm-cs-trace-disasm.py | 1 - 1 file changed, 1