[PATCH linux-next] selftests/powerpc: remove duplicate include

From: Changcheng Deng <redacted>

Clean up the following includecheck warning:

./tools/testing/selftests/powerpc/tm/tm-poison.c: inttypes.h is included
more than once.

No functional change.

Reported-by: Zeal Robot <redacted>
Signed-off-by: Changcheng Deng <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 29e5f26..27c083a 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"
 

-- 
1.8.3.1

On 8/23/21 9:05 PM, CGEL wrote:

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

thanks,
-- Shuah

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

On 8/23/21 9:05 PM, CGEL wrote:

quoted

From: Changcheng Deng <redacted>

Clean up the following includecheck warning:

./tools/testing/selftests/powerpc/tm/tm-poison.c: inttypes.h is included
more than once.

No functional change.

Reported-by: Zeal Robot <redacted>
Signed-off-by: Changcheng Deng <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 29e5f26..27c083a 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:

From: Changcheng Deng <redacted>
Signed-off-by: Changcheng Deng <redacted>

Christophe

On 8/24/21 10:35 AM, Christophe Leroy wrote:

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

quoted

On 8/23/21 9:05 PM, CGEL wrote:

quoted

From: Changcheng Deng <redacted>

Clean up the following includecheck warning:

./tools/testing/selftests/powerpc/tm/tm-poison.c: inttypes.h is included
more than once.

No functional change.

Reported-by: Zeal Robot <redacted>
Signed-off-by: Changcheng Deng <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 29e5f26..27c083a 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:

From: Changcheng Deng <redacted>
Signed-off-by: Changcheng Deng <redacted>

Yeah. My bad.

thanks,
-- Shuah