Messages in this thread | | | Subject | Re: [PATCH v5 6/9] coresight: add support for CPU debug module | From | Sudeep Holla <> | Date | Thu, 30 Mar 2017 16:47:59 +0100 |
| |
On 29/03/17 15:56, Mike Leach wrote:
[...] > > No - EDPRCR_COREPURQ and EDPRCR_CORENPDRQ have different semantics and purposes > > EDPRCR_COREPURQ is in the debug power domain an is tied to an external > debug request that should be an input to the external (to the PE) > system power controller. > The requirement is that the system power controller powers up the core > domain and does not power it down while it remains asserted. > > EDPRCR_CORENPDRQ is in the core power domain and thus to the specific > core only. This ensures that any power control software running on > that core should emulate a power down if this is set to one. > > We cannot know the power control design of the system, so the safe > solution is to set both bits. >
+1
I agree that's the safe bet.
-- Regards, Sudeep
| |