Has anybody tried building latest linux-omap master ? The build is breaking for other OMAP processors. CC arch/arm/mach-omap2/id.o arch/arm/mach-omap2/id.c: In function 'omap3_cpuinfo': arch/arm/mach-omap2/id.c:269: error: implicit declaration of function 'cpu_is_omap3630' make[1]: *** [arch/arm/mach-omap2/id.o] Error 1 make: *** [arch/arm/mach-omap2] Error 2 This is because of " 0a9b95f21995aa3cdda82ebc6e77b0b2ab401861" omap: Introduce OMAP3630 Below patch from Vikram fixes the build break. > -----Original Message----- > From: linux-omap-owner@xxxxxxxxxxxxxxx [mailto:linux-omap- > owner@xxxxxxxxxxxxxxx] On Behalf Of Pandita, Vikram > Sent: Tuesday, October 13, 2009 2:38 AM > To: Menon, Nishanth > Cc: linux-omap@xxxxxxxxxxxxxxx > Subject: RE: [PATCH] [RFC] omap: 3630: default cpu_is_omap3630 to zero > > > > >-----Original Message----- > >From: Menon, Nishanth > >Sent: Monday, October 12, 2009 4:05 PM > >To: Pandita, Vikram > >Cc: linux-omap@xxxxxxxxxxxxxxx > >Subject: RE: [PATCH] [RFC] omap: 3630: default cpu_is_omap3630 to zero > > > >> -----Original Message----- > >> From: Pandita, Vikram > >> Sent: Monday, October 12, 2009 3:51 PM > >> > >> make default cpu_is_omap3630() return zero > >> > >> Signed-off-by: Vikram Pandita <vikram.pandita@xxxxxx> > >> --- > >> arch/arm/plat-omap/include/mach/cpu.h | 2 ++ > >> 1 files changed, 2 insertions(+), 0 deletions(-) > >> > >> diff --git a/arch/arm/plat-omap/include/mach/cpu.h b/arch/arm/plat- > >> omap/include/mach/cpu.h > >> index da9e8f8..940946e 100644 > >> --- a/arch/arm/plat-omap/include/mach/cpu.h > >> +++ b/arch/arm/plat-omap/include/mach/cpu.h > >> @@ -322,6 +322,7 @@ IS_OMAP_TYPE(3430, 0x3430) > >> #define cpu_is_omap2423() 0 > >> #define cpu_is_omap2430() 0 > >> #define cpu_is_omap3430() 0 > >> +#define cpu_is_omap3630() 0 > >> > >> /* > >> * Whether we have MULTI_OMAP1 or not, we still need to distinguish > >> @@ -386,6 +387,7 @@ IS_OMAP_TYPE(3430, 0x3430) > >> (omap3_has_sgx()) & \ > >> (!omap3_has_iva())) > >> # define cpu_is_omap3530 (cpu_is_omap3430()) > >> +# undef cpu_is_omap3630() > >> # define cpu_is_omap3630() is_omap363x() > >> #endif > >Dumb question: why is this needed? cpu_is_omap3530,15,25,03 don't seems > to declare these.. > > If in some file, you want to distinguish between 3630 vs 3430, > and the build is for 3430 only, then cpu_is_omap3630() should return 0. > > Eg: opp table allocation based on run time check > > Omap35xx may also need it for the opp in future I guess. > -- 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