[PATCH linux-next] powerpc/tm: remove duplicate include in tm-poison.c

From: yong yiran <redacted>

'inttypes.h' included in 'tm-poison.c' is duplicated.
Remove all but the first include of inttypes.h from tm-poison.c.

Reported-by: Zeal Robot <redacted>
Signed-off-by: yong yiran <redacted>
---
 tools/testing/selftests/powerpc/tm/tm-poison.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/tools/testing/selftests/powerpc/tm/tm-poison.c b/tools/testing/selftests/powerpc/tm/tm-poison.c
index 29e5f26af7b9..27c083a03d1f 100644
--- a/tools/testing/selftests/powerpc/tm/tm-poison.c
+++ b/tools/testing/selftests/powerpc/tm/tm-poison.c

@@ -20,7 +20,6 @@
 #include <sched.h>
 #include <sys/types.h>
 #include <signal.h>
-#include <inttypes.h>
 
 #include "tm.h"
 

-- 
2.25.1

On 8/5/21 12:52 AM, cgel.zte@gmail.com wrote:

We can't accept this patch. The from and Signed-off-by don't match.

thanks,
-- Shuah

Le 24/08/2021 à 16:40, Shuah Khan a écrit :

On 8/5/21 12:52 AM, cgel.zte@gmail.com wrote:

quoted

From: yong yiran <redacted>

'inttypes.h' included in 'tm-poison.c' is duplicated.
Remove all but the first include of inttypes.h from tm-poison.c.

Reported-by: Zeal Robot <redacted>
Signed-off-by: yong yiran <redacted>
---
  tools/testing/selftests/powerpc/tm/tm-poison.c | 1 -
  1 file changed, 1 deletion(-)

diff --git a/tools/testing/selftests/powerpc/tm/tm-poison.c 

b/tools/testing/selftests/powerpc/tm/tm-poison.c
index 29e5f26af7b9..27c083a03d1f 100644

--- a/tools/testing/selftests/powerpc/tm/tm-poison.c
+++ b/tools/testing/selftests/powerpc/tm/tm-poison.c

@@ -20,7 +20,6 @@

  #include <sched.h>
  #include <sys/types.h>
  #include <signal.h>
-#include <inttypes.h>
  #include "tm.h"

We can't accept this patch. The from and Signed-off-by don't match.

As far as I can see they match. You have:

From: yong yiran <redacted>
Signed-off-by: yong yiran <redacted>

Christophe Leroy [off-list ref] writes:

Le 24/08/2021 à 16:40, Shuah Khan a écrit :

quoted

On 8/5/21 12:52 AM, cgel.zte@gmail.com wrote:

quoted

From: yong yiran <redacted>

'inttypes.h' included in 'tm-poison.c' is duplicated.
Remove all but the first include of inttypes.h from tm-poison.c.

Reported-by: Zeal Robot <redacted>
Signed-off-by: yong yiran <redacted>
---
  tools/testing/selftests/powerpc/tm/tm-poison.c | 1 -
  1 file changed, 1 deletion(-)

diff --git a/tools/testing/selftests/powerpc/tm/tm-poison.c 

b/tools/testing/selftests/powerpc/tm/tm-poison.c
index 29e5f26af7b9..27c083a03d1f 100644

--- a/tools/testing/selftests/powerpc/tm/tm-poison.c
+++ b/tools/testing/selftests/powerpc/tm/tm-poison.c

@@ -20,7 +20,6 @@

  #include <sched.h>
  #include <sys/types.h>
  #include <signal.h>
-#include <inttypes.h>
  #include "tm.h"

We can't accept this patch. The from and Signed-off-by don't match.

As far as I can see they match. You have:

From: yong yiran <redacted>
Signed-off-by: yong yiran <redacted>

Regardless I already have a patch queued to fix this, from a different
CI bot.

cheers