On 06/20/2012 06:45 AM, Bhupesh SHARMA wrote: > Hi David, > >> -----Original Message----- >> From: David Miller [mailto:davem@davemloft.net] >> Sent: Wednesday, June 20, 2012 10:07 AM >> To: Bhupesh SHARMA >> Cc: sfr@canb.auug.org.au; netdev@vger.kernel.org; linux- >> next@vger.kernel.org; linux-kernel@vger.kernel.org; >> federico.vaga@gmail.com; Giancarlo ASNAGHI; wg@grandegger.com; >> mkl@pengutronix.de >> Subject: Re: linux-next: build failure after merge of the net-next tree >> >> From: Bhupesh SHARMA >> Date: Wed, 20 Jun 2012 12:27:11 +0800 >> >>> clk_get/clk_put* variants are usually used by ARM platforms. >>> Protecting their calls under macro 'CONFIG_HAVE_CLK' should solve the >> problem. >> >> No, we don't pepper foo.c files with crappy ifdefs. > > So, whether adding a check in Kconfig for HAVE_CLK be a proper solution ? > But that will limit the compilation of this driver for only platforms which are ARM based. > > One may need to support this driver on x86 like platforms also.. I think the driver is currently used/tested only on ARM only, so what about adding a HAVE_CLK dependency, until Viresh Kumar's patches are available in net-next. Later we can remove it. regards, Marc -- Pengutronix e.K. | Marc Kleine-Budde | Industrial Linux Solutions | Phone: +49-231-2826-924 | Vertretung West/Dortmund | Fax: +49-5121-206917-5555 | Amtsgericht Hildesheim, HRA 2686 | http://www.pengutronix.de |