From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757959AbcASXjF (ORCPT ); Tue, 19 Jan 2016 18:39:05 -0500 Received: from mailout2.w1.samsung.com ([210.118.77.12]:56747 "EHLO mailout2.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757208AbcASXiz (ORCPT ); Tue, 19 Jan 2016 18:38:55 -0500 X-AuditID: cbfec7f5-f79b16d000005389-bd-569ec90da0bf Subject: Re: [PATCH v2 2/3] clk: s2mps11: merge two for loop in one To: Andi Shyti , linux-samsung-soc@vger.kernel.org References: <1453193552-4761-1-git-send-email-andi.shyti@samsung.com> <1453193552-4761-3-git-send-email-andi.shyti@samsung.com> Cc: Sangbeom Kim , Michael Turquette , Stephen Boyd , linux-kernel@vger.kernel.org, linux-clk@vger.kernel.org, Andi Shyti , Yadwinder Singh Brar From: Krzysztof Kozlowski Message-id: <569EC90D.9040409@samsung.com> Date: Wed, 20 Jan 2016 08:38:53 +0900 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.5.1 MIME-version: 1.0 In-reply-to: <1453193552-4761-3-git-send-email-andi.shyti@samsung.com> Content-type: text/plain; charset=windows-1252 Content-transfer-encoding: 7bit X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFnrLLMWRmVeSWpSXmKPExsVy+t/xy7q8J+eFGWycY2ax/cgzVovFP54z Wbx+YWjxseceq8XlXXPYLGac38dkcfGUq8XFFV+YLH6c6WaxmPu7kdWBy+P9jVZ2j8t9vUwe 15d8YvbYOesuu0ffllWMHp83yQWwRXHZpKTmZJalFunbJXBl/FjTz1jwgb3i96Me1gbGPrYu Rk4OCQETiR+v5jBC2GISF+6tB4sLCSxllNiwCcjmArKfMkqs7lrGDJIQFnCWOD3tABOILSLg JTH14QlGiKJGRonuXcfBHGaBNiaJk++ns4BUsQkYS2xevgRsLK+AlsTHKx3sIDaLgKrE7ouL wWpEBSIkDnd2sUPUCEr8mHwPLM4p4CbRvP0f0GYOoKF6EvcvaoGEmQXkJTavecs8gVFgFpKO WQhVs5BULWBkXsUomlqaXFCclJ5rpFecmFtcmpeul5yfu4kREglfdzAuPWZ1iFGAg1GJh5ex eV6YEGtiWXFl7iFGCQ5mJRHes1uAQrwpiZVVqUX58UWlOanFhxilOViUxHln7nofIiSQnliS mp2aWpBaBJNl4uCUamCU6Vg0U15Fp8PhxfOov2oL4l3zbtSZRIXysxT0PC/a9yuyK/z8vZTl ko3XnvX18JQrf7bZfG+huGMGi6nGioeHHP2PfZuXO/1oiZXv8pSZn/IfvWp6KB5teCj3/Gvz NU6/1YuCLXp0ms0dvc4Y32+YEMUtE8/EVPrnXSBnc17ogdYD53uPvFJiKc5INNRiLipOBAD2 ZCg3gAIAAA== Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 19.01.2016 17:52, Andi Shyti wrote: > the driver already loops once, there is no reason to loop again /s/the/The/ > for a different purpose. Merge the second loop into the first. > > Signed-off-by: Andi Shyti > --- > drivers/clk/clk-s2mps11.c | 6 ------ > 1 file changed, 6 deletions(-) > > diff --git a/drivers/clk/clk-s2mps11.c b/drivers/clk/clk-s2mps11.c > index 22d4df2..cae92fc 100644 > --- a/drivers/clk/clk-s2mps11.c > +++ b/drivers/clk/clk-s2mps11.c > @@ -202,12 +202,6 @@ static int s2mps11_clk_probe(struct platform_device *pdev) > ret = -ENOMEM; > goto err_reg; > } > - } > - > - for (i = 0; i < S2MPS11_CLKS_NUM; i++) { > - /* Skip clocks not present on S2MPS14 */ > - if (i == S2MPS11_CLK_CP && hwid == S2MPS14X) You changed this line in patch 1/3 - meaningless. This is a indication that this patch should go before 1st. BR, Krzysztof > - continue; > clk_table[i] = s2mps11_clks[i].clk; > } > >