Received-SPF: pass (google.com: domain of linux-kernel+bounces-73836-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
Message-ID: <6e31af22-b1b5-43d6-85ee-6818dc8e8eac@linuxfoundation.org>
Date: Tue, 20 Feb 2024 16:48:23 -0700
Precedence: bulk
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH] tools: selftests: Remove unnecessary semicolons
Content-Language: en-US
To: Thorsten Blum <thorsten.blum@toblux.com>, Shuah Khan <shuah@kernel.org>
Cc: Mykola Lysenko <mykolal@fb.com>, Martin KaFai Lau <martin.lau@linux.dev>,
 linux-kselftest@vger.kernel.org, linux-kernel@vger.kernel.org,
 Shuah Khan <skhan@linuxfoundation.org>
References: <20240213122317.1883-2-thorsten.blum@toblux.com>
From: Shuah Khan <skhan@linuxfoundation.org>
In-Reply-To: <20240213122317.1883-2-thorsten.blum@toblux.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit

On 2/13/24 05:23, Thorsten Blum wrote:
> Remove unnecessary semicolons reported by Coccinelle/coccicheck and the
> semantic patch at scripts/coccinelle/misc/semicolon.cocci.
> 
> Signed-off-by: Thorsten Blum <thorsten.blum@toblux.com>
> ---
>   tools/testing/selftests/bpf/benchs/bench_local_storage_create.c | 2 +-
>   tools/testing/selftests/kvm/access_tracking_perf_test.c         | 2 +-
>   tools/testing/selftests/timers/nanosleep.c                      | 2 +-
>   tools/testing/selftests/timers/nsleep-lat.c                     | 2 +-
>   tools/testing/selftests/timers/set-timer-lat.c                  | 2 +-
>   5 files changed, 5 insertions(+), 5 deletions(-)
> 

Thank you for finding the problems and fixing them.

Please split these up and generate separate patches.
Run getmaintainers.pl to get the right people to send the
patch.

Taking all these changes in one single patch will result in
conflicts between the trees and more importantly if changes
to one file have to reverted for some reason, it makes it
easier have separate patches.

thanks,
-- Shuah