From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Subject: Re: [PATCH v3 2/4] clk: s2mps11: allocate only one structure for clock init To: Andi Shyti , linux-samsung-soc@vger.kernel.org References: <1453284864-8082-1-git-send-email-andi.shyti@samsung.com> <1453284864-8082-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 , Jaehoon Chung From: Krzysztof Kozlowski Message-id: <56A02519.4050207@samsung.com> Date: Thu, 21 Jan 2016 09:23:53 +0900 MIME-version: 1.0 In-reply-to: <1453284864-8082-3-git-send-email-andi.shyti@samsung.com> Content-type: text/plain; charset=windows-1252 List-ID: On 20.01.2016 19:14, Andi Shyti wrote: > The driver allocates three structures, s2mpsxx_clk_init, for > three different clock types (s2mps11, s2mps13 and s2mps14). They > are quite similar but they differ only by the name. Only one of > these structures is used, while the others lie unused in the > memory. > > The clock's name, though, is not such a meaningful information > and by assigning the same name to the initial data we can avoid > over allocation. The common name chosen will be s2mps11, > coherently with the device driver name, instead of the clock > device. > > Therefore, remove the structures associated to s2mps13 and > s2mps14 and use only the one referred to s2mps11 for all kind of > clocks. > > Signed-off-by: Andi Shyti > Suggested-by: Krzysztof Kozlowski > --- > drivers/clk/clk-s2mps11.c | 51 +++++++---------------------------------------- > 1 file changed, 7 insertions(+), 44 deletions(-) > Reviewed-by: Krzysztof Kozlowski Best regards, Krzysztof