Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752920AbbBYAyG (ORCPT ); Tue, 24 Feb 2015 19:54:06 -0500 Received: from mail-ob0-f181.google.com ([209.85.214.181]:33024 "EHLO mail-ob0-f181.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752721AbbBYAyE (ORCPT ); Tue, 24 Feb 2015 19:54:04 -0500 MIME-Version: 1.0 In-Reply-To: <1424769751-4633-1-git-send-email-ideal.song@samsung.com> References: <1424769751-4633-1-git-send-email-ideal.song@samsung.com> Date: Wed, 25 Feb 2015 09:54:02 +0900 Message-ID: Subject: Re: [PATCH] clk: samsung: Add CLKOUT driver support for Exynos3250 SoC. From: Tomasz Figa To: Inha Song Cc: "linux-samsung-soc@vger.kernel.org" , linux-kernel , linux-arm-kernel , Kukjin Kim , Stephen Boyd , Michael Turquette , Sylwester Nawrocki , Chanwoo Choi Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2082 Lines: 50 Hi Inha, Thanks for the patch. Please see my comments inline. 2015-02-24 18:22 GMT+09:00 Inha Song : > This patch add CLKOUT driver support for Exynos3250 SoC. Could you please add a little more information? I know that it might be pretty obvious to people familiar with this driver and/or hardware, but it might be a good idea to explicitly say that the CLKOUT controller is compatible with Exynos4, so only a new compatible string is added. On the other hand, do you really need to add a new compatible string if an existing one can be reused? The reason why the DT property is called "compatible" is to be able to use the same compatible strings on different devices, because they are compatible, even though the string might have its name after only one of them. If there is some additional reason to add a new compatible string, please write this in commit message. > > Signed-off-by: Inha Song > --- > drivers/clk/samsung/clk-exynos-clkout.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/clk/samsung/clk-exynos-clkout.c b/drivers/clk/samsung/clk-exynos-clkout.c > index 3a7cb25..1c02e73 100644 > --- a/drivers/clk/samsung/clk-exynos-clkout.c > +++ b/drivers/clk/samsung/clk-exynos-clkout.c > @@ -142,6 +142,8 @@ CLK_OF_DECLARE(exynos4212_clkout, "samsung,exynos4212-pmu", > exynos4_clkout_init); > CLK_OF_DECLARE(exynos4412_clkout, "samsung,exynos4412-pmu", > exynos4_clkout_init); > +CLK_OF_DECLARE(exynos3250_clkout, "samsung,exynos3250-pmu", > + exynos4_clkout_init); Are you sure that the PMU DEBUG register on Exynos3250 is indeed compatible with Exynos4 and not with newer SoCs? AFAIR, the only difference was the number of bits (4 on Exynos4 and 5 on Exynos5?) of the main mux. Best regards, Tomasz -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/