Re: [PATCH v3 01/25] dt-bindings: soc: qcom: Add bindings for APR bus

Srinivas Kandagatla
2018-03-02T13:13:42Z

Thanks for the review,

On 01/03/18 20:34, Mark Brown wrote:
> On Thu, Feb 22, 2018 at 10:03:42AM +0000, Srinivas Kandagatla wrote:
>> On 22/02/18 00:14, Rob Herring wrote:
>
>>> Am I saying a single DT node for this? Yes, perhaps.
>> Yes, I will give that a go.
>>
>> So we will endup having something like this in DT for one frontend and
>> backend dailink:
>
> Let's not start encoding DPCM into DT, DPCM is very much an
> implemntation detail of the current stack which we're gradually pushing
> towards replacing with something better. What we want to be doing is
> just treating components inside the SoC the same as components in a
> CODEC, the routing within a SoC being the same as in a CODEC and
> similarly for externally connected devices.
>
>> fe@1 {
>> is-fe;
>> link-name = "MultiMedia1";
>
> In particular having the concept of "front end" in the DT is *very*
> implementation specific. We should just be describing the connections
> that exist in the system.
Yep, it makes sense. is-fe flag will be removed in next version, we can
determine this based on codec dai, so from DT side FE or BE should not
be any different.

Thanks,
srini

>