From mboxrd@z Thu Jan  1 00:00:00 1970
From: Santosh Shilimkar <santosh.shilimkar@ti.com>
Subject: RE: [PATCH] OMAP4: PM: Set static dependency between MPUSS and EMIF
Date: Wed, 9 Mar 2011 10:29:33 +0530
Message-ID: <8b5b26707cf6ad19fb6b37960d8ce04d@mail.gmail.com>
References: <1299596946-31334-1-git-send-email-santosh.shilimkar@ti.com><alpine.DEB.2.00.1103081619500.15132@utopia.booyaka.com>
 <87ei6hnoda.fsf@ti.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1
Return-path: <linux-omap-owner@vger.kernel.org>
Received: from na3sys009aog114.obsmtp.com ([74.125.149.211]:49841 "EHLO
	na3sys009aog114.obsmtp.com" rhost-flags-OK-OK-OK-OK)
	by vger.kernel.org with ESMTP id S932200Ab1CIE7g (ORCPT
	<rfc822;linux-omap@vger.kernel.org>); Tue, 8 Mar 2011 23:59:36 -0500
Received: by mail-qw0-f43.google.com with SMTP id 7so147320qwb.2
        for <linux-omap@vger.kernel.org>; Tue, 08 Mar 2011 20:59:34 -0800 (PST)
In-Reply-To: <87ei6hnoda.fsf@ti.com>
Sender: linux-omap-owner@vger.kernel.org
List-Id: linux-omap@vger.kernel.org
To: Kevin Hilman <khilman@ti.com>, Paul Walmsley <paul@pwsan.com>
Cc: Rajendra Nayak <rnayak@ti.com>, linux-omap@vger.kernel.org

> -----Original Message-----
> From: Kevin Hilman [mailto:khilman@ti.com]
> Sent: Wednesday, March 09, 2011 5:42 AM
> To: Paul Walmsley
> Cc: Rajendra Nayak; Santosh Shilimkar; linux-omap@vger.kernel.org
> Subject: Re: [PATCH] OMAP4: PM: Set static dependency between MPUSS
> and EMIF
>
> Paul Walmsley <paul@pwsan.com> writes:
>
> > On Tue, 8 Mar 2011, Santosh Shilimkar wrote:
> >
> >> As per OMAP4430 TRM, the dynamic dependency between MEMIF and
> MPUSS
> >> clockdomains is enable by default. Refer register
> CM_MPU_DYNAMICDEP
> >> description for details.
> >>
> >> But it doesn't seems to work as expected and MPUSS doesn't wakeup
> >> from off-mode if the static dependency is not set between MPUSS
> and
> >> EMIF clockdomains. i.e CM_MPU_STATICDEP.MEMIF_STATDEP = 0
> >>
> >> The issue is under investigation with hardware team.
> >>
> >> Signed-off-by: Rajendra Nayak <rnayak@ti.com>
> >> Signed-off-by: Santosh Shilimkar <santosh.shilimkar@ti.com>
> >> Cc: Paul Walmsley <paul@pwsan.com>
> >> Cc: Kevin Hilman <khilman@ti.com>
> >
> > Nice work!
> >
> > Kevin, you want to take this one?
>
> Yup
>
> > My only comment is that it might be nice to add a pr_info("OMAP4
> > PM: Temporary static dependency added between EMIF and MPUSS");
> > in there to reduce the risk that the issue will be forgotten...
> >
> > Acked-by: Paul Walmsley <paul@pwsan.com>
>
> Santosh,
>
> Please update with the pr_info() as suggested by Paul, repost with
> linux-arm-kernel in Cc, add Paul's ack and I'll queue for 2.6.39.
>
Sure.

Regards
Santosh