linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] ARM: OMAP2+: AM33xx: Add missing reset status info to GFX hwmod
@ 2013-05-04 18:48 Vaibhav Hiremath
  2013-05-17 11:08 ` Hiremath, Vaibhav
  2013-06-09  7:14 ` Paul Walmsley
  0 siblings, 2 replies; 5+ messages in thread
From: Vaibhav Hiremath @ 2013-05-04 18:48 UTC (permalink / raw)
  To: linux-arm-kernel

GFX has a reset status register (PRM_GFX.RM_GFX_RSTST),
so update the GFX hwmod data with .rstst_off and .st_shift
information.

Signed-off-by: Vaibhav Hiremath <hvaibhav@ti.com>
Cc: Paul Walmsley <paul@pwsan.com>
---
 arch/arm/mach-omap2/omap_hwmod_33xx_data.c |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/arch/arm/mach-omap2/omap_hwmod_33xx_data.c b/arch/arm/mach-omap2/omap_hwmod_33xx_data.c
index d1cf3ab..38c7b04 100644
--- a/arch/arm/mach-omap2/omap_hwmod_33xx_data.c
+++ b/arch/arm/mach-omap2/omap_hwmod_33xx_data.c
@@ -329,7 +329,7 @@ static struct omap_hwmod_class am33xx_gfx_hwmod_class = {
 };
 
 static struct omap_hwmod_rst_info am33xx_gfx_resets[] = {
-	{ .name = "gfx", .rst_shift = 0 },
+	{ .name = "gfx", .rst_shift = 0, .st_shift = 0},
 };
 
 static struct omap_hwmod_irq_info am33xx_gfx_irqs[] = {
@@ -347,6 +347,7 @@ static struct omap_hwmod am33xx_gfx_hwmod = {
 		.omap4	= {
 			.clkctrl_offs	= AM33XX_CM_GFX_GFX_CLKCTRL_OFFSET,
 			.rstctrl_offs	= AM33XX_RM_GFX_RSTCTRL_OFFSET,
+			.rstst_offs	= AM33XX_RM_GFX_RSTST_OFFSET,
 			.modulemode	= MODULEMODE_SWCTRL,
 		},
 	},
-- 
1.7.0.4

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* [PATCH] ARM: OMAP2+: AM33xx: Add missing reset status info to GFX hwmod
  2013-05-04 18:48 [PATCH] ARM: OMAP2+: AM33xx: Add missing reset status info to GFX hwmod Vaibhav Hiremath
@ 2013-05-17 11:08 ` Hiremath, Vaibhav
  2013-05-17 17:24   ` Tony Lindgren
  2013-06-09  7:14 ` Paul Walmsley
  1 sibling, 1 reply; 5+ messages in thread
From: Hiremath, Vaibhav @ 2013-05-17 11:08 UTC (permalink / raw)
  To: linux-arm-kernel


> -----Original Message-----
> From: Hiremath, Vaibhav
> Sent: Sunday, May 05, 2013 12:18 AM
> To: linux-omap at vger.kernel.org
> Cc: tony at atomide.com; linux-arm-kernel at lists.infradead.org; Hiremath,
> Vaibhav; Paul Walmsley
> Subject: [PATCH] ARM: OMAP2+: AM33xx: Add missing reset status info to
> GFX hwmod
> 
> GFX has a reset status register (PRM_GFX.RM_GFX_RSTST),
> so update the GFX hwmod data with .rstst_off and .st_shift
> information.
> 
> Signed-off-by: Vaibhav Hiremath <hvaibhav@ti.com>
> Cc: Paul Walmsley <paul@pwsan.com>
> ---
>  arch/arm/mach-omap2/omap_hwmod_33xx_data.c |    3 ++-
>  1 files changed, 2 insertions(+), 1 deletions(-)
> 
> diff --git a/arch/arm/mach-omap2/omap_hwmod_33xx_data.c
> b/arch/arm/mach-omap2/omap_hwmod_33xx_data.c
> index d1cf3ab..38c7b04 100644
> --- a/arch/arm/mach-omap2/omap_hwmod_33xx_data.c
> +++ b/arch/arm/mach-omap2/omap_hwmod_33xx_data.c
> @@ -329,7 +329,7 @@ static struct omap_hwmod_class
> am33xx_gfx_hwmod_class = {
>  };
> 
>  static struct omap_hwmod_rst_info am33xx_gfx_resets[] = {
> -	{ .name = "gfx", .rst_shift = 0 },
> +	{ .name = "gfx", .rst_shift = 0, .st_shift = 0},
>  };
> 
>  static struct omap_hwmod_irq_info am33xx_gfx_irqs[] = {
> @@ -347,6 +347,7 @@ static struct omap_hwmod am33xx_gfx_hwmod = {
>  		.omap4	= {
>  			.clkctrl_offs	= AM33XX_CM_GFX_GFX_CLKCTRL_OFFSET,
>  			.rstctrl_offs	= AM33XX_RM_GFX_RSTCTRL_OFFSET,
> +			.rstst_offs	= AM33XX_RM_GFX_RSTST_OFFSET,
>  			.modulemode	= MODULEMODE_SWCTRL,
>  		},
>  	},
> --

Tony and Paul,

Any Update on this one as well?

Thanks,
Vaibhav

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [PATCH] ARM: OMAP2+: AM33xx: Add missing reset status info to GFX hwmod
  2013-05-17 11:08 ` Hiremath, Vaibhav
@ 2013-05-17 17:24   ` Tony Lindgren
  2013-05-20  5:50     ` Hiremath, Vaibhav
  0 siblings, 1 reply; 5+ messages in thread
From: Tony Lindgren @ 2013-05-17 17:24 UTC (permalink / raw)
  To: linux-arm-kernel

* Hiremath, Vaibhav <hvaibhav@ti.com> [130517 04:13]:
> 
> > -----Original Message-----
> > From: Hiremath, Vaibhav
> > Sent: Sunday, May 05, 2013 12:18 AM
> > To: linux-omap at vger.kernel.org
> > Cc: tony at atomide.com; linux-arm-kernel at lists.infradead.org; Hiremath,
> > Vaibhav; Paul Walmsley
> > Subject: [PATCH] ARM: OMAP2+: AM33xx: Add missing reset status info to
> > GFX hwmod
> > 
> > GFX has a reset status register (PRM_GFX.RM_GFX_RSTST),
> > so update the GFX hwmod data with .rstst_off and .st_shift
> > information.
> > 
> > Signed-off-by: Vaibhav Hiremath <hvaibhav@ti.com>
> > Cc: Paul Walmsley <paul@pwsan.com>
> > ---
> >  arch/arm/mach-omap2/omap_hwmod_33xx_data.c |    3 ++-
> >  1 files changed, 2 insertions(+), 1 deletions(-)
> > 
> > diff --git a/arch/arm/mach-omap2/omap_hwmod_33xx_data.c
> > b/arch/arm/mach-omap2/omap_hwmod_33xx_data.c
> > index d1cf3ab..38c7b04 100644
> > --- a/arch/arm/mach-omap2/omap_hwmod_33xx_data.c
> > +++ b/arch/arm/mach-omap2/omap_hwmod_33xx_data.c
> > @@ -329,7 +329,7 @@ static struct omap_hwmod_class
> > am33xx_gfx_hwmod_class = {
> >  };
> > 
> >  static struct omap_hwmod_rst_info am33xx_gfx_resets[] = {
> > -	{ .name = "gfx", .rst_shift = 0 },
> > +	{ .name = "gfx", .rst_shift = 0, .st_shift = 0},
> >  };
> > 
> >  static struct omap_hwmod_irq_info am33xx_gfx_irqs[] = {
> > @@ -347,6 +347,7 @@ static struct omap_hwmod am33xx_gfx_hwmod = {
> >  		.omap4	= {
> >  			.clkctrl_offs	= AM33XX_CM_GFX_GFX_CLKCTRL_OFFSET,
> >  			.rstctrl_offs	= AM33XX_RM_GFX_RSTCTRL_OFFSET,
> > +			.rstst_offs	= AM33XX_RM_GFX_RSTST_OFFSET,
> >  			.modulemode	= MODULEMODE_SWCTRL,
> >  		},
> >  	},
> > --
> 
> Tony and Paul,
> 
> Any Update on this one as well?

Is this needed for v3.10 as a fix? If so, it should describe the
regression or error.

Regards,

Tony

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [PATCH] ARM: OMAP2+: AM33xx: Add missing reset status info to GFX hwmod
  2013-05-17 17:24   ` Tony Lindgren
@ 2013-05-20  5:50     ` Hiremath, Vaibhav
  0 siblings, 0 replies; 5+ messages in thread
From: Hiremath, Vaibhav @ 2013-05-20  5:50 UTC (permalink / raw)
  To: linux-arm-kernel

> -----Original Message-----
> From: Tony Lindgren [mailto:tony at atomide.com]
> Sent: Friday, May 17, 2013 10:55 PM
> To: Hiremath, Vaibhav
> Cc: linux-omap at vger.kernel.org; linux-arm-kernel at lists.infradead.org;
> Paul Walmsley
> Subject: Re: [PATCH] ARM: OMAP2+: AM33xx: Add missing reset status info
> to GFX hwmod
> 
> * Hiremath, Vaibhav <hvaibhav@ti.com> [130517 04:13]:
> >
> > > -----Original Message-----
> > > From: Hiremath, Vaibhav
> > > Sent: Sunday, May 05, 2013 12:18 AM
> > > To: linux-omap at vger.kernel.org
> > > Cc: tony at atomide.com; linux-arm-kernel at lists.infradead.org;
> Hiremath,
> > > Vaibhav; Paul Walmsley
> > > Subject: [PATCH] ARM: OMAP2+: AM33xx: Add missing reset status info
> to
> > > GFX hwmod
> > >
> > > GFX has a reset status register (PRM_GFX.RM_GFX_RSTST),
> > > so update the GFX hwmod data with .rstst_off and .st_shift
> > > information.
> > >
> > > Signed-off-by: Vaibhav Hiremath <hvaibhav@ti.com>
> > > Cc: Paul Walmsley <paul@pwsan.com>
> > > ---
> > >  arch/arm/mach-omap2/omap_hwmod_33xx_data.c |    3 ++-
> > >  1 files changed, 2 insertions(+), 1 deletions(-)
> > >
> > > diff --git a/arch/arm/mach-omap2/omap_hwmod_33xx_data.c
> > > b/arch/arm/mach-omap2/omap_hwmod_33xx_data.c
> > > index d1cf3ab..38c7b04 100644
> > > --- a/arch/arm/mach-omap2/omap_hwmod_33xx_data.c
> > > +++ b/arch/arm/mach-omap2/omap_hwmod_33xx_data.c
> > > @@ -329,7 +329,7 @@ static struct omap_hwmod_class
> > > am33xx_gfx_hwmod_class = {
> > >  };
> > >
> > >  static struct omap_hwmod_rst_info am33xx_gfx_resets[] = {
> > > -	{ .name = "gfx", .rst_shift = 0 },
> > > +	{ .name = "gfx", .rst_shift = 0, .st_shift = 0},
> > >  };
> > >
> > >  static struct omap_hwmod_irq_info am33xx_gfx_irqs[] = {
> > > @@ -347,6 +347,7 @@ static struct omap_hwmod am33xx_gfx_hwmod = {
> > >  		.omap4	= {
> > >  			.clkctrl_offs	= AM33XX_CM_GFX_GFX_CLKCTRL_OFFSET,
> > >  			.rstctrl_offs	= AM33XX_RM_GFX_RSTCTRL_OFFSET,
> > > +			.rstst_offs	= AM33XX_RM_GFX_RSTST_OFFSET,
> > >  			.modulemode	= MODULEMODE_SWCTRL,
> > >  		},
> > >  	},
> > > --
> >
> > Tony and Paul,
> >
> > Any Update on this one as well?
> 
> Is this needed for v3.10 as a fix? If so, it should describe the
> regression or error.
> 

Certainly this is fix, but no impact on kernel boot.
I will describe it more in the description and send next version shortly.

Thanks,
Vaibhav

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [PATCH] ARM: OMAP2+: AM33xx: Add missing reset status info to GFX hwmod
  2013-05-04 18:48 [PATCH] ARM: OMAP2+: AM33xx: Add missing reset status info to GFX hwmod Vaibhav Hiremath
  2013-05-17 11:08 ` Hiremath, Vaibhav
@ 2013-06-09  7:14 ` Paul Walmsley
  1 sibling, 0 replies; 5+ messages in thread
From: Paul Walmsley @ 2013-06-09  7:14 UTC (permalink / raw)
  To: linux-arm-kernel

On Sun, 5 May 2013, Vaibhav Hiremath wrote:

> GFX has a reset status register (PRM_GFX.RM_GFX_RSTST),
> so update the GFX hwmod data with .rstst_off and .st_shift
> information.
> 
> Signed-off-by: Vaibhav Hiremath <hvaibhav@ti.com>
> Cc: Paul Walmsley <paul@pwsan.com>

Thanks, queued.

- Paul

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2013-06-09  7:14 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-05-04 18:48 [PATCH] ARM: OMAP2+: AM33xx: Add missing reset status info to GFX hwmod Vaibhav Hiremath
2013-05-17 11:08 ` Hiremath, Vaibhav
2013-05-17 17:24   ` Tony Lindgren
2013-05-20  5:50     ` Hiremath, Vaibhav
2013-06-09  7:14 ` Paul Walmsley

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).