Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Thu, 12 Nov 2020 17:18:20 +0100
From:   Krzysztof Kozlowski <krzk@kernel.org>
To:     Sylwester Nawrocki <snawrocki@kernel.org>
Cc:     Sylwester Nawrocki <s.nawrocki@samsung.com>,
        Tomasz Figa <tomasz.figa@gmail.com>,
        Chanwoo Choi <cw00.choi@samsung.com>,
        Michael Turquette <mturquette@baylibre.com>,
        Stephen Boyd <sboyd@kernel.org>,
        linux-samsung-soc@vger.kernel.org, linux-clk@vger.kernel.org,
        linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH] clk: samsung: allow building the clkout driver as module
Message-ID: <20201112161820.GA17700@kozik-lap>
References: <20201110193749.261367-1-krzk@kernel.org>
 <92ccc476-6afd-4be9-7c72-40c9a9218b03@kernel.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
In-Reply-To: <92ccc476-6afd-4be9-7c72-40c9a9218b03@kernel.org>
Precedence: bulk

On Thu, Nov 12, 2020 at 03:36:35PM +0100, Sylwester Nawrocki wrote:
> On 11/10/20 20:37, Krzysztof Kozlowski wrote:
> > The Exynos clock output driver can be built as module (it does not have
> > to be part of core init process) for better customization.  Adding a
> > KConfig entry allows also compile testing for build coverage.
> > 
> > Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> 
> This needs to go through your tree due to dependencies on your previous
> patches, so
> 
> Acked-by: Sylwester Nawrocki <s.nawrocki@samsung.com>

Yes, thanks.

> 
> > ---
> >   drivers/clk/samsung/Kconfig             | 10 ++++++++++
> >   drivers/clk/samsung/Makefile            |  2 +-
> >   drivers/clk/samsung/clk-exynos-clkout.c |  1 +
> >   3 files changed, 12 insertions(+), 1 deletion(-)
> > 
> > diff --git a/drivers/clk/samsung/Kconfig b/drivers/clk/samsung/Kconfig
> > index 57d4b3f20417..b6b2cb209543 100644
> > --- a/drivers/clk/samsung/Kconfig
> > +++ b/drivers/clk/samsung/Kconfig
> > @@ -19,6 +19,16 @@ config EXYNOS_AUDSS_CLK_CON
> >   	  on some Exynos SoC variants. Choose M or Y here if you want to
> >   	  use audio devices such as I2S, PCM, etc.
> >   
> > +config EXYNOS_CLK_OUT
> 
> Perhaps change it EXYNOS_CLKOUT for a better match with the SoC documentation? 

Sure, I applied it with name fixup.

Best regards,
Krzysztof