Re: [alsa-devel] [PATCH 4/6] soundwire: stream: do not update parameters during DISABLED-PREPARED transition

[Pierre-Louis Bossart]

> > +	/*
> > +	 * when the stream is DISABLED, this means sdw_prepare_stream()
> > +	 * is called as a result of an underflow or a resume operation.
> > +	 * In this case, the bus parameters shall not be recomputed, but
> > +	 * still need to be re-applied
> > +	 */
> > +	if (stream->state == SDW_STREAM_DISABLED)
> > +		update_params = false;
>
> Should this not be handled by the caller..? I do not like to deduce this
> here as the info is already available in dai driver, so go ahead and
> propagate it and get it from caller when it is required..

No, this update_params boolean is used later on to modify the bandwidth 
computation. These values are not accessible to the caller (and should 
absolutely be kept private/opaque).