From patchwork Fri Mar 14 14:59:02 2025 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Mike Looijmans X-Patchwork-Id: 14017031 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id EC29DC35FF3 for ; Fri, 14 Mar 2025 15:01:43 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:References:MIME-Version: Content-Type:Content-Transfer-Encoding:Message-ID:Date:Subject:CC:To:From: Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From:Resent-Sender :Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:List-Owner; bh=DasCPADCDjnV2jDDFR92iEghmdulkaTefmYegBGMoq8=; b=AsUqAr9RgA48Rvt49LwBVgZVBq qcq5h38lDto0ZcCDeoXtbNJgPTD4A+/UEufX40cJJiGhs4nISKdDnNYRJU6UudwxuEjgmM0ila21j DFb73xs8zTlq9Cr9Z7HoKwrMKY39EHIl6IJMVFQiSbjteOaLbhO54XnKjuH4FNGd/xeOxhNDbc013 R5i200vbZdlwoemfusejWwl6gkDqNx7WI214nsVMlRl7Agx1oFc2Gz6XN2yjbGZPuonVw6gg314Vd e/OvAPbZPVnqeDKbaed3we0J7hOlRP9UsHo2iTM8qDqPCqANiYwI0u+JfoKOgsITgugojTdIdQoLv yadLORKw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1tt6XW-0000000EUHQ-1MWE; Fri, 14 Mar 2025 15:01:34 +0000 Received: from mail-am6eur05on2072a.outbound.protection.outlook.com ([2a01:111:f403:2612::72a] helo=EUR05-AM6-obe.outbound.protection.outlook.com) by bombadil.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1tt6Vp-0000000ETw7-1oN8 for linux-arm-kernel@lists.infradead.org; Fri, 14 Mar 2025 14:59:50 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=tO2BIhMqHZVHIOJbwsCGjysNHxNRHfGdDBpWeW1VpZpn7yvrc6E/WSQgQSuxb0WRtaAgItR5qtqI8Bx3ea1y6ELDhv5FBVuXOjNRBFUHROTebxFRmW8oY5qV6LlIWTQSMgC0dKODlKBbP9Cg46YL86osmtn/9oiShvh+xwaN5nChImj/od37LnFoTQYQVwaOqx5D9FaoUPTUy0Mx7ibHM31oVFf+g+N7W2CPifIXmWBZWaoJj1vqheu9gZKcS8wYPpYwFpdl5pPY3kIAVXNL/I4pjUxumzKK0d2WWRAStXIaZBEurinENoVM+ws4PVCrj420xSy/Qn1qQR0MoVu75w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=DasCPADCDjnV2jDDFR92iEghmdulkaTefmYegBGMoq8=; b=KOpBtHb3zo5/Rs2D6KfYp6sPLOBR5SywGWX6QRsStSZT8vYC0hvDstO8Tp/Aan9m72GoCaJw7ba2gqJNZxfJ/R/m0Ou8BngNysoIznVtjmZ80py3JDQ/b1AdUIbjcnZtgtCLLvRWOpCEdYzDpXvuinhBJbFeidtSurlZ5evx/gCLSiOw9TyBK4tPct+7bFoYCEwrhClf6Gqv6ZLOO8GuzFvgkhvL/Ci7ZayYgBP20Sg62YUTrYuFiouJ5Pm9X1ASuM7QbH5yRiFru+wjwF3BvQGvN94KRuJBUNYNZcg+g+c5lAc4KW2GBOsgCJYn0ZncLWOg8+hRxtweCqgc61FgMQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 13.93.42.39) smtp.rcpttodomain=vger.kernel.org smtp.mailfrom=topic.nl; dmarc=pass (p=none sp=none pct=100) action=none header.from=topic.nl; dkim=none (message not signed); arc=none (0) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=topic.nl; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=DasCPADCDjnV2jDDFR92iEghmdulkaTefmYegBGMoq8=; b=cSDebQ9zag0fleVIarqgBRZHG1os8lTuC1cfk4Oly1Wf9z6hTUahrxbMuhA+rW0jb/11v1QxXlpgIcl5II5W5BN3XYY4Ce/S3gbKqLsx6SSh32Lwb+9lxvlvyI26+8iNUB5Ec5IIIHA2dNgANmAi6jKdREsw2WHVyhQehxC9WhE5u9SABVBzvSZEydU/tdJ2+n8ccgBhigi+qSMOm/MuKhRMeNeYKB4AYKv98/fodBbwoNXkg5dLG7HhM6UU1h49Db1XekmYLAl1GE7ZDi6es1BBQTQfChhchQpQFT/GApvnCbaXdSvbNyOMqC9aB00rSKFsNy8fkdHKPX4Y3kujJQ== Received: from AS9PR06CA0422.eurprd06.prod.outlook.com (2603:10a6:20b:49e::6) by AS8PR04MB8120.eurprd04.prod.outlook.com (2603:10a6:20b:3f1::12) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8511.28; Fri, 14 Mar 2025 14:59:42 +0000 Received: from AM3PEPF00009B9F.eurprd04.prod.outlook.com (2603:10a6:20b:49e:cafe::93) by AS9PR06CA0422.outlook.office365.com (2603:10a6:20b:49e::6) with Microsoft SMTP Server (version=TLS1_3, cipher=TLS_AES_256_GCM_SHA384) id 15.20.8534.28 via Frontend Transport; Fri, 14 Mar 2025 14:59:42 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 13.93.42.39) smtp.mailfrom=topic.nl; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=topic.nl; Received-SPF: Pass (protection.outlook.com: domain of topic.nl designates 13.93.42.39 as permitted sender) receiver=protection.outlook.com; client-ip=13.93.42.39; helo=westeu12-emailsignatures-cloud.codetwo.com; pr=C Received: from westeu12-emailsignatures-cloud.codetwo.com (13.93.42.39) by AM3PEPF00009B9F.mail.protection.outlook.com (10.167.16.24) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8534.20 via Frontend Transport; Fri, 14 Mar 2025 14:59:41 +0000 Received: from DUZPR83CU001.outbound.protection.outlook.com (40.93.64.9) by westeu12-emailsignatures-cloud.codetwo.com with CodeTwo SMTP Server (TLS12) via SMTP; Fri, 14 Mar 2025 14:59:41 +0000 Authentication-Results-Original: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=topic.nl; Received: from AS8PR04MB8644.eurprd04.prod.outlook.com (2603:10a6:20b:42b::12) by DU4PR04MB10483.eurprd04.prod.outlook.com (2603:10a6:10:565::15) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8511.31; Fri, 14 Mar 2025 14:59:39 +0000 Received: from AS8PR04MB8644.eurprd04.prod.outlook.com ([fe80::e86d:f110:534e:480a]) by AS8PR04MB8644.eurprd04.prod.outlook.com ([fe80::e86d:f110:534e:480a%5]) with mapi id 15.20.8511.026; Fri, 14 Mar 2025 14:59:39 +0000 From: Mike Looijmans To: linux-pci@vger.kernel.org CC: Mike Looijmans , Bjorn Helgaas , =?utf-8?q?Krzysztof_Wilczy=C5=84ski?= , Lorenzo Pieralisi , Manivannan Sadhasivam , Michal Simek , Rob Herring , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: [PATCH] pcie-xilinx: Support reset GPIO and wait for link-up status Date: Fri, 14 Mar 2025 15:59:02 +0100 Message-ID: <20250314145933.27902-1-mike.looijmans@topic.nl> X-Mailer: git-send-email 2.43.0 X-ClientProxiedBy: AM0PR10CA0126.EURPRD10.PROD.OUTLOOK.COM (2603:10a6:208:e6::43) To AS8PR04MB8644.eurprd04.prod.outlook.com (2603:10a6:20b:42b::12) MIME-Version: 1.0 X-MS-TrafficTypeDiagnostic: AS8PR04MB8644:EE_|DU4PR04MB10483:EE_|AM3PEPF00009B9F:EE_|AS8PR04MB8120:EE_ X-MS-Office365-Filtering-Correlation-Id: 0369ea7c-3f6d-4f0e-c500-08dd6308d9a2 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam-Untrusted: BCL:0;ARA:13230040|52116014|376014|1800799024|366016|38350700014; X-Microsoft-Antispam-Message-Info-Original: JZS3fSnyBQntrSu3919hOoDYRHOW4is1FdiUw+B4ieLzBiJwlmxXY1p/UcfHV+lJv62+lld293htehF9SdgrckYwPj0F1oDj+Rdl5Esh3h3YMeyEpyIN5QisNSSbpHGthj2UU50Wjwp+A4OPWZAI4lm87YEqx7uWDN0NljU5RLkCJjpx32RVKMW1EKwRs7bbXvcxUGEwovTuyZr3LutVTsNjglkRSzg6GHdMovel9cuw6Ea/xpqA/nnwol34myuYA//Kqhpl2/hJNrqIi4t2AcgsQJlv7RcPWvtfQYkwRLUB5eWi3lNiXskyiqZpOdqGubaONF5X74EPpsRcWLVUgXnrCwQOMT8yhNvDqsvBWG8PJPyhID1NP4yaF7SSh2HehSwcJyI8syKMG0PjxZyMuxlre+JEtvUqiPQx3Lg4WzKnksqJqrzBrRNSGWcHzSMeGV23Ic2IcSWY6aS7iJ0ytEmHhicps4+7GQ4iSk9gcnTw9AfVlQECZxXo0zDSFNTQAc/yXqmy/QBnBR9gQPch/rvQM98c9KFBbtAPQJ4/oAWbY4adQMzRbR6U9HECYxRdGDXirPUKUhVtJMsP/xb9fdWN4xb7vWGzLZ6vT6oEg5Mjs+aK1AZo8Nfu2tm1O8uHUumU6UCzJJ31BiAcWG8Jlr/Wm8Ar7OfjNDeUp1gSklW5UM0Qpx130JS/O0ITkyBlhXPeFmkfbhtrXlj7tcoMe0FJa9Hm2J6kxMqs7CXmOYkU4LFs0Q7wRKtpfEJqlvzzi2xhxq+jmNLFAYxUFJosUImhRNRpc2prlEAt9G7ojOBuQRvYRekDFGAXIKqgB4+Q140xyyGXfMl3UlOSCsZU3JwCEzvDmXQ1lKQ/QIe9iLmv2AG9ma8X0QEJJR6/hpWSdrDSV7omkLb3EmSCwmucnt0Tc0X/Fe8urhfD91pZdBqdohdF6OF/NL+FRd5Jdp13tW067fwB6713fz7mXgKCnmdWnE/lkAWl8mPeWfxL3v/A4+GjhiyZ9Ws6ei5RT3K192YmaFL1krMvKO6jIV5JvI3WizSXEMmG9hFCAH2J349CGn81rdekWzN0cuXVKBjKJrNwmzkMKIttHWMTXSwGLvDxbYbgM1ndBa40uYFY9UeWK/4IZ+PRj+TfhD//Sqhphp8OZqbQP4rTLWGzQ2ODJfGu3NLfjAx6kFnI2Mn4ypypn8gaeEhQV0hTLez3SMDRZjzjuoqc/dMbXw4Hac5jJ6TWFBbYen6289n5BcPQclKXiaFtxN33gUN6f8zqn1rbdYH7FUBddLDkqegu++DHl0z51IFVrozCAR//+yhj/DZrRi+IRM4tmy8v+95+Ta8nXckXbmwobOxUflf3a4bR2Ptys4cvBV4x4no+GFFI7j+mvK7qzw3cq1kp9bL3o+toSIm7+NPDUEjiYah9uZUR+Q8eYkAUZvx1kI/HOEcXHvKTvXrmC6+BRW/COsil2wUd X-Forefront-Antispam-Report-Untrusted: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:AS8PR04MB8644.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(52116014)(376014)(1800799024)(366016)(38350700014);DIR:OUT;SFP:1102; X-MS-Exchange-Transport-CrossTenantHeadersStamped: DU4PR04MB10483 X-CodeTwo-MessageID: f3928d99-3d05-4cb2-aae3-0b2d49200134.20250314145941@westeu12-emailsignatures-cloud.codetwo.com References: <1b153bce-a66a-45ee-a5c6-963ea6fb1c82.949ef384-8293-46b8-903f-40a477c056ae.c4ab15e9-561e-402d-bf39-9e852485e4fa@emailsignatures365.codetwo.com> X-CodeTwoProcessed: true X-EOPAttributedMessage: 0 X-MS-Exchange-Transport-CrossTenantHeadersStripped: AM3PEPF00009B9F.eurprd04.prod.outlook.com X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id-Prvs: 06cdbf93-5668-4437-9c56-08dd6308d813 X-Microsoft-Antispam: BCL:0;ARA:13230040|14060799003|82310400026|35042699022|36860700013|1800799024|376014|13003099007; X-Microsoft-Antispam-Message-Info: rlVh9MODRyPRFUeLqN5eaGNCJDbJOSd1zTa7Zja3i8HqtqZn5iGr91nsl6/1gKMeaqba5+hzlcW07xk7x2M21I+zVjvcUIjcw0YcAqV5cNt4ojxRTbYJbdnVPitcVmrFpFOvxiHPQETphp338XmrQwG4hmpXAfKFO2ScLDR7vpAHjB5ViBor26s3AAwKbQuUCaxsHREIhcWrgQuqof5oG7INGENAYadskb1MBadFktIpCg8QVdJGCfLmPa9KOisDkU84MYQlTw2awNrgePw/Me7G+26Zkm6Xu++g4dyRU+n9VXee7IlpI10Z0A3puDwwGJnq4PGZ5uVU+uPJ4vLXYp9O4L+0aKx+l46bmr23Os4D8qRaOGWfwiDsE8DhU9Ej/zRky2ARFFxHN5qJrFSpOOhOR4u5o3TJCPL9kkxZncoThUhx4eTZkwCfy94vce0YVZXgf9/VO2FzGDD6JkOT4HX+1ZBvbRoVjH4sJI32+/a9BzsKkrJdYYLD+zgypmpP6WIO4DDpH2RNkdrQBHzSDsInPyLaHCOKQCcUXlRimKfI9/5RL7n005tPYJA7lPXpscQbJwVHMZgpP069g3NCLpjt+IhBt/9iRqav4Y220ZeFecbqtCYc3NIYmQA7CtDZRk0eIuq6JO7OPOxsR0whXevAsbdf/N8mLhTYeBEVd6gTHzS6yBPbqBasB9sfT0yDrH65U7hnO+ElxLM30v/4qzlHDiRsatEV0hWoCOZhfC6xA+KAwMgPH0dnYBggqkT9Y6jof0VaKuJFG2cRObek9YYzL2JsEVFcETkKZVLEDXI+l9NiB7nQ1yiPIJY33lRQBew7It8BozlJSfxNdbc/CISZi2LrkH0Tk5CvD+Wbwnu669Wo9wtibEEyKi072nUZ9bArfnksspk5zwRK9fe8p5A+BJESYWS1jrvIzbTjWDTlMM4gFMlL1WFQnsu5tcXcUy2Ff+pCH0qMR94r/K0JXxIZ93c3DBoyqRnQLrIn+XtD8w/W/7INd3Jck2U7gQrRZAEMjoaY3YaG6lpk1eFPwOFd9htrX6hFtemB9wAbTzaIe64igLYQggrKWncoUEEMKdQ9EEkVD1uc7CuV5vTi0Xy9FEqCbsFRJm07N+PPS++u/TxrR4EpNIXgm3QhW7TKOfnjMfrx3wPtNw8fX8G+beAeGWlS6xaCN/4PgCirozNmAGGdO7bDcmVca9laRraZCXBir2kFXTa/9d6DOzZ/InZZGtYEBzfKIAC5UWsmodHZuCA+9ZglipkpEozTJgckeKHCgcG4ALgJ0nEWTHbcGxhnCc2guYxqWMqSbPtGkI9PbQHuzLjhJF9uvyHli5uMQ/+IrEHVHYdrg3JV95vCsi9FIzNMiA2naDASTwkJMzcTDKCZIwD1t4zek5HxDJFvffyj7TaCLwS01fSWHEm3Fb1qEsYy5HQ248lYMRoM2L4= X-Forefront-Antispam-Report: CIP:13.93.42.39;CTRY:NL;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:westeu12-emailsignatures-cloud.codetwo.com;PTR:westeu12-emailsignatures-cloud.codetwo.com;CAT:NONE;SFS:(13230040)(14060799003)(82310400026)(35042699022)(36860700013)(1800799024)(376014)(13003099007);DIR:OUT;SFP:1102; X-OriginatorOrg: topic.nl X-MS-Exchange-CrossTenant-OriginalArrivalTime: 14 Mar 2025 14:59:41.8533 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 0369ea7c-3f6d-4f0e-c500-08dd6308d9a2 X-MS-Exchange-CrossTenant-Id: 449607a5-3517-482d-8d16-41dd868cbda3 X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=449607a5-3517-482d-8d16-41dd868cbda3;Ip=[13.93.42.39];Helo=[westeu12-emailsignatures-cloud.codetwo.com] X-MS-Exchange-CrossTenant-AuthSource: AM3PEPF00009B9F.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: AS8PR04MB8120 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20250314_075949_472459_C8B907AA X-CRM114-Status: GOOD ( 17.83 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Support providing the PERST reset signal through a devicetree binding. Thus the system no longer relies on external components to perform the bus reset. When the driver loads, the transceiver may still be in the state of setting up a link. Wait for that to complete before continuing. This fixes that the PCIe core does not work when loading the PL bitstream from userspace. There's only milliseconds between the FPGA boot and the core initializing in that case, and the link won't be up yet. The design works when the FPGA was programmed in the bootloader, as that will give the system hundreds of milliseconds to boot. As the PCIe spec mentions about 120 ms time to establish a link, we'll allow up to 200ms before giving up. Signed-off-by: Mike Looijmans --- drivers/pci/controller/pcie-xilinx.c | 25 ++++++++++++++++++++++++- 1 file changed, 24 insertions(+), 1 deletion(-) diff --git a/drivers/pci/controller/pcie-xilinx.c b/drivers/pci/controller/pcie-xilinx.c index 0b534f73a942..cd09deba0ddc 100644 --- a/drivers/pci/controller/pcie-xilinx.c +++ b/drivers/pci/controller/pcie-xilinx.c @@ -15,8 +15,10 @@ #include #include #include +#include #include #include +#include #include #include #include @@ -126,6 +128,14 @@ static inline bool xilinx_pcie_link_up(struct xilinx_pcie *pcie) XILINX_PCIE_REG_PSCR_LNKUP) ? 1 : 0; } +static int xilinx_pci_wait_link_up(struct xilinx_pcie *pcie) +{ + u32 val; + + return readl_poll_timeout(pcie->reg_base + XILINX_PCIE_REG_PSCR, val, + (val & XILINX_PCIE_REG_PSCR_LNKUP), 2000, 200000); +} + /** * xilinx_pcie_clear_err_interrupts - Clear Error Interrupts * @pcie: PCIe port information @@ -492,8 +502,21 @@ static int xilinx_pcie_init_irq_domain(struct xilinx_pcie *pcie) static void xilinx_pcie_init_port(struct xilinx_pcie *pcie) { struct device *dev = pcie->dev; + struct gpio_desc *perst_gpio; + + perst_gpio = devm_gpiod_get_optional(dev, "reset", GPIOD_OUT_HIGH); + if (IS_ERR(perst_gpio)) { + dev_err(dev, "gpio request failed: %d\n", PTR_ERR(perst_gpio)); + perst_gpio = NULL; + } + + if (perst_gpio) { + usleep_range(10, 20); /* Assert the reset for ~10 us */ + gpiod_set_value_cansleep(perst_gpio, 0); + usleep_range(1000, 2000); + } - if (xilinx_pcie_link_up(pcie)) + if (!xilinx_pci_wait_link_up(pcie)) dev_info(dev, "PCIe Link is UP\n"); else dev_info(dev, "PCIe Link is DOWN\n");