From mboxrd@z Thu Jan 1 00:00:00 1970 From: Leo Yan Subject: Re: [PATCH v5 6/9] coresight: add support for CPU debug module Date: Thu, 30 Mar 2017 21:51:45 +0800 Message-ID: <20170330135145.GA26911@leoy-linaro> References: <1490466197-29163-1-git-send-email-leo.yan@linaro.org> <1490466197-29163-7-git-send-email-leo.yan@linaro.org> <20170328165010.GA21937@linaro.org> <20170329015423.GA5035@leoy-linaro> <20170330010343.GA31843@leoy-linaro> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-doc-owner@vger.kernel.org To: Suzuki K Poulose Cc: Mike Leach , Mathieu Poirier , Jonathan Corbet , Rob Herring , Mark Rutland , Wei Xu , Catalin Marinas , Will Deacon , Andy Gross , David Brown , Michael Turquette , Stephen Boyd , Guodong Xu , John Stultz , linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-arm-msm@vger.kernel.org, linux-soc@vger.kernel.org, linux-clk@vger.kernel.org, Sudeep Holla List-Id: devicetree@vger.kernel.org On Thu, Mar 30, 2017 at 10:00:30AM +0100, Suzuki K Poulose wrote: [...] > Leo, > > Also, it would be good to restore the PRCR register back to the original state > after we read the registers (if we changed them). I am exploring ways to make > use of this feature on demand (e.g, tie it to sysrq-t or sysrq-l) and not just > at panic time. So it would be good to have the state restored to not affect the > normal functioning even after dumping the registers. Got it. Will take care for this. Thanks, Leo Yan > PS: I have a small hack to hook this into a sysrq-x at runtime, but on a second thought > it is better not to consume the key and rather tie it to something which already exist, > as mentioned above. > > Thanks > Suzuki