From mboxrd@z Thu Jan 1 00:00:00 1970 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 92351C3DA4A for ; Mon, 19 Aug 2024 14:47:07 +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:In-Reply-To: Content-Type:References:Message-ID:Subject:Cc:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=9WOygTGMW4OmiUVg0LifCCjfWY5YUu48DSGmHXhrAug=; b=a3GTmqIfHbnrT9erP49Rq6znx4 hKkk//oVhzXHtAIed2csO2K10KlNmhQW0p3kD0I0sPOOTJEuyjMMyXNyVaPLF1+2FXBEHRcltmxqb AOZvMOJvzxePnOS8JnBNljkch6pshgCfN7PJ32PxAMmJ5UZTug1D/F88fxjrSgLKN+SOuod8ht9U6 Rdk5KM72X/aZDlMJC7tjUH9E0se21bAEjQjBqXvRj7N6qBi02XrqZLY+MuwYNXyz76XYlbFKGKhPS 3gxDlKaoupP8BrbmQklm8U482Ls76Ei17/pVQzaGi/FbxHZKeqVOPP7w9a0Sq18vnlMzK4wEW4YNP kdd0Ey2g==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sg3eq-00000001ocu-2zY8; Mon, 19 Aug 2024 14:46:56 +0000 Received: from mail-westeuropeazlp170110003.outbound.protection.outlook.com ([2a01:111:f403:c201::3] helo=AS8PR04CU009.outbound.protection.outlook.com) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1sg3dP-00000001o8y-3Nii for linux-arm-kernel@lists.infradead.org; Mon, 19 Aug 2024 14:45:29 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=wx+0AYirMiTO+4kfOGopAGSNB0K9YsjoDojmaIJ9+aB5bAdDh+ZMlvmZRtD1yGIHTtV1RxA621LyayBYLLJqUKY+BjvJCHd/Vs/laPdESm2CFUUTTrg00JbexPyB6CqlJBUcGRp4V1YH8otqoQi6spcICbPVjt9hNpxgT0UPlvrk7XjbTmQ7AMb57vaeSkdlJWTk966y1FVwVGUpFXcEtDfK72ABvgnzLN8nniZl5etc14a+ETtq9klVdNBRRSIbTGm3peULAgshr23Sw82Gv15QAsSPyhPVw68fTNJTaVImpe/Sw2NQ87QrzozHWRXMmDjN3dgy1WCDIWsR94umrA== 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=9WOygTGMW4OmiUVg0LifCCjfWY5YUu48DSGmHXhrAug=; b=K7T7T9u+vmYWOWwyrk3amj4lFgxAo/o+Nms0kmUdZb7ok+qPN0GwGAZGKqMRhkbPUPqCqxPjKmd/YXbimYIRWr3HUk2zHsg5rF86zkUOBv8M++ndbI8Gpwpyb16gpOkeLBxBsWTbzHzdia3olFTn8G4+javLzIg1sw2YANi9NS1lxolYLnd+9BORrx+peAGK8vAc552HrpW0NZE+5NVKWrgLRxr8XQKB7wovcOL/niaz5D2poGLr+4DQa8bFnEDBhXkzzYzUeATXAq4vq36dSqLYOvrCibfGuDMJsQ0h56XTzul6jTUzaLtXXa107TKlx7x9aH6pIZkpEkdeGVDnqA== 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=9WOygTGMW4OmiUVg0LifCCjfWY5YUu48DSGmHXhrAug=; b=S6V8OTnndGEFKnWT+iJLA3XrrN67YDQ6gqu4U7YdidvPD46NJkPpppp8/o+iDKUa7JUky7c9YARUrMhN5fpDeU/tl7s2IJe6cZRH1MpsRAZ2ESq/xwdDUx5ZSxU64pSGcl3ldksiv08PLr8Hy6aG5KC3NrskK7E+azVkoUNJ4BL7OcF1C3oNKzVGCS52U98hvE2DZllIJN1txijqeU1bxKkJwDoWko2a0/rprYQmKjyk94i7oxc0F+6zHNhMTgtu3XlivDfxe4pFvuWq94B/KfNWLxoVj8iHYx487yVkQSDxnW1BBV33AQp1rEu0/vvZyPw0yl44Ob/OrCDPGUccRQ== 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 DUZPR04MB9796.eurprd04.prod.outlook.com (2603:10a6:10:4e1::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7875.21; Mon, 19 Aug 2024 14:45:23 +0000 Received: from PAXPR04MB9642.eurprd04.prod.outlook.com ([fe80::9126:a61e:341d:4b06]) by PAXPR04MB9642.eurprd04.prod.outlook.com ([fe80::9126:a61e:341d:4b06%3]) with mapi id 15.20.7875.019; Mon, 19 Aug 2024 14:45:23 +0000 Date: Mon, 19 Aug 2024 10:45:13 -0400 From: Frank Li To: Stefan Eichenberger Cc: hongxing.zhu@nxp.com, l.stach@pengutronix.de, lpieralisi@kernel.org, kw@linux.com, robh@kernel.org, bhelgaas@google.com, shawnguo@kernel.org, s.hauer@pengutronix.de, kernel@pengutronix.de, festevam@gmail.com, francesco.dolcini@toradex.com, linux-pci@vger.kernel.org, linux-arm-kernel@lists.infradead.org, imx@lists.linux.dev, linux-kernel@vger.kernel.org, Stefan Eichenberger Subject: Re: [PATCH v1 2/3] PCI: imx6: move the wait for clock stabilization to enable ref clk Message-ID: References: <20240819090428.17349-1-eichest@gmail.com> <20240819090428.17349-3-eichest@gmail.com> Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20240819090428.17349-3-eichest@gmail.com> X-ClientProxiedBy: SJ0PR03CA0288.namprd03.prod.outlook.com (2603:10b6:a03:39e::23) To PAXPR04MB9642.eurprd04.prod.outlook.com (2603:10a6:102:240::14) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: PAXPR04MB9642:EE_|DUZPR04MB9796:EE_ X-MS-Office365-Filtering-Correlation-Id: 97a9ba98-7470-430b-fe67-08dcc05d8e16 X-LD-Processed: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635,ExtAddr X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|366016|1800799024|376014|7416014|52116014|38350700014; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?g8NtHLeNKwwjEyhaBfzcMdW2xFvgHOxcbVtJ5drx/0o896UiQOMMhu7d6Ew1?= =?us-ascii?Q?6WGkKhICWTOduLzc43pUZSnftYy+CehihZgrLB3nW5wXEsTMhuJj3ftFhg9T?= =?us-ascii?Q?mdYtQaQbeycpl4HdgH6eWX9LDfhpOuW99FYiGV8Vg+Tgx2RcjFfNRG6v0OMg?= =?us-ascii?Q?TbsTsF5x7jpiKG0YQQHHLM9vqlHtWWe/JAVWTLkozGlLs6YljTjpgYDM5N8L?= =?us-ascii?Q?A3olG0CA7+XlGGnH2PfjFdvta1HvoplKhTvXzqPPDg7r18B5n3/7opdqcmTq?= =?us-ascii?Q?tdTmyzSqzHSm9VMqQbtxqVSok4HnU3E4QL4j6nsYBnhHuTblZcq6C6+zSF78?= =?us-ascii?Q?QqlcQFivdemIpyhQC3INKpJcSDzjtaN3zxhCXM20bkum5k27M82y0fb2ur91?= =?us-ascii?Q?BbVfcN7I8LNmqsr0wuOIdRSL7MWyNd9XKimu3anrRSUgJGiCTGGo3WeA/jV9?= =?us-ascii?Q?Rno1cBplix+0U9uZv9k/2HRqr2aiuBxt8u25AOUi6eSuOG3Y8sMsA06N8GRC?= =?us-ascii?Q?nS5Pu0C2kV40fD/EqQhmGbJ6sNbBxVuziSrHORuH/CB5qNQmtmTZMjmPYbxm?= =?us-ascii?Q?3QSreOg1mPqyl2rScE7NT57HJzib6QzmuIQzTv0XEa7NWAScyHeUW77VqSkx?= =?us-ascii?Q?pw3aeZTcTniDiF2OImGTu9W1fUxibJ9kyqsh4GgbT8JI71bGUquXn0ng77DX?= =?us-ascii?Q?/xLTSywKrmN0R/v34rw5F/AJ+XM0k1ilU+NJcjls3qmC71TRaVRwdTEElVYy?= =?us-ascii?Q?7+RS6GcZz/JYCblsI290YS9LBw7GMFst4iAaGVQOq7fQPN17lFm9mDYrUK+t?= =?us-ascii?Q?WyTDPdP2742I4Zcq8FVO202cWj7YjeyOUrUIBObvD2K1eYX6l0fUu2b1uWaF?= =?us-ascii?Q?lPn/QXqpLIT6V+fHsiizyReqkRsuUQH+Vcbcb93tIJBBCSojuowfwNxJ+MF+?= =?us-ascii?Q?dHBLOWdptEH66j3WsogiAFi/o4g92F8DHN0Pj5SrbKP/c0OHgBT6KCcfAoZ5?= =?us-ascii?Q?QYUGIGWzqKqqSrBzzHbj1crn7ZIkfOv6uZ2blDHoZHvUy5e1aXl4RqTGICwG?= =?us-ascii?Q?RmU2VuU2lru5W/tnIazKz5ILSXhju6kY4ubzpfUb3vqnaU5p9xYkKG9DPTa+?= =?us-ascii?Q?fsnU6WGWyTKEzF3ZuAl4A+chDzozp2QBs3HTn+x3ELHBkSv8NXQeSwKGfb9a?= =?us-ascii?Q?Ow1j0Gimnmfnke1j4YyJwGU6EGiRQ72Bb2xf2+/SJBHvzkP3X0ax2+6iDawc?= =?us-ascii?Q?Ktg3D7zWwLA1F7Lnutw9R3q4mAWbt1BOh1ygRy/l7cL25nDAphr+122mMjSH?= =?us-ascii?Q?pzxLNeV2eVmclMDPEXfWvZU8EscS5+0pps6O46VmjtDXEgeKUBn1czJ8C5z0?= =?us-ascii?Q?SXP1AN0No9fl2WhfYJK8c1jYG6oNqlTLNVe/kECrNsvzUNfSLA=3D=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)(366016)(1800799024)(376014)(7416014)(52116014)(38350700014);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?qSIqxwhp4hcSBmPu71zrh2y48Ajh3M8z3YHIDE7wAqjwNUzzU6aEk/EiESGq?= =?us-ascii?Q?L2SBb+xqrsQZvDk1IB2dcB7ikVMtBOpxoiEvqsu7nVIoKJyjvGNjZ1Q0Qpuj?= =?us-ascii?Q?kEWz3H4AfaZaV0eFBR5J6WWHg3Ddf3CShYwGuReqNngaCbAM1eRI6Mk1UoAa?= =?us-ascii?Q?e21mhEr6eBofhCWWkiqneXv5HG9aJsFrWC/PeMLLvNCr3cmMBkB7XerJfjlc?= =?us-ascii?Q?JFTVlXoQiF7ZkpRb9p9M1qDGA3Dg0NxqO1lxQKkONWAn3xjteNK3BrbhilPz?= =?us-ascii?Q?bwIyckWAvHonatBCUq2MHkG9gzqDr7g3tQwfL548q71K/72GDnJ3dQr/m3Vy?= =?us-ascii?Q?wSJvX/zEXNjd77XOYdticUM19OVBle/aOrr7/hwCbY5wNho3Ig9x9egp1XcL?= =?us-ascii?Q?5kPCBk6fdlDsPTHcHunG2zDuuYLXsYl8WopahEF66PCca1t+nDJmRwAVY2jS?= =?us-ascii?Q?5YvO+alTN9tsGZPwkoSeAfLAvqJt65cZP/WKsRTR1o+ET3fC5eTkeo1THKJz?= =?us-ascii?Q?gTQW5A/UZhc3Ok53is+5FwZzeqPwQBgBN77olX3Of2kX2PVCHp3ycKLaccOQ?= =?us-ascii?Q?3iJE2thlPklxYk9bu4PCcgqR+1iai9ZBR0KvklVd9ItlCM27hEATQxbMLhfo?= =?us-ascii?Q?oaHUvbdaBxny0WPxzsov5mUfpE7m7y1ya7GvfU1DF7LMMVGckr40HDqJQjWl?= =?us-ascii?Q?HIGS3bhunGcIYNL9tdqcWCCWEyHFTbHvx9lvahlamfWYGYdRaCPb4pIc05Sq?= =?us-ascii?Q?808DxmKa95RBTdj97A0N8gGvFo7ffFaZ6Waax2pEBHxt24XEEIIlxAcXn9pY?= =?us-ascii?Q?xgLmM9ZGsX00tJYrYV3oEZYGOZM+Aj64oP08hhlb4zx+zFDcRcu0K4y5mWe2?= =?us-ascii?Q?VK3ajov2W4mFSbBZ5eJShbimq9+j9DL2aQeciHmN8Gp73sdMwUi1XE+h4mCm?= =?us-ascii?Q?5iULwwUuFGJdbsG9L9OS9ZBkSv4O5jAk5uT4BXwUpIlNXOf9TAs/do/CrXe0?= =?us-ascii?Q?F+5ZNk2ZDoepOkLuo3N3NGFiYkf5uhjqutcr5C+XdKxjtuXBfHUDgkLmyX2T?= =?us-ascii?Q?O/4CybKss8QlMCgr45rmpBxZufk9cUSDYr9dyvVRhfHLgggaot9FER+9vgQQ?= =?us-ascii?Q?S0aAuNhbwGwelfpC/qij2Ef4PUbrEMFS1EPCUnmtYkc45JSNBkTtXwtZeyk6?= =?us-ascii?Q?QPTWLmklfKsUPLObmKktSERLzzKPZRDqUBP4X4ppIqLoCMQPouDe2ObDrR9j?= =?us-ascii?Q?uoHbG8iGoK3KXcsb09vuLPaUKpZNsdP06cHBtGVRlyOmrorokn6lM1nikKQC?= =?us-ascii?Q?enwNGKnG8wSGRxbVc2YRckMxvTSoudvnSstu4y65MguPqWFdwBYsHP8vmsuN?= =?us-ascii?Q?riH+tA+nil4XUSNeLZAKsfM/Qf5SK8vplCmYYc06OWYvkayDmFZAa2YYzWBS?= =?us-ascii?Q?4qcEJTwvw8iNFPyNG56+NaW+eMMWZypeCRziPCi1n9Kq/T4lf+Ulqc6xJH/p?= =?us-ascii?Q?iGIPw+m/gqNSmJPvUPw6zvj6XdsiIIU0np2iSuSbaRiK8Baq5yGMMGsNoAUA?= =?us-ascii?Q?DnDRubsAHH/wRwcJFLUMuJ9GSetNRlmyugTkle/q?= X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: 97a9ba98-7470-430b-fe67-08dcc05d8e16 X-MS-Exchange-CrossTenant-AuthSource: PAXPR04MB9642.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 19 Aug 2024 14:45:23.0361 (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: nW7dPc1ZWas+sp5tcAyRUSMm0NKGYgHw1yNrl87EpvMTUqqHp6d7KKla20bLa0GrQaCwlXQOOuDvfOGBtRzaqw== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DUZPR04MB9796 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240819_074527_873464_C5750F21 X-CRM114-Status: GOOD ( 18.61 ) 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 On Mon, Aug 19, 2024 at 11:03:18AM +0200, Stefan Eichenberger wrote: > From: Stefan Eichenberger > > After enabling the ref clock, we should wait for the clock to stabilize. > To eliminate the need for code duplication in the future, move the > usleep to the enable_ref_clk function. Logically, it's better wait in imx6_pcie_clk_enable(). But not sure why it can reduce duplication. Frank > > Signed-off-by: Stefan Eichenberger > --- > drivers/pci/controller/dwc/pci-imx6.c | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/drivers/pci/controller/dwc/pci-imx6.c b/drivers/pci/controller/dwc/pci-imx6.c > index fda704d82431f..f17561791e35a 100644 > --- a/drivers/pci/controller/dwc/pci-imx6.c > +++ b/drivers/pci/controller/dwc/pci-imx6.c > @@ -632,6 +632,9 @@ static int imx6_pcie_enable_ref_clk(struct imx6_pcie *imx6_pcie) > break; > } > > + /* allow the clocks to stabilize */ > + usleep_range(200, 500); > + > return ret; > } > > @@ -672,8 +675,6 @@ static int imx6_pcie_clk_enable(struct imx6_pcie *imx6_pcie) > goto err_ref_clk; > } > > - /* allow the clocks to stabilize */ > - usleep_range(200, 500); > return 0; > > err_ref_clk: > -- > 2.43.0 >