From nobody Tue Dec 2 02:19:53 2025 Received: from mx0a-0031df01.pphosted.com (mx0a-0031df01.pphosted.com [205.220.168.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 19B3B302CB9 for ; Wed, 19 Nov 2025 18:10:25 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=205.220.168.131 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763575835; cv=none; b=qxzf49+RJBoVBZtxnF6DK+EFxTDRyiBHmPPpkAk+FtGwSVgsjlkG4Ipn7zRqGJLr27iiYQYRclUz8npFEYEAy4njBbFr7+coBa1TZ2iD63naGL/O59kVgopIH/lgafNCKMbb0D8LpsaE7k/u7dmgEzxHGJ04oK3S5FsfPEe1Iug= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763575835; c=relaxed/simple; bh=aB8AQU5T79GE8yEjbX3yEliqZgfm3QNVjb6a8THRBX4=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=ZdXV/PywKzQHJXnT1/DB+UAEQ2KUdJaxJj6gwnMOnOg2E2J/OWVUMsFdnmEFibj1mAefqz3bEy+guNUpU43SPUDqGJLHOx1C00HF1Sh9uHXxem6aMSd1XeXRfKIm+pnjqk0HFx1It/gIMkrh1e+DQcFApSFfYNbdFNfXCUlA21o= 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=ZNdcde/H; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b=FmBt7nJr; arc=none smtp.client-ip=205.220.168.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="ZNdcde/H"; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b="FmBt7nJr" Received: from pps.filterd (m0279863.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.11/8.18.1.11) with ESMTP id 5AJFkPXI2061048 for ; Wed, 19 Nov 2025 18:10:23 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=qualcomm.com; h= cc:content-transfer-encoding:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to; s=qcppdkim1; bh= M13z2nhJ5XNNt8OBEkRgQFiRfSl+F/tnKBCTf6qisp4=; b=ZNdcde/Hv4gTLz0i qagLhtLdd8qfSoQA9bdy3IT1DNzeNom0L6tzmK3uVTUOqz7Layo1L8GUa+C3+887 95IwVoRFqaF+7CkN48W3m++4O77PPGk13WHKmcv2DpDS/S3jCTEtiZRnukxtQtMm o55GtzE/N0HeBacYmy77nrzkxdxqQ58NRWVaT26lo0ixlKObJZheKQ6c6yglyfEv oFDNkmbSHpSkYh0GnMAvnDJ289yxGtRaJEuNxHf9oXKY5IdRLPX2UvbwxO1yv05T jtQF14G+ivgPDPtftN6q8ewpqFfPcpj7VKgGfww1TgM7If2nEYQMo7VfD5NWK/jw HcMLcw== Received: from mail-pl1-f200.google.com (mail-pl1-f200.google.com [209.85.214.200]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 4ah9fu9ym6-1 (version=TLSv1.3 cipher=TLS_AES_128_GCM_SHA256 bits=128 verify=NOT) for ; Wed, 19 Nov 2025 18:10:23 +0000 (GMT) Received: by mail-pl1-f200.google.com with SMTP id d9443c01a7336-297f48e81b8so163215ad.0 for ; Wed, 19 Nov 2025 10:10:23 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oss.qualcomm.com; s=google; t=1763575823; x=1764180623; darn=vger.kernel.org; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:from:to:cc:subject:date:message-id :reply-to; bh=M13z2nhJ5XNNt8OBEkRgQFiRfSl+F/tnKBCTf6qisp4=; b=FmBt7nJrAyrpxmpjHVTozMqJPiB+DJBY8RKEWjcI7gC/FrcHr0w3/xAUnBSe6Q80an JtawBYwTZ2XO2pkI7jvlJcv2XH2dO3rYfh+Y0Bup0TgWi7jVeI+ermTSU9SRaSmkCuhL zbke4HnqklVOgPzftx1Yd4lxNcHCzTZYoiqHZB4RlMANXx2I/rS3jmuoIz5C/3pV2Z7O Yazi4SnZpkIBUxc1u2SfrDHhcf4zCvOvNc8gmpDoBEk2QRmfeD+RgFl0wVBp2xNsodWw hEuYxA6MnYEYPIs25v88UQZx/F/RqB/BfR/e5LLMnTqp7E8JNhhVsZD3820sHFrDaw0L emxQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1763575823; x=1764180623; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:x-gm-gg:x-gm-message-state:from:to :cc:subject:date:message-id:reply-to; bh=M13z2nhJ5XNNt8OBEkRgQFiRfSl+F/tnKBCTf6qisp4=; b=TH9QK5rz0d6NmnkTj6druK3f11XZCi2QabMhEMtWNr9BCzXS36+YY4LUAa4C7pCQ02 kWT03/0YOORJwGtg68V9M3ae2fTDw5u5qSMcsMquT9K0GFia+5AONldSbCSwx1KBtNXV YH1nj6DTSX9AGanL8yiXJsEpcAjTrEdx+PczLKm8lHKfcFnEanu6NXVyb6N3aeN4V2QP VRXXfxNPtYPUNTQsGQwmCfp7Lwz/LVGp4uoo/nLDIZch6STxoQHn2xcNK3l9GkXSkGNw rI7nEe3+mwbgbzv+C96y9Z6VM6t+2/6mh5P/1+DkPzeWedGBXQOiKCeBLBPOTHVVATs4 Djjw== X-Forwarded-Encrypted: i=1; AJvYcCXptIW1ErLIc/ZPoApbdRte7bs3SDrbiOcsrOJKJUGbcERY11e8/i/4YqPxDHPHvVAJoYoAmvAWF/BPAkY=@vger.kernel.org X-Gm-Message-State: AOJu0YzstUDFl2TIelNdMQn6/KHt2wtjSAiTiZr5CJRep6U7qdGGMYJ0 GY+M4JdIFhGkgzyx9xf0UIOE9v1NEBwXy4nlrDzYiwC9NCHNSzEroS2msMm1Bjs2t6mFx0e3btu lPvInav/CNMnIM3eOFdLV8Yn13lo4KL3Tnpd+yvRdrkEyUX6YqFkQSfeFninQEODqg88= X-Gm-Gg: ASbGncuaQ6DigC+ls3WZGPVvdfzoKsSd9ntNdmk1W81Dt0o9MCmlCMYpW0I2wyP+7+d 7OG/cO9tBJnfteh+NZkP9C+SPrqZMU/OywIxQPc8y+96tyAU+PbqUAhSkivdSNZjVsUn2VbjAUh 1NlystXXIgGJ9955tUFXkDh+IwQfZrOrvwLiylD69ijbi/kNHutBAEkabAE5Lh9Zs3ujHtOh7FL ISD84rUHRIQu+w08lObJW48QHfsSPX4r6c54HCRTzC1OBugZg0W0ajK7OYWc62GI+1zPkyQlXbP 9RiLCKwudZvJcoT6JeoRIRG1Uu6MdOrruaV3Js44vFQToT7z/ZL4vSGQns5ikQS3vKznx/HUYvT F87mEx7kc+QviZcMK7Z9Jkj+YvMXDcg96zEOAoak= X-Received: by 2002:a17:903:1a90:b0:269:b2e5:900d with SMTP id d9443c01a7336-29b5c0b3ca9mr918585ad.5.1763575822644; Wed, 19 Nov 2025 10:10:22 -0800 (PST) X-Google-Smtp-Source: AGHT+IH7ZN9s2BxP3STc5lsVm3YHhRFutnEWOCIP3ZR8jYDADQVNljE4X6IilASjwkOZu5k9soId1g== X-Received: by 2002:a17:903:1a90:b0:269:b2e5:900d with SMTP id d9443c01a7336-29b5c0b3ca9mr918165ad.5.1763575822195; Wed, 19 Nov 2025 10:10:22 -0800 (PST) Received: from [192.168.1.102] ([120.56.197.63]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-29b5b111016sm1408865ad.6.2025.11.19.10.10.17 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 19 Nov 2025 10:10:21 -0800 (PST) From: Manivannan Sadhasivam Date: Wed, 19 Nov 2025 23:40:07 +0530 Subject: [PATCH 1/2] PCI: dwc: Skip PME_Turn_Off broadcast and L2/L3 transition during suspend if link is not up Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20251119-pci-dwc-suspend-rework-v1-1-aad104828562@oss.qualcomm.com> References: <20251119-pci-dwc-suspend-rework-v1-0-aad104828562@oss.qualcomm.com> In-Reply-To: <20251119-pci-dwc-suspend-rework-v1-0-aad104828562@oss.qualcomm.com> To: Jingoo Han , Manivannan Sadhasivam , Lorenzo Pieralisi , =?utf-8?q?Krzysztof_Wilczy=C5=84ski?= , Rob Herring , Bjorn Helgaas Cc: linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org, vincent.guittot@linaro.org, zhangsenchuan@eswincomputing.com, Manivannan Sadhasivam X-Mailer: b4 0.14.3 X-Developer-Signature: v=1; a=openpgp-sha256; l=1100; i=manivannan.sadhasivam@oss.qualcomm.com; h=from:subject:message-id; bh=aB8AQU5T79GE8yEjbX3yEliqZgfm3QNVjb6a8THRBX4=; b=owEBbQGS/pANAwAKAVWfEeb+kc71AcsmYgBpHggDw7uT50wCuHAJeUJpbSYo00637n44xxTrW qCCA3gYjZmJATMEAAEKAB0WIQRnpUMqgUjL2KRYJ5dVnxHm/pHO9QUCaR4IAwAKCRBVnxHm/pHO 9RIyB/0eWUbFMn4mUD4N851/q3ujQ8LdgHgJhkb49RcDa2GLr4w653BIi2hEXIQGwCEMKclTn92 8T0/OwibDSct5/qwZeTuqU9WIJeK2B+NuTfCkkkSSROuvPVKlBTYR8aC/HmtQG6skiDZ+PPAhhQ UgpE9lQNP+gYCgSqhUR/BZeuXnW1YP64KEJKRM27siyKpcn/1gjfrvnRoXaFrhrbDksaeN/jCq7 CUxd0i58XbOTyrTerOyFdplxX7yOrA9U0u5qtPa+IRDG0yk3fYANuUAoComfRgekR9i3Q75jChY LK1//zeGVzzga6lsXC1leSORSqGvFnlfHBhMwE1/Lc7ax6bb X-Developer-Key: i=manivannan.sadhasivam@oss.qualcomm.com; a=openpgp; fpr=C668AEC3C3188E4C611465E7488550E901166008 X-Proofpoint-Spam-Details-Enc: AW1haW4tMjUxMTE5MDE0NCBTYWx0ZWRfX3WUSWJbjJl3O jjRgLznRuvIVt3u0fC1P9gAJozwflAndFbXb50hrfXHLsJ88xrx1jC1IDJEf6GV+I2TyUDCD9gL dQVoSvOZkSWSrKJD5iSOQLoJFo5hvim4yyGZlMESxjzLGoUACowQ6/vN7gVJP525RgrrL+riPXd mqN93Wa8yARPTgudI9k2kWXVGob+KP70SZ5ZcfBSI2zslv+sgUxCu0plsAYYP0Q12bqihn44A5u 2rn6HNTzwK+gwzqFSOaqyxo5cLjFuj2SPI/xdySVX17IF+10UNXS07WEFORSJ6xdtdvKnoo+ZIS +chjlrRbB6Y4qaljJDp/wDwHU+Ff9BAByrKVdP4tg== X-Authority-Analysis: v=2.4 cv=KZTfcAYD c=1 sm=1 tr=0 ts=691e080f cx=c_pps a=IZJwPbhc+fLeJZngyXXI0A==:117 a=xeX0Tm50+WxWVv+NCdhSGA==:17 a=IkcTkHD0fZMA:10 a=6UeiqGixMTsA:10 a=s4-Qcg_JpJYA:10 a=VkNPw1HP01LnGYTKEx00:22 a=EUspDBNiAAAA:8 a=aqWwvIXvDE_FxjdaEBsA:9 a=QEXdDO2ut3YA:10 a=uG9DUKGECoFWVXl0Dc02:22 X-Proofpoint-ORIG-GUID: A328wAkzfhiGoY5b1E7qyNPnazyCwTwS X-Proofpoint-GUID: A328wAkzfhiGoY5b1E7qyNPnazyCwTwS X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1121,Hydra:6.1.9,FMLib:17.12.100.49 definitions=2025-11-19_05,2025-11-18_02,2025-10-01_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 suspectscore=0 adultscore=0 lowpriorityscore=0 spamscore=0 phishscore=0 malwarescore=0 bulkscore=0 classifier=typeunknown authscore=0 authtc= authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.22.0-2510240001 definitions=main-2511190144 During system suspend, if the PCIe link is not up, then there is no need to broadcast PME_Turn_Off message and wait for L2/L3 transition. So skip them. Signed-off-by: Manivannan Sadhasivam Reviewed-by: Frank Li Tested-by: Vincent Guittot --- drivers/pci/controller/dwc/pcie-designware-host.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/drivers/pci/controller/dwc/pcie-designware-host.c b/drivers/pc= i/controller/dwc/pcie-designware-host.c index 20c9333bcb1c..8fe3454f3b13 100644 --- a/drivers/pci/controller/dwc/pcie-designware-host.c +++ b/drivers/pci/controller/dwc/pcie-designware-host.c @@ -1129,6 +1129,9 @@ int dw_pcie_suspend_noirq(struct dw_pcie *pci) u32 val; int ret; =20 + if (!dw_pcie_link_up(pci)) + goto stop_link; + /* * If L1SS is supported, then do not put the link into L2 as some * devices such as NVMe expect low resume latency. @@ -1162,6 +1165,7 @@ int dw_pcie_suspend_noirq(struct dw_pcie *pci) */ udelay(1); =20 +stop_link: dw_pcie_stop_link(pci); if (pci->pp.ops->deinit) pci->pp.ops->deinit(&pci->pp); --=20 2.48.1 From nobody Tue Dec 2 02:19:53 2025 Received: from mx0a-0031df01.pphosted.com (mx0a-0031df01.pphosted.com [205.220.168.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 209F726F2AA for ; Wed, 19 Nov 2025 18:10:31 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=205.220.168.131 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763575838; cv=none; b=PAaJO1w59R37YmducOeFwHHFq4XeBYQznrk+TjCEDOWWq55tm/uFOKbb3VwBri9MdXV22W7GxSAYrEkQPfX8d+z0PlnVuJEnkGXC3oEVMoj7L4k93p4NYs/h7Foy05wJmYjPFlmZCp1fzMpcNBPYKq5oedPJup+p2ROwtrLte2M= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763575838; c=relaxed/simple; bh=Cf0vCAl5snTvRcvHYoqSDh6FCeDaD461npSza6EkeXo=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=aNQB4gvTsuzmHMEhDY9JI4RxhZpPZXqaYD5Ppl7R4P/G/M1ec3PxWLhXWrTlDGyymUoNitgc+jIhX/8+Nu9gZlUIoa7U7pSuSNxetFwXqsAU89BIq1JUpwLtqk+5IVnP3MrPFjL3xc+GfZSqnrvR+jT707XzCq4SetfPivkLLYY= 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=NT2/O0sE; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b=jjcUpBkg; arc=none smtp.client-ip=205.220.168.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="NT2/O0sE"; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b="jjcUpBkg" Received: from pps.filterd (m0279862.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.11/8.18.1.11) with ESMTP id 5AJFkXoJ2112450 for ; Wed, 19 Nov 2025 18:10:29 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=qualcomm.com; h= cc:content-transfer-encoding:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to; s=qcppdkim1; bh= NVTn4Wtau+BZ+/gvjqlvfniNUMfKB1rE87mf5c2GkIU=; b=NT2/O0sEvIIpcRO9 KYQt1yGAxO8rNS8QA5IMHe4aEwtu7Yo+SMslOfZHwts4aDR/IME2LzWUnypJcYHy BMZH1RTbTnP8YUTE0z++3KP5x18XTfAjTpKGa/uwHe0HM5OB+HVTmtsMZ7FPOc9H dj+d1ljAQnEB8txQoOw5znZeEpHjLI1ntDoGaKdpCMzoTk5Xi5CWS27E8aFPLrKg o1C9VSe0X/0gcSEvsfoMfqRUr+rqjZSmLjGMSUg8ftEHBQUGP7/twyzE2I5xPVp2 V6xNUMYlQ6hYg5mRidpmRmTpSnopOG7qC2EKeiO91Kfz8KKDhr7hPS4QwpCINqPG aT95NA== Received: from mail-pl1-f198.google.com (mail-pl1-f198.google.com [209.85.214.198]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 4ah65t2j9h-1 (version=TLSv1.3 cipher=TLS_AES_128_GCM_SHA256 bits=128 verify=NOT) for ; Wed, 19 Nov 2025 18:10:29 +0000 (GMT) Received: by mail-pl1-f198.google.com with SMTP id d9443c01a7336-29557f43d56so178345ad.3 for ; Wed, 19 Nov 2025 10:10:29 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oss.qualcomm.com; s=google; t=1763575828; x=1764180628; darn=vger.kernel.org; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:from:to:cc:subject:date:message-id :reply-to; bh=NVTn4Wtau+BZ+/gvjqlvfniNUMfKB1rE87mf5c2GkIU=; b=jjcUpBkgNkLBHjnWaav6yICtMcBh1fzVMY/1xOHShN85Nc7oVuccDuxrY6FH7UTDhm 1QHDqkD+oEwjVxnoYMUAvb6j+vRSXeEuYBA1pYm/bsKYznNnsXNHTsmYO6LNQL1l6C7C 4QeRT5DuY7JjHrp9VnhKlyTNJJkUkfzN+Pivsf2UgcHpW6RRjRDqPXrkv62aiRUjjOmO rg2dMU2xAq912znB3jp9F1CYR/67EFBW9I+PeJvN9sDpTs1JJf7vOYQaVuei1t6LXO/l SJrKMyVSbBFTODPSnpL8M3XOecRBIP0Iz/L+RSYlIQ3pt13yj5S+Ut335n+F96CNagtq NORA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1763575828; x=1764180628; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:x-gm-gg:x-gm-message-state:from:to :cc:subject:date:message-id:reply-to; bh=NVTn4Wtau+BZ+/gvjqlvfniNUMfKB1rE87mf5c2GkIU=; b=q9Ozry/shbGUQ1jJ9pHCXBF8/L+1QUNAvx9mWNaC1Ps2B69OnQX+K2niK9efRlekVS QB+GEkkVTLaIYFEorqmTKuTIQTWGVpCKCvU4Nhz8HuwlEb4MURV60Lj/47A0+K81yoo8 MdqW2We7CqYSXQW4NbhvqPrPWyDG6BugG4yTJ7Fmps+Ph8zXzqVubzMYa4Rwjw4Ej6gK AISxLWmGtLEeBiGM6D4DM1zgkNm3o4I70tRviENpw1p+ODk0s86lg9VvfmShjJ4mAcSb gTJ/I+taNqod58chWhNOxr0xOMjmA2oT5/EpnvEpqlj0+UM+IYN5cFrXuqTIiKjTU6Hz hXJw== X-Forwarded-Encrypted: i=1; AJvYcCVPvOwuCJa8Ifh2eut9D1BxDdXDonaew3bDxUScnvAslOAs4DUeXI5pZ4YPUmW++qVM5+tFk9GtpcfiOSs=@vger.kernel.org X-Gm-Message-State: AOJu0YyeWIXM7BT9mz7EbM82m7hGviubQW5RRDwnyDYVmwQe3a8JeqAm /sjnoSTY4lB6cvQ0LvAuSAVDmEJtGLfeoZJptvPEX0r3sfgWK1umlaBwJr67Ncze5jYP6hK+ogj YYYmp9e/EUtcjgmy0vmgBUXlmTuK+TS3uHy5NQHmvaNz4zdpUd8qbZgf2RDsxnamjze0U06rCUo E= X-Gm-Gg: ASbGnct50NP4O9upVPHkXHbGhsp9CJzJow33HzX/q60CPK1hwIdHV1db2VI+C55KuUy XBPFqvhXl7EtBLbePYHev/CPMyWZ161d9l9pVQyS+ja5gFD11df3wUxNQl3xz3n05cLnbfJUmkW 49rV8og4OgbW+jz+125isXcDwhSCmSc7fCRqSoyA568iXR6yXxxgoVcR9GutGkPH6P7orenw/AM P61KWHezarvy8b3AU9R7W5wKR9L5CEq0SY/LQPVcigArRlDlW9phr2NiPsQuOyk19Uh90k/XCDr sGJSLH69IuT/ew7fPXdo7o75VPcaJeHaHcSYnwiVbWnPeA2aQ0ncjUAF4MyKry+KuXbQJ9cZor+ BKC/tUeFkA6TxiVHsMiK+jQdMw99/P+eTvHndDNU= X-Received: by 2002:a17:903:1745:b0:297:e267:c4c1 with SMTP id d9443c01a7336-29b5b13e439mr2714845ad.55.1763575828328; Wed, 19 Nov 2025 10:10:28 -0800 (PST) X-Google-Smtp-Source: AGHT+IEZw0/65yXbHehlekHt+yFKCCXDOMWobOiCHVD+AV0GO+o43yJA4Or2y1cJKFlSuOyj30JjSg== X-Received: by 2002:a17:903:1745:b0:297:e267:c4c1 with SMTP id d9443c01a7336-29b5b13e439mr2714585ad.55.1763575827796; Wed, 19 Nov 2025 10:10:27 -0800 (PST) Received: from [192.168.1.102] ([120.56.197.63]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-29b5b111016sm1408865ad.6.2025.11.19.10.10.22 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 19 Nov 2025 10:10:27 -0800 (PST) From: Manivannan Sadhasivam Date: Wed, 19 Nov 2025 23:40:08 +0530 Subject: [PATCH 2/2] PCI: dwc: Do not return failure from dw_pcie_wait_for_link() if link is in Detect/Poll state Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20251119-pci-dwc-suspend-rework-v1-2-aad104828562@oss.qualcomm.com> References: <20251119-pci-dwc-suspend-rework-v1-0-aad104828562@oss.qualcomm.com> In-Reply-To: <20251119-pci-dwc-suspend-rework-v1-0-aad104828562@oss.qualcomm.com> To: Jingoo Han , Manivannan Sadhasivam , Lorenzo Pieralisi , =?utf-8?q?Krzysztof_Wilczy=C5=84ski?= , Rob Herring , Bjorn Helgaas Cc: linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org, vincent.guittot@linaro.org, zhangsenchuan@eswincomputing.com, Manivannan Sadhasivam X-Mailer: b4 0.14.3 X-Developer-Signature: v=1; a=openpgp-sha256; l=2652; i=manivannan.sadhasivam@oss.qualcomm.com; h=from:subject:message-id; bh=Cf0vCAl5snTvRcvHYoqSDh6FCeDaD461npSza6EkeXo=; b=owEBbQGS/pANAwAKAVWfEeb+kc71AcsmYgBpHggD9y2nMg+sO9+ekIKG7d3/hu2igXML1Pytq eivFLhFiOGJATMEAAEKAB0WIQRnpUMqgUjL2KRYJ5dVnxHm/pHO9QUCaR4IAwAKCRBVnxHm/pHO 9WtLB/43To7RvUQDZ9SZLwlAmaPnM++cTTYx1Pn6mQ+QDM4fxeib9sxVbcCMZ1V/1lJwNljdUCx 0vpvP7WHw7+3ewMxD9nwEQfyHiCfvhL3Nz+1YR9qa70FCkb5B/Ky1pRD6HFlOvmllik+2ke3ZfN iNAuD2X1xPxHjLciDrWNjGd6axxf+5pfOmxQ5geXUR1ZOiLEnijekGWXdD7So77QKKsH5BGJ0Mn KMVaAjPgmIaRWhm8AfGzaErlbCqb3oWJAmbJ6qAX78t5uxYK7Qohbh2y5Ioe3n8sSpLc80OyOHl JNzP/uDMDpmAW9Q6opjsBlTFRCsyiHDEUUUvloKZ2HboeFOm X-Developer-Key: i=manivannan.sadhasivam@oss.qualcomm.com; a=openpgp; fpr=C668AEC3C3188E4C611465E7488550E901166008 X-Authority-Analysis: v=2.4 cv=avK/yCZV c=1 sm=1 tr=0 ts=691e0815 cx=c_pps a=MTSHoo12Qbhz2p7MsH1ifg==:117 a=xeX0Tm50+WxWVv+NCdhSGA==:17 a=IkcTkHD0fZMA:10 a=6UeiqGixMTsA:10 a=s4-Qcg_JpJYA:10 a=VkNPw1HP01LnGYTKEx00:22 a=EUspDBNiAAAA:8 a=mPLJVkjgqw7R4SvY3gYA:9 a=QEXdDO2ut3YA:10 a=GvdueXVYPmCkWapjIL-Q:22 X-Proofpoint-Spam-Details-Enc: AW1haW4tMjUxMTE5MDE0NCBTYWx0ZWRfXwyF6AncFRass mzASE+906MkssrPQE3ew7ofKb5hW9aWZnTyyMxU7J/+azEdJmctZvZ2B0ogibE3JBJHDMQyoNBX vii5MG/s9K7EIqaiW2KZY/uLHNeABfiiapLCD8MIPVhmSmWLsq+PH8DQL0zRT2J2hkgBcP5Z1E9 kwY7FP78h2NpChsykGt9qpN90djWAuZmfcIYYRZEGll5UN1GMdCgFb34Os0XvE36h7cJcoQusEr 8Jsqd7SYYTpBm0CKBGB50CbJT6/pqdEAkeSW2y/TQWnPq5T45WvgPvLjf5p+rZNdBCO8gDbLUH5 PqtymDTMhrfCjnfkh/SB1w6m1b35J5yLh3wBC8CUk4WDRkD0PYGBK782q6MNyeVx/oHm0B9yUAL WjEFLYm7gmDj3X9a12N3iQ9AVX0G9A== X-Proofpoint-GUID: uweZjh2CfkF4igiTh5a6zQf2HBU9OBRp X-Proofpoint-ORIG-GUID: uweZjh2CfkF4igiTh5a6zQf2HBU9OBRp X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1121,Hydra:6.1.9,FMLib:17.12.100.49 definitions=2025-11-19_05,2025-11-18_02,2025-10-01_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 clxscore=1015 spamscore=0 suspectscore=0 phishscore=0 priorityscore=1501 adultscore=0 impostorscore=0 lowpriorityscore=0 malwarescore=0 bulkscore=0 classifier=typeunknown authscore=0 authtc= authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.22.0-2510240001 definitions=main-2511190144 dw_pcie_wait_for_link() API waits for the link to be up and returns failure if the link is not up within the 1 second interval. But if there was no device connected to the bus, then the link up failure would be expected. In that case, the callers might want to skip the failure in a hope that the link will be up later when a device gets connected. One of the callers, dw_pcie_host_init() is currently skipping the failure irrespective of the link state, in an assumption that the link may come up later. But this assumption is wrong, since LTSSM states other than Detect and Poll during link training phase are considered to be fatal and the link needs to be retrained. So to avoid callers making wrong assumptions, skip returning failure from dw_pcie_wait_for_link() if the link is in Detect or Poll state after timeout and also check the return value of the API in dw_pcie_host_init(). Signed-off-by: Manivannan Sadhasivam --- drivers/pci/controller/dwc/pcie-designware-host.c | 8 +++++--- drivers/pci/controller/dwc/pcie-designware.c | 8 ++++++++ 2 files changed, 13 insertions(+), 3 deletions(-) diff --git a/drivers/pci/controller/dwc/pcie-designware-host.c b/drivers/pc= i/controller/dwc/pcie-designware-host.c index 8fe3454f3b13..8c4845fd24aa 100644 --- a/drivers/pci/controller/dwc/pcie-designware-host.c +++ b/drivers/pci/controller/dwc/pcie-designware-host.c @@ -671,9 +671,11 @@ int dw_pcie_host_init(struct dw_pcie_rp *pp) * If there is no Link Up IRQ, we should not bypass the delay * because that would require users to manually rescan for devices. */ - if (!pp->use_linkup_irq) - /* Ignore errors, the link may come up later */ - dw_pcie_wait_for_link(pci); + if (!pp->use_linkup_irq) { + ret =3D dw_pcie_wait_for_link(pci); + if (ret) + goto err_stop_link; + } =20 ret =3D pci_host_probe(bridge); if (ret) diff --git a/drivers/pci/controller/dwc/pcie-designware.c b/drivers/pci/con= troller/dwc/pcie-designware.c index c644216995f6..fe13c6b10ccb 100644 --- a/drivers/pci/controller/dwc/pcie-designware.c +++ b/drivers/pci/controller/dwc/pcie-designware.c @@ -651,6 +651,14 @@ int dw_pcie_wait_for_link(struct dw_pcie *pci) } =20 if (retries >=3D PCIE_LINK_WAIT_MAX_RETRIES) { + /* + * If the link is in Detect or Poll state, it indicates that no + * device is connected. So return success to allow the device to + * show up later. + */ + if (dw_pcie_get_ltssm(pci) <=3D DW_PCIE_LTSSM_DETECT_WAIT) + return 0; + dev_info(pci->dev, "Phy link never came up\n"); return -ETIMEDOUT; } --=20 2.48.1