RE: [PATCH 1/1] ARM: OMAP4: PM: Add check for avoiding dependency related update.

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

 



Hi Paul,


> -----Original Message-----
> From: Paul Walmsley [mailto:paul@xxxxxxxxx]
> Sent: Wednesday, December 30, 2009 2:02 PM
> To: Pagare, Abhijit
> Cc: linux-omap@xxxxxxxxxxxxxxx; linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
> Subject: Re: [PATCH 1/1] ARM: OMAP4: PM: Add check for avoiding dependency
> related update.
> 
> Hello Abhijit,
> 
> On Wed, 23 Dec 2009, Abhijit Pagare wrote:
> 
> > A check is added for avoiding the sleep/wakeup dependency updates
> > for OMAP4 as the structures for the dependencies are currently absent.
> 
> As a threshold matter, code and data added in temporary hack patches like
> this need to be marked with XXX or FIXME or something similar to indicate
> that these are not intended features, but rather, temporary lines that
> need to be removed in the near term.

I think I missed that. I will add the FIXME indication to the check below. I think that this check is essential for OMAP4 kernel boot-up as long as the dependency patches are not pushed in.

> 
> But really, I'd rather not apply this patch.  The best way forward is to
> apply a patch that adds the OMAP4 wakeup and sleep dependencies
> themselves.  Have you had a chance to look at the series that I sent you a
> few weeks ago off-list that reorganizes the wakeup and sleep dependencies
> to determine if it is adequate for you to add the OMAP4 wakeup/sleep
> dependencies afterwards?

I had a look at the dependency patches that you had sent to me earlier and it seems feasible to add the OMAP4 related dependency data on top of those patches. A small change exists but can be taken care of.

Regards,
Abhijit

> 
> 
> - Paul
> 
> >
> > Signed-off-by: Abhijit Pagare <abhijitpagare@xxxxxx>
> > Cc: Paul Walmsley <paul@xxxxxxxxx>
> > ---
> >
> > Compiled and Boot Tested on OMAP4430 simulator, ES1 Chip & OMAP3430 SDP.
> > And also Compiled for OMAP2430 and OMAP2420
> >
> > This patch has a dependency on the earlier set of OMAP4 clock-domain
> patches.
> >
> >  arch/arm/mach-omap2/clockdomain.c |   10 ++++++----
> >  1 files changed, 6 insertions(+), 4 deletions(-)
> >
> > diff --git a/arch/arm/mach-omap2/clockdomain.c b/arch/arm/mach-
> omap2/clockdomain.c
> > index 8c9e78c..6ec25ee 100644
> > --- a/arch/arm/mach-omap2/clockdomain.c
> > +++ b/arch/arm/mach-omap2/clockdomain.c
> > @@ -507,8 +507,9 @@ void omap2_clkdm_allow_idle(struct clockdomain
> *clkdm)
> >  	pr_debug("clockdomain: enabling automatic idle transitions for
> %s\n",
> >  		 clkdm->name);
> >
> > -	if (atomic_read(&clkdm->usecount) > 0)
> > -		_clkdm_add_autodeps(clkdm);
> > +	if (!cpu_is_omap44xx())
> > +		if (atomic_read(&clkdm->usecount) > 0)
> > +			_clkdm_add_autodeps(clkdm);
> >
> >  	_omap2_clkdm_set_hwsup(clkdm, 1);
> >
> > @@ -540,8 +541,9 @@ void omap2_clkdm_deny_idle(struct clockdomain
> *clkdm)
> >
> >  	_omap2_clkdm_set_hwsup(clkdm, 0);
> >
> > -	if (atomic_read(&clkdm->usecount) > 0)
> > -		_clkdm_del_autodeps(clkdm);
> > +	if (!cpu_is_omap44xx())
> > +		if (atomic_read(&clkdm->usecount) > 0)
> > +			_clkdm_del_autodeps(clkdm);
> >  }
> >
> >
> > --
> > 1.5.4.7
> >
> 
> 
> - 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