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 8556CC433EF for ; Wed, 13 Jul 2022 09:08:26 +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: Date:Cc:To:From:Subject:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=aICjGXTJ/0CjthrcV21e5pyaFCqrqiA83yTroYvthlg=; b=dU0kOYntjP4JPW H7qABWgUQuxgAOPJpJ7S6vDpVMPlvuuarLhc9s50oXtVfes4aKXrxYj39aW5SERu7q4Gt4CJgWV9y 9JSIQzgGT/5n/geQNOXKXqgnqrpZjFVQJX8Uihd34J2Chv6QEb5WjGI3IwsK0ZwgTVlz8zDv2adO4 NqF2R5ZarrX4Zd1+iKk1HZT1/216MDO8qv45+VMv+MJbcvY9No0pV6/dxv3BfCsHEINp7qLeCises TDo1FRDf6K2NY9qFlNgj2HeK2UmVoa3s/QB+lwP2F1AR40WHtdXXX18Z+Ym7/NNkCItu/TPJ8qEgF ArZoNyO1kUcm4egks6IA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1oBYL4-0023qS-Od; Wed, 13 Jul 2022 09:07:22 +0000 Received: from metis.ext.pengutronix.de ([2001:67c:670:201:290:27ff:fe1d:cc33]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1oBYL1-0023kf-Q3 for linux-arm-kernel@lists.infradead.org; Wed, 13 Jul 2022 09:07:21 +0000 Received: from gallifrey.ext.pengutronix.de ([2001:67c:670:201:5054:ff:fe8d:eefb] helo=[IPv6:::1]) by metis.ext.pengutronix.de with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1oBYKr-0000bg-3E; Wed, 13 Jul 2022 11:07:09 +0200 Message-ID: Subject: Re: [PATCH v14 17/17] PCI: imx6: Reformat suspend callback to keep symmetric with resume From: Lucas Stach To: Richard Zhu , bhelgaas@google.com, robh+dt@kernel.org, broonie@kernel.org, lorenzo.pieralisi@arm.com, festevam@gmail.com, francesco.dolcini@toradex.com Cc: linux-pci@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, kernel@pengutronix.de, linux-imx@nxp.com Date: Wed, 13 Jul 2022 11:07:08 +0200 In-Reply-To: <1656645935-1370-18-git-send-email-hongxing.zhu@nxp.com> References: <1656645935-1370-1-git-send-email-hongxing.zhu@nxp.com> <1656645935-1370-18-git-send-email-hongxing.zhu@nxp.com> User-Agent: Evolution 3.40.4 (3.40.4-1.fc34) MIME-Version: 1.0 X-SA-Exim-Connect-IP: 2001:67c:670:201:5054:ff:fe8d:eefb X-SA-Exim-Mail-From: l.stach@pengutronix.de X-SA-Exim-Scanned: No (on metis.ext.pengutronix.de); SAEximRunCond expanded to false X-PTX-Original-Recipient: linux-arm-kernel@lists.infradead.org X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220713_020719_885865_D710DD6E X-CRM114-Status: GOOD ( 21.31 ) 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: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Am Freitag, dem 01.07.2022 um 11:25 +0800 schrieb Richard Zhu: > Create imx6_pcie_stop_link() and imx6_pcie_host_exit() functions. > Encapsulate clocks, regulators disables and PHY uninitialization into > imx6_pcie_host_exit(). > To keep suspend/resume symmetric as much as possible, invoke these two > new created functions in suspend callback. > > To be symmetric with imx6_pcie_host_exit(), move imx6_pcie_clk_enable() > to imx6_pcie_host_init() from imx6_pcie_deassert_core_reset(). > > Signed-off-by: Richard Zhu Reviewed-by: Lucas Stach > --- > drivers/pci/controller/dwc/pci-imx6.c | 60 ++++++++++++++++----------- > 1 file changed, 36 insertions(+), 24 deletions(-) > > diff --git a/drivers/pci/controller/dwc/pci-imx6.c b/drivers/pci/controller/dwc/pci-imx6.c > index 1cf8bf9035f2..bf8992a6c238 100644 > --- a/drivers/pci/controller/dwc/pci-imx6.c > +++ b/drivers/pci/controller/dwc/pci-imx6.c > @@ -703,13 +703,6 @@ static int imx6_pcie_deassert_core_reset(struct imx6_pcie *imx6_pcie) > { > struct dw_pcie *pci = imx6_pcie->pci; > struct device *dev = pci->dev; > - int ret; > - > - ret = imx6_pcie_clk_enable(imx6_pcie); > - if (ret) { > - dev_err(dev, "unable to enable pcie clocks: %d\n", ret); > - return ret; > - } > > switch (imx6_pcie->drvdata->variant) { > case IMX8MQ: > @@ -905,6 +898,14 @@ static int imx6_pcie_start_link(struct dw_pcie *pci) > return 0; > } > > +static void imx6_pcie_stop_link(struct dw_pcie *pci) > +{ > + struct device *dev = pci->dev; > + > + /* Turn off PCIe LTSSM */ > + imx6_pcie_ltssm_disable(dev); > +} > + > static int imx6_pcie_host_init(struct pcie_port *pp) > { > struct dw_pcie *pci = to_dw_pcie_from_pp(pp); > @@ -922,11 +923,17 @@ static int imx6_pcie_host_init(struct pcie_port *pp) > return ret; > } > } > + ret = imx6_pcie_clk_enable(imx6_pcie); > + if (ret) { > + dev_err(dev, "unable to enable pcie clocks: %d\n", ret); > + goto err_reg_disable; > + } > + > if (imx6_pcie->phy) { > ret = phy_power_on(imx6_pcie->phy); > if (ret) { > dev_err(dev, "pcie phy power up failed.\n"); > - goto err_reg_disable; > + goto err_clk_disable; > } > } > > @@ -947,17 +954,33 @@ static int imx6_pcie_host_init(struct pcie_port *pp) > > return 0; > > -err_clk_disable: > - imx6_pcie_clk_disable(imx6_pcie); > err_phy_off: > if (imx6_pcie->phy) > phy_power_off(imx6_pcie->phy); > +err_clk_disable: > + imx6_pcie_clk_disable(imx6_pcie); > err_reg_disable: > if (imx6_pcie->vpcie) > regulator_disable(imx6_pcie->vpcie); > return ret; > } > > +static void imx6_pcie_host_exit(struct pcie_port *pp) > +{ > + struct dw_pcie *pci = to_dw_pcie_from_pp(pp); > + struct imx6_pcie *imx6_pcie = to_imx6_pcie(pci); > + > + if (imx6_pcie->phy) { > + if (phy_power_off(imx6_pcie->phy)) > + dev_err(pci->dev, "unable to power off PHY\n"); > + phy_exit(imx6_pcie->phy); > + } > + imx6_pcie_clk_disable(imx6_pcie); > + > + if (imx6_pcie->vpcie) > + regulator_disable(imx6_pcie->vpcie); > +} > + > static const struct dw_pcie_host_ops imx6_pcie_host_ops = { > .host_init = imx6_pcie_host_init, > }; > @@ -1007,25 +1030,14 @@ static void imx6_pcie_pm_turnoff(struct imx6_pcie *imx6_pcie) > static int imx6_pcie_suspend_noirq(struct device *dev) > { > struct imx6_pcie *imx6_pcie = dev_get_drvdata(dev); > + struct pcie_port *pp = &imx6_pcie->pci->pp; > > if (!(imx6_pcie->drvdata->flags & IMX6_PCIE_FLAG_SUPPORTS_SUSPEND)) > return 0; > > imx6_pcie_pm_turnoff(imx6_pcie); > - imx6_pcie_ltssm_disable(dev); > - imx6_pcie_clk_disable(imx6_pcie); > - switch (imx6_pcie->drvdata->variant) { > - case IMX8MM: > - if (phy_power_off(imx6_pcie->phy)) > - dev_err(dev, "unable to power off PHY\n"); > - phy_exit(imx6_pcie->phy); > - break; > - default: > - break; > - } > - > - if (imx6_pcie->vpcie) > - regulator_disable(imx6_pcie->vpcie); > + imx6_pcie_stop_link(imx6_pcie->pci); > + imx6_pcie_host_exit(pp); > > return 0; > } _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel