linux-kernel - Re: [PATCH v4 09/10] dt-bindings: PCI: qcom,pcie-sc7280: Add 'global' interrupt

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] [day] [month] [year] [list]

Message-ID: <652d8d87-2338-4842-829a-cb79532471bc@kernel.org>
Date: Wed, 5 Mar 2025 13:14:44 +0100
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Jorge Ramirez <jorge.ramirez@....qualcomm.com>,
 Bjorn Andersson <andersson@...nel.org>
Cc: Krishna Chaitanya Chundru <krishna.chundru@....qualcomm.com>,
 Bjorn Helgaas <bhelgaas@...gle.com>,
 Lorenzo Pieralisi <lpieralisi@...nel.org>,
 Krzysztof Wilczyński <kw@...ux.com>,
 Manivannan Sadhasivam <manivannan.sadhasivam@...aro.org>,
 Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>,
 Conor Dooley <conor+dt@...nel.org>,
 chaitanya chundru <quic_krichai@...cinc.com>,
 Konrad Dybcio <konradybcio@...nel.org>, cros-qcom-dts-watchers@...omium.org,
 Jingoo Han <jingoohan1@...il.com>, Bartosz Golaszewski <brgl@...ev.pl>,
 quic_vbadigan@...cnic.com, amitk@...nel.org, dmitry.baryshkov@...aro.org,
 linux-pci@...r.kernel.org, devicetree@...r.kernel.org,
 linux-kernel@...r.kernel.org, linux-arm-msm@...r.kernel.org
Subject: Re: [PATCH v4 09/10] dt-bindings: PCI: qcom,pcie-sc7280: Add 'global'
 interrupt

On 05/03/2025 08:36, Jorge Ramirez wrote:
>> I think the patch context ("properties") is confusing here, but it looks
>> to me that these are in sync: interrupts is defined to have 8 items, and
>> interrupt-names is a list of msi0 through msi7.
>>
>> @Krishna, these two last patches (adding the global interrupt) doesn't
>> seem strongly connected to the switch patches. So, if Krzysztof agrees
>> with above assessment, please submit them separately (i.e. a new series,
>> 2 patches, v5).
> 
> um, but without these two patches, the functionality is broken requiring
> users to manually rescan the pci bus (ie, via sysfs) to see what is
> behind the bridge.
> 
> shouldnt the set include all the necessary patches? 

Broken? Then the patchset should be NAKed because nothing can depend on
the DTS and the DTS should not depend on the drivers without some sort
of explanation, either.

Also nothing in the commit msg explains this dependency, so I don't get
the problem here.

Best regards,
Krzysztof