Re: [PATCH 1/2] dt-bindings: net: qcom,ipa: describe IPA v4.5 interconnects

From: David Heidelberg
Date: Tue Oct 26 2021 - 12:35:08 EST


Hello,

sent fixed version "[PATCH v2] dt-bindings: net: qcom,ipa: IPA does support up to two iommus".

Thank you Alex

David

P.S. I have some basic structure for qcom,smp2p.yaml, but to make it right take some effort. If someone want to, I can send it as WIP or just into private email.
Best regards
David Heidelberg

On Tue, Oct 26 2021 at 09:17:43 -0500, Alex Elder <elder@xxxxxxxx> wrote:
On 10/21/21 5:11 PM, David Heidelberg wrote:
Hello Alex,

it's make dtbs_check (for me with ARCH=arm)

David

Thank you, I see the errors now. I am gathering information
so I can fix the interconnect issue for IPA v4.5 (SDX55).

Your other suggested change (increasing the allowed number of
iommus) is the right thing to do, but it seems you need to
specify "minItems = 1" as well to avoid the error Rob pointed
out. You should post version two of that patch (only), or if
you would prefer I do that, say so.

Another error that shows up is that no "qcom,smp2p" schema
matches. I'm pretty sure that's simply because the binding
"soc/qcom/qcom,smp2p.txt" has not bee converted to YAML.

-Alex