From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lee Jones Subject: Re: [PATCH 8/9] isdn: Remove unused variable causing a compile build warning Date: Mon, 5 Nov 2012 10:44:50 +0100 Message-ID: <20121105094450.GA5220@gmail.com> References: <20121103224832.GA24262@gmail.com> <1352024091.1434.17.camel@x61.thuisdomein> <20121104105332.GC27124@gmail.com> <20121104.123013.700345271120279281.davem@davemloft.net> <20121105084419.GC2699@gmail.com> <1352106704.1434.25.camel@x61.thuisdomein> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: David Miller , linux-kernel@vger.kernel.org, isdn@linux-pingi.de, netdev@vger.kernel.org To: Paul Bolle Return-path: Content-Disposition: inline In-Reply-To: <1352106704.1434.25.camel@x61.thuisdomein> Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org On Mon, 05 Nov 2012, Paul Bolle wrote: > On Mon, 2012-11-05 at 09:44 +0100, Lee Jones wrote: > > On Sun, 04 Nov 2012, David Miller wrote: > > > I think the most appropriate thing to do is make CONFIG_ISDN depe= nd > > > upon CONFIG_NETDEVICES in the Kconfig file. > >=20 > > ... and then remove the CONFIG_NETDEVICES guards in isdn_common.c? >=20 > If ISDN would depend on NETDEVICES, ISDN_I4L would too, since it depe= nds > on ISDN. In that case CONFIG_NETDEVICES would always be true when > compiling isdn_common.c. That would make these guards pointless. (The > dependency of ISDN_PPP on NETDEVICES would then also be pointless.) I'll submit a patch based on your comments. Thanks for your time Paul. --=20 Lee Jones Linaro ST-Ericsson Landing Team Lead Linaro.org =E2=94=82 Open source software for ARM SoCs =46ollow Linaro: Facebook | Twitter | Blog