On Fri, Apr 13, 2012 at 10:01:52AM +0800, Shawn Guo wrote: > On 13 April 2012 09:22, Paul Gortmaker <paul.gortmaker@xxxxxxxxxxxxx> wrote: > > Hi Mark, > > > > The commit in linux-next: > > > > ----------------- > > commit b2bc9e0ee349db84035d8141e85babc7986594d7 > > Author: Mark Brown <broonie@xxxxxxxxxxxxx> > > Date: Mon Apr 2 12:30:35 2012 +0100 > > > > ARM: 7366/1: amba: Remove AMBA level regulator support > > ----------------- > > > > is pointed at by git bisect for the u8500_defconfig build failure: > > > > drivers/dma/ste_dma40.c:2708:3: error: implicit declaration of function > > 'regulator_disable' [-Werror=implicit-function-declaration] > > drivers/dma/ste_dma40.c:2747:3: error: implicit declaration of function > > 'regulator_enable' [-Werror=implicit-function-declaration] > > drivers/dma/ste_dma40.c:3260:3: error: implicit declaration of function > > 'regulator_get' [-Werror=implicit-function-declaration] > > drivers/dma/ste_dma40.c:3271:4: error: implicit declaration of function > > 'regulator_put' [-Werror=implicit-function-declaration] > > > That's because Mark's patch removes "#include > <linux/regulator/consumer.h>" included indirectly by > drivers/dma/ste_dma40.c from linux/amba/bus.h. The ste_dma40.c should > really directly includes consumer.h. Okay, I'll drop this patch because it's causing regressions, so it can't be pushed as a 'fix' during -rc. -- To unsubscribe from this list: send the line "unsubscribe linux-next" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
