Re: [PATCH 02/13] soundwire: Add support for SoundWire stream management

[Vinod Koul <vinod.koul@intel.com>]

On Thu, Mar 29, 2018 at 08:57:59PM -0500, Pierre-Louis Bossart wrote:
> On 3/28/18 4:38 AM, Vinod Koul wrote:

> >+/**
> >+ * sdw_slave_runtime: Runtime Stream parameters for Slave
> >+ *
> >+ * @slave: Slave handle
> >+ * @direction: Data direction w.r.t Slave
> >+ * @ch_count: Channel count of the Slave w.r.t stream
> 
> same here, I flagged all this already as needing to be fixed. Something's
> not right here, looks to me like you send the wrong version of the
> patches...

Somehow missed this bit will fix

> >+struct sdw_master_runtime {
> >+	struct sdw_bus *bus;
> >+	struct sdw_stream_runtime *stream;
> >+	unsigned int ch_count;
> >+	struct list_head slave_rt_list;
> >+	struct list_head bus_node;
> >+};
> 
> no direction?

Yes no direction here. Why would sdw_master_runtime need direction.

If you look closely we do have direction but in data ports.

One can argue we can move direction up here and remove from data ports,
that seems logical, is that something you would like to see?

> >+enum sdw_data_direction {
> >+	SDW_DATA_DIR_IN = 0,
> >+	SDW_DATA_DIR_OUT = 1,
> >+};
> 
> WTH? Didn't we agree that we would use SoundWire concepts instead of
> redefining new concepts.

So we will make this TX/RX as discussed.

-- 
~Vinod