Message ID | 20121114164801.GQ6801@atomide.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
* Tony Lindgren <tony@atomide.com> [121114 08:50]: > * Tomi Valkeinen <tomi.valkeinen@ti.com> [121114 03:47]: > > On 2012-11-14 11:26, Russell King - ARM Linux wrote: > > > OMAP* allnoconfig fails: > > > > > > arch/arm/mach-omap2/built-in.o: In function `omap_dss_set_min_bus_tput': > > > twl-common.c:(.text+0x1e08): undefined reference to `omap_pm_set_min_bus_tput' > > > arch/arm/mach-omap2/built-in.o: In function `omap_hwmod_init_postsetup': > > > twl-common.c:(.init.text+0x8f8): undefined reference to `omap_pm_if_early_init' > > > arch/arm/mach-omap2/built-in.o: In function `omap_serial_init_port': > > > twl-common.c:(.init.text+0x1284): undefined reference to `omap_pm_get_dev_context_loss_count' > > > arch/arm/mach-omap2/built-in.o: In function `omap_timer_init': > > > twl-common.c:(.init.text+0x1544): undefined reference to `omap_pm_get_dev_context_loss_count' > > > arch/arm/mach-omap2/built-in.o: In function `omap2_common_pm_init': > > > twl-common.c:(.init.text+0x1af0): undefined reference to `omap_pm_if_init' > > > arch/arm/mach-omap2/built-in.o: In function `omap2_gpio_dev_init': > > > twl-common.c:(.init.text+0x2168): undefined reference to `omap_pm_get_dev_context_loss_count' > > > arch/arm/mach-omap2/built-in.o: In function `omap_display_init': > > > twl-common.c:(.init.text+0x25cc): undefined reference to `omap_pm_get_dev_context_loss_count' > > > > I'm not able to reproduce this. I took v3.7-rc5, and the omap4430-sdp > > noconfig > > (http://www.arm.linux.org.uk/developer/build/file.php?type=config&idx=2711), > > and it builds fine for me. > > It happens if in arm-soc/for-next and rmk/for-next. Looks like the > CONFIG_OMAP_PM_NOOP can't be under CONFIG_PM in the makefile where I moved it > in commit 6e740f9a8. This should say just "It happens in arm-soc/for-next". > Looks like in my test configs I run make oldnoconfig on Russell's seed > config, and I do get CONFIG_PM=y set while Russell's generated config > does not have that. No ideas yet why oldnoconfig add CONFIG_PM=y.. > > Anyways, patch below to make it behave like earlier. > > Regards, > > Tony > > > From: Tony Lindgren <tony@atomide.com> > Date: Wed, 14 Nov 2012 08:40:00 -0800 > Subject: [PATCH] ARM: OMAP: Fix compile for OMAP_PM_NOOP if PM is not selected > > Commit 6e740f9a8 (ARM: OMAP: Move omap-pm-noop.c local to mach-omap2) > moved omap-pm-noop to be local to mach-omap2. However, the makefile > entry got placed within ifeq ($(CONFIG_PM),y) which was not the > case earlier. > > Fix the issue by moving it out of the ifeq ($(CONFIG_PM),y) in > the makefile as these stubs are needed also when PM is not set. > > Reported-by: Russell King <rmk+kernel@arm.linux.org.uk> > Signed-off-by: Tony Lindgren <tony@atomide.com> > > --- a/arch/arm/mach-omap2/Makefile > +++ b/arch/arm/mach-omap2/Makefile > @@ -68,6 +68,8 @@ obj-$(CONFIG_ARCH_OMAP4) += opp4xxx_data.o > endif > > # Power Management > +obj-$(CONFIG_OMAP_PM_NOOP) += omap-pm-noop.o > + > ifeq ($(CONFIG_PM),y) > obj-$(CONFIG_ARCH_OMAP2) += pm24xx.o > obj-$(CONFIG_ARCH_OMAP2) += sleep24xx.o > @@ -75,7 +77,6 @@ obj-$(CONFIG_ARCH_OMAP3) += pm34xx.o sleep34xx.o > obj-$(CONFIG_ARCH_OMAP4) += pm44xx.o omap-mpuss-lowpower.o > obj-$(CONFIG_SOC_OMAP5) += omap-mpuss-lowpower.o > obj-$(CONFIG_PM_DEBUG) += pm-debug.o > -obj-$(CONFIG_OMAP_PM_NOOP) += omap-pm-noop.o > > obj-$(CONFIG_POWER_AVS_OMAP) += sr_device.o > obj-$(CONFIG_POWER_AVS_OMAP_CLASS3) += smartreflex-class3.o > -- > To unsubscribe from this list: send the line "unsubscribe linux-omap" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
* Tony Lindgren <tony@atomide.com> [121114 09:02]: > * Tony Lindgren <tony@atomide.com> [121114 08:50]: > > * Tomi Valkeinen <tomi.valkeinen@ti.com> [121114 03:47]: > > > On 2012-11-14 11:26, Russell King - ARM Linux wrote: > > > > OMAP* allnoconfig fails: > > > > > > > > arch/arm/mach-omap2/built-in.o: In function `omap_dss_set_min_bus_tput': > > > > twl-common.c:(.text+0x1e08): undefined reference to `omap_pm_set_min_bus_tput' > > > > arch/arm/mach-omap2/built-in.o: In function `omap_hwmod_init_postsetup': > > > > twl-common.c:(.init.text+0x8f8): undefined reference to `omap_pm_if_early_init' > > > > arch/arm/mach-omap2/built-in.o: In function `omap_serial_init_port': > > > > twl-common.c:(.init.text+0x1284): undefined reference to `omap_pm_get_dev_context_loss_count' > > > > arch/arm/mach-omap2/built-in.o: In function `omap_timer_init': > > > > twl-common.c:(.init.text+0x1544): undefined reference to `omap_pm_get_dev_context_loss_count' > > > > arch/arm/mach-omap2/built-in.o: In function `omap2_common_pm_init': > > > > twl-common.c:(.init.text+0x1af0): undefined reference to `omap_pm_if_init' > > > > arch/arm/mach-omap2/built-in.o: In function `omap2_gpio_dev_init': > > > > twl-common.c:(.init.text+0x2168): undefined reference to `omap_pm_get_dev_context_loss_count' > > > > arch/arm/mach-omap2/built-in.o: In function `omap_display_init': > > > > twl-common.c:(.init.text+0x25cc): undefined reference to `omap_pm_get_dev_context_loss_count' > > > > > > I'm not able to reproduce this. I took v3.7-rc5, and the omap4430-sdp > > > noconfig > > > (http://www.arm.linux.org.uk/developer/build/file.php?type=config&idx=2711), > > > and it builds fine for me. > > > > It happens if in arm-soc/for-next and rmk/for-next. Looks like the > > CONFIG_OMAP_PM_NOOP can't be under CONFIG_PM in the makefile where I moved it > > in commit 6e740f9a8. > > This should say just "It happens in arm-soc/for-next". > > > Looks like in my test configs I run make oldnoconfig on Russell's seed > > config, and I do get CONFIG_PM=y set while Russell's generated config > > does not have that. No ideas yet why oldnoconfig add CONFIG_PM=y.. Looks like the way to do this not to use oldnoconfig but to do: $ make KCONFIG_ALLCONFIG=../configs/rmk-omap4430-sdp-noconfig allnoconfig It seems that oldnoconfig will pick the new options that have default y? Regards, Tony -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Wed, Nov 14, 2012 at 09:17:31AM -0800, Tony Lindgren wrote: > Looks like the way to do this not to use oldnoconfig but to do: > > $ make KCONFIG_ALLCONFIG=../configs/rmk-omap4430-sdp-noconfig allnoconfig > > It seems that oldnoconfig will pick the new options that have default y? That is the correct way to tell Kconfig to use the seed configuration - that forces the options in the ALLCONFIG file to the values in there in the final configuration while setting the rest of the options as desired by the all*config target. -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
--- a/arch/arm/mach-omap2/Makefile +++ b/arch/arm/mach-omap2/Makefile @@ -68,6 +68,8 @@ obj-$(CONFIG_ARCH_OMAP4) += opp4xxx_data.o endif # Power Management +obj-$(CONFIG_OMAP_PM_NOOP) += omap-pm-noop.o + ifeq ($(CONFIG_PM),y) obj-$(CONFIG_ARCH_OMAP2) += pm24xx.o obj-$(CONFIG_ARCH_OMAP2) += sleep24xx.o @@ -75,7 +77,6 @@ obj-$(CONFIG_ARCH_OMAP3) += pm34xx.o sleep34xx.o obj-$(CONFIG_ARCH_OMAP4) += pm44xx.o omap-mpuss-lowpower.o obj-$(CONFIG_SOC_OMAP5) += omap-mpuss-lowpower.o obj-$(CONFIG_PM_DEBUG) += pm-debug.o -obj-$(CONFIG_OMAP_PM_NOOP) += omap-pm-noop.o obj-$(CONFIG_POWER_AVS_OMAP) += sr_device.o obj-$(CONFIG_POWER_AVS_OMAP_CLASS3) += smartreflex-class3.o