From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752524AbcGSIaj (ORCPT ); Tue, 19 Jul 2016 04:30:39 -0400 Received: from lb3-smtp-cloud2.xs4all.net ([194.109.24.29]:36204 "EHLO lb3-smtp-cloud2.xs4all.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752391AbcGSIac (ORCPT ); Tue, 19 Jul 2016 04:30:32 -0400 Subject: Re: [PATCH 2/2] [media] cec: add RC_CORE dependency To: Arnd Bergmann , Mauro Carvalho Chehab References: <20160719081040.2685845-1-arnd@arndb.de> <20160719081040.2685845-2-arnd@arndb.de> Cc: Hans Verkuil , Greg Kroah-Hartman , Kamil Debski , linux-media@vger.kernel.org, devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org From: Hans Verkuil Message-ID: <578DE51E.8080604@xs4all.nl> Date: Tue, 19 Jul 2016 10:30:22 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Icedove/38.5.0 MIME-Version: 1.0 In-Reply-To: <20160719081040.2685845-2-arnd@arndb.de> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 07/19/16 10:10, Arnd Bergmann wrote: > We cannot build the cec driver when the RC core is a module > and cec is built-in: > > drivers/staging/built-in.o: In function `cec_allocate_adapter': > :(.text+0x134): undefined reference to `rc_allocate_device' > drivers/staging/built-in.o: In function `cec_register_adapter': > :(.text+0x304): undefined reference to `rc_register_device' > > This adds an explicit dependency to avoid this case. We still > allow building when CONFIG_RC_CORE is disabled completely, > as the driver has checks for this case itself. This makes no sense: the rc_allocate_device and rc_register_device are under: #if IS_REACHABLE(CONFIG_RC_CORE) So it shouldn't be enabled at all, should it? Regards, Hans > > Signed-off-by: Arnd Bergmann > --- > I originally submitted this on June 29, but it may have gotten > lost as out of the three patch series, one patch got replaced > and another patch got applied, but nothing happened on this one. > --- > drivers/staging/media/cec/Kconfig | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/staging/media/cec/Kconfig b/drivers/staging/media/cec/Kconfig > index 21457a1f6c9f..c623bd32a5b8 100644 > --- a/drivers/staging/media/cec/Kconfig > +++ b/drivers/staging/media/cec/Kconfig > @@ -1,6 +1,7 @@ > config MEDIA_CEC > bool "CEC API (EXPERIMENTAL)" > depends on MEDIA_SUPPORT > + depends on RC_CORE || !RC_CORE > select MEDIA_CEC_EDID > ---help--- > Enable the CEC API. >