Re: [PATCH v3 0/6] dt-bindings: PCI: qcom: move to dedicated schema (part one)

From: Krzysztof Kozlowski
Date: Wed Feb 14 2024 - 06:56:29 EST


On 26/01/2024 09:56, Krzysztof Kozlowski wrote:
> Hi,
>
> Changes in v3:
> - sm8450: add missing allOf: to common schema, which also fixes issue
> reported by Rob's robot.
> - Link to v2: https://lore.kernel.org/r/20240125-dt-bindings-pci-qcom-split-v2-0-6b58efd91a7a@xxxxxxxxxx
>
> Changes in v2:
> - Switch on SM8[123456]50 to 8 MSI interrupts.
> - Simplify SM8450 clocks.
> - Add Acks/Rb.
> - Link to v1: https://lore.kernel.org/r/20240108-dt-bindings-pci-qcom-split-v1-0-d541f05f4de0@xxxxxxxxxx
>
> DTS fixes for interrupts will be send separately
>
> The qcom,pcie.yaml containing all devices results in huge allOf: section
> with a lot of if:then: clauses making review and changes quite
> difficult.
>
> Split common parts into common schema and then move few devices to
> dedicated files, so that each file will be easier to review.
>
> I did not split/move all devices yet, so if this gets accepted I plan to
> send more patches.

Krzysztof W., Bjorn H., Lorenzo,

Any comments from your side? If not, could you apply the series? I
already have work on top of this and other people are sending patches
touching same diff-context, so they should rebase on top of this.

Best regards,
Krzysztof