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 268BDCF6491 for ; Fri, 27 Sep 2024 23:56:11 +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:In-Reply-To:Content-Type: MIME-Version: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:References: List-Owner; bh=rj5mDycSqQTvHYqGkfixdH+WL4ucro3YKpZKQEDMe90=; b=P9wcjj/t9v35fn FawUMj8IT4Nw7wbEJDxGR79MNulGgtyZjJp1kCRCDRDDv/GZ6rmRWY+J9wW/xFczE1uteTSPOnd6+ FXajmFTZitIE5oUL2TdFJUJYCARQZqCgfbIovdQqwdaD/5R3cGWnIs4cK49mrll2mKdn0Fwp94l37 7GZhC0XR2CG4/pgctBoyHBX2o5+yCcF/dSzUyY0ubFT/zwzA5gP+NcWxNVGvnjXWY2xsYmHLK0zkT SMHg5qRhv+LURm02/bdi0Aoliqi8GHvhEsHDbwILB0llUMR680AJOFA9HraUMRR3k86H5IXXy157t bo7WW946BOCTIKZSW8Mw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1suKob-0000000CLx8-1ezw; Fri, 27 Sep 2024 23:56:01 +0000 Received: from dfw.source.kernel.org ([2604:1380:4641:c500::1]) by bombadil.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1suKnP-0000000CLoQ-0dIa for linux-arm-kernel@lists.infradead.org; Fri, 27 Sep 2024 23:54:48 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by dfw.source.kernel.org (Postfix) with ESMTP id 524F15C56A0; Fri, 27 Sep 2024 23:54:42 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id AF9CCC4CEC4; Fri, 27 Sep 2024 23:54:45 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1727481286; bh=xuljhnvBCGrJKOmRNT+0jGIXwSOHGJy+hMu8r7QwgC8=; h=Date:From:To:Cc:Subject:In-Reply-To:From; b=Wgc0B/X6N9Wpa23OFUqvPsmyDacOOMPxTRWZca6USb44APmwCH1LTlX7CSftlXD4n THkDexcFUOI0t5CKfrY6D3Eh82TGVYEF7DXQHd4l0vMQMOLL5u+oSPn0dSL45o1faK lYqrTokvnKclH8/5lK81DgwIvEfLK0yjwM2TImkuz3Nt3Y+df5AAY5wT3qzPx8ljsj T0pCkQfsyYdR6OnMRVfqAAiLc+0D1vpeBfDXc9saqJUa1mPtSXOnSwqY8ig3aKwW5x fmNDlzhoPAgPXkjv6G63HDci1RpDcJOy+BkT4Yi25d243V1zpOs/dWStLKAHcwKxwy 97mYpxIHWERHA== Date: Fri, 27 Sep 2024 18:54:44 -0500 From: Bjorn Helgaas To: Frank Li Cc: Rob Herring , Saravana Kannan , Jingoo Han , Manivannan Sadhasivam , Lorenzo Pieralisi , Krzysztof =?utf-8?Q?Wilczy=C5=84ski?= , Bjorn Helgaas , Richard Zhu , Lucas Stach , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-pci@vger.kernel.org, linux-arm-kernel@lists.infradead.org, imx@lists.linux.dev Subject: Re: [PATCH v2 3/3] PCI: imx6: Remove cpu_addr_fixup() Message-ID: <20240927235444.GA98792@bhelgaas> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20240926-pci_fixup_addr-v2-3-e4524541edf4@nxp.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240927_165447_305614_07A74416 X-CRM114-Status: GOOD ( 22.64 ) 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 Thu, Sep 26, 2024 at 12:47:15PM -0400, Frank Li wrote: > Remove cpu_addr_fixup() because dwc common driver already handle address > translate. > > Signed-off-by: Frank Li > --- > Change from v1 to v2 > - set using_dtbus_info true > --- > drivers/pci/controller/dwc/pci-imx6.c | 22 ++-------------------- > 1 file changed, 2 insertions(+), 20 deletions(-) > > diff --git a/drivers/pci/controller/dwc/pci-imx6.c b/drivers/pci/controller/dwc/pci-imx6.c > index 1e58c24137e7f..94f3411352bf0 100644 > --- a/drivers/pci/controller/dwc/pci-imx6.c > +++ b/drivers/pci/controller/dwc/pci-imx6.c > @@ -82,7 +82,6 @@ enum imx_pcie_variants { > #define IMX_PCIE_FLAG_HAS_PHY_RESET BIT(5) > #define IMX_PCIE_FLAG_HAS_SERDES BIT(6) > #define IMX_PCIE_FLAG_SUPPORT_64BIT BIT(7) > -#define IMX_PCIE_FLAG_CPU_ADDR_FIXUP BIT(8) > > #define imx_check_flag(pci, val) (pci->drvdata->flags & val) > > @@ -1015,22 +1014,6 @@ static void imx_pcie_host_exit(struct dw_pcie_rp *pp) > regulator_disable(imx_pcie->vpcie); > } > > -static u64 imx_pcie_cpu_addr_fixup(struct dw_pcie *pcie, u64 cpu_addr) > -{ > - struct imx_pcie *imx_pcie = to_imx_pcie(pcie); > - struct dw_pcie_rp *pp = &pcie->pp; > - struct resource_entry *entry; > - > - if (!(imx_pcie->drvdata->flags & IMX_PCIE_FLAG_CPU_ADDR_FIXUP)) > - return cpu_addr; > - > - entry = resource_list_first_type(&pp->bridge->windows, IORESOURCE_MEM); > - if (!entry) > - return cpu_addr; > - > - return cpu_addr - entry->offset; > -} > - > static const struct dw_pcie_host_ops imx_pcie_host_ops = { > .init = imx_pcie_host_init, > .deinit = imx_pcie_host_exit, > @@ -1039,7 +1022,6 @@ static const struct dw_pcie_host_ops imx_pcie_host_ops = { > static const struct dw_pcie_ops dw_pcie_ops = { > .start_link = imx_pcie_start_link, > .stop_link = imx_pcie_stop_link, > - .cpu_addr_fixup = imx_pcie_cpu_addr_fixup, This is tremendous, thank you very much for doing this! Have you looked at the other users of .cpu_addr_fixup()? It looks like cadence, dra7xx, artpec6, intel-gw, and visconti all use it. Do we know whether any of them have to deal with DTs that don't describe the correct translations? It would be even better if we could fix them all and we didn't need using_dtbus_info. > }; > > static void imx_pcie_ep_init(struct dw_pcie_ep *ep) > @@ -1459,6 +1441,7 @@ static int imx_pcie_probe(struct platform_device *pdev) > if (ret) > return ret; > > + pci->using_dtbus_info = true; > if (imx_pcie->drvdata->mode == DW_PCIE_EP_TYPE) { > ret = imx_add_pcie_ep(imx_pcie, pdev); > if (ret < 0) > @@ -1598,8 +1581,7 @@ static const struct imx_pcie_drvdata drvdata[] = { > }, > [IMX8Q] = { > .variant = IMX8Q, > - .flags = IMX_PCIE_FLAG_HAS_PHYDRV | > - IMX_PCIE_FLAG_CPU_ADDR_FIXUP, > + .flags = IMX_PCIE_FLAG_HAS_PHYDRV, > .clk_names = imx8q_clks, > .clks_cnt = ARRAY_SIZE(imx8q_clks), > }, > > -- > 2.34.1 >