| Message ID | 20210805065255.628170-1-yong.yiran@zte.com.cn |
|---|---|
| State | Superseded |
| Headers | show |
| Series | [linux-next] powerpc/tm: remove duplicate include in tm-poison.c | expand |
On 8/5/21 12:52 AM, cgel.zte@gmail.com wrote: > From: yong yiran <yong.yiran@zte.com.cn> > > '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 <zealci@zte.com.cn> > Signed-off-by: yong yiran <yong.yiran@zte.com.cn> > --- > 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. thanks, -- Shuah
Le 24/08/2021 à 16:40, Shuah Khan a écrit : > On 8/5/21 12:52 AM, cgel.zte@gmail.com wrote: >> From: yong yiran <yong.yiran@zte.com.cn> >> >> '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 <zealci@zte.com.cn> >> Signed-off-by: yong yiran <yong.yiran@zte.com.cn> >> --- >> 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 <yong.yiran@zte.com.cn> Signed-off-by: yong yiran <yong.yiran@zte.com.cn>
Christophe Leroy <christophe.leroy@csgroup.eu> writes: > Le 24/08/2021 à 16:40, Shuah Khan a écrit : >> On 8/5/21 12:52 AM, cgel.zte@gmail.com wrote: >>> From: yong yiran <yong.yiran@zte.com.cn> >>> >>> '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 <zealci@zte.com.cn> >>> Signed-off-by: yong yiran <yong.yiran@zte.com.cn> >>> --- >>> 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 <yong.yiran@zte.com.cn> > Signed-off-by: yong yiran <yong.yiran@zte.com.cn> Regardless I already have a patch queued to fix this, from a different CI bot. cheers
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"