Re: [PATCHv2 3/4] coresight: replicator: Reset replicator if context is lost

[Sai Prakash Ranjan]

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@xxxxxxxxxxxxxx> 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@xxxxxxxxxx>
> > Signed-off-by: Sai Prakash Ranjan <saiprakash.ranjan@xxxxxxxxxxxxxx>
> > ---
> >
> > 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@xxxxxxxxxx>

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