From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751540AbbFYOPm (ORCPT ); Thu, 25 Jun 2015 10:15:42 -0400 Received: from mail1.windriver.com ([147.11.146.13]:58778 "EHLO mail1.windriver.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751137AbbFYOPh (ORCPT ); Thu, 25 Jun 2015 10:15:37 -0400 Message-ID: <558C0C82.3080709@windriver.com> Date: Thu, 25 Jun 2015 10:13:22 -0400 From: Paul Gortmaker User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 MIME-Version: 1.0 To: , Greg Kroah-Hartman CC: Magnus Damm , Simon Horman , Geert Uytterhoeven , "linux-next@vger.kernel.org" Subject: Re: [PATCH v2] staging: make board support depend on OF_IRQ and CLKDEV_LOOKUP References: <1434842903-24708-1-git-send-email-paul.gortmaker@windriver.com> In-Reply-To: <1434842903-24708-1-git-send-email-paul.gortmaker@windriver.com> Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit X-Originating-IP: [128.224.56.57] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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 > Cc: Simon Horman > Cc: Geert Uytterhoeven > Acked-by: Geert Uytterhoeven > Cc: Greg Kroah-Hartman > Signed-off-by: Paul Gortmaker > --- > [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. > >