On Wed, Apr 12, 2023 at 03:27:24PM +0200, Lorenzo Pieralisi wrote: > On Wed, Apr 12, 2023 at 04:59:18PM +0530, Manivannan Sadhasivam wrote: > > [...] > > > > > This is fine. The plan is to remove the "iommus" property from Qcom PCI binding > > > > since we have removed the usage of that property from devicetree [1]. Initially > > > > the iommu properties were not documented at all in the binding. But commit, > > > > "dt-bindings: PCI: qcom: Add SM8550 compatible" added them to the binding to > > > > satisfy dtbs check. But in parallel, the patch removing "iommus" property from > > > > dts got merged to qcom tree. > > > > > > > > So now we have 2 options here: > > > > > > > > 1. Amend the commit "dt-bindings: PCI: qcom: Add SM8550 compatible" to remove > > > > the "iommus" property. > > > > > > > > 2. I will submit a separate patch removing that property. > > > > > > > > Lorenzo, let me know what works for you. Sorry for the mess! Confusion happened > > > > due to patches getting applied without sync. > > > > > > You can inline a diff here for (1), I will amend the commit. > > > > > > > Here you go: > > > > diff --git a/Documentation/devicetree/bindings/pci/qcom,pcie.yaml b/Documentation/devicetree/bindings/pci/qcom,pcie.yaml > > index a1318a4ecadf..1548a7114732 100644 > > --- a/Documentation/devicetree/bindings/pci/qcom,pcie.yaml > > +++ b/Documentation/devicetree/bindings/pci/qcom,pcie.yaml > > @@ -60,9 +60,6 @@ properties: > > minItems: 1 > > maxItems: 8 > > > > - iommus: > > - maxItems: 1 > > - > > iommu-map: > > maxItems: 2 > > Done, please check, thanks. > LGTM, thanks! - Mani > Lorenzo -- மணிவண்ணன் சதாசிவம்
