Messages in this thread | ![/](/images/icornerl.gif) | | Date | Fri, 8 Nov 2019 09:59:40 +0530 | From | Vinod Koul <> | Subject | Re: [alsa-devel] [PATCH 1/4] soundwire: sdw_slave: add new fields to track probe status |
| |
On 04-11-19, 08:32, Pierre-Louis Bossart wrote: > > > On 11/2/19 11:56 PM, Vinod Koul wrote: > > On 23-10-19, 16:06, Pierre-Louis Bossart wrote: > > > Changes to the sdw_slave structure needed to solve race conditions on > > > driver probe. > > > > Can you please explain the race you have observed, it would be a very > > useful to document it as well > > the races are explained in the [PATCH 00/18] soundwire: code hardening and > suspend-resume support series.
It would make sense to explain it here as well to give details to reviewers, there is nothing wrong with too much detail!
> > > > > > The functionality is added in the next patch. > > > > which one..? > > [PATCH 00/18] soundwire: code hardening and suspend-resume support
Yeah great! let me play detective with 18 patch series. I asked for a patch and got a series!
Again, please help the maintainer to help you. We would love to see this merged as well, but please step up and give more details in cover letter and changelogs. I shouldn't need to do guesswork and scan through the inbox to find the context!
-- ~Vinod
| ![\](/images/icornerr.gif) |