RE: [PATCH] OMAP4: PM: Set static dependency between MPUSS and EMIF

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

 



> -----Original Message-----
> From: Kevin Hilman [mailto:khilman@xxxxxx]
> Sent: Wednesday, March 09, 2011 5:42 AM
> To: Paul Walmsley
> Cc: Rajendra Nayak; Santosh Shilimkar; linux-omap@xxxxxxxxxxxxxxx
> Subject: Re: [PATCH] OMAP4: PM: Set static dependency between MPUSS
> and EMIF
>
> Paul Walmsley <paul@xxxxxxxxx> 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@xxxxxx>
> >> Signed-off-by: Santosh Shilimkar <santosh.shilimkar@xxxxxx>
> >> Cc: Paul Walmsley <paul@xxxxxxxxx>
> >> Cc: Kevin Hilman <khilman@xxxxxx>
> >
> > 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@xxxxxxxxx>
>
> 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
--
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