Re: [PATCH v6 0/8] dt-bindings: YAMLify pci/qcom,pcie schema

From: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
To: Rob Herring <robh@kernel.org>
Cc: Andy Gross <agross@kernel.org>,
	Bjorn Andersson <bjorn.andersson@linaro.org>,
	Krzysztof Kozlowski <krzk@kernel.org>,
	Jingoo Han <jingoohan1@gmail.com>,
	Gustavo Pimentel <gustavo.pimentel@synopsys.com>,
	Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>,
	Bjorn Helgaas <bhelgaas@google.com>,
	Stanimir Varbanov <svarbanov@mm-sol.com>,
	Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>,
	Vinod Koul <vkoul@kernel.org>,
	linux-arm-msm@vger.kernel.org, linux-pci@vger.kernel.org,
	devicetree@vger.kernel.org
Subject: Re: [PATCH v6 0/8] dt-bindings: YAMLify pci/qcom,pcie schema
Date: Wed, 11 May 2022 13:13:28 +0300	[thread overview]
Message-ID: <CAA8EJpriMcP4uQ3fjyiCKY+uc82ctXe2VrjO1psPDcp-P++Nhw@mail.gmail.com> (raw)
In-Reply-To: <YnqXxNxFhf/odyka@robh.at.kernel.org>

On Tue, 10 May 2022 at 19:50, Rob Herring <robh@kernel.org> wrote:
>
> On Fri, May 06, 2022 at 06:20:59PM +0300, Dmitry Baryshkov wrote:
> > Convert pci/qcom,pcie schema to YAML description. The first patch
> > introduces several warnings which are fixed by the other patches in the
> > series.
> >
> > Note regarding the snps,dw-pcie compatibility. The Qualcomm PCIe
> > controller uses Synopsys PCIe IP core. However it is not just fused to
> > the address space. Accessing PCIe registers requires several clocks and
> > regulators to be powered up. Thus it can be assumed that the qcom,pcie
> > bindings are not fully compatible with the snps,dw-pcie schema.
> >
> > Changes since v5:
> >  - s/stance/stanza (pointed out by Bjorn Helgaas)
> >
> > Changes since v4:
> >  - Change subjects to follow convention (suggested by Bjorn Helgaas)
> >
> > Changes since v3:
> >  - Rebase on linux-next to include sm8150 patches
> >
> > Changes since v2 (still kudos to Krzyshtof):
> >  - Readded reg-names conversion patch
> >  - Mention wake-gpio update in the commit message
> >  - Remove extra quotes in the schema
> >
> > Changes since v1 (all kudos to Krzyshtof):
> >  - Dropped the reg-names patch. It will be handled separately
> >  - Squashed the snps,dw-pcie removal (from schema) into the first patch
> >  - Replaced deprecated perst-gpio and wake-gpio with perst-gpios and
> >    wake-gpios in the examples and in DT files
> >  - Moved common clocks/clock-names, resets/reset-names and power-domains
> >    properties to the top level of the schema, leaving only platform
> >    specifics in the conditional branches
> >  - Dropped iommu-map/iommu-map-mask for now
> >  - Added (missed) interrupt-cells, clocks, clock-names, resets,
> >    reset-names properties to the required list (resets/reset-names are
> >    removed in the next patch, as they are not used on msm8996)
> >  - Fixed IRQ flags in the examples
> >  - Merged apq8064/ipq8064 into the single condition statement
> >  - Added extra empty lines
> >
> > Dmitry Baryshkov (8):
> >   dt-bindings: PCI: qcom: Convert to YAML
> >   dt-bindings: PCI: qcom: Do not require resets on msm8996 platforms
> >   dt-bindings: PCI: qcom: Specify reg-names explicitly
> >   dt-bindings: PCI: qcom: Add schema for sc7280 chipset
> >   arm64: dts: qcom: stop using snps,dw-pcie falback
> >   arm: dts: qcom: stop using snps,dw-pcie falback
> >   arm: dts: qcom-*: replace deprecated perst-gpio with perst-gpios
> >   arm64: dts: qcom: replace deprecated perst-gpio with perst-gpios
> >
> >  .../devicetree/bindings/pci/qcom,pcie.txt     | 398 ----------
> >  .../devicetree/bindings/pci/qcom,pcie.yaml    | 714 ++++++++++++++++++
>
> What tree do these apply to because they don't apply to rc1. I'm
> assuming the PCI tree and Lorenzo should take them.

The series depends on the patch in Lorenzo's tree (sm8150 bindings),
so I'd assume it would be natural to merge these patches through his
tree too.

-- 
With best wishes
Dmitry