From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: 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 In-Reply-To: <1453284864-8082-5-git-send-email-andi.shyti@samsung.com> List-ID: 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