Hi Kevin, On Tue, Mar 8, 2011 at 1:12 AM, Kevin Hilman <khilman@xxxxxx> wrote: > Lesly A M <leslyam@xxxxxx> writes: > >> Using TWL4030 power scripts to control the LDOs and DCDC regulators >> while going to low power mode. >> >> Signed-off-by: Lesly A M <leslyam@xxxxxx> > > Minor: Re: Subject: OMAP is an acronym, please capitalize. Ok >> Cc: Nishanth Menon <nm@xxxxxx> >> Cc: David Derrick <dderrick@xxxxxx> >> Cc: Samuel Ortiz <sameo@xxxxxxxxxxxxxxx> >> --- >> arch/arm/mach-omap2/Makefile | 3 +- >> arch/arm/mach-omap2/board-3430sdp.c | 93 +---------------------------------- >> 2 files changed, 4 insertions(+), 92 deletions(-) >> >> diff --git a/arch/arm/mach-omap2/Makefile b/arch/arm/mach-omap2/Makefile >> index 742ca67..bb11329 100644 >> --- a/arch/arm/mach-omap2/Makefile >> +++ b/arch/arm/mach-omap2/Makefile >> @@ -175,7 +175,8 @@ obj-$(CONFIG_MACH_OMAP3_PANDORA) += board-omap3pandora.o \ >> hsmmc.o >> obj-$(CONFIG_MACH_OMAP_3430SDP) += board-3430sdp.o \ >> hsmmc.o \ >> - board-flash.o >> + board-flash.o \ >> + twl4030.o > > This should not be board-specific in the Makefile, otherwise it will > have to be added for every single board that uses it. > > Rather, it should probably be built as long as CONFIG_TWL4030_POWER=y. > Also, please be sure to build test when that option is disabled as well. Ok, I will make the changes as mentioned in next thread ... Thanks & Regards, Lesly A M -- 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
