Messages in this thread | | | Date | Thu, 21 May 2020 22:33:13 +0530 | From | Sai Prakash Ranjan <> | Subject | Re: [PATCHv2 3/4] coresight: replicator: Reset replicator if context is lost |
| |
Hi Mike,
On 2020-05-21 21:38, Mike Leach wrote: > Hi Sai, > > On Tue, 19 May 2020 at 14:36, Sai Prakash Ranjan > <saiprakash.ranjan@codeaurora.org> wrote: >> >> On some QCOM SoCs, replicators in Always-On domain loses its >> context as soon as the clock is disabled. Currently as a part >> of pm_runtime workqueue, clock is disabled after the replicator >> is initialized by amba_pm_runtime_suspend assuming that context >> is not lost which is not true for replicators with such >> limitations. So add a new property "qcom,replicator-loses-context" >> to identify such replicators and reset them. >> >> Suggested-by: Mike Leach <mike.leach@linaro.org> >> Signed-off-by: Sai Prakash Ranjan <saiprakash.ranjan@codeaurora.org> >> --- >> >> Added Mike's suggested by for parts other than the DT property. >> Perhaps I should add Co-developed-by Mike since the full skeletal >> was given by Mike. I can add that if required on the next version. >>
[...]
> > Suggested-by is just fine - I didn't actually try out my suggested > code after all! > > Reviewed-by Mike Leach <mike.leach@linaro.org>
Sure, thanks for the review.
Thanks, Sai
-- QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation
| |