On Tue, Apr 10, 2018 at 10:47:24AM -0500, Pierre-Louis Bossart wrote:
Again we are not writing a spec but code and comments to explain the code. The above is not related to "this" series so we should not add it here. When we add device-device support, multi-master support these can be added
We are adding blocks in a giant building, but you are only looking at the giant picture but not the block in discussion!
no, you have keep mentions to multi-link that are questionable. Either you remove them or you state it's reserved for future use.
I have clarified on bus_node above, can you check and tell me if there is anything else?
The code pattern for the bank switch was inspired by multi-link.
As you pointed out, we are fixing that too
Okay so we are going to track if master and slaves runtimes are configured by adding a flag in each of them. Once all the components are configured the stream stated will be updated. Does that sound ok?
How do you detect that all components are configured? We have a distributed allocation/configuration with multiple actors, I don't what triggers the transition to a new state. see more below.
+Configuration state of stream. Operations performed before entering in +this state:
- (1) The resources allocated for stream information in
SDW_STREAM_ALLOCATED
state are updated here. This includes stream parameters, Master(s)and Slave(s) runtime information associated with current stream.- (2) All the Master(s) and Slave(s) associated with current stream provide
the port information to Bus which includes port numbers allocated byMaster(s) and Slave(s) for current stream and their channel mask.so how can Master ports be configured if it is added to a stream *after* the CONFIGURED state?
Yes you have a valid point wrt documentation, will update it.
It's not just the documentation, I wonder if it actually works. You would want to make sure all masters and slaves are allocated before configuring them, otherwise the error handling flow is just awful to deal with. You could end-up in a situation where all Slaves are configured but the master allocation fails.
Ok
If the ALLOCATED and CONFIGURED states are clearly separated out, then what if the flag you mentioned used for?
As you rightly pointed out that we have distributed allocation/configuration with multiple actors. So to ensure that all actors are properly configured before we move stream state to CONFIGURED, we need to check and to do so the flag will be very useful :-)