From: Joachim Eastwood <manabian@gmail.com> To: Vladimir Zapolskiy <vz@mleia.com> Cc: Rob Herring <robh+dt@kernel.org>, Stephen Boyd <sboyd@codeaurora.org>, Michael Turquette <mturquette@baylibre.com>, Arnd Bergmann <arnd@arndb.de>, Russell King <linux@arm.linux.org.uk>, Roland Stigge <stigge@antcom.de>, linux-clk@vger.kernel.org, devicetree@vger.kernel.org, "linux-arm-kernel@lists.infradead.org" <linux-arm-kernel@lists.infradead.org> Subject: Re: [PATCH 08/11] clk: lpc18xx: add NXP specific common clock framework selection Date: Sun, 22 Nov 2015 21:38:08 +0100 [thread overview] Message-ID: <CAGhQ9VzrGA3jZBKiZzDr=6pGUnbs0aHdKxsxwmkyMWMT+xYghg@mail.gmail.com> (raw) In-Reply-To: <1447981511-29653-9-git-send-email-vz@mleia.com> Hi Vladimir, On 20 November 2015 at 02:05, Vladimir Zapolskiy <vz@mleia.com> wrote: > The change adds COMMON_CLK_NXP configuration symbol and enables it for > NXP LPC18XX architecture, this is needed to reuse drivers/clk/nxp > folder for NXP common clock framework drivers other than LPC18XX one. > > Signed-off-by: Vladimir Zapolskiy <vz@mleia.com> > Cc: Joachim Eastwood <manabian@gmail.com> > --- > arch/arm/Kconfig | 1 + > drivers/clk/Kconfig | 5 +++++ > drivers/clk/Makefile | 2 +- > 3 files changed, 7 insertions(+), 1 deletion(-) > > diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig > index 0365cbb..c318277 100644 > --- a/arch/arm/Kconfig > +++ b/arch/arm/Kconfig > @@ -952,6 +952,7 @@ config ARCH_LPC18XX > select ARCH_HAS_RESET_CONTROLLER > select ARM_AMBA > select CLKSRC_LPC32XX > + select COMMON_CLK COMMON_CLK is already selected by ARM_SINGLE_ARMV7M which ARCH_LPC18XX depends on so this hunk should not be necessary. > select PINCTRL > help > Support for NXP's LPC18xx Cortex-M3 and LPC43xx Cortex-M4 > diff --git a/drivers/clk/Kconfig b/drivers/clk/Kconfig > index c3e3a02..7fc1eb9 100644 > --- a/drivers/clk/Kconfig > +++ b/drivers/clk/Kconfig > @@ -175,6 +175,11 @@ config COMMON_CLK_PWM > Adapter driver so that any PWM output can be (mis)used as clock signal > at 50% duty cycle. > > +config COMMON_CLK_NXP > + def_bool COMMON_CLK && ARCH_LPC18XX > + ---help--- > + Support for clock providers on NXP platforms. > + > config COMMON_CLK_PXA > def_bool COMMON_CLK && ARCH_PXA > ---help--- > diff --git a/drivers/clk/Makefile b/drivers/clk/Makefile > index 820714c..15603c1 100644 > --- a/drivers/clk/Makefile > +++ b/drivers/clk/Makefile > @@ -62,8 +62,8 @@ endif > obj-$(CONFIG_PLAT_ORION) += mvebu/ > obj-$(CONFIG_ARCH_MESON) += meson/ > obj-$(CONFIG_ARCH_MXS) += mxs/ > -obj-$(CONFIG_ARCH_LPC18XX) += nxp/ > obj-$(CONFIG_MACH_PISTACHIO) += pistachio/ > +obj-$(CONFIG_COMMON_CLK_NXP) += nxp/ > obj-$(CONFIG_COMMON_CLK_PXA) += pxa/ > obj-$(CONFIG_COMMON_CLK_QCOM) += qcom/ > obj-$(CONFIG_ARCH_ROCKCHIP) += rockchip/ > -- Other than the one comment above it looks good to me: Acked-by: Joachim Eastwood <manabian@gmail.com> I'll try to find the time look through the other patches you have for lpc32xx. I also have a EA LPC3250 dev kit v2 (http://embeddedartists.com/products/kits/lpc3250_kit_v2.php) in my collection. regards, Joachim Eastwood
WARNING: multiple messages have this Message-ID (diff)
From: manabian@gmail.com (Joachim Eastwood) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH 08/11] clk: lpc18xx: add NXP specific common clock framework selection Date: Sun, 22 Nov 2015 21:38:08 +0100 [thread overview] Message-ID: <CAGhQ9VzrGA3jZBKiZzDr=6pGUnbs0aHdKxsxwmkyMWMT+xYghg@mail.gmail.com> (raw) In-Reply-To: <1447981511-29653-9-git-send-email-vz@mleia.com> Hi Vladimir, On 20 November 2015 at 02:05, Vladimir Zapolskiy <vz@mleia.com> wrote: > The change adds COMMON_CLK_NXP configuration symbol and enables it for > NXP LPC18XX architecture, this is needed to reuse drivers/clk/nxp > folder for NXP common clock framework drivers other than LPC18XX one. > > Signed-off-by: Vladimir Zapolskiy <vz@mleia.com> > Cc: Joachim Eastwood <manabian@gmail.com> > --- > arch/arm/Kconfig | 1 + > drivers/clk/Kconfig | 5 +++++ > drivers/clk/Makefile | 2 +- > 3 files changed, 7 insertions(+), 1 deletion(-) > > diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig > index 0365cbb..c318277 100644 > --- a/arch/arm/Kconfig > +++ b/arch/arm/Kconfig > @@ -952,6 +952,7 @@ config ARCH_LPC18XX > select ARCH_HAS_RESET_CONTROLLER > select ARM_AMBA > select CLKSRC_LPC32XX > + select COMMON_CLK COMMON_CLK is already selected by ARM_SINGLE_ARMV7M which ARCH_LPC18XX depends on so this hunk should not be necessary. > select PINCTRL > help > Support for NXP's LPC18xx Cortex-M3 and LPC43xx Cortex-M4 > diff --git a/drivers/clk/Kconfig b/drivers/clk/Kconfig > index c3e3a02..7fc1eb9 100644 > --- a/drivers/clk/Kconfig > +++ b/drivers/clk/Kconfig > @@ -175,6 +175,11 @@ config COMMON_CLK_PWM > Adapter driver so that any PWM output can be (mis)used as clock signal > at 50% duty cycle. > > +config COMMON_CLK_NXP > + def_bool COMMON_CLK && ARCH_LPC18XX > + ---help--- > + Support for clock providers on NXP platforms. > + > config COMMON_CLK_PXA > def_bool COMMON_CLK && ARCH_PXA > ---help--- > diff --git a/drivers/clk/Makefile b/drivers/clk/Makefile > index 820714c..15603c1 100644 > --- a/drivers/clk/Makefile > +++ b/drivers/clk/Makefile > @@ -62,8 +62,8 @@ endif > obj-$(CONFIG_PLAT_ORION) += mvebu/ > obj-$(CONFIG_ARCH_MESON) += meson/ > obj-$(CONFIG_ARCH_MXS) += mxs/ > -obj-$(CONFIG_ARCH_LPC18XX) += nxp/ > obj-$(CONFIG_MACH_PISTACHIO) += pistachio/ > +obj-$(CONFIG_COMMON_CLK_NXP) += nxp/ > obj-$(CONFIG_COMMON_CLK_PXA) += pxa/ > obj-$(CONFIG_COMMON_CLK_QCOM) += qcom/ > obj-$(CONFIG_ARCH_ROCKCHIP) += rockchip/ > -- Other than the one comment above it looks good to me: Acked-by: Joachim Eastwood <manabian@gmail.com> I'll try to find the time look through the other patches you have for lpc32xx. I also have a EA LPC3250 dev kit v2 (http://embeddedartists.com/products/kits/lpc3250_kit_v2.php) in my collection. regards, Joachim Eastwood
next prev parent reply other threads:[~2015-11-22 20:38 UTC|newest] Thread overview: 67+ messages / expand[flat|nested] mbox.gz Atom feed top 2015-11-20 1:05 [PATCH 00/11] clk: lpc32xx: add clock support for NXP LPC32xx Vladimir Zapolskiy 2015-11-20 1:05 ` Vladimir Zapolskiy 2015-11-20 1:05 ` Vladimir Zapolskiy 2015-11-20 1:05 ` [PATCH 02/11] dt-bindings: clock: add description of LPC32xx USB clock controller Vladimir Zapolskiy 2015-11-20 1:05 ` Vladimir Zapolskiy 2015-11-20 16:41 ` Rob Herring 2015-11-20 16:41 ` Rob Herring 2015-11-20 18:14 ` Vladimir Zapolskiy 2015-11-20 18:14 ` Vladimir Zapolskiy 2015-11-20 1:05 ` [PATCH 03/11] dt-bindings: clock: add NXP LPC32xx clock list for consumers Vladimir Zapolskiy 2015-11-20 1:05 ` Vladimir Zapolskiy 2015-11-20 13:56 ` Arnd Bergmann 2015-11-20 13:56 ` Arnd Bergmann 2015-11-20 13:56 ` Arnd Bergmann 2015-11-20 17:58 ` Vladimir Zapolskiy 2015-11-20 17:58 ` Vladimir Zapolskiy 2015-11-20 21:07 ` Arnd Bergmann 2015-11-20 21:07 ` Arnd Bergmann 2015-11-21 18:53 ` Vladimir Zapolskiy 2015-11-21 18:53 ` Vladimir Zapolskiy 2015-11-20 1:05 ` [PATCH 04/11] arm: dts: lpc32xx: add device nodes for external oscillators Vladimir Zapolskiy 2015-11-20 1:05 ` Vladimir Zapolskiy 2015-11-20 1:05 ` [PATCH 05/11] arm: dts: lpc32xx: add clock controller device node Vladimir Zapolskiy 2015-11-20 1:05 ` Vladimir Zapolskiy [not found] ` <1447981511-29653-1-git-send-email-vz-ChpfBGZJDbMAvxtiuMwx3w@public.gmane.org> 2015-11-20 1:05 ` [PATCH 01/11] dt-bindings: clock: add description of LPC32xx clock controller Vladimir Zapolskiy 2015-11-20 1:05 ` Vladimir Zapolskiy 2015-11-20 1:05 ` Vladimir Zapolskiy 2015-11-20 13:58 ` Arnd Bergmann 2015-11-20 13:58 ` Arnd Bergmann 2015-11-20 18:01 ` Vladimir Zapolskiy 2015-11-20 18:01 ` Vladimir Zapolskiy 2015-11-20 18:01 ` Vladimir Zapolskiy 2015-11-20 20:03 ` Arnd Bergmann 2015-11-20 20:03 ` Arnd Bergmann 2015-11-20 1:05 ` [PATCH 06/11] arm: dts: lpc32xx: add clock properties to device nodes Vladimir Zapolskiy 2015-11-20 1:05 ` Vladimir Zapolskiy 2015-11-20 1:05 ` Vladimir Zapolskiy 2015-11-20 1:05 ` [PATCH 07/11] arm: dts: lpc32xx: add USB clock controller Vladimir Zapolskiy 2015-11-20 1:05 ` Vladimir Zapolskiy 2015-11-20 1:05 ` Vladimir Zapolskiy 2015-11-20 1:05 ` [PATCH 08/11] clk: lpc18xx: add NXP specific common clock framework selection Vladimir Zapolskiy 2015-11-20 1:05 ` Vladimir Zapolskiy 2015-11-20 1:05 ` Vladimir Zapolskiy 2015-11-22 20:38 ` Joachim Eastwood [this message] 2015-11-22 20:38 ` Joachim Eastwood 2015-11-22 20:38 ` Joachim Eastwood 2015-11-20 1:05 ` [PATCH 11/11] arm: dts: lpc32xx: remove clock frequency property from UART device nodes Vladimir Zapolskiy 2015-11-20 1:05 ` Vladimir Zapolskiy 2015-11-20 1:05 ` Vladimir Zapolskiy 2015-11-20 1:05 ` [PATCH 09/11] clk: lpc32xx: add common clock framework driver Vladimir Zapolskiy 2015-11-20 1:05 ` Vladimir Zapolskiy 2015-11-20 14:04 ` Arnd Bergmann 2015-11-20 14:04 ` Arnd Bergmann 2015-11-20 18:07 ` Vladimir Zapolskiy 2015-11-20 18:07 ` Vladimir Zapolskiy 2015-11-20 18:07 ` Vladimir Zapolskiy 2015-11-20 20:20 ` Arnd Bergmann 2015-11-20 20:20 ` Arnd Bergmann 2015-11-29 13:00 ` Vladimir Zapolskiy 2015-11-29 13:00 ` Vladimir Zapolskiy 2015-11-20 1:05 ` [PATCH 10/11] arm: lpc32xx: switch to common clock framework Vladimir Zapolskiy 2015-11-20 1:05 ` Vladimir Zapolskiy 2016-02-09 21:01 ` [PATCH 00/11] clk: lpc32xx: add clock support for NXP LPC32xx Sylvain Lemieux [not found] ` <loom.20160209T215103-78-eS7Uydv5nfjZ+VzJOa5vwg@public.gmane.org> 2016-02-10 0:19 ` Vladimir Zapolskiy 2016-02-10 0:19 ` Vladimir Zapolskiy [not found] ` <56BA81F7.4080006-ChpfBGZJDbMAvxtiuMwx3w@public.gmane.org> 2016-02-10 14:25 ` Sylvain Lemieux 2016-02-10 14:25 ` Sylvain Lemieux
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to='CAGhQ9VzrGA3jZBKiZzDr=6pGUnbs0aHdKxsxwmkyMWMT+xYghg@mail.gmail.com' \ --to=manabian@gmail.com \ --cc=arnd@arndb.de \ --cc=devicetree@vger.kernel.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-clk@vger.kernel.org \ --cc=linux@arm.linux.org.uk \ --cc=mturquette@baylibre.com \ --cc=robh+dt@kernel.org \ --cc=sboyd@codeaurora.org \ --cc=stigge@antcom.de \ --cc=vz@mleia.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.