[PATCH] ARM: OMAP2+: am33xx: Make am33xx as a separate class

[Hiremath, Vaibhav]

On Wed, May 09, 2012 at 01:38:34, Tony Lindgren wrote:
> * Vaibhav Hiremath <hvaibhav at ti.com> [120508 08:32]:
> > --- a/arch/arm/mach-omap2/Kconfig
> > +++ b/arch/arm/mach-omap2/Kconfig
> > @@ -123,8 +123,11 @@ config SOC_OMAPTI81XX
> >  
> >  config SOC_OMAPAM33XX
> >  	bool "AM33XX support"
> > -	depends on ARCH_OMAP3
> > +	depends on ARCH_OMAP2PLUS
> 
> This is not needed, it's inside if ARCH_OMAP2PLUS block.
> 

Good point, I will remove it.


> > --- a/arch/arm/mach-omap2/Makefile
> > +++ b/arch/arm/mach-omap2/Makefile
> > @@ -16,6 +16,7 @@ secure-common				= omap-smc.o omap-secure.o
> >  obj-$(CONFIG_ARCH_OMAP2) += $(omap-2-3-common) $(hwmod-common)
> >  obj-$(CONFIG_ARCH_OMAP3) += $(omap-2-3-common) $(hwmod-common) $(secure-common)
> >  obj-$(CONFIG_ARCH_OMAP4) += prm44xx.o $(hwmod-common) $(secure-common)
> > +obj-$(CONFIG_SOC_OMAPAM33XX) += irq.o $(hwmod-common)
> >  
> >  ifneq ($(CONFIG_SND_OMAP_SOC_MCBSP),)
> >  obj-y += mcbsp.o
> > @@ -93,6 +94,7 @@ obj-$(CONFIG_ARCH_OMAP4)		+= prcm.o cm2xxx_3xxx.o cminst44xx.o \
> >  					   cm44xx.o prcm_mpu44xx.o \
> >  					   prminst44xx.o vc44xx_data.o \
> >  					   vp44xx_data.o prm44xx.o
> > +obj-$(CONFIG_SOC_OMAPAM33XX)		+= prcm.o
> >  
> >  # OMAP voltage domains
> >  voltagedomain-common			:= voltage.o vc.o vp.o
> > @@ -102,6 +104,7 @@ obj-$(CONFIG_ARCH_OMAP3)		+= $(voltagedomain-common) \
> >  					   voltagedomains3xxx_data.o
> >  obj-$(CONFIG_ARCH_OMAP4)		+= $(voltagedomain-common) \
> >  					   voltagedomains44xx_data.o
> > +obj-$(CONFIG_SOC_OMAPAM33XX)		+= $(voltagedomain-common)
> >  
> >  # OMAP powerdomain framework
> >  powerdomain-common			+= powerdomain.o powerdomain-common.o
> > @@ -116,6 +119,7 @@ obj-$(CONFIG_ARCH_OMAP3)		+= $(powerdomain-common) \
> >  obj-$(CONFIG_ARCH_OMAP4)		+= $(powerdomain-common) \
> >  					   powerdomain44xx.o \
> >  					   powerdomains44xx_data.o
> > +obj-$(CONFIG_SOC_OMAPAM33XX)		+= $(powerdomain-common)
> >  
> >  # PRCM clockdomain control
> >  clockdomain-common			+= clockdomain.o \
> > @@ -132,6 +136,7 @@ obj-$(CONFIG_ARCH_OMAP3)		+= $(clockdomain-common) \
> >  obj-$(CONFIG_ARCH_OMAP4)		+= $(clockdomain-common) \
> >  					   clockdomain44xx.o \
> >  					   clockdomains44xx_data.o
> > +obj-$(CONFIG_SOC_OMAPAM33XX)		+= $(clockdomain-common)
> >  
> >  # Clock framework
> >  obj-$(CONFIG_ARCH_OMAP2)		+= $(clock-common) clock2xxx.o \
> > @@ -149,6 +154,7 @@ obj-$(CONFIG_ARCH_OMAP3)		+= $(clock-common) clock3xxx.o \
> >  					   clkt_iclk.o
> >  obj-$(CONFIG_ARCH_OMAP4)		+= $(clock-common) clock44xx_data.o \
> >  					   dpll3xxx.o dpll44xx.o
> > +obj-$(CONFIG_SOC_OMAPAM33XX)		+= $(clock-common) dpll3xxx.o
> >  
> >  # OMAP2 clock rate set data (old "OPP" data)
> >  obj-$(CONFIG_SOC_OMAP2420)		+= opp2420_data.o
> 
> Can you please rebase this on what's now in the cleanup branch at commit
> 382ae00105d9989a821f458084e0428bce10b969? That should simplify the Makefile
> changes a bit.
> 

Yes, will do it and submit next version today.

> Other than that looks good to me.
> 

Thanks,
Vaibhav