Hi Tony
The issue I have with that it does not then follow the binding doc :)
See this part in Documentation/devicetree/bindings/graph.txt:
"If a single port is connected to more than one remote device, an 'endpoint' child node must be provided for each link."
Isn't the I2C TDM case the same as "single port connecected to more than one remote device" rather than multiple ports?
To me it seems we're currently only handling the multiple ports case, and not multiple endpoints for a port. Other than fixing that, things should work just as earlier with my two patches. That is unless I accidentally broke something.
So just trying to correct the binding usage. Or am I missing something?
I'm not 100% sure your "I2C TDM case", but you can check multi-endpoint sample on "Example: Multi DAI with DPCM" below. "pcm3168a" is using multi-endpoint. Does this help you ?
https://patchwork.kernel.org/patch/10712877/
Best regards --- Kuninori Morimoto