From patchwork Fri Sep 24 08:05:15 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Matthias Schiffer X-Patchwork-Id: 12514359 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-18.2 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,USER_AGENT_GIT autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id F1F0BC433EF for ; Fri, 24 Sep 2021 08:08:03 +0000 (UTC) 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 mail.kernel.org (Postfix) with ESMTPS id C09C0610CF for ; Fri, 24 Sep 2021 08:08:03 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org C09C0610CF Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=ew.tq-group.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=lists.infradead.org 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:MIME-Version:List-Subscribe:List-Help: List-Post:List-Archive:List-Unsubscribe:List-Id: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:In-Reply-To:References: List-Owner; bh=jjuWrWZSOk8BC9fuTUGusdD+6VycApX/2OAcm1iYF/w=; b=CiCwpzJtzw0cd9 qIYtJlqzu+FVSf/kD0khYCyqYczK7XDru3qNNgy70+LZGhPrU2cwQ/xW6PHQ4LBYoJLn1w/+LNR05 F/jWX4iPDGZa3LzmUplmd5QqGVumXDP2g2MSWSGDXrjPXXsucOcHGnHH/s4lLeDVYj2OCJNrFy2yL AdXbxLZdPGVCbEB9HYRD/Ddb8saJgWXmWfW9PiVQCWUiCXuj7mXg1vE+21v66yPBz5LH79OTTm8sl m4q3I9KGLDeRZ++xm+bPRuPMtR+5mCDmgDNqP3Z9L5tW2Vr9lFltupjYd/fA7C4JwI5W1DhJjqLIv V7/52ruXBaQLDzGnFnOA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1mTgDJ-00DR40-1E; Fri, 24 Sep 2021 08:05:45 +0000 Received: from mx1.tq-group.com ([93.104.207.81]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1mTgDF-00DR3D-5f for linux-arm-kernel@lists.infradead.org; Fri, 24 Sep 2021 08:05:43 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=tq-group.com; i=@tq-group.com; q=dns/txt; s=key1; t=1632470741; x=1664006741; h=from:to:cc:subject:date:message-id; bh=qJ9EaKrEBGYgsWxnX3MhIVjCcoom979TP/eYeQZJVsI=; b=j2E5SJsmG57eLeMe30u8tzgjcbrKEtnE1+drSbXjKfLYfpyNfii4rXML piwJN3qSDXHqTzWnt8UaGQ+8pP2p859bugxSluIUxcngG7w95GYfcb1v8 MDFopfaT9h4f9L3a81rBA/rIqc9OpF6XxUdD+3wr8kAl0+DLbOGP6xg2P lRORTdBDyuU7hBHIjCCRsgmxyRapvnqRRt+9bsgTOs75VJ3uiPf9hFh7d 9bpao0/yMRC0kOdu24WigFu9Z2h6nxJ86iG0IoANKNn8ITDfwPhSajNJM bYGMS139lcelDkSG3R94RZHCR0R3EE9rN338eXWJB37zIK/bXRTXIdQU/ A==; X-IronPort-AV: E=Sophos;i="5.85,319,1624312800"; d="scan'208";a="19688819" Received: from unknown (HELO tq-pgp-pr1.tq-net.de) ([192.168.6.15]) by mx1-pgp.tq-group.com with ESMTP; 24 Sep 2021 10:05:36 +0200 Received: from mx1.tq-group.com ([192.168.6.7]) by tq-pgp-pr1.tq-net.de (PGP Universal service); Fri, 24 Sep 2021 10:05:36 +0200 X-PGP-Universal: processed; by tq-pgp-pr1.tq-net.de on Fri, 24 Sep 2021 10:05:36 +0200 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=tq-group.com; i=@tq-group.com; q=dns/txt; s=key1; t=1632470736; x=1664006736; h=from:to:cc:subject:date:message-id; bh=qJ9EaKrEBGYgsWxnX3MhIVjCcoom979TP/eYeQZJVsI=; b=G+K+9oePcnlrWNSApGGgorKYm/XRgB96idXqguVWevM+PMP/4j5VT+sp wupae+BbV1GttIDMJFizskDq/22u2gqBEHql24nn5xtwq3Btb/9nZR494 OZRv9KG0l4mpJiZai0FCPc7HmpJ8k87rJM0eGHf7Lt9WWkoSszw+UduOY GES2xpzhrfOM3mF8GRrcuTw1p8yB64o2gBE9JKbeX9TexfzDuNdgfbB3o gzkI570Rg9pE+kWE5BCN1o//itdZTPhhdvjevbI+ALNIVzjWRFGgqWt7A W/lMzbAnyfpF0c462Cd2GA+sZw3MnNGmCATAnHL8tKCh/y2Ou3/Kgsslh w==; X-IronPort-AV: E=Sophos;i="5.85,319,1624312800"; d="scan'208";a="19688818" Received: from vtuxmail01.tq-net.de ([10.115.0.20]) by mx1.tq-group.com with ESMTP; 24 Sep 2021 10:05:36 +0200 Received: from schifferm-ubuntu4.tq-net.de (schifferm-ubuntu4.tq-net.de [10.121.48.12]) by vtuxmail01.tq-net.de (Postfix) with ESMTPA id 3DC38280070; Fri, 24 Sep 2021 10:05:36 +0200 (CEST) From: Matthias Schiffer To: Richard Zhu , Lucas Stach , Shawn Guo , Sascha Hauer Cc: Lorenzo Pieralisi , Rob Herring , =?utf-8?q?Krzysztof_Wilczy=C5=84ski?= , Bjorn Helgaas , Pengutronix Kernel Team , Fabio Estevam , NXP Linux Team , linux-pci@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Matthias Schiffer , Markus Niebel Subject: [RFC] PCI: imx6: add support for internal oscillator on i.MX7D Date: Fri, 24 Sep 2021 10:05:15 +0200 Message-Id: <81c77a29362433fc5629ada442f0489046ce1051.1632319151.git.matthias.schiffer@ew.tq-group.com> X-Mailer: git-send-email 2.17.1 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210924_010541_590905_DA0651AA X-CRM114-Status: GOOD ( 26.06 ) 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: , MIME-Version: 1.0 Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Adds support for a DT property fsl,internal-osc to select the internal oscillator for the PCIe PHY. Signed-off-by: Matthias Schiffer Cc: Markus Niebel --- Okay, so while this patch is nice and short, I'm note sure if it's a good solution, hence I submit it as an RFC. It is roughly based on code from older linux-imx versions [1] - although it seems this feature was never supported on i.MX7D even by linux-imx (possibly because of compliance issues with the internal clock, however I haven't found a definitive erratum backing this), but only on other SoC like i.MX6QP. The device tree binding docs of the driver are somewhat lacking, but looking at [1] it seems that an external reference clock takes the place of the "pcie_bus" clock - various pieces of the driver skip enabling/disabling this clock when an external clock is configured. From this I've come to the conclusion that the clock settings in imx7d.dtsi do not really make sense: The pcie_bus clock is configured to PLL_ENET_MAIN_100M_CLK, but this seems wrong for both internal and external reference clocks: - For the internal clock, the correct clock should be PCIE_PHY_ROOT_CLK according to the reference manual - The external clocks, this should refer to an actual external clock, or possibly a fixed-clock node I would be great if someone with more insight into this could chime in and tell me if my reasoning here is correct or not. Unfortunately I only have our MBa7x at my disposal for further experimentation. This board does not have an external reference clock for the PCIe PHY, so I cannot test the behaviour for settings that use an external clock. Without this patch (and adding the new flag to the MBa7x DTS), the boot will hang while waiting for the PCIe link to come up. So, for the actual question (given that my thoughts above make any sense): How do we want to implement this? 1. A simple boolean flag, like this patch provides 2. Allow Device Trees not to specify a "pcie_bus" clock at all, meaning it should use the internal clock 3. Special handling when the "pcie_bus" clock is configured to PCIE_PHY_ROOT_CLK - is such a thing even possible, or is this breaking the clock driver's abstraction too much? 4. Something more involved, with a proper clock sel as the source for "pcie_bus" Solution 4. seems difficult to implement nicely, as the PCIe driver also fiddles with IMX7D_GPR12_PCIE_PHY_REFCLK_SEL for power management: the clock selection is switched back to the internal clock in imx6_pcie_clk_disable(), which also disables its source PCIE_PHY_ROOT_CLK, effectively gating the clock. Regards, Matthias [1] https://source.codeaurora.org/external/imx/linux-imx/tree/drivers/pci/host/pci-imx6.c?h=imx_4.1.15_2.0.0_ga drivers/pci/controller/dwc/pci-imx6.c | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/drivers/pci/controller/dwc/pci-imx6.c b/drivers/pci/controller/dwc/pci-imx6.c index 80fc98acf097..021499b9ee7c 100644 --- a/drivers/pci/controller/dwc/pci-imx6.c +++ b/drivers/pci/controller/dwc/pci-imx6.c @@ -83,6 +83,7 @@ struct imx6_pcie { struct regulator *vpcie; struct regulator *vph; void __iomem *phy_base; + bool internal_osc; /* power domain for pcie */ struct device *pd_pcie; @@ -637,7 +638,9 @@ static void imx6_pcie_init_phy(struct imx6_pcie *imx6_pcie) break; case IMX7D: regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, - IMX7D_GPR12_PCIE_PHY_REFCLK_SEL, 0); + IMX7D_GPR12_PCIE_PHY_REFCLK_SEL, + imx6_pcie->internal_osc ? + IMX7D_GPR12_PCIE_PHY_REFCLK_SEL : 0); break; case IMX6SX: regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR12, @@ -1130,6 +1133,9 @@ static int imx6_pcie_probe(struct platform_device *pdev) &imx6_pcie->tx_swing_low)) imx6_pcie->tx_swing_low = 127; + if (of_property_read_bool(node, "fsl,internal-osc")) + imx6_pcie->internal_osc = true; + /* Limit link speed */ pci->link_gen = 1; ret = of_property_read_u32(node, "fsl,max-link-speed", &pci->link_gen);