From patchwork Wed May 15 10:01:01 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Thomas Richard X-Patchwork-Id: 13665037 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 ADA54C25B78 for ; Wed, 15 May 2024 10:01:53 +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:Cc:To:MIME-Version:Message-Id:Date: Subject:From:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References: List-Owner; bh=vJiUTrozj3HyIi2IypzvtGBiZ2zdYTHGNXUAeR2QYQI=; b=yYvHOALB/lDelA BUlSZuAYgUx+wIZ/64ziLNbCofIYb0jz38WhHJqQx5GQjp1V040OsqCW5G+r9o1JWv49+WetuusuM PPUP4RtMm1xTI6xN+3h3jh/bu2gG0ncKOoKbukwytYKi1iu9aqNzOJ2xW+Xht7eo/MP4LezufHx1E 9vALzrHv7F8jbjEwRFk6IbE6cxDPZ2xyiJFfnP9kVruer+IjnKlRiCLPOHQ92Ep2v/RF9L8VYTo+O l64zW9kYHkieVpKhn7Jjeey2zrC5IsUdEvHn3GsWe0b7n1QMt2iWuRJxiZAgmr+Kc8maHjn5WUrTL giR52z3kOQtk/5ZzvP0A==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1s7BS5-00000001AIw-0rfN; Wed, 15 May 2024 10:01:37 +0000 Received: from relay8-d.mail.gandi.net ([217.70.183.201]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1s7BRv-00000001ACj-3MU7 for linux-arm-kernel@lists.infradead.org; Wed, 15 May 2024 10:01:30 +0000 Received: by mail.gandi.net (Postfix) with ESMTPSA id CB85F1BF20F; Wed, 15 May 2024 10:01:17 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1715767282; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=/Tq48lIYa1mOy6g80ZOBX+iF4hlu1ufcE8TIs/96ovo=; b=f4CxYDhDZ698OMTEiqRorYG8oJMx6MuqJzAPvcYVzvA1P6oiZpWjtIAAiS13q2n2N3zorf fTMHpkJ3hPmS+gY3qJBZNRbOflrMMBeFZkdTfu82mb2MO5XzmDrXWJ9VmSZFqJypE+9GH/ xjYOllL9CrlVmEAtOSPy5uYFx3pyzFsctsUIRy/5GZVPiHKGxI2PEXiCEZEDToUE8d0DU4 EBjRh+ty+7r1bI+Yj5A8V+gBY/pLmaNlGd6RexoOA5q0u1DV3vUEkM/DTVC/XiAmEAfvqP 4YjT8t0ZXzIQ1AGtkCVjTOlII6/g8+LsuVybrXFTkG4NRss5I3RhFlFeHjvEpA== From: Thomas Richard Subject: [PATCH v6 00/12] Add suspend to ram support for PCIe on J7200 Date: Wed, 15 May 2024 12:01:01 +0200 Message-Id: <20240102-j7200-pcie-s2r-v6-0-4656ef6e6d66@bootlin.com> MIME-Version: 1.0 X-B4-Tracking: v=1; b=H4sIAN2HRGYC/33NzU7EIBQF4FeZsBYDFyitK9/DuIDLxWKcMoGm0 Uz67sLsTGOX5/58584qlUSVvVzurNCWaspLC8PTheHslg/iKbTMQIAWUgD/tCAEv2EiXqFwQi/ dZCdSxrL25F0l7otbcO5vV1dXKn1xKxTT96Pp7b3lOdU1l59H8Sb79N+OTXLBR03GBGfaiXj1O a9faXnGfGUd2+AcgA6QjjYAjIRwBNQ5oBpgEEg7FR3KeAT0OaAbMEQZjVKTwNEcAXMOmAZoP6I erJSBwl9g3/dffb7tKMsBAAA= To: Linus Walleij , Bartosz Golaszewski , Tony Lindgren , Aaro Koskinen , Janusz Krzysztofik , Vignesh R , Andi Shyti , Peter Rosin , Lorenzo Pieralisi , =?utf-8?q?Krzysztof_Wilczy?= =?utf-8?q?=C5=84ski?= , Rob Herring , Bjorn Helgaas , Siddharth Vadapalli Cc: linux-gpio@vger.kernel.org, linux-kernel@vger.kernel.org, linux-omap@vger.kernel.org, linux-i2c@vger.kernel.org, linux-pci@vger.kernel.org, linux-arm-kernel@lists.infradead.org, gregory.clement@bootlin.com, theo.lebrun@bootlin.com, thomas.petazzoni@bootlin.com, u-kumar1@ti.com, Thomas Richard , Bartosz Golaszewski , Andy Shevchenko , Wolfram Sang , Francesco Dolcini X-Mailer: b4 0.12.0 X-GND-Sasl: thomas.richard@bootlin.com X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240515_030128_230398_E641C9B2 X-CRM114-Status: GOOD ( 21.48 ) 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 This adds suspend to ram support for the PCIe (RC mode) on J7200 platform. This 6th iteration fixes a compile issue in the i2c-omap driver if CONFIG_PM_SLEEP is not set. A new patch was added to remove the __maybe_unused attribute of omap_i2c_runtime_suspend() and omap_i2c_runtime_resume(). Regards, Thomas Signed-off-by: Thomas Richard --- Changes in v6: - i2c-omap: add a patch to remove __maybe_unused attribute of omap_i2c_runtime_suspend() and omap_i2c_runtime_resume() - i2c-omap: fix compile issue if CONFIG_PM_SLEEP is not set - Link to v5: https://lore.kernel.org/r/20240102-j7200-pcie-s2r-v5-0-4b8c46711ded@bootlin.com Changes in v5: - all: series rebased on Linux 6.9-rc1 - pinctrl-single: patch removed (already applied to the pinctrl tree) - phy: patches moved to a dedicated series. - pci: add T_PERST_CLK_US macro. - pci-j721e: update the comments about T_PERST_CLK_US. - Link to v4: https://lore.kernel.org/r/20240102-j7200-pcie-s2r-v4-0-6f1f53390c85@bootlin.com Changes in v4: - all: use SoB/Co-developed-by for patches initially developed by Théo Lebrun. - pinctrl-single: squash the two commits. - i2c-omap: fix line lenghts of the comment in omap_i2c_suspend(). - mux: mux_chip_resume() return 0 or at the first error. - phy-j721e-wiz: clean code around dev_err_probe(). - phy-j721e-wiz: use REF_CLK_100MHZ macros. - pci: fix subject line for all PCI patches. - pci-cadence: use fsleep() instead of usleep_range(). - pci-cadence: remove cdns_torrent_clk_cleanup() call in cdns_torrent_phy_resume_noirq(). - pci-j721e: add a patch to use dev_err_probe() instead of dev_err() in the probe(). - pci-j721e: fix unordered header files. - pci-j721e: remove some log spammers. - pci-j721e: add a missing clock disable in j721e_pcie_resume_noirq(). - pci-j721e: simplify the patch "Add reset GPIO to struct j721e_pcie" - Link to v3: https://lore.kernel.org/r/20240102-j7200-pcie-s2r-v3-0-5c2e4a3fac1f@bootlin.com Changes in v3: - pinctrl-single: split patch in two parts, a first patch to remove the dead code, a second to move suspend()/resume() callbacks to noirq. - i2c-omap: add a comments above the suspend_noirq() callback. - mux: now mux_chip_resume() try to restores all muxes, then return 0 if all is ok or the first failure. - mmio: fix commit message. - phy-j721e-wiz: add a patch to use dev_err_probe() instead of dev_err() in the wiz_clock_init() function. - phy-j721e-wiz: remove probe boolean for the wiz_clock_init(), rename the function to wiz_clock_probe(), extract hardware configuration part in a new function wiz_clock_init(). - phy-cadence-torrent: use dev_err_probe() and fix commit messages - pcie-cadence-host: remove probe boolean for the cdns_pcie_host_setup() function and extract the link setup part in a new function cdns_pcie_host_link_setup(). - pcie-cadence-host: make cdns_pcie_host_init() global. - pci-j721e: use the cdns_pcie_host_link_setup() cdns_pcie_host_init() functions in the resume_noirq() callback. - Link to v2: https://lore.kernel.org/r/20240102-j7200-pcie-s2r-v2-0-8e4f7d228ec2@bootlin.com Changes in v2: - all: fix commits messages. - all: use DEFINE_NOIRQ_DEV_PM_OPS and pm_sleep_ptr macros. - all: remove useless #ifdef CONFIG_PM. - pinctrl-single: drop dead code - mux: add mux_chip_resume() function in mux core. - mmio: resume sequence is now a call to mux_chip_resume(). - phy-cadence-torrent: fix typo in resume sequence (reset_control_assert() instead of reset_control_put()). - phy-cadence-torrent: use PHY instead of phy. - pci-j721e: do not shadow cdns_pcie_host_setup return code in resume sequence. - pci-j721e: drop dead code. - Link to v1: https://lore.kernel.org/r/20240102-j7200-pcie-s2r-v1-0-84e55da52400@bootlin.com --- Thomas Richard (9): gpio: pca953x: move suspend()/resume() to suspend_noirq()/resume_noirq() i2c: omap: switch to NOIRQ_SYSTEM_SLEEP_PM_OPS() and RUNTIME_PM_OPS() i2c: omap: wakeup the controller during suspend() callback mux: add mux_chip_resume() function PCI: cadence: Extract link setup sequence from cdns_pcie_host_setup() PCI: cadence: Set cdns_pcie_host_init() global PCI: j721e: Use dev_err_probe() in the probe() function PCI: Add T_PERST_CLK_US macro PCI: j721e: Use T_PERST_CLK_US macro Théo Lebrun (3): mux: mmio: add resume support PCI: j721e: Add reset GPIO to struct j721e_pcie PCI: j721e: Add suspend and resume support drivers/gpio/gpio-pca953x.c | 7 +- drivers/i2c/busses/i2c-omap.c | 36 ++++-- drivers/mux/core.c | 29 +++++ drivers/mux/mmio.c | 12 ++ drivers/pci/controller/cadence/pci-j721e.c | 121 ++++++++++++++++++--- drivers/pci/controller/cadence/pcie-cadence-host.c | 44 +++++--- drivers/pci/controller/cadence/pcie-cadence.h | 12 ++ drivers/pci/pci.h | 3 + include/linux/mux/driver.h | 1 + 9 files changed, 221 insertions(+), 44 deletions(-) --- base-commit: cebffb4fba6177b15e60e28e1ac17fc4efb2f86f change-id: 20240102-j7200-pcie-s2r-ecb1a979e357 Best regards,