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

From: Michael Ellerman
Date: Thu Aug 26 2021 - 10:26:44 EST


Christophe Leroy <christophe.leroy@xxxxxxxxxx> writes:
> Le 24/08/2021 à 16:40, Shuah Khan a écrit :
>> On 8/5/21 12:52 AM, cgel.zte@xxxxxxxxx wrote:
>>> From: yong yiran <yong.yiran@xxxxxxxxxx>
>>>
>>> '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@xxxxxxxxxx>
>>> Signed-off-by: yong yiran <yong.yiran@xxxxxxxxxx>
>>> ---
>>>   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@xxxxxxxxxx>
> Signed-off-by: yong yiran <yong.yiran@xxxxxxxxxx>

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

cheers