From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932437AbcA3Aer (ORCPT ); Fri, 29 Jan 2016 19:34:47 -0500 Received: from smtp.codeaurora.org ([198.145.29.96]:36486 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756750AbcA3AeN (ORCPT ); Fri, 29 Jan 2016 19:34:13 -0500 Date: Fri, 29 Jan 2016 16:34:12 -0800 From: Stephen Boyd To: Andi Shyti Cc: linux-samsung-soc@vger.kernel.org, Sangbeom Kim , Krzysztof Kozlowski , Michael Turquette , linux-kernel@vger.kernel.org, linux-clk@vger.kernel.org, Andi Shyti , Yadwinder Singh Brar , Jaehoon Chung Subject: Re: [PATCH v3 4/4] clk: s2mps11: remove redundant code Message-ID: <20160130003412.GK12841@codeaurora.org> References: <1453284864-8082-1-git-send-email-andi.shyti@samsung.com> <1453284864-8082-5-git-send-email-andi.shyti@samsung.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1453284864-8082-5-git-send-email-andi.shyti@samsung.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 01/20, Andi Shyti wrote: > The definition of s2mps11_name is meant to resolve the name of a > given clock. Remove it because the clocks have the same name we > can get it directly from the s2mps11_clks_init structure. > > While in the probe function the s2mps11_clks is used only to > iterate through the s2mps11_clks. The naming itself brings > confusion and the readability does not improve much. > > Signed-off-by: Andi Shyti > --- Applied to clk-next -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project