Re: [PATCH 2/2] AM35xx: Add clock support for new modules on AM35xx

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



One more thing.

On Tue, 19 Jan 2010, Paul Walmsley wrote:

> On Tue, 19 Jan 2010, Ranjith Lohithakshan wrote:
>
> > diff --git a/arch/arm/mach-omap2/cm-regbits-34xx.h b/arch/arm/mach-omap2/cm-regbits-34xx.h
> > index 6923deb..4ff0e24 100644
> > --- a/arch/arm/mach-omap2/cm-regbits-34xx.h
> > +++ b/arch/arm/mach-omap2/cm-regbits-34xx.h
> > @@ -168,6 +168,12 @@
> >  #define OMAP3430_EN_SDRC				(1 << 1)
> >  #define OMAP3430_EN_SDRC_SHIFT				1
> >  
> > +/* AM35XX specific CM_ICLKEN1_CORE bits */
> > +#define AM35XX_EN_IPSS					(1 << 4)

For masks like the above, please put _MASK on the end of these macros.
I realize that this file is not consistent, but that's the practice that 
we should use going forward, and soon a patch will appear to fix the
old-style mask macros in these files.

> > +#define AM35XX_EN_IPSS_SHIFT				4
> > +#define AM35XX_EN_UART4 				(1 << 23)

same as above

> > +#define AM35XX_EN_UART4_SHIFT				23
> > +
> >  /* CM_ICLKEN2_CORE */
> >  #define OMAP3430_EN_PKA					(1 << 4)
> >  #define OMAP3430_EN_PKA_SHIFT				4
> > @@ -220,6 +226,10 @@
> >  #define OMAP3430_ST_SSI_STDBY_SHIFT			0
> >  #define OMAP3430_ST_SSI_STDBY_MASK			(1 << 0)
> >  
> > +/* AM35xx specific CM_IDLEST1_CORE bits */
> > +#define AM35XX_ST_IPSS_SHIFT				5
> > +#define AM35XX_ST_IPSS_MASK 				(1 << 5)
> > +
> >  /* CM_IDLEST2_CORE */
> >  #define OMAP3430_ST_PKA_SHIFT				4
> >  #define OMAP3430_ST_PKA_MASK				(1 << 4)


- Paul
--
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

[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux