On Sat, Apr 21, 2018 at 06:02:52AM -0700, Pierre-Louis Bossart wrote:
On 4/18/18 3:58 AM, Vinod Koul wrote:
+static int sdw_config_stream(struct device *dev,
struct sdw_stream_runtime *stream,struct sdw_stream_config *stream_config, bool is_slave)+{
- /*
* Update the stream rate, channel and bps based on data* source. For more than one data source (multilink),* match the rate, bps, stream type and increment number of channels.*/- if ((stream->params.rate) &&
so if the rate is zero there is no error?
(stream->params.rate != stream_config->frame_rate)) {dev_err(dev, "rate not matching, stream:%s", stream->name);return -EINVAL;- }
- if ((stream->params.bps) &&
same here, what is the intent behind the first test?
IIRC this was to check for valid values, but am not too sure, let me get back to you on these two..
(stream->params.bps != stream_config->bps)) {dev_err(dev, "bps not matching, stream:%s", stream->name);return -EINVAL;- }
- stream->type = stream_config->type;
- stream->params.rate = stream_config->frame_rate;
- stream->params.bps = stream_config->bps;
- if (is_slave)
stream->params.ch_count += stream_config->ch_count;Add comment or TODO that this does not work in device-to-device communication. This is a known limitation that needs to be tracked.
Yes limitation for a feature that we dont support yet. So i dont think we need to do anything atm for this. When the support for device-to-device shows up, that would update this and other conditions valid only for specific cases.
+/**
- sdw_stream_state: Stream states
- @SDW_STREAM_RELEASED: Stream released
- @SDW_STREAM_ALLOCATED: New stream allocated.
- @SDW_STREAM_CONFIGURED: Stream configured
- @SDW_STREAM_PREPARED: Stream prepared
- @SDW_STREAM_ENABLED: Stream enabled
- @SDW_STREAM_DISABLED: Stream disabled
- @SDW_STREAM_DEPREPARED: Stream de-prepared
- */
+enum sdw_stream_state {
- SDW_STREAM_ALLOCATED = 0,
with the kzalloc used in sdw_alloc_stream() the default state is ALLOCATED...You may want to use values which don't include zero.
How would that help. If we use kzalloc then stream init to SDW_STREAM_ALLOCATED is no op. If we dont then it initializes... so seems better to me!