[PATCH] staging: make board support depend on CLKDEV_LOOKUP

[PATCH] staging: make board support depend on CLKDEV_LOOKUP

[Paul Gortmaker]

Building allmodconfig for arch/cris currently fails with:

drivers/built-in.o: In function `board_staging_register_clock':
drivers/staging/board/board.c:131: undefined reference to `clk_add_alias'
make: *** [vmlinux] Error 1

The clk_add_alias lives in drivers/clk/clkdev.c and that file
is only compiled for CONFIG_CLKDEV_LOOKUP, so it would seem
we need to add a dependency on that.

Cc: Magnus Damm <damm+renesas@opensource.se>
Cc: Simon Horman <horms+renesas@verge.net.au>
Cc: Geert Uytterhoeven <geert+renesas@glider.be>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com>
---
 drivers/staging/board/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/board/Kconfig b/drivers/staging/board/Kconfig
index b8ee81840666..4ff5a795055f 100644
--- a/drivers/staging/board/Kconfig
+++ b/drivers/staging/board/Kconfig
@@ -1,6 +1,6 @@
 config STAGING_BOARD
 	bool "Staging Board Support"
-	depends on OF_ADDRESS
+	depends on OF_ADDRESS && CLKDEV_LOOKUP
 	help
 	  Select to enable per-board staging support code.
 
-- 
2.2.1

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH] staging: make board support depend on CLKDEV_LOOKUP

[Geert Uytterhoeven]

Hi Paul,

On Sat, Jun 20, 2015 at 9:27 PM, Paul Gortmaker
<paul.gortmaker@windriver.com> wrote:
> Building allmodconfig for arch/cris currently fails with:
>
> drivers/built-in.o: In function `board_staging_register_clock':
> drivers/staging/board/board.c:131: undefined reference to `clk_add_alias'
> make: *** [vmlinux] Error 1
>
> The clk_add_alias lives in drivers/clk/clkdev.c and that file
> is only compiled for CONFIG_CLKDEV_LOOKUP, so it would seem
> we need to add a dependency on that.

Thanks!

There seems to be another missing dependency on OF_IRQ:

drivers/built-in.o: In function `board_staging_gic_fixup_resources':
(.init.text+0x21c2): undefined reference to `irq_create_of_mapping'

> Cc: Magnus Damm <damm+renesas@opensource.se>
> Cc: Simon Horman <horms+renesas@verge.net.au>
> Cc: Geert Uytterhoeven <geert+renesas@glider.be>
> Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com>

Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>

> ---
>  drivers/staging/board/Kconfig | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/staging/board/Kconfig b/drivers/staging/board/Kconfig
> index b8ee81840666..4ff5a795055f 100644
> --- a/drivers/staging/board/Kconfig
> +++ b/drivers/staging/board/Kconfig
> @@ -1,6 +1,6 @@
>  config STAGING_BOARD
>         bool "Staging Board Support"
> -       depends on OF_ADDRESS
> +       depends on OF_ADDRESS && CLKDEV_LOOKUP
>         help
>           Select to enable per-board staging support code.

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH v2] staging: make board support depend on OF_IRQ and CLKDEV_LOOKUP

[Paul Gortmaker]

Building allmodconfig for arch/cris currently fails with:

drivers/built-in.o: In function `board_staging_register_clock':
drivers/staging/board/board.c:131: undefined reference to `clk_add_alias'
make: *** [vmlinux] Error 1

The clk_add_alias lives in drivers/clk/clkdev.c and that file
is only compiled for CONFIG_CLKDEV_LOOKUP, so it would seem
we need to add a dependency on that.

Geert also reported seeing this in his build coverage:

  There seems to be another missing dependency on OF_IRQ:

  drivers/built-in.o: In function `board_staging_gic_fixup_resources':
  (.init.text+0x21c2): undefined reference to `irq_create_of_mapping'

so we might as well fix that at the same time since it is
on the same line.

Cc: Magnus Damm <damm+renesas@opensource.se>
Cc: Simon Horman <horms+renesas@verge.net.au>
Cc: Geert Uytterhoeven <geert+renesas@glider.be>
Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com>
---
[v2: fold in Geert's OF_IRQ addition suggestion.]

 drivers/staging/board/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/board/Kconfig b/drivers/staging/board/Kconfig
index b8ee81840666..3f287c48e082 100644
--- a/drivers/staging/board/Kconfig
+++ b/drivers/staging/board/Kconfig
@@ -1,6 +1,6 @@
 config STAGING_BOARD
 	bool "Staging Board Support"
-	depends on OF_ADDRESS
+	depends on OF_ADDRESS && OF_IRQ && CLKDEV_LOOKUP
 	help
 	  Select to enable per-board staging support code.
 
-- 
2.2.1

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH v2] staging: make board support depend on OF_IRQ and CLKDEV_LOOKUP

[Paul Gortmaker]

On 15-06-20 07:28 PM, Paul Gortmaker wrote:
> Building allmodconfig for arch/cris currently fails with:
> 
> drivers/built-in.o: In function `board_staging_register_clock':
> drivers/staging/board/board.c:131: undefined reference to `clk_add_alias'
> make: *** [vmlinux] Error 1

Just checking this didn't fall through the cracks, as its absence
causes several arch breakage on linux-next for several days now:

(.init.text+0xf968): undefined reference to `clk_add_alias'

 
x86:     http://kisskb.ellerman.id.au/kisskb/buildresult/12448913/
powerpc: http://kisskb.ellerman.id.au/kisskb/buildresult/12449785/
parisc:  http://kisskb.ellerman.id.au/kisskb/buildresult/12449940/

and possibly others I've missed.

Paul.
--

> 
> The clk_add_alias lives in drivers/clk/clkdev.c and that file
> is only compiled for CONFIG_CLKDEV_LOOKUP, so it would seem
> we need to add a dependency on that.
> 
> Geert also reported seeing this in his build coverage:
> 
>   There seems to be another missing dependency on OF_IRQ:
> 
>   drivers/built-in.o: In function `board_staging_gic_fixup_resources':
>   (.init.text+0x21c2): undefined reference to `irq_create_of_mapping'
> 
> so we might as well fix that at the same time since it is
> on the same line.
> 
> Cc: Magnus Damm <damm+renesas@opensource.se>
> Cc: Simon Horman <horms+renesas@verge.net.au>
> Cc: Geert Uytterhoeven <geert+renesas@glider.be>
> Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>
> Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com>
> ---
> [v2: fold in Geert's OF_IRQ addition suggestion.]
> 
>  drivers/staging/board/Kconfig | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/staging/board/Kconfig b/drivers/staging/board/Kconfig
> index b8ee81840666..3f287c48e082 100644
> --- a/drivers/staging/board/Kconfig
> +++ b/drivers/staging/board/Kconfig
> @@ -1,6 +1,6 @@
>  config STAGING_BOARD
>  	bool "Staging Board Support"
> -	depends on OF_ADDRESS
> +	depends on OF_ADDRESS && OF_IRQ && CLKDEV_LOOKUP
>  	help
>  	  Select to enable per-board staging support code.
>  
> 

Re: [PATCH v2] staging: make board support depend on OF_IRQ and CLKDEV_LOOKUP

[Paul Gortmaker]

On 15-06-20 07:28 PM, Paul Gortmaker wrote:
> Building allmodconfig for arch/cris currently fails with:
> 
> drivers/built-in.o: In function `board_staging_register_clock':
> drivers/staging/board/board.c:131: undefined reference to `clk_add_alias'
> make: *** [vmlinux] Error 1

Just checking this didn't fall through the cracks, as its absence
causes several arch breakage on linux-next for several days now:

(.init.text+0xf968): undefined reference to `clk_add_alias'

 
x86:     http://kisskb.ellerman.id.au/kisskb/buildresult/12448913/
powerpc: http://kisskb.ellerman.id.au/kisskb/buildresult/12449785/
parisc:  http://kisskb.ellerman.id.au/kisskb/buildresult/12449940/

and possibly others I've missed.

Paul.
--

> 
> The clk_add_alias lives in drivers/clk/clkdev.c and that file
> is only compiled for CONFIG_CLKDEV_LOOKUP, so it would seem
> we need to add a dependency on that.
> 
> Geert also reported seeing this in his build coverage:
> 
>   There seems to be another missing dependency on OF_IRQ:
> 
>   drivers/built-in.o: In function `board_staging_gic_fixup_resources':
>   (.init.text+0x21c2): undefined reference to `irq_create_of_mapping'
> 
> so we might as well fix that at the same time since it is
> on the same line.
> 
> Cc: Magnus Damm <damm+renesas@opensource.se>
> Cc: Simon Horman <horms+renesas@verge.net.au>
> Cc: Geert Uytterhoeven <geert+renesas@glider.be>
> Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>
> Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com>
> ---
> [v2: fold in Geert's OF_IRQ addition suggestion.]
> 
>  drivers/staging/board/Kconfig | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/staging/board/Kconfig b/drivers/staging/board/Kconfig
> index b8ee81840666..3f287c48e082 100644
> --- a/drivers/staging/board/Kconfig
> +++ b/drivers/staging/board/Kconfig
> @@ -1,6 +1,6 @@
>  config STAGING_BOARD
>  	bool "Staging Board Support"
> -	depends on OF_ADDRESS
> +	depends on OF_ADDRESS && OF_IRQ && CLKDEV_LOOKUP
>  	help
>  	  Select to enable per-board staging support code.
>  
> 

Re: [PATCH v2] staging: make board support depend on OF_IRQ and CLKDEV_LOOKUP

[Guenter Roeck]

On Thu, Jun 25, 2015 at 10:13:22AM -0400, Paul Gortmaker wrote:
> On 15-06-20 07:28 PM, Paul Gortmaker wrote:
> > Building allmodconfig for arch/cris currently fails with:
> > 
> > drivers/built-in.o: In function `board_staging_register_clock':
> > drivers/staging/board/board.c:131: undefined reference to `clk_add_alias'
> > make: *** [vmlinux] Error 1
> 
> Just checking this didn't fall through the cracks, as its absence
> causes several arch breakage on linux-next for several days now:
> 
> (.init.text+0xf968): undefined reference to `clk_add_alias'
> 
>  
> x86:     http://kisskb.ellerman.id.au/kisskb/buildresult/12448913/
> powerpc: http://kisskb.ellerman.id.au/kisskb/buildresult/12449785/
> parisc:  http://kisskb.ellerman.id.au/kisskb/buildresult/12449940/
> 
This problem is now seen in mainline.

Also affected: alpha:allmodconfig, mips:allmodconfig, s390:allmodconfig.

and applying the patch fixes the problem.

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

Guenter