Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1316919 > unrolled thread

[PATCH] watchdog: imgdpc: select WATCHDOG_CORE

Started byArnd Bergmann <arnd@arndb.de>
First post2016-01-25 17:10 +0100
Last post2016-01-25 17:50 +0100
Articles 3 — 3 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH] watchdog: imgdpc: select WATCHDOG_CORE Arnd Bergmann <arnd@arndb.de> - 2016-01-25 17:10 +0100
    Re: [PATCH] watchdog: imgdpc: select WATCHDOG_CORE James Hogan <james.hogan@imgtec.com> - 2016-01-25 17:20 +0100
    Re: [PATCH] watchdog: imgdpc: select WATCHDOG_CORE Guenter Roeck <linux@roeck-us.net> - 2016-01-25 17:50 +0100

#1316919 — [PATCH] watchdog: imgdpc: select WATCHDOG_CORE

FromArnd Bergmann <arnd@arndb.de>
Date2016-01-25 17:10 +0100
Subject[PATCH] watchdog: imgdpc: select WATCHDOG_CORE
Message-ID<qURSj-6uy-31@gated-at.bofh.it>
The imgpdc_wdt driver can be built on all architectures with
CONFIG_COMPILE_TEST, but fails if no other watchdog driver is
enabled:

drivers/watchdog/built-in.o: In function `pdc_wdt_remove':
imgpdc_wdt.c:(.text+0x74): undefined reference to `watchdog_unregister_device'

This adds the normal 'select WATCHDOG_CORE' that is needed to
ensure the driver always builds cleanly.

Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
 drivers/watchdog/Kconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig
index 4f0e7be0da34..0dc61f1a552e 100644
--- a/drivers/watchdog/Kconfig
+++ b/drivers/watchdog/Kconfig
@@ -1383,6 +1383,7 @@ config IMGPDC_WDT
 	tristate "Imagination Technologies PDC Watchdog Timer"
 	depends on HAS_IOMEM
 	depends on METAG || MIPS || COMPILE_TEST
+	select WATCHDOG_CORE
 	help
 	  Driver for Imagination Technologies PowerDown Controller
 	  Watchdog Timer.
-- 
2.7.0

[toc] | [next] | [standalone]


#1316953

FromJames Hogan <james.hogan@imgtec.com>
Date2016-01-25 17:20 +0100
Message-ID<qUS20-6yn-53@gated-at.bofh.it>
In reply to#1316919

[Multipart message — attachments visible in raw view] — view raw

On Mon, Jan 25, 2016 at 05:06:27PM +0100, Arnd Bergmann wrote:
> The imgpdc_wdt driver can be built on all architectures with
> CONFIG_COMPILE_TEST, but fails if no other watchdog driver is
> enabled:
> 
> drivers/watchdog/built-in.o: In function `pdc_wdt_remove':
> imgpdc_wdt.c:(.text+0x74): undefined reference to `watchdog_unregister_device'
> 
> This adds the normal 'select WATCHDOG_CORE' that is needed to
> ensure the driver always builds cleanly.
> 
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>

Reviewed-by: James Hogan <james.hogan@imgtec.com>

Cheers
James

> ---
>  drivers/watchdog/Kconfig | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig
> index 4f0e7be0da34..0dc61f1a552e 100644
> --- a/drivers/watchdog/Kconfig
> +++ b/drivers/watchdog/Kconfig
> @@ -1383,6 +1383,7 @@ config IMGPDC_WDT
>  	tristate "Imagination Technologies PDC Watchdog Timer"
>  	depends on HAS_IOMEM
>  	depends on METAG || MIPS || COMPILE_TEST
> +	select WATCHDOG_CORE
>  	help
>  	  Driver for Imagination Technologies PowerDown Controller
>  	  Watchdog Timer.
> -- 
> 2.7.0
> 

[toc] | [prev] | [next] | [standalone]


#1317003

FromGuenter Roeck <linux@roeck-us.net>
Date2016-01-25 17:50 +0100
Message-ID<qUSv2-6M7-55@gated-at.bofh.it>
In reply to#1316919
On 01/25/2016 08:06 AM, Arnd Bergmann wrote:
> The imgpdc_wdt driver can be built on all architectures with
> CONFIG_COMPILE_TEST, but fails if no other watchdog driver is
> enabled:
>
> drivers/watchdog/built-in.o: In function `pdc_wdt_remove':
> imgpdc_wdt.c:(.text+0x74): undefined reference to `watchdog_unregister_device'
>
> This adds the normal 'select WATCHDOG_CORE' that is needed to
> ensure the driver always builds cleanly.
>
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>

Reviewed-by: Guenter Roeck <linux@roeck-us.net>

> ---
>   drivers/watchdog/Kconfig | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig
> index 4f0e7be0da34..0dc61f1a552e 100644
> --- a/drivers/watchdog/Kconfig
> +++ b/drivers/watchdog/Kconfig
> @@ -1383,6 +1383,7 @@ config IMGPDC_WDT
>   	tristate "Imagination Technologies PDC Watchdog Timer"
>   	depends on HAS_IOMEM
>   	depends on METAG || MIPS || COMPILE_TEST
> +	select WATCHDOG_CORE
>   	help
>   	  Driver for Imagination Technologies PowerDown Controller
>   	  Watchdog Timer.
>

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web