linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] ARM: prima2: always enable reset controller
@ 2016-02-25 16:27 Arnd Bergmann
  2016-02-25 17:25 ` Philipp Zabel
  0 siblings, 1 reply; 3+ messages in thread
From: Arnd Bergmann @ 2016-02-25 16:27 UTC (permalink / raw)
  To: Barry Song; +Cc: linux-arm-kernel, Philipp Zabel, Arnd Bergmann, linux-kernel

The atlas7 clock controller driver registers a reset controller
for itself, which causes a link error when the subsystem is
disabled:

drivers/built-in.o: In function `atlas7_clk_init':
drivers/clk/sirf/clk-atlas7.c:1681: undefined reference to `reset_controller_register'

As the clk driver does not have a Kconfig symbol for itself
but it always built-in when the platform is enabled, we have
to ensure that the reset controller subsystem is also built-in
in this case.

Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Fixes: 301c5d29402e ("clk: sirf: add CSR atlas7 clk and reset support")
---
 arch/arm/mach-prima2/Kconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/mach-prima2/Kconfig b/arch/arm/mach-prima2/Kconfig
index 59cafaf097c0..86b7c859784f 100644
--- a/arch/arm/mach-prima2/Kconfig
+++ b/arch/arm/mach-prima2/Kconfig
@@ -2,6 +2,7 @@ menuconfig ARCH_SIRF
 	bool "CSR SiRF"
 	depends on ARCH_MULTI_V6_V7
 	select ARCH_HAS_RESET_CONTROLLER
+	select RESET_CONTROLLER
 	select ARCH_REQUIRE_GPIOLIB
 	select GENERIC_IRQ_CHIP
 	select NO_IOPORT_MAP
-- 
2.7.0

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] ARM: prima2: always enable reset controller
  2016-02-25 16:27 [PATCH] ARM: prima2: always enable reset controller Arnd Bergmann
@ 2016-02-25 17:25 ` Philipp Zabel
  2016-02-26 16:46   ` Arnd Bergmann
  0 siblings, 1 reply; 3+ messages in thread
From: Philipp Zabel @ 2016-02-25 17:25 UTC (permalink / raw)
  To: Arnd Bergmann; +Cc: Barry Song, linux-arm-kernel, linux-kernel

Am Donnerstag, den 25.02.2016, 17:27 +0100 schrieb Arnd Bergmann:
> The atlas7 clock controller driver registers a reset controller
> for itself, which causes a link error when the subsystem is
> disabled:
> 
> drivers/built-in.o: In function `atlas7_clk_init':
> drivers/clk/sirf/clk-atlas7.c:1681: undefined reference to `reset_controller_register'
> 
> As the clk driver does not have a Kconfig symbol for itself
> but it always built-in when the platform is enabled, we have
> to ensure that the reset controller subsystem is also built-in
> in this case.
> 
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> Fixes: 301c5d29402e ("clk: sirf: add CSR atlas7 clk and reset support")

Acked-by: Philipp Zabel <p.zabel@pengutronix.de>

regards
Philipp

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] ARM: prima2: always enable reset controller
  2016-02-25 17:25 ` Philipp Zabel
@ 2016-02-26 16:46   ` Arnd Bergmann
  0 siblings, 0 replies; 3+ messages in thread
From: Arnd Bergmann @ 2016-02-26 16:46 UTC (permalink / raw)
  To: Philipp Zabel; +Cc: Barry Song, linux-arm-kernel, linux-kernel

On Thursday 25 February 2016 18:25:32 Philipp Zabel wrote:
> Am Donnerstag, den 25.02.2016, 17:27 +0100 schrieb Arnd Bergmann:
> > The atlas7 clock controller driver registers a reset controller
> > for itself, which causes a link error when the subsystem is
> > disabled:
> > 
> > drivers/built-in.o: In function `atlas7_clk_init':
> > drivers/clk/sirf/clk-atlas7.c:1681: undefined reference to `reset_controller_register'
> > 
> > As the clk driver does not have a Kconfig symbol for itself
> > but it always built-in when the platform is enabled, we have
> > to ensure that the reset controller subsystem is also built-in
> > in this case.
> > 
> > Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> > Fixes: 301c5d29402e ("clk: sirf: add CSR atlas7 clk and reset support")
> 
> Acked-by: Philipp Zabel <p.zabel@pengutronix.de>
> 

Applied to next/fixes-non-critical, thanks!

	Arnd

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2016-02-26 16:47 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-02-25 16:27 [PATCH] ARM: prima2: always enable reset controller Arnd Bergmann
2016-02-25 17:25 ` Philipp Zabel
2016-02-26 16:46   ` Arnd Bergmann

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).