| Message ID | 20191023210657.32440-3-pierre-louis.bossart@linux.intel.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | soundwire: update ASoC interfaces | expand |
On 23-10-19, 16:06, Pierre-Louis Bossart wrote: > We need an async mechanism to prevent access to Slaves that are not > fully-enumerated. > > init_completion() will be invoked when the Slave becomes UNATTACHED, > and complete() will be invoked when the state become ATTACHED. Any > read/write before this status change will be delayed with a > wait_for_completion(). > > Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com> > --- > include/linux/soundwire/sdw.h | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/include/linux/soundwire/sdw.h b/include/linux/soundwire/sdw.h > index a381a596212b..1381edfaa206 100644 > --- a/include/linux/soundwire/sdw.h > +++ b/include/linux/soundwire/sdw.h > @@ -565,6 +565,7 @@ struct sdw_slave { > u16 dev_num; > bool probed; > struct completion probe_complete; > + struct completion enumeration_complete; Which series/patch uses this..? Thanks
On 11/3/19 12:22 AM, Vinod Koul wrote: > On 23-10-19, 16:06, Pierre-Louis Bossart wrote: >> We need an async mechanism to prevent access to Slaves that are not >> fully-enumerated. >> >> init_completion() will be invoked when the Slave becomes UNATTACHED, >> and complete() will be invoked when the state become ATTACHED. Any >> read/write before this status change will be delayed with a >> wait_for_completion(). >> >> Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com> >> --- >> include/linux/soundwire/sdw.h | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/include/linux/soundwire/sdw.h b/include/linux/soundwire/sdw.h >> index a381a596212b..1381edfaa206 100644 >> --- a/include/linux/soundwire/sdw.h >> +++ b/include/linux/soundwire/sdw.h >> @@ -565,6 +565,7 @@ struct sdw_slave { >> u16 dev_num; >> bool probed; >> struct completion probe_complete; >> + struct completion enumeration_complete; > > Which series/patch uses this..? [PATCH 00/18] soundwire: code hardening and suspend-resume support
diff --git a/include/linux/soundwire/sdw.h b/include/linux/soundwire/sdw.h index a381a596212b..1381edfaa206 100644 --- a/include/linux/soundwire/sdw.h +++ b/include/linux/soundwire/sdw.h @@ -565,6 +565,7 @@ struct sdw_slave { u16 dev_num; bool probed; struct completion probe_complete; + struct completion enumeration_complete; }; #define dev_to_sdw_dev(_dev) container_of(_dev, struct sdw_slave, dev)
We need an async mechanism to prevent access to Slaves that are not fully-enumerated. init_completion() will be invoked when the Slave becomes UNATTACHED, and complete() will be invoked when the state become ATTACHED. Any read/write before this status change will be delayed with a wait_for_completion(). Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com> --- include/linux/soundwire/sdw.h | 1 + 1 file changed, 1 insertion(+)