From patchwork Mon Jul 29 20:18:17 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Frank Li X-Patchwork-Id: 13745763 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 1B25EC3DA61 for ; Mon, 29 Jul 2024 20:35:13 +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:MIME-Version:Cc:To: In-Reply-To:References:Message-Id:Content-Transfer-Encoding:Content-Type: Subject:Date:From:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=IBwLRFPfd+0byJrsW+igL3Gd6ozPrhPd3TitZlHjCHM=; b=cv2plbGgZiyksPXX/cNXl1y0ZY AWBO9gJkDtCQUtTUgN9ffWhDUcY5cf0CN5prNMPIOf4iLyfN3NMcRS8IYrlgODPMHyyYMd2mW/TYp N4nM9rQzBIARemuAStJkuAjXqITFbgb2OLspg5+ww6LZNOJL4ylgrhOMvyIoh2I2s701nUpkOxR58 1Sjb/da6oD9P8aEe8qpVvdrmNljxzYZMNesXBwRuEdIcrGYy9PHsWd+VIi761YXyW4cZ09jC1ubiS TLX4UCTaMG5D4IVHvY529sdbw0Ra2GPIi7A4eTu7/PeQP8986DjZLGlKkGhwp/hMvT6iXANQ61MNW 9fetVerA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sYX59-0000000Cfot-1m6m; Mon, 29 Jul 2024 20:34:59 +0000 Received: from mail-db5eur02on20601.outbound.protection.outlook.com ([2a01:111:f403:2608::601] helo=EUR02-DB5-obe.outbound.protection.outlook.com) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1sYX4i-0000000CfnF-3t6X for linux-arm-kernel@lists.infradead.org; Mon, 29 Jul 2024 20:34:34 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=U8p87ruLpFvkif9bPJmyTeVpdHisL7UoscE9uL6Y8BJ6uEOOQBQe4L8NRGF3+/+h3VMXL0/QtLpbf1OkaoT/a36uqkrU/ahepcUhSKQbj3pTAAS4dnDZWTkr0ovFPvUNMwk5To3I3NVKeut2dFvXJmHYs5yHqIf5ZwNTp6XeOydBKSKCvHJiYE+0TmEV9vCZG1WDnzZa+rmdEs/r2uEZ41rPMirCNzetNe4eh0EIrZkJYLObY6IOetZA9Wgtyb56W8zTx8R8NKmRVdhhG0rVAyH2bv+28j2FXaKm7td9areO4gZwb2Fpccak701T59Jo1p+98pM5YmnxIx2PPdphTA== 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=IBwLRFPfd+0byJrsW+igL3Gd6ozPrhPd3TitZlHjCHM=; b=T4aw5KUrKVJocWNMxB4aOocYy/G+aHSLxwS9fvT++JZ2H5J4JCHyb6ic1RDhDdVPcaScI+pwHd+jxfmN6R4f1C6nYY5pERY6GQIrDUVkgZd82v31Jo/EUgg+vUosuVrKYg4lnNMA3vVDpi33y3IyLVY6S6B/7iLYb9kQ2vaurTrIGr/0Wyzt0X1YnSpfPk3voW+xCgYgTK+vr40oNKXrQdk8iaH4Q/+8rBLMBAO1NhvdRFvbC9r+AdOUMuqOImdAwbC4ayRY15Y84y52s9qEZXNxqcZfUK4dcGnf7XtZo2Q8A+nARXGEgnr8bXz9aHy65fnZMMiCc5k8ZO8r8rMAkw== 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=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=IBwLRFPfd+0byJrsW+igL3Gd6ozPrhPd3TitZlHjCHM=; b=ap+1KBwSyeEpPsOpbQNhTy+2+y+1nI4pVrvwOSD2NX4ivw73r9xldX3f73RW/aEhBYEw9YIAP4yalyvA7UU4FEJJEQdrxWV4TQb0sbGMsWguVA1pVcYT+mgUR/lwUpg1MFFuE5J2c8sJ6sTQzyIHWCowxF3AwcKsQuFUqGYkXp74HvPj6BeOrHSci3uiMyf2z7p9m+nDuOorr7cdIjtHIUE21kL/obEd2nzusgFu4V6BGl7UkvW8hxmXEGThf85i4WZOcmnbbNTcNq8552/JVb/p5JVGoqOBmhyoX4C9ZD+F7HqA1cW5qooxtK/GSm4rJVbP+9/GoICA0BWcBNEUTw== Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=nxp.com; Received: from PAXPR04MB9642.eurprd04.prod.outlook.com (2603:10a6:102:240::14) by DBAPR04MB7382.eurprd04.prod.outlook.com (2603:10a6:10:1ab::17) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7807.27; Mon, 29 Jul 2024 20:19:40 +0000 Received: from PAXPR04MB9642.eurprd04.prod.outlook.com ([fe80::9126:a61e:341d:4b06]) by PAXPR04MB9642.eurprd04.prod.outlook.com ([fe80::9126:a61e:341d:4b06%5]) with mapi id 15.20.7807.026; Mon, 29 Jul 2024 20:19:40 +0000 From: Frank Li Date: Mon, 29 Jul 2024 16:18:17 -0400 Subject: [PATCH v8 10/11] PCI: imx6: Call common PHY API to set mode, speed, and submode Message-Id: <20240729-pci2_upstream-v8-10-b68ee5ef2b4d@nxp.com> References: <20240729-pci2_upstream-v8-0-b68ee5ef2b4d@nxp.com> In-Reply-To: <20240729-pci2_upstream-v8-0-b68ee5ef2b4d@nxp.com> To: Richard Zhu , Lucas Stach , Lorenzo Pieralisi , =?utf-8?q?Krzysztof_Wilczy?= =?utf-8?q?=C5=84ski?= , Rob Herring , Bjorn Helgaas , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , NXP Linux Team , Philipp Zabel , Liam Girdwood , Mark Brown , Manivannan Sadhasivam , Krzysztof Kozlowski , Conor Dooley Cc: linux-pci@vger.kernel.org, imx@lists.linux.dev, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, bpf@vger.kernel.org, devicetree@vger.kernel.org, Frank Li X-Mailer: b4 0.13-dev-e586c X-Developer-Signature: v=1; a=ed25519-sha256; t=1722284317; l=2619; i=Frank.Li@nxp.com; s=20240130; h=from:subject:message-id; bh=xnegrrqq+Q1Ekgre7IoRsuq7FwdKEEV8MnDreE+OHeQ=; b=3FmqDsHSTKKkzp+9sX7JBJOjNmCre53DefCsAe67ExslztvYHsn8f4rmrntF3dj1hUY4im/Qz GITWKhfUMSGCU0nCedF5e4B4e/G+HHiAiwjoLqsBpkEQmB8MD6SFm4W X-Developer-Key: i=Frank.Li@nxp.com; a=ed25519; pk=I0L1sDUfPxpAkRvPKy7MdauTuSENRq+DnA+G4qcS94Q= X-ClientProxiedBy: SJ0PR03CA0069.namprd03.prod.outlook.com (2603:10b6:a03:331::14) To PAXPR04MB9642.eurprd04.prod.outlook.com (2603:10a6:102:240::14) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: PAXPR04MB9642:EE_|DBAPR04MB7382:EE_ X-MS-Office365-Filtering-Correlation-Id: ec0118d5-3f30-49fc-d490-08dcb00bc6ad X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|376014|7416014|366016|52116014|1800799024|38350700014|921020; X-Microsoft-Antispam-Message-Info: =?utf-8?q?6Y8Vko0eY3aPhIMDdn4EVe0fXREWk5S?= =?utf-8?q?BHhc43GZZaeuFLt+pvXSumKIGffD0AVKjDimnnwa58MZ5BGa+2/F6oKNn2qTr7RoA?= =?utf-8?q?ezu5YSqHR+aHP3/78VzSSUCpbvQfsuuVKQgxtOnGuNkmec+dem/IXj4uqZE2VupuY?= =?utf-8?q?X0TFSG/c6Sfo50Xs7NeHKs4nvpsfCVXiR7afNnSCmXeXgY5OpzM9gcWeidgNVq9LJ?= =?utf-8?q?A+bY/wBqPt4o0bwi/JnQ2pUuCpBcr0l6cd4Y8Vokr/fCdNzNPSfSTThI5zCS9SB3U?= =?utf-8?q?h1hbv9FvExilJPI3aG7qrvgnl/7Zvz6juuD7412xW+rAlNTebBNVpifYdZ3GMSW8x?= =?utf-8?q?3vb7PTZVn7BfhJtQcHMk3qCQDatXzUkohNOSRM32/ci1osaGy9I8VUByVd6FlItPY?= =?utf-8?q?spuxeknuGE5S5ztIG2+KQ3BYYhrWcNTBZ3qxTV1MQb5304PWfp+UPWVAldTOrgk4A?= =?utf-8?q?18za7MqsBfVvFrJlFl1gBbtKiBlISVBFX+ktZwXh+fIGrXl15OLq4Z3NvB7CCd/D6?= =?utf-8?q?pyvcSJNOxp8+XXdjrKyMjwGP5m/F+ekjQEECddDFTGjElZngI/UBk/SPaIhgBrnZg?= =?utf-8?q?8D9EkoYddsBd0kTK17ZT1w5IAX9fjZg6zFC65LFI7o6DHOANIvaHEMtgE6loTgk0f?= =?utf-8?q?14D2gElUIurF1+eZoWqTDUKdoyRFgSGJT5bisf5Vo6MxjhypYyioYqN95FtYjhNbB?= =?utf-8?q?VABsksnInPc54JSFP2DajDO2sV/wbWWoqw3OAyJrXaxxqenWnaujEHQ1/1dLzicA7?= =?utf-8?q?FuS3NHROwsp3W4LetIK+MzwrS+H8vVx0iFRDDkfjEJrDTqG0x8KiPgkV2hYaPVsym?= =?utf-8?q?FbjZN3HLksKr7RXDQrGYHXqJMCO51RAyfaNnvz42mINxl5F5ZZyMkpdjqr9P8eBP3?= =?utf-8?q?hel6r1BUfdMm71MMsd+G3DDNiRtl8ztUiO8CNfpcJ2VlExUst2La2cDqvSr6LJGNN?= =?utf-8?q?WctiKFEhXjrZvJN+INpoTqrY0iPffFQzMFjPZlVlQphcwX0c9s1fo5xCMAThCVHmc?= =?utf-8?q?uHnwiN8enkUOeIUpHx/LQ8HLJN/G4AzQSHe3ybPA4RmwJxfpY5hUL2K/n/zmuLjUG?= =?utf-8?q?6jz4nHlfCuL2yYtpazBiYAu6syOZCefho5cdrvwm7DIHLg5vwXUM3wqlCpKxnb1++?= =?utf-8?q?N1+BuY8fxu8UdNEV115OXRRPK/gHUfFyLJedUUrV4CCdmR5v2Rxaib+YMcYA7nh/E?= =?utf-8?q?9Ej/wS6dqH3WttwnZhNo7FQO+IE535BmmDAS1W36VxDeYXxn4yoxShq8si+UTtL0V?= =?utf-8?q?jdzLTA5CUgxX5ao04Ia7sPTpkP0KSLaeEAVul4AGnzU/IXNcHbF4xhd7lrpdiVlnG?= =?utf-8?q?HtAY1vfsgD12kfVX2wmkTkMKtJtLuPlwNiA1/l1P5+ujkLVVBhLt/W0=3D?= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:PAXPR04MB9642.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(376014)(7416014)(366016)(52116014)(1800799024)(38350700014)(921020);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?q?3rPjfHFh/EpuddG2xRL1XU/Lc1hG?= =?utf-8?q?YKMrpPZ2bal4cjwqoSCSZwMk+Afm/qzFfjn3cimLNv10oo3YDJAPPvUMes/jKPv34?= =?utf-8?q?p28rB5ib9qscsQINQkZTyAObdvyj+29huAz86QktoPN91iejHuFBNLB7J/Ps5TODZ?= =?utf-8?q?03w4g3YMn3JvNZbUhYhINDeXE9gads10a0AHKIYD2YKtBMuAKPZXNZMTwAN5Y6NWy?= =?utf-8?q?b+b7pc9Z8/gkIfdF1cx7evzP9idLR69NLujAHV2xpr1KXpiAJYlwhwHh5Nm/SUUDv?= =?utf-8?q?9qZNE4h5X7ZbEUucgSXZLNJUUAyJFD6FtqISkdf0svkn1Smu9A5y4MrxmPPSWTsMS?= =?utf-8?q?Edx/EBQDAINb/yr9xgUByw0pngmv6ma6hP1REx2CzwcY4DI3CPDQQJJYw1S2FXHFN?= =?utf-8?q?fddQf+BclzDlcUri5bM5QqobwszpwxzQP8/k/4K2W1VtT6k7l0F3l+74lnp5/YD2I?= =?utf-8?q?GJ11Qq+gjkhfS7DV6ZjE8pNYZEUW+um0b1XyP1vNQCafcIAerDv4bv1BMnIdzxV7j?= =?utf-8?q?SAUeun+cwQqnZ3rxTDLXKwMa0eVp8UMK4aXjvyjT35ViAsEOP1r3hkJuKzHxOVUiX?= =?utf-8?q?zUoqryiXyBtUa3br8Si/ojTTwtM5n/rBEtmXxjY9igsU/QETRkYqXfg60+oArY0zI?= =?utf-8?q?gmOPQ19vkKaKKsHRg5N0SYe9aK/7iuqPnrjHcyaxi4zMdGtkH2iuBdNjBqhdy9x/5?= =?utf-8?q?XC/fYX98ppyhqOQuiaY60sNW/wOkBZtws89c7mf+j1Np1XS79I6rQC05CRpzdIdX3?= =?utf-8?q?wjwV37+jmOKLQVHkBq8O9mfZn11IDnN8iEv52diMdTaDVyYuIV4dYf9GYcQc3BWQ3?= =?utf-8?q?7spauEJeVEuP6u9vrZ4XariOQorQz6XG/gcAFK2VmJs6bMoAVXXTRakdtiDdavNUp?= =?utf-8?q?ytjsa9tEDwvZ8PPXWbDf+GQ+x+UAuz1FsqVyIykW/y+x9CisBlZjIVD135btwCWCU?= =?utf-8?q?6fFXaAoUEP75qzoFvqnPQGfYXM/Uop1shmvS9hrrfdqyCrDFK0Rwlz0/GtiuanZI0?= =?utf-8?q?DIOkI5B0unNSK8l1oluoaeuN+j7RiDsfBnQrsTNBQf1MRR3+0PfLeKGfoEltejbdS?= =?utf-8?q?M3wJGWKiRk9VUJjQOgufLDprGTUUYbawMJKFPp/0aov1ZAbeqfW1Yscy1Pgav/9YE?= =?utf-8?q?3RobtYk/X/PjPztLI4cZVkAmQbdjGm4AFJ+Jqe7+6/l38C7qmwzbW5Fqg/0flLjYt?= =?utf-8?q?ELRcPHK51AKUlz1F1oGAc5xk6Yhv69rz89gr3YhKjO7uX8DvtuSzKpIZ6GDmGoADh?= =?utf-8?q?hFHCpNLsVrf5wmU/97u5fqelWwhrfOD+LkoAgKKUiH0G+YHosm93TidpdKM62X2XQ?= =?utf-8?q?ZyAaTnMvwzaDsUP3bczf9dpbGekFvpvssJXFAZpYIZIwvA1R7df3471VEnlJRy+C1?= =?utf-8?q?9xSOjZ7hcAKTMw9FgxSinX4zH7AWXpdJJZBUj2z7btDBZ7z2NZbb3DT19yQ0Ebu9S?= =?utf-8?q?IkWREBqokI13v34DGpYQtH09REQ8GywnlVBamZiLlQbCD9vjJBDrB5zLxwPPYrD0R?= =?utf-8?q?Lfg+vJU3Obn3?= X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: ec0118d5-3f30-49fc-d490-08dcb00bc6ad X-MS-Exchange-CrossTenant-AuthSource: PAXPR04MB9642.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 29 Jul 2024 20:19:40.5169 (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: Pw83kBQPGbL1dZfbBLglZMVmw7+67gVVExWGmdwkOLZRAk2p6lIHO7qOdhrkneLVbYeaVgmYhCpmh1m6QwBIzg== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DBAPR04MB7382 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240729_133432_983817_0ECC040F X-CRM114-Status: GOOD ( 15.29 ) 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 Invoke the common PHY API to configure mode, speed, and submode. While these functions are optional in the PHY interface, they are necessary for certain PHY drivers. Lack of support for these functions in a PHY driver does not cause harm. Reviewed-by: Manivannan Sadhasivam Signed-off-by: Frank Li --- drivers/pci/controller/dwc/pci-imx6.c | 16 ++++++++++++++++ 1 file changed, 16 insertions(+) diff --git a/drivers/pci/controller/dwc/pci-imx6.c b/drivers/pci/controller/dwc/pci-imx6.c index ccb7cdae32756..91aab0288fdcb 100644 --- a/drivers/pci/controller/dwc/pci-imx6.c +++ b/drivers/pci/controller/dwc/pci-imx6.c @@ -28,6 +28,7 @@ #include #include #include +#include #include #include #include @@ -227,6 +228,10 @@ static void imx_pcie_configure_type(struct imx_pcie *imx_pcie) id = imx_pcie->controller_id; + /* If mode_mask is 0, then generic PHY driver is used to set the mode */ + if (!drvdata->mode_mask[0]) + return; + /* If mode_mask[id] is zero, means each controller have its individual gpr */ if (!drvdata->mode_mask[id]) id = 0; @@ -802,7 +807,11 @@ static void imx_pcie_ltssm_enable(struct device *dev) { struct imx_pcie *imx_pcie = dev_get_drvdata(dev); const struct imx_pcie_drvdata *drvdata = imx_pcie->drvdata; + u8 offset = dw_pcie_find_capability(imx_pcie->pci, PCI_CAP_ID_EXP); + u32 tmp; + tmp = dw_pcie_readl_dbi(imx_pcie->pci, offset + PCI_EXP_LNKCAP); + phy_set_speed(imx_pcie->phy, FIELD_GET(PCI_EXP_LNKCAP_SLS, tmp)); if (drvdata->ltssm_mask) regmap_update_bits(imx_pcie->iomuxc_gpr, drvdata->ltssm_off, drvdata->ltssm_mask, drvdata->ltssm_mask); @@ -815,6 +824,7 @@ static void imx_pcie_ltssm_disable(struct device *dev) struct imx_pcie *imx_pcie = dev_get_drvdata(dev); const struct imx_pcie_drvdata *drvdata = imx_pcie->drvdata; + phy_set_speed(imx_pcie->phy, 0); if (drvdata->ltssm_mask) regmap_update_bits(imx_pcie->iomuxc_gpr, drvdata->ltssm_off, drvdata->ltssm_mask, 0); @@ -950,6 +960,12 @@ static int imx_pcie_host_init(struct dw_pcie_rp *pp) goto err_clk_disable; } + ret = phy_set_mode_ext(imx_pcie->phy, PHY_MODE_PCIE, PHY_MODE_PCIE_RC); + if (ret) { + dev_err(dev, "unable to set PCIe PHY mode\n"); + goto err_phy_exit; + } + ret = phy_power_on(imx_pcie->phy); if (ret) { dev_err(dev, "waiting for PHY ready timeout!\n");