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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <90a50ab4-a513-48af-b13a-bba082e49540@linaro.org>
Date: Wed, 14 Feb 2024 12:56:12 +0100
From: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
To: Lorenzo Pieralisi <lpieralisi@...nel.org>,
 Krzysztof Wilczyński <kw@...ux.com>,
 Bjorn Helgaas <bhelgaas@...gle.com>,
 Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
 Manivannan Sadhasivam <mani@...nel.org>
Cc: Bjorn Andersson <andersson@...nel.org>, Conor Dooley
 <conor+dt@...nel.org>, Konrad Dybcio <konrad.dybcio@...aro.org>,
 Rob Herring <robh+dt@...nel.org>, linux-arm-msm@...r.kernel.org,
 linux-pci@...r.kernel.org, devicetree@...r.kernel.org,
 linux-kernel@...r.kernel.org, Rob Herring <robh@...nel.org>,
 Manivannan Sadhasivam <manivannan.sadhasivam@...aro.org>
Subject: Re: [PATCH v3 0/6] dt-bindings: PCI: qcom: move to dedicated schema
 (part one)

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@linaro.org
> 
> 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@linaro.org
> 
> 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