Re: [alsa-devel] [PATCH 0/2] Graph fixes for using multiple endpoints per port

2018-12-12 Thread Tony Lindgren
* Kuninori Morimoto [181213 01:06]: > > Hi Tony > > > > According to OF graph maintainer, he said that > > > counting port / endpoint are not guaranteed, > > > and we need to use "reg". > > > (It is the reason of b6f3fc005a2c8b425d7a0973b43bef05890bf479) > > > > > > If you could double checked,

Re: [alsa-devel] [PATCH 0/2] Graph fixes for using multiple endpoints per port

2018-12-12 Thread Kuninori Morimoto
Hi Tony > > According to OF graph maintainer, he said that > > counting port / endpoint are not guaranteed, > > and we need to use "reg". > > (It is the reason of b6f3fc005a2c8b425d7a0973b43bef05890bf479) > > > > If you could double checked, and we could agreed > > that "reg" is the solution fo

Re: [alsa-devel] [PATCH 0/2] Graph fixes for using multiple endpoints per port

2018-12-12 Thread Tony Lindgren
* Kuninori Morimoto [181213 00:24]: > (snip) > > foo = <&ep 0>; > > bar = <&ep 1>; > > Ahh, it looks nice idea ! > but hmm..., it seems dtc doesn't allow us to use #address-cells = <2> > for "remote-endpoint". OK. Yeah let's scrap that then, it does not suit for endpoints it seems. Thanks for ch

Re: [alsa-devel] [PATCH 0/2] Graph fixes for using multiple endpoints per port

2018-12-12 Thread Kuninori Morimoto
Hi Tony > /* Codec has 1 DAI */ > Codec { > port { > #address-cells = <2>; > #size-cells = <1>; > > ep: endpoint { > remote-endpoint = ; > }; >

Re: [alsa-devel] [PATCH 0/2] Graph fixes for using multiple endpoints per port

2018-12-12 Thread Tony Lindgren
* Kuninori Morimoto [181212 06:52]: > > Hi Tony, again > > > > > https://patchwork.kernel.org/patch/10712877/ > > > > > > Hmm, so do you have multiple separate ports at the "&sound" node > > > hardware? If so then yeah multiple ports make sense. > > > > > > But if you only a single phys

Re: [alsa-devel] [PATCH 0/2] Graph fixes for using multiple endpoints per port

2018-12-11 Thread Kuninori Morimoto
Hi Tony, again > > > https://patchwork.kernel.org/patch/10712877/ > > > > Hmm, so do you have multiple separate ports at the "&sound" node > > hardware? If so then yeah multiple ports make sense. > > > > But if you only a single physical (I2S?) port at the > > "&sound" node hardware, then IMO