From mboxrd@z Thu Jan 1 00:00:00 1970 From: Igor Grinberg Subject: Re: [PATCH V2 3/4] ARM: AM335x: Fix warning in timer.c Date: Thu, 29 Nov 2012 09:10:48 +0200 Message-ID: <50B70A78.3010704@compulab.co.il> References: <1354140300-16825-1-git-send-email-jon-hunter@ti.com> <1354140300-16825-4-git-send-email-jon-hunter@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Received: from softlayer.compulab.co.il ([50.23.254.55]:41562 "EHLO compulab.co.il" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751100Ab2K2HKz (ORCPT ); Thu, 29 Nov 2012 02:10:55 -0500 In-Reply-To: <1354140300-16825-4-git-send-email-jon-hunter@ti.com> Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: Jon Hunter Cc: Tony Lindgren , linux-omap , linux-arm On 11/29/12 00:04, Jon Hunter wrote: > When compiling the kernel with configuration options ... >=20 > # CONFIG_ARCH_OMAP2 is not set > # CONFIG_ARCH_OMAP3 is not set > # CONFIG_ARCH_OMAP4 is not set > # CONFIG_SOC_OMAP5 is not set > CONFIG_SOC_AM33XX=3Dy >=20 > ... the following build warning is seen. >=20 > CC arch/arm/mach-omap2/timer.o > arch/arm/mach-omap2/timer.c:395:19: warning: =E2=80=98omap2_sync32k= _clocksource_init=E2=80=99 > defined but not used [-Wunused-function] >=20 > This issue was introduced by commit 6f80b3b (ARM: OMAP2+: timer: remo= ve > CONFIG_OMAP_32K_TIMER) where the omap2_sync32k_clocksource_init() is = no > longer referenced by the timer initialisation function for the AM335x > device as it has no 32k-sync timer. >=20 > Fix this by adding the "__maybe_unused" compiler directive to the > omap2_sync32k_clocksource_init() function to indicate that this funct= ion > may be used for certain configurations. >=20 > Cc: Igor Grinberg >=20 > Signed-off-by: Jon Hunter Acked-by: Igor Grinberg --=20 Regards, Igor. -- To unsubscribe from this list: send the line "unsubscribe linux-omap" i= n the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html From mboxrd@z Thu Jan 1 00:00:00 1970 From: grinberg@compulab.co.il (Igor Grinberg) Date: Thu, 29 Nov 2012 09:10:48 +0200 Subject: [PATCH V2 3/4] ARM: AM335x: Fix warning in timer.c In-Reply-To: <1354140300-16825-4-git-send-email-jon-hunter@ti.com> References: <1354140300-16825-1-git-send-email-jon-hunter@ti.com> <1354140300-16825-4-git-send-email-jon-hunter@ti.com> Message-ID: <50B70A78.3010704@compulab.co.il> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 11/29/12 00:04, Jon Hunter wrote: > When compiling the kernel with configuration options ... > > # CONFIG_ARCH_OMAP2 is not set > # CONFIG_ARCH_OMAP3 is not set > # CONFIG_ARCH_OMAP4 is not set > # CONFIG_SOC_OMAP5 is not set > CONFIG_SOC_AM33XX=y > > ... the following build warning is seen. > > CC arch/arm/mach-omap2/timer.o > arch/arm/mach-omap2/timer.c:395:19: warning: ?omap2_sync32k_clocksource_init? > defined but not used [-Wunused-function] > > This issue was introduced by commit 6f80b3b (ARM: OMAP2+: timer: remove > CONFIG_OMAP_32K_TIMER) where the omap2_sync32k_clocksource_init() is no > longer referenced by the timer initialisation function for the AM335x > device as it has no 32k-sync timer. > > Fix this by adding the "__maybe_unused" compiler directive to the > omap2_sync32k_clocksource_init() function to indicate that this function > may be used for certain configurations. > > Cc: Igor Grinberg > > Signed-off-by: Jon Hunter Acked-by: Igor Grinberg -- Regards, Igor.