* [PATCH] AM35xx: Add AM35xx specific control module registers
@ 2009-11-03 17:37 Ranjith Lohithakshan
2009-12-14 22:26 ` Paul Walmsley
0 siblings, 1 reply; 4+ messages in thread
From: Ranjith Lohithakshan @ 2009-11-03 17:37 UTC (permalink / raw)
To: linux-omap; +Cc: ranjithl
AM3517/05 has a few additional control module registers defined mainly
to control the new IP's. This patch adds support for those new registers.
Signed-off-by: Ranjith Lohithakshan <ranjithl@ti.com>
---
arch/arm/plat-omap/include/plat/control.h | 17 +++++++++++++++++
1 files changed, 17 insertions(+), 0 deletions(-)
diff --git a/arch/arm/plat-omap/include/plat/control.h b/arch/arm/plat-omap/include/plat/control.h
index fdb6300..9ba907a 100644
--- a/arch/arm/plat-omap/include/plat/control.h
+++ b/arch/arm/plat-omap/include/plat/control.h
@@ -147,6 +147,15 @@
#define OMAP343X_CONTROL_PBIAS_LITE (OMAP2_CONTROL_GENERAL + 0x02b0)
#define OMAP343X_CONTROL_TEMP_SENSOR (OMAP2_CONTROL_GENERAL + 0x02b4)
+/* OMAP3517/AM3517 only CONTROL_GENERAL register offsets */
+#define OMAP3517_CONTROL_MSUSPENDMUX_6 (OMAP2_CONTROL_GENERAL + 0x0038)
+#define OMAP3517_CONTROL_DEVCONF2 (OMAP2_CONTROL_GENERAL + 0x0310)
+#define OMAP3517_CONTROL_DEVCONF3 (OMAP2_CONTROL_GENERAL + 0x0314)
+#define OMAP3517_CONTROL_CBA_PRIORITY (OMAP2_CONTROL_GENERAL + 0x0320)
+#define OMAP3517_CONTROL_LVL_INTR_CLEAR (OMAP2_CONTROL_GENERAL + 0x0324)
+#define OMAP3517_CONTROL_IP_SW_RESET (OMAP2_CONTROL_GENERAL + 0x0328)
+#define OMAP3517_CONTROL_IP_CLK_CTRL (OMAP2_CONTROL_GENERAL + 0x032C)
+
/* 34xx D2D idle-related pins, handled by PM core */
#define OMAP3_PADCONF_SAD2D_MSTANDBY 0x250
#define OMAP3_PADCONF_SAD2D_IDLEACK 0x254
@@ -238,6 +247,14 @@
#define FEAT_NEON 0
#define FEAT_NEON_NONE 1
+/* OMAP3517_CONTROL_IP_CLK_CTRL bits */
+#define OMAP3517_USBOTG_VBUSP_CLK_SHIFT 0
+#define OMAP3517_CPGMAC_VBUSP_CLK_SHIFT 1
+#define OMAP3517_VPFE_VBUSP_CLK_SHIFT 2
+#define OMAP3517_HECC_VBUSP_CLK_SHIFT 3
+#define OMAP3517_USBOTG_FCLK_SHIFT 8
+#define OMAP3517_CPGMAC_FCLK_SHIFT 9
+#define OMAP3517_VPFE_FCLK_SHIFT 10
#ifndef __ASSEMBLY__
#if defined(CONFIG_ARCH_OMAP2) || defined(CONFIG_ARCH_OMAP3) || \
--
1.6.2.4
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] AM35xx: Add AM35xx specific control module registers
2009-11-03 17:37 [PATCH] AM35xx: Add AM35xx specific control module registers Ranjith Lohithakshan
@ 2009-12-14 22:26 ` Paul Walmsley
2009-12-14 22:33 ` Paul Walmsley
0 siblings, 1 reply; 4+ messages in thread
From: Paul Walmsley @ 2009-12-14 22:26 UTC (permalink / raw)
To: Ranjith Lohithakshan; +Cc: linux-omap
Hello Ranjith,
again, I regret the delay -
On Tue, 3 Nov 2009, Ranjith Lohithakshan wrote:
> AM3517/05 has a few additional control module registers defined mainly
> to control the new IP's. This patch adds support for those new registers.
>
> Signed-off-by: Ranjith Lohithakshan <ranjithl@ti.com>
It looks like the IP_CLK_CTRL register is called IPSS_CLK_CTRL, according
to the TRM. Should this be renamed in the patch? If so, I'll be happy to
do it before queuing it up, or if you like you can send an updated patch.
- Paul
> diff --git a/arch/arm/plat-omap/include/plat/control.h b/arch/arm/plat-omap/include/plat/control.h
> index fdb6300..9ba907a 100644
> --- a/arch/arm/plat-omap/include/plat/control.h
> +++ b/arch/arm/plat-omap/include/plat/control.h
> @@ -147,6 +147,15 @@
> #define OMAP343X_CONTROL_PBIAS_LITE (OMAP2_CONTROL_GENERAL + 0x02b0)
> #define OMAP343X_CONTROL_TEMP_SENSOR (OMAP2_CONTROL_GENERAL + 0x02b4)
>
> +/* OMAP3517/AM3517 only CONTROL_GENERAL register offsets */
> +#define OMAP3517_CONTROL_MSUSPENDMUX_6 (OMAP2_CONTROL_GENERAL + 0x0038)
> +#define OMAP3517_CONTROL_DEVCONF2 (OMAP2_CONTROL_GENERAL + 0x0310)
> +#define OMAP3517_CONTROL_DEVCONF3 (OMAP2_CONTROL_GENERAL + 0x0314)
> +#define OMAP3517_CONTROL_CBA_PRIORITY (OMAP2_CONTROL_GENERAL + 0x0320)
> +#define OMAP3517_CONTROL_LVL_INTR_CLEAR (OMAP2_CONTROL_GENERAL + 0x0324)
> +#define OMAP3517_CONTROL_IP_SW_RESET (OMAP2_CONTROL_GENERAL + 0x0328)
> +#define OMAP3517_CONTROL_IP_CLK_CTRL (OMAP2_CONTROL_GENERAL + 0x032C)
> +
> /* 34xx D2D idle-related pins, handled by PM core */
> #define OMAP3_PADCONF_SAD2D_MSTANDBY 0x250
> #define OMAP3_PADCONF_SAD2D_IDLEACK 0x254
> @@ -238,6 +247,14 @@
> #define FEAT_NEON 0
> #define FEAT_NEON_NONE 1
>
> +/* OMAP3517_CONTROL_IP_CLK_CTRL bits */
> +#define OMAP3517_USBOTG_VBUSP_CLK_SHIFT 0
> +#define OMAP3517_CPGMAC_VBUSP_CLK_SHIFT 1
> +#define OMAP3517_VPFE_VBUSP_CLK_SHIFT 2
> +#define OMAP3517_HECC_VBUSP_CLK_SHIFT 3
> +#define OMAP3517_USBOTG_FCLK_SHIFT 8
> +#define OMAP3517_CPGMAC_FCLK_SHIFT 9
> +#define OMAP3517_VPFE_FCLK_SHIFT 10
>
> #ifndef __ASSEMBLY__
> #if defined(CONFIG_ARCH_OMAP2) || defined(CONFIG_ARCH_OMAP3) || \
> --
> 1.6.2.4
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
- Paul
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] AM35xx: Add AM35xx specific control module registers
2009-12-14 22:26 ` Paul Walmsley
@ 2009-12-14 22:33 ` Paul Walmsley
2009-12-15 8:15 ` Ranjith Lohithakshan
0 siblings, 1 reply; 4+ messages in thread
From: Paul Walmsley @ 2009-12-14 22:33 UTC (permalink / raw)
To: Ranjith Lohithakshan; +Cc: linux-omap
Hi Ranjith,
one other question.
On Mon, 14 Dec 2009, Paul Walmsley wrote:
> On Tue, 3 Nov 2009, Ranjith Lohithakshan wrote:
>
> > AM3517/05 has a few additional control module registers defined mainly
> > to control the new IP's. This patch adds support for those new registers.
Are these registers 3517-specific, or are they common to both AM3517 and
3505? If the latter, than please use "AM35XX" or something similar as the
prefix for the macros below.
- Paul
> > diff --git a/arch/arm/plat-omap/include/plat/control.h b/arch/arm/plat-omap/include/plat/control.h
> > index fdb6300..9ba907a 100644
> > --- a/arch/arm/plat-omap/include/plat/control.h
> > +++ b/arch/arm/plat-omap/include/plat/control.h
> > @@ -147,6 +147,15 @@
> > #define OMAP343X_CONTROL_PBIAS_LITE (OMAP2_CONTROL_GENERAL + 0x02b0)
> > #define OMAP343X_CONTROL_TEMP_SENSOR (OMAP2_CONTROL_GENERAL + 0x02b4)
> >
> > +/* OMAP3517/AM3517 only CONTROL_GENERAL register offsets */
> > +#define OMAP3517_CONTROL_MSUSPENDMUX_6 (OMAP2_CONTROL_GENERAL + 0x0038)
> > +#define OMAP3517_CONTROL_DEVCONF2 (OMAP2_CONTROL_GENERAL + 0x0310)
> > +#define OMAP3517_CONTROL_DEVCONF3 (OMAP2_CONTROL_GENERAL + 0x0314)
> > +#define OMAP3517_CONTROL_CBA_PRIORITY (OMAP2_CONTROL_GENERAL + 0x0320)
> > +#define OMAP3517_CONTROL_LVL_INTR_CLEAR (OMAP2_CONTROL_GENERAL + 0x0324)
> > +#define OMAP3517_CONTROL_IP_SW_RESET (OMAP2_CONTROL_GENERAL + 0x0328)
> > +#define OMAP3517_CONTROL_IP_CLK_CTRL (OMAP2_CONTROL_GENERAL + 0x032C)
> > +
> > /* 34xx D2D idle-related pins, handled by PM core */
> > #define OMAP3_PADCONF_SAD2D_MSTANDBY 0x250
> > #define OMAP3_PADCONF_SAD2D_IDLEACK 0x254
> > @@ -238,6 +247,14 @@
> > #define FEAT_NEON 0
> > #define FEAT_NEON_NONE 1
> >
> > +/* OMAP3517_CONTROL_IP_CLK_CTRL bits */
> > +#define OMAP3517_USBOTG_VBUSP_CLK_SHIFT 0
> > +#define OMAP3517_CPGMAC_VBUSP_CLK_SHIFT 1
> > +#define OMAP3517_VPFE_VBUSP_CLK_SHIFT 2
> > +#define OMAP3517_HECC_VBUSP_CLK_SHIFT 3
> > +#define OMAP3517_USBOTG_FCLK_SHIFT 8
> > +#define OMAP3517_CPGMAC_FCLK_SHIFT 9
> > +#define OMAP3517_VPFE_FCLK_SHIFT 10
> >
> > #ifndef __ASSEMBLY__
> > #if defined(CONFIG_ARCH_OMAP2) || defined(CONFIG_ARCH_OMAP3) || \
> > --
> > 1.6.2.4
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] AM35xx: Add AM35xx specific control module registers
2009-12-14 22:33 ` Paul Walmsley
@ 2009-12-15 8:15 ` Ranjith Lohithakshan
0 siblings, 0 replies; 4+ messages in thread
From: Ranjith Lohithakshan @ 2009-12-15 8:15 UTC (permalink / raw)
To: Paul Walmsley; +Cc: linux-omap@vger.kernel.org
Hi Paul,
On Tue, 15-Dec-09 4:03 AM +0530, Paul Walmsley wrote:
> Hi Ranjith,
>
> one other question.
>
> On Mon, 14 Dec 2009, Paul Walmsley wrote:
>
>> On Tue, 3 Nov 2009, Ranjith Lohithakshan wrote:
>>
>>> AM3517/05 has a few additional control module registers defined mainly
>>> to control the new IP's. This patch adds support for those new registers.
>
> Are these registers 3517-specific, or are they common to both AM3517 and
> 3505? If the latter, than please use "AM35XX" or something similar as the
> prefix for the macros below.
>
Yes, these are common to 3517 & 3505. I will rename these to AM35XX,
incorporate your IPSS_CLK_CTRL comment and generate a v2 version of
this patch shortly.
- Ranjith
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2009-12-15 8:15 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-11-03 17:37 [PATCH] AM35xx: Add AM35xx specific control module registers Ranjith Lohithakshan
2009-12-14 22:26 ` Paul Walmsley
2009-12-14 22:33 ` Paul Walmsley
2009-12-15 8:15 ` Ranjith Lohithakshan
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox