Messages in this thread | | | From | Mathieu Poirier <> | Date | Wed, 14 Aug 2019 09:04:22 -0600 | Subject | Re: [PATCH v3] coresight: Serialize enabling/disabling a link device. |
| |
On Tue, 13 Aug 2019 at 18:36, Yabin Cui <yabinc@google.com> wrote: > > > The patch isn't difficult to do but does go deeper in each link > > drivers (ETF, funnel, replicator). Let me know if you are not > > comfortable with the idea and I will see what I can do on my side. > > I am comfortable with the idea. I chose to add lock in coresight_enable_link() just because it > is the smallest change. But a refactor probably is better for maintainance.
Yes, that is my prime concern.
> > Feel free to upload a new patch fixing the problem. I can help to test it.
I have a few things on my plate at this time so you might beat me to the punch. I'll see how the next few days go.
Mathieu
| |