From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934094AbdHYQG0 (ORCPT ); Fri, 25 Aug 2017 12:06:26 -0400 Received: from mailout.easymail.ca ([64.68.200.34]:54961 "EHLO mailout.easymail.ca" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933744AbdHYQGZ (ORCPT ); Fri, 25 Aug 2017 12:06:25 -0400 Reply-To: shuah@kernel.org Subject: Re: [PATCH] selftests/timers: make loop consistent with array size To: Benjamin Gaignard , john.stultz@linaro.org, tglx@linutronix.de, sboyd@codeaurora.org, linux-kernel@vger.kernel.org, linux-kselftest@vger.kernel.org, Shuah Khan , Shuah Khan References: <1503504425-22808-1-git-send-email-benjamin.gaignard@linaro.org> From: Shuah Khan Message-ID: <9e18e516-8577-8b0e-9f8c-37d08a44d958@kernel.org> Date: Fri, 25 Aug 2017 10:06:11 -0600 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.1.1 MIME-Version: 1.0 In-Reply-To: <1503504425-22808-1-git-send-email-benjamin.gaignard@linaro.org> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 08/23/2017 10:07 AM, Benjamin Gaignard wrote: > clocksource_list array is defined as char [10][30] so > to initialise it we only have to iterate 10 times. > > Signed-off-by: Benjamin Gaignard > --- > tools/testing/selftests/timers/clocksource-switch.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/tools/testing/selftests/timers/clocksource-switch.c b/tools/testing/selftests/timers/clocksource-switch.c > index 5ff1653..56fbfc1 100644 > --- a/tools/testing/selftests/timers/clocksource-switch.c > +++ b/tools/testing/selftests/timers/clocksource-switch.c > @@ -61,7 +61,7 @@ int get_clocksources(char list[][30]) > > close(fd); > > - for (i = 0; i < 30; i++) > + for (i = 0; i < 10; i++) > list[i][0] = '\0'; > > head = buf; > Good catch. Thanks. Applied to linux-kselftest next for 4.14-rc1. thanks, -- Shuah