Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1427561 > unrolled thread
| Started by | Keerthy <j-keerthy@ti.com> |
|---|---|
| First post | 2016-06-21 12:20 +0200 |
| Last post | 2016-06-22 10:10 +0200 |
| Articles | 4 — 4 participants |
Back to article view | Back to linux.kernel
[PATCH] ARM: AM33xx: PRM: Remove wrongly defined RSTST offset for PER Domain Keerthy <j-keerthy@ti.com> - 2016-06-21 12:20 +0200
Re: [PATCH] ARM: AM33xx: PRM: Remove wrongly defined RSTST offset for PER Domain Suman Anna <s-anna@ti.com> - 2016-06-22 00:30 +0200
Re: [PATCH] ARM: AM33xx: PRM: Remove wrongly defined RSTST offset for PER Domain Keerthy <a0393675@ti.com> - 2016-06-22 07:30 +0200
Re: [PATCH] ARM: AM33xx: PRM: Remove wrongly defined RSTST offset for PER Domain Tony Lindgren <tony@atomide.com> - 2016-06-22 10:10 +0200
| From | Keerthy <j-keerthy@ti.com> |
|---|---|
| Date | 2016-06-21 12:20 +0200 |
| Subject | [PATCH] ARM: AM33xx: PRM: Remove wrongly defined RSTST offset for PER Domain |
| Message-ID | <rMqWJ-4rb-5@gated-at.bofh.it> |
As per the TRM: http://www.ti.com/lit/ug/spruh73m/spruh73m.pdf offset 0x4 is reserved for PRM_PER. Hence removing the wrongly defined address offset. Signed-off-by: Keerthy <j-keerthy@ti.com> --- arch/arm/mach-omap2/prm33xx.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/arch/arm/mach-omap2/prm33xx.h b/arch/arm/mach-omap2/prm33xx.h index 2bc4ec5..66302c6 100644 --- a/arch/arm/mach-omap2/prm33xx.h +++ b/arch/arm/mach-omap2/prm33xx.h @@ -52,8 +52,6 @@ /* PRM.PER_PRM register offsets */ #define AM33XX_RM_PER_RSTCTRL_OFFSET 0x0000 #define AM33XX_RM_PER_RSTCTRL AM33XX_PRM_REGADDR(AM33XX_PRM_PER_MOD, 0x0000) -#define AM33XX_RM_PER_RSTST_OFFSET 0x0004 -#define AM33XX_RM_PER_RSTST AM33XX_PRM_REGADDR(AM33XX_PRM_PER_MOD, 0x0004) #define AM33XX_PM_PER_PWRSTST_OFFSET 0x0008 #define AM33XX_PM_PER_PWRSTST AM33XX_PRM_REGADDR(AM33XX_PRM_PER_MOD, 0x0008) #define AM33XX_PM_PER_PWRSTCTRL_OFFSET 0x000c -- 1.9.1
[toc] | [next] | [standalone]
| From | Suman Anna <s-anna@ti.com> |
|---|---|
| Date | 2016-06-22 00:30 +0200 |
| Subject | Re: [PATCH] ARM: AM33xx: PRM: Remove wrongly defined RSTST offset for PER Domain |
| Message-ID | <rMClb-3lP-19@gated-at.bofh.it> |
| In reply to | #1427561 |
Hi Keerthy, On 06/21/2016 05:08 AM, Keerthy wrote: > As per the TRM: http://www.ti.com/lit/ug/spruh73m/spruh73m.pdf > offset 0x4 is reserved for PRM_PER. Hence removing the wrongly > defined address offset. Thanks for the patch. These macros are not used anywhere, so it should be safe to remove these. I have tested this patch with an off-tree PRUSS driver releasing the reset, it works as expected without throwing any issues. That said, the current omap_hwmod code assumes default rstst_offs and st_shift and tries to still write some value into the RSTCTRL register in am33xx_prm_deassert_hardreset(), but it didn't have any side-affects though on AM33xx. This is what affected the AM437x due to incorrect RSTST offset value. That behavior is independent of this patch though. Tested-by: Suman Anna <s-anna@ti.com> regards Suman > Signed-off-by: Keerthy <j-keerthy@ti.com> > --- > arch/arm/mach-omap2/prm33xx.h | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/arch/arm/mach-omap2/prm33xx.h b/arch/arm/mach-omap2/prm33xx.h > index 2bc4ec5..66302c6 100644 > --- a/arch/arm/mach-omap2/prm33xx.h > +++ b/arch/arm/mach-omap2/prm33xx.h > @@ -52,8 +52,6 @@ > /* PRM.PER_PRM register offsets */ > #define AM33XX_RM_PER_RSTCTRL_OFFSET 0x0000 > #define AM33XX_RM_PER_RSTCTRL AM33XX_PRM_REGADDR(AM33XX_PRM_PER_MOD, 0x0000) > -#define AM33XX_RM_PER_RSTST_OFFSET 0x0004 > -#define AM33XX_RM_PER_RSTST AM33XX_PRM_REGADDR(AM33XX_PRM_PER_MOD, 0x0004) > #define AM33XX_PM_PER_PWRSTST_OFFSET 0x0008 > #define AM33XX_PM_PER_PWRSTST AM33XX_PRM_REGADDR(AM33XX_PRM_PER_MOD, 0x0008) > #define AM33XX_PM_PER_PWRSTCTRL_OFFSET 0x000c >
[toc] | [prev] | [next] | [standalone]
| From | Keerthy <a0393675@ti.com> |
|---|---|
| Date | 2016-06-22 07:30 +0200 |
| Subject | Re: [PATCH] ARM: AM33xx: PRM: Remove wrongly defined RSTST offset for PER Domain |
| Message-ID | <rMITD-7ER-7@gated-at.bofh.it> |
| In reply to | #1428230 |
On Wednesday 22 June 2016 03:55 AM, Suman Anna wrote: > Hi Keerthy, > > On 06/21/2016 05:08 AM, Keerthy wrote: >> As per the TRM: http://www.ti.com/lit/ug/spruh73m/spruh73m.pdf >> offset 0x4 is reserved for PRM_PER. Hence removing the wrongly >> defined address offset. > > Thanks for the patch. These macros are not used anywhere, so it should > be safe to remove these. I have tested this patch with an off-tree PRUSS > driver releasing the reset, it works as expected without throwing any > issues. > > That said, the current omap_hwmod code assumes default rstst_offs and > st_shift and tries to still write some value into the RSTCTRL register > in am33xx_prm_deassert_hardreset(), but it didn't have any side-affects > though on AM33xx. This is what affected the AM437x due to incorrect > RSTST offset value. That behavior is independent of this patch though. > > Tested-by: Suman Anna <s-anna@ti.com> Thanks for testing! > > regards > Suman > >> Signed-off-by: Keerthy <j-keerthy@ti.com> >> --- >> arch/arm/mach-omap2/prm33xx.h | 2 -- >> 1 file changed, 2 deletions(-) >> >> diff --git a/arch/arm/mach-omap2/prm33xx.h b/arch/arm/mach-omap2/prm33xx.h >> index 2bc4ec5..66302c6 100644 >> --- a/arch/arm/mach-omap2/prm33xx.h >> +++ b/arch/arm/mach-omap2/prm33xx.h >> @@ -52,8 +52,6 @@ >> /* PRM.PER_PRM register offsets */ >> #define AM33XX_RM_PER_RSTCTRL_OFFSET 0x0000 >> #define AM33XX_RM_PER_RSTCTRL AM33XX_PRM_REGADDR(AM33XX_PRM_PER_MOD, 0x0000) >> -#define AM33XX_RM_PER_RSTST_OFFSET 0x0004 >> -#define AM33XX_RM_PER_RSTST AM33XX_PRM_REGADDR(AM33XX_PRM_PER_MOD, 0x0004) >> #define AM33XX_PM_PER_PWRSTST_OFFSET 0x0008 >> #define AM33XX_PM_PER_PWRSTST AM33XX_PRM_REGADDR(AM33XX_PRM_PER_MOD, 0x0008) >> #define AM33XX_PM_PER_PWRSTCTRL_OFFSET 0x000c >> >
[toc] | [prev] | [next] | [standalone]
| From | Tony Lindgren <tony@atomide.com> |
|---|---|
| Date | 2016-06-22 10:10 +0200 |
| Subject | Re: [PATCH] ARM: AM33xx: PRM: Remove wrongly defined RSTST offset for PER Domain |
| Message-ID | <rMLot-Tc-1@gated-at.bofh.it> |
| In reply to | #1427561 |
* Keerthy <j-keerthy@ti.com> [160621 03:11]: > As per the TRM: http://www.ti.com/lit/ug/spruh73m/spruh73m.pdf > offset 0x4 is reserved for PRM_PER. Hence removing the wrongly > defined address offset. Applying into omap-for-v4.8/dt thanks. Tony
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web