From patchwork Thu Jul 20 13:58:34 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Frank Li X-Patchwork-Id: 13320561 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 84EF9EB64DD for ; Thu, 20 Jul 2023 13:59:27 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: 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: List-Owner; bh=dgk9l2TC0w4w0G4pi4E2Nd4wqf7GTKIJk/2MtG4AI7Q=; b=lI8cHw2EQqq4bm 0RUhHc45qHEEZ8jOqkHt+2HQy8xv6Lzt3JqTerccCMxbq56CP0X1VjutwfcD3rEu3rwU6F1I0w1W9 WUOENuWjhupR1ayayvJ36uee55W76UA7QEXgrn/2eSjx0TIDf+0hN74KifyvPQ6Id9skHvZev/h1u jCyyMD4hFCiiItjZa0K4Xu8QgjUZmE2ziUQ/nD3YSeiwIVR5U8TfG6cv3AqfQlgwnqxsdtHCxG5XD tuRmsbHuwyG0aWJHwmOX2CqzUq1Gtycj3iA6BP8t0QX+MuOGfvm8LfSu4uBxrU6BPem+0FrXQyOqj RyNy2W8Mkc9sAZO59D4w==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qMUBQ-00BJyz-2k; Thu, 20 Jul 2023 13:59:08 +0000 Received: from mail-am6eur05on20624.outbound.protection.outlook.com ([2a01:111:f400:7e1b::624] helo=EUR05-AM6-obe.outbound.protection.outlook.com) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qMUBI-00BJto-2t for linux-arm-kernel@lists.infradead.org; Thu, 20 Jul 2023 13:59:02 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=EuOvbtORuQbONwj/CQ5pdmFzwRJXIj/qYOcMuh4hckRGwf/gw9GDMaoydHyYdDN4qOr8ZdfiiAGWn4uAzF/QSpEe+nejPqFxctISStwunaAEa5BHvTouHB3Q+fVgOsAJP8uy8V+uy80DI9voxexeO1boGKaJ2AiN+BdKqFP5BGPXPUoWyA6/MatJOVCrwNFGLEsoSlbUvj1nNMVEmNIckXR99y0C5h8me8whmD62l+DtD6OaG7DY2WeJmUFxq/gApN+dlcefc2R/+Dm5eOyuW/1L72oa2HJOyKXlZ9xVOBsvxJBNfNEMBKjjSHB3IsrKtzj46/fs50vK1gy1IujqmA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; 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=Vvvbx1BEBpinxsumUX48rt0sTeC+wt2vpJoWkExyF8s=; b=FsKIwVXt9R2loVF0ctRx7OeNm7fIsFBQUCVQVwwSTdlg/bTdIEwn7PFg6O/KKgdhyS8juLqx5TiezwFLskZjhGsXTbhUUek2ntTfyAbcMGkOjryM9t/CbBtsVYU8118MOPztH4Ntqf2+2SoGKtL2GW062X7C4OH/mp8YGM6eCQ5NhpKLjib8UsEBORkJrks1KeIXLmM/oygnstRfNIxdtQ/kZEZwhKEagakuLGjSBhtwH2pbcuwFG60Oh5rpiOTYpOrVuKf9IP0VxvkOPgpnOPzjLieKs1zrlOPCytGjGXdAI9RXuCc9j5W2sE5HQrzawxjw432T5vPsW81wtyFhAw== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nxp.com; dmarc=pass action=none header.from=nxp.com; dkim=pass header.d=nxp.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=Vvvbx1BEBpinxsumUX48rt0sTeC+wt2vpJoWkExyF8s=; b=VyWIFSNUchTMXIR6jVBq9j/HPP2hROpEIufTxMzzbuazefrTEMHbmDh23uSoyskqRA9yeJT6IuRnezhgf7uuUVCzad4ZeKlcisjaGWRlvQ5u1PtD7tEKXddS5vCMYwoZXSDENTrd3O1Kp7zQ7BU4cjlGkIuvWGhgf6P/MoKPSlc= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=nxp.com; Received: from AM6PR04MB4838.eurprd04.prod.outlook.com (2603:10a6:20b:4::16) by GVXPR04MB9973.eurprd04.prod.outlook.com (2603:10a6:150:112::9) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6588.31; Thu, 20 Jul 2023 13:58:58 +0000 Received: from AM6PR04MB4838.eurprd04.prod.outlook.com ([fe80::d0d5:3604:98da:20b1]) by AM6PR04MB4838.eurprd04.prod.outlook.com ([fe80::d0d5:3604:98da:20b1%7]) with mapi id 15.20.6609.024; Thu, 20 Jul 2023 13:58:58 +0000 From: Frank Li To: frank.li@nxp.com, lpieralisi@kernel.org Cc: Zhiqiang.Hou@nxp.com, bhelgaas@google.com, imx@lists.linux.dev, kernel-janitors@vger.kernel.org, kw@linux.com, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-pci@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, mani@kernel.org, markus.elfring@web.de, minghuan.Lian@nxp.com, mingkai.hu@nxp.com, robh@kernel.org, roy.zang@nxp.com, xiaowei.bao@nxp.com Subject: [PATCH v4 2/2] PCI: layerscape: Add the workaround for lost link capabilities during reset. Date: Thu, 20 Jul 2023 09:58:34 -0400 Message-Id: <20230720135834.1977616-2-Frank.Li@nxp.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20230720135834.1977616-1-Frank.Li@nxp.com> References: <20230720135834.1977616-1-Frank.Li@nxp.com> X-ClientProxiedBy: SJ0PR05CA0012.namprd05.prod.outlook.com (2603:10b6:a03:33b::17) To AM6PR04MB4838.eurprd04.prod.outlook.com (2603:10a6:20b:4::16) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: AM6PR04MB4838:EE_|GVXPR04MB9973:EE_ X-MS-Office365-Filtering-Correlation-Id: 838d610e-1fd4-4576-74ca-08db89297672 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: xXA7F94dlh9zBxN5MbPsHXe8419ukd3JNd+YDwFPNxq+N0WnglCC76LefS289LL+ArZoskmoT4LszCRqnM/cVISVIulmYYwhK5DZKK9go/cTN6/0Gicsly6Er8IrPvqeOBqwQtTlFqIwpe0aEMAqSZdIfhIaUveolyv5xWkEsDW0+k3g7Uw0C9ie98XMJOzHGNhMChdEmtoC3LmiEeq9vOWsUA7t1Ze8vcSYgvfpPYlVZn8CeLCSMDGovNJVq/DoTy1TpnZQKGGaYsbGh7hjPNHccSXQT6yOa1cex6WWlX8Khs+MXAueWpDYY1AvfPhF63fVj1uG2nl2NwwVZ3MigKRnB25eaJNo7t4dQqShCh4gQBdOsngTr2vv1uv705cojI8dGuDS1ZD1oDyGHz5A6s+OEC4smgGFDAavG5YvWBV9oDAPRZvx3jxjh1o4HnJ32rHhPNmK+49RUWlYlwi2Mw07WIttQeNh0u7jMX5fudhVDvpNi3Dj2EoW8dQiSKLjwRxh/BiAAAEbtIVwGtedhuDN31okRwp8H/n3ywGoZeuR0fs3QSutO1uE8+Qlsksnu7DhT3KK2CcSHkV/PclWiM4aTlXY20CqeiKWSPQNtk+alb145RRt9CXj2g4UdIgI X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:AM6PR04MB4838.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230028)(4636009)(136003)(39860400002)(396003)(366004)(346002)(376002)(451199021)(66556008)(478600001)(6666004)(6486002)(52116002)(186003)(36756003)(316002)(6512007)(2616005)(5660300002)(6506007)(2906002)(1076003)(38100700002)(8936002)(66476007)(66946007)(7416002)(4326008)(8676002)(41300700001)(26005)(38350700002)(83380400001)(86362001);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: fHJdL8bMcVOfndCBdGaTAqykQ6csCH9N2TmjCX/ARNUIB+MjdyUFcKY5DsSuWJdgpu7zmo4PLjYjc/S61KwN2+18vk1HPruziDjayKxq0VuqDcyCq9VFeF+9b5gV+KXFUhW7P/iMgsQbKrF/gUuK5YgXEmjKi2AayHhg5bsc901b3a1CQU0vZF5MUHHw441kjEPu/5eChrRdHZQgtjMfH41wiRpWR49X6g+yqPNdHJc7PLA6R3zctD7EI1//bK9sPdmXRA+WdYArpGJrYJANQ2m8EMEaQnkFfYqA0N3A/mm44IE7xNH2z3xEJkg6KWcp8uHdAbXpQrq8txW5kiYmcY1M6iHmT+rwqSn0HQVR9+UU+0J3ZKadQhXLWFbcyHgv8EZncgfJ7T2q3KqOqL8S1Udiql+MEvokn8EvVbD8bYcXrZwPS7VZgVWUopXrG04cZ31d4oeIxvBXxnRAxHECUxqepDO/Tzn8uYoQ1tyj68oqnPD9MR15H508DnF/tnKErhpz4En9RLZ77vWcUKBC3niWYiFDuIsBNmqDq2JMlBfueALhdFepp9rE8/cQBIq6CjGl4YrnlOa8tr+vobW54P7SrktsZDVxTjKcYlcgfptqwjRHmcPiJwjXJnIAxLZvRkUy5GFRzhtcRFSgbcSkjetUTLjXUO2cw5i/Red/nj1XBwbbqaQGdVp6ueIFO/VJxxRvnXyH/dP5diItRQRMcd/2jZTo/EOjcKnkEsFnUwRjJdyQTn27GfReBj0G/wFuWy3QmAdxLWRAUt12bj12sHpst3eJDOAwXg8YRWtVz1fCtaKUVXDHMLiS/UtwGYe/3b8UgU1PGRCcL9J5kDkHCgRVUASpi/G531fP250xnEnRK/FITSuTiwOWiX4fWU4dL4OTZ5a4Z/gOBciTjyFPRE9lY9hmAx8n4P6z6Su7hedFcrm1lQNf9/f5vXIvnUYBdfF2V3WPouFWaVJxjv+EsqvpCscj4o7AFLog9hJX3rAGJWEL1EF1eO9V1ePZdC5dR9pqADOMC9e9eFveteEP1lt/O4rjwCE8ZwV6ymyWqhu80GBCm+oK8c5Kj916PHL/AsKatvpxl4y8pJnpHbl22GgnjXdhuWjlKmDwD1hKURjBiO5VP6sOPa5Kp1MX1V/mkNBJCUzpQzJ6h1btohX8xV9IwTDJiKiKsyJ+n8AMArHxYfanVWGGPR2c0UNZzgxEBPjwaMW3+huRtoLzlCnOdCyMArhpYyXARvZP3/ltXQh+CFLH7lSXm5D+a/8sumqxBV0c9ITtZ3JdG5yUhmhWc2usoxJlhIxBQrgLjgUGiulzjgplCd6sihKArXbGDW4OX+B9q8Y12CzJNHk9i3Vb77150TQkSZSQhHElcH9zfVFZoypz1u/dA5Z/lGeR3lTvAKLbtzNUMAX5arF9TQhSj6u48osMAZX/KcRG3pc0Yn+flohw6MmwxuwYLZrKetFzr5ZTqwkdh/ReWy/JDR7GS8pjjSbE0IiJk+YEey+6zF4KOOOUCI/BCzQKVoV95am6tBQbPbEpZ6sWQ5pWLirsI33uJlUAUX6TRjmXWiQLt76U4qbeInhVEoQsCkvjG8Fp X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: 838d610e-1fd4-4576-74ca-08db89297672 X-MS-Exchange-CrossTenant-AuthSource: AM6PR04MB4838.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 20 Jul 2023 13:58:57.9125 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: tTk6OWYVdAhxQv4XFX7CK7fLGb2d8sswtOEKewFDYcSKTc1GfnNhorAhw3Ngant3lcK+FDlMLOkNqqVdYx6fwA== X-MS-Exchange-Transport-CrossTenantHeadersStamped: GVXPR04MB9973 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230720_065900_938966_60C20E9D X-CRM114-Status: GOOD ( 13.58 ) 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 From: Xiaowei Bao A workaround for the issue where the PCI Express Endpoint (EP) controller loses the values of the Maximum Link Width and Supported Link Speed from the Link Capabilities Register, which initially configured by the Reset Configuration Word (RCW) during a link-down or hot reset event. Fixes: a805770d8a22 ("PCI: layerscape: Add EP mode support") Acked-by: Manivannan Sadhasivam Signed-off-by: Xiaowei Bao Signed-off-by: Hou Zhiqiang Signed-off-by: Frank Li --- change from v3 to v4 - fix wrong commit a debug code, change pr_err to pr_dbg change from v2 to v3 - fix subject typo capabilities change from v1 to v2: - add comments at restore register - add fixes tag - dw_pcie_writew_dbi to dw_pcie_writel_dbi .../pci/controller/dwc/pci-layerscape-ep.c | 19 +++++++++++++++++++ 1 file changed, 19 insertions(+) diff --git a/drivers/pci/controller/dwc/pci-layerscape-ep.c b/drivers/pci/controller/dwc/pci-layerscape-ep.c index e0969ff2ddf7..b1faf41a2fae 100644 --- a/drivers/pci/controller/dwc/pci-layerscape-ep.c +++ b/drivers/pci/controller/dwc/pci-layerscape-ep.c @@ -45,6 +45,7 @@ struct ls_pcie_ep { struct pci_epc_features *ls_epc; const struct ls_pcie_ep_drvdata *drvdata; int irq; + u32 lnkcap; bool big_endian; }; @@ -73,6 +74,7 @@ static irqreturn_t ls_pcie_ep_event_handler(int irq, void *dev_id) struct ls_pcie_ep *pcie = dev_id; struct dw_pcie *pci = pcie->pci; u32 val, cfg; + u8 offset; val = ls_lut_readl(pcie, PEX_PF0_PME_MES_DR); ls_lut_writel(pcie, PEX_PF0_PME_MES_DR, val); @@ -81,6 +83,19 @@ static irqreturn_t ls_pcie_ep_event_handler(int irq, void *dev_id) return IRQ_NONE; if (val & PEX_PF0_PME_MES_DR_LUD) { + + offset = dw_pcie_find_capability(pci, PCI_CAP_ID_EXP); + + /* + * The values of the Maximum Link Width and Supported Link + * Speed from the Link Capabilities Register will be lost + * during link down or hot reset. Restore initial value + * that configured by the Reset Configuration Word (RCW). + */ + dw_pcie_dbi_ro_wr_en(pci); + dw_pcie_writel_dbi(pci, offset + PCI_EXP_LNKCAP, pcie->lnkcap); + dw_pcie_dbi_ro_wr_dis(pci); + cfg = ls_lut_readl(pcie, PEX_PF0_CONFIG); cfg |= PEX_PF0_CFG_READY; ls_lut_writel(pcie, PEX_PF0_CONFIG, cfg); @@ -216,6 +231,7 @@ static int __init ls_pcie_ep_probe(struct platform_device *pdev) struct ls_pcie_ep *pcie; struct pci_epc_features *ls_epc; struct resource *dbi_base; + u8 offset; int ret; pcie = devm_kzalloc(dev, sizeof(*pcie), GFP_KERNEL); @@ -252,6 +268,9 @@ static int __init ls_pcie_ep_probe(struct platform_device *pdev) platform_set_drvdata(pdev, pcie); + offset = dw_pcie_find_capability(pci, PCI_CAP_ID_EXP); + pcie->lnkcap = dw_pcie_readl_dbi(pci, offset + PCI_EXP_LNKCAP); + ret = dw_pcie_ep_init(&pci->ep); if (ret) return ret;