* Russell King - ARM Linux <linux@xxxxxxxxxxxxxxxx> [090115 12:37]: > On Tue, Jan 13, 2009 at 04:35:29PM +0200, Tony Lindgren wrote: > > * Tony Lindgren <tony@xxxxxxxxxxx> [090112 16:37]: > > > Hi, > > > > > > Here are some omap fixes for review. Some of these patches > > > were posted earlier. > > > > If no other comments, here's the pull request: > > Unfortunately, this breaks stuff. Whereas before pulling this I get: > > CC drivers/usb/core/hub.o > drivers/usb/core/hub.c: In function `usb_configure_device_otg': > drivers/usb/core/hub.c:1557: error: too few arguments to function `usb_port_suspend' > > while building my omap config, I now get this much earlier in the build: > > CC arch/arm/mach-omap1/board-h2.o > arch/arm/mach-omap1/board-h2.c:259: error: `OMAP_DMA_UART3_RX' undeclared here (not in a function) > arch/arm/mach-omap1/board-h2.c:259: error: initializer element is not constant > arch/arm/mach-omap1/board-h2.c:259: error: (near initialization for `h2_irda_data.rx_channel') > arch/arm/mach-omap1/board-h2.c:260: error: `OMAP_DMA_UART3_TX' undeclared here (not in a function) > arch/arm/mach-omap1/board-h2.c:260: error: initializer element is not constant > arch/arm/mach-omap1/board-h2.c:260: error: (near initialization for `h2_irda_data.tx_channel') > arch/arm/mach-omap1/board-h2.c:375: warning: 'h2_nand_dev_ready' defined but not used > > This has appeared because in 9f94ad8ab33417f98b363491d4730a67f4d92831, > the include of mach/omap-alsa.h was removed, which includes mach/dma.h, > and mach/dma.h is needed for these definitions. > > It would be far better to omit mach/dma.h from mach/omap-alsa.h since > mach/dma.h isn't strictly required for mach/omap-alsa.h (the file seems > to contain nothing that requires anything from mach/dma.h) and then these > kinds of accidents can be avoided. > > With that fixed, that then brings us on to: > > CC arch/arm/plat-omap/mcbsp.o > arch/arm/plat-omap/mcbsp.c: In function `omap_mcbsp_config': > arch/arm/plat-omap/mcbsp.c:177: error: `OMAP_MCBSP_REG_XCCR' undeclared (first use in this function) > arch/arm/plat-omap/mcbsp.c:177: error: (Each undeclared identifier is reported only once > arch/arm/plat-omap/mcbsp.c:177: error: for each function it appears in.) > arch/arm/plat-omap/mcbsp.c:178: error: `OMAP_MCBSP_REG_RCCR' undeclared (first use in this function) > > which is caused by 82b30192282d9b3da37d98af4b308d989ed6b92e. > > Not pulling. Hmm, I thought I build all the defconfigs before posting the series. Will update in just few mins. Tony -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html