From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mx0b-0031df01.pphosted.com (mx0b-0031df01.pphosted.com [205.220.180.131]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 7C3411DB92A for ; Wed, 5 Mar 2025 07:36:31 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=205.220.180.131 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1741160193; cv=none; b=tfm44YU2bfG0O3qC8Bj+RFXm3tvqwSP6mFjtIm6a6LsPMWCfA322eOIE1h8+qqv+Am1LvS0w6sESb5P/QEbi8YRbX1wCapBk2Jmaf5l8WLfO/f3n/dgLzkW2VjTmxC/N1QfRLKVAK8WERB2pt8qma9QP3VXkinCyAmJZtCb5FyI= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1741160193; c=relaxed/simple; bh=BTVy2qtZxarzMPWFhSUSPZJ3OCz22mtLdewDBNlhhWY=; h=From:Date:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=iAS7aSrwUYVw+Qrhwxyv98yyeF48icBJGGYgIbIWeCXkRn3M+1UXJjwvQHgImuedHzlUg72AfjL+/mx8gxk4gVLt5Rp5hzuZMePC3o3PbDsyhpwZj0FZiQQPCeNmpX9qNvD3pNBtbDRrhlg/W8VUFR+UCq69wNlL8eITHnapZkg= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=oss.qualcomm.com; spf=pass smtp.mailfrom=oss.qualcomm.com; dkim=pass (2048-bit key) header.d=qualcomm.com header.i=@qualcomm.com header.b=hDAGl+/m; arc=none smtp.client-ip=205.220.180.131 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=oss.qualcomm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=oss.qualcomm.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=qualcomm.com header.i=@qualcomm.com header.b="hDAGl+/m" Received: from pps.filterd (m0279868.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.2/8.18.1.2) with ESMTP id 524N9wMR005533 for ; Wed, 5 Mar 2025 07:36:30 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=qualcomm.com; h= cc:content-type:date:from:in-reply-to:message-id:mime-version :references:subject:to; s=qcppdkim1; bh=T1d16P+U/rXcTJW+B5kAc654 E3IZiLRurO5HUBWmlmQ=; b=hDAGl+/mMCzOFP9OeK5dqkMi0Rl7fk5sEiJJQLz6 lwglXtry0wIfXPc9cLb/gy1yGVM7O/0KA6XkjQTrtIbcPwpMStwJy7MtcwjeuV2f t5O5jFBVHexsOt0YYpXBX7be6Fnb1pkZGJpFEzoaYDroVlSeqSVDVyk4/zhCOFIQ aIwte6gt7QNqj3w9bB8KvJznGcZMM+v2C7YzpaHR9dyAJBAI+0l3srBqTNx5IWgs 96le4JCIIIQYkes5zKxH0tDH7k9U+4kp4EJepKBUGm2I1/d72Kqk9Bcej3QlHdQy jwtMtS3ttyX0rUovlxNtdiG9RVoYD4ihMmV4cXlnaDq8bA== Received: from mail-qv1-f70.google.com (mail-qv1-f70.google.com [209.85.219.70]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 455p6v4g86-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128 verify=NOT) for ; Wed, 05 Mar 2025 07:36:30 +0000 (GMT) Received: by mail-qv1-f70.google.com with SMTP id 6a1803df08f44-6e8e171fd66so32035546d6.0 for ; Tue, 04 Mar 2025 23:36:30 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1741160189; x=1741764989; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:date:from:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=T1d16P+U/rXcTJW+B5kAc654E3IZiLRurO5HUBWmlmQ=; b=Rdff/di7U/5QxQCfL3mSSp4gGFuGxIW2aeF10ATmVEndKOwNlw36RVJoEAlmNymQkC yGqdIuo5N9nwkGaMNLWfrAOwomWJsZCGiFF8PHT9deC9Dadoo/Jrq58T3VoEzfU/9XxZ xzphHr/EjBrTddXpSV4wx1Zbotc+pFNMxwRgjWs1zmsECXsPYk+LvqI4Ru1jTkdQLTwW XLSyBz7/u3OgdPktd8c3fhq7jmt7RW0rLBGij1BHHZ+mo0tr7vYHsv44lHxiLheN2yKi 7cQxhAhnNbxgWHo1k+OokZlHafr4WbrJNVOM8OQZ4VAyuODRBN/vLWj4WCSuHgxfksOW PNYg== X-Forwarded-Encrypted: i=1; AJvYcCVvkNsCdw0w2v/lrU19R5+pQ0da46izyQRb7zvnIhLMUHt63kRpeJMlJbjscsH0G6ap8Vvae2cfQDE=@vger.kernel.org X-Gm-Message-State: AOJu0YxalJ/rxeOgCTFHgGxLDjO4Ju29+m/ZJ9yZy30OYLXAq0IqvX79 idy2Tn81+eCNf9mQVPQjxlCnOtWHdbrH9jP+X9dApeA+kRZKApI78EiC0xjlFw+mwEwLel9cN4J Xejb0QS7dPDn83WULa5jqE+uTKMrL+ORFGm3uoz88Z/T1N2mQZCrsIklYyHo= X-Gm-Gg: ASbGnctqD1AAgLx/X3XskwynaDtvT/BoELYftvzj6Qh4x1LB/5idjJQs1kOC/0vUGGn nnGNhnf7NrAe0Fp3+ICQq3m9iPamEX2F0LqHbJMbCelx6RSzQ976Ah5TRT/hKfi5r7a/HL8Uxy4 SXuu+hIlyBPD0qCaJIKIs7NFhAMZNePOHkJIxxs0y++fWZHDQ2xXblTWS72KBS955cLd1AFw5wl Ob+Zt0EBcSdJYkcskbKyNhnjslsZok6AacPLVM/1YFB7NhWSZRIFbWe02Lu7mdSvPyzwEVe/wJO AeusvC120AjxvmvHavyFUQ5wo/auahi4s7CWlFhwhU3F1mkC X-Received: by 2002:ad4:5f87:0:b0:6e4:2f4c:f2d2 with SMTP id 6a1803df08f44-6e8e6d3585bmr33462016d6.31.1741160189285; Tue, 04 Mar 2025 23:36:29 -0800 (PST) X-Google-Smtp-Source: AGHT+IEg7o+3YrqSXsCnQ/x9TphdfEbcLiAgOEKHpqGxdPXvtfHan6pbMvY6utfj+hLuaIquk3bL0A== X-Received: by 2002:ad4:5f87:0:b0:6e4:2f4c:f2d2 with SMTP id 6a1803df08f44-6e8e6d3585bmr33461786d6.31.1741160188837; Tue, 04 Mar 2025 23:36:28 -0800 (PST) Received: from trex (54.red-79-144-194.dynamicip.rima-tde.net. [79.144.194.54]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-43bd426c33asm9222345e9.3.2025.03.04.23.36.27 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 04 Mar 2025 23:36:28 -0800 (PST) From: Jorge Ramirez X-Google-Original-From: Jorge Ramirez Date: Wed, 5 Mar 2025 08:36:26 +0100 To: Bjorn Andersson Cc: Krzysztof Kozlowski , Krishna Chaitanya Chundru , Bjorn Helgaas , Lorenzo Pieralisi , Krzysztof =?utf-8?Q?Wilczy=C5=84ski?= , Manivannan Sadhasivam , Rob Herring , Krzysztof Kozlowski , Conor Dooley , chaitanya chundru , Konrad Dybcio , cros-qcom-dts-watchers@chromium.org, Jingoo Han , Bartosz Golaszewski , quic_vbadigan@quicnic.com, amitk@kernel.org, dmitry.baryshkov@linaro.org, linux-pci@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org, jorge.ramirez@oss.qualcomm.com Subject: Re: [PATCH v4 09/10] dt-bindings: PCI: qcom,pcie-sc7280: Add 'global' interrupt Message-ID: References: <20250225-qps615_v4_1-v4-0-e08633a7bdf8@oss.qualcomm.com> <20250225-qps615_v4_1-v4-9-e08633a7bdf8@oss.qualcomm.com> <20250226-enlightened-chachalaca-of-artistry-2de5ea@krzk-bin> Precedence: bulk X-Mailing-List: linux-pci@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-Proofpoint-GUID: ex33ZSJ48pq4936BloRxtT6Hpejwx9T- X-Proofpoint-ORIG-GUID: ex33ZSJ48pq4936BloRxtT6Hpejwx9T- X-Authority-Analysis: v=2.4 cv=fatXy1QF c=1 sm=1 tr=0 ts=67c7fefe cx=c_pps a=oc9J++0uMp73DTRD5QyR2A==:117 a=cl0az7d0LwC7qAhl51RXhA==:17 a=kj9zAlcOel0A:10 a=Vs1iUdzkB0EA:10 a=EUspDBNiAAAA:8 a=9WSsZrPlu9oLnKen9MIA:9 a=CjuIK1q_8ugA:10 a=iYH6xdkBrDN1Jqds4HTS:22 X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1093,Hydra:6.0.680,FMLib:17.12.68.34 definitions=2025-03-05_03,2025-03-05_01,2024-11-22_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 clxscore=1011 bulkscore=0 spamscore=0 priorityscore=1501 malwarescore=0 impostorscore=0 lowpriorityscore=0 mlxlogscore=999 mlxscore=0 adultscore=0 suspectscore=0 phishscore=0 classifier=spam authscore=0 authtc=n/a authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.19.0-2502100000 definitions=main-2503050059 On 26/02/25 10:29:43, Bjorn Andersson wrote: > On Wed, Feb 26, 2025 at 08:32:42AM +0100, Krzysztof Kozlowski wrote: > > On Tue, Feb 25, 2025 at 03:04:06PM +0530, Krishna Chaitanya Chundru wrote: > > > Qcom PCIe RC controllers are capable of generating 'global' SPI interrupt > > > to the host CPU. This interrupt can be used by the device driver to handle > > > PCIe link specific events such as Link up and Link down, which give the > > > driver a chance to start bus enumeration on its own when link is up and > > > initiate link training if link goes to a bad state. The PCIe driver can > > > still work without this interrupt but it will provide a nice user > > > experience when device gets plugged and removed. > > > > > > Hence, document it in the binding along with the existing MSI interrupts. > > > Global interrupt is parsed as optional in driver, so adding it in bindings > > > will not break the ABI. > > > > > > Signed-off-by: Krishna Chaitanya Chundru > > > --- > > > Documentation/devicetree/bindings/pci/qcom,pcie-sc7280.yaml | 8 +++++--- > > > 1 file changed, 5 insertions(+), 3 deletions(-) > > > > > > diff --git a/Documentation/devicetree/bindings/pci/qcom,pcie-sc7280.yaml b/Documentation/devicetree/bindings/pci/qcom,pcie-sc7280.yaml > > > index 76cb9fbfd476..7ae09ba8da60 100644 > > > --- a/Documentation/devicetree/bindings/pci/qcom,pcie-sc7280.yaml > > > +++ b/Documentation/devicetree/bindings/pci/qcom,pcie-sc7280.yaml > > > @@ -54,7 +54,7 @@ properties: > > > > > > interrupts: > > > minItems: 8 > > > - maxItems: 8 > > > + maxItems: 9 > > > > > > interrupt-names: > > > items: > > > @@ -66,6 +66,7 @@ properties: > > > - const: msi5 > > > - const: msi6 > > > - const: msi7 > > > + - const: global > > > > Either context is missing or these are not synced with interrupts. > > > > 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? > > Regards, > Bjorn > > > Best regards, > > Krzysztof > >