From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 2E0832036FD; Mon, 17 Mar 2025 22:12:01 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1742249522; cv=none; b=MFG4LSlHQToco31zUQMdLU0+jZ7+n6WyP8EunYKp44tqHATdQ/Z5sJZmG9EHGKKlHZ2CQk72Kc7eIw26k2FmD6sEfgDP9RkbkYF+1Witd5rJGcsMBX6ymKIAXyq9KJsC0L3bXvtLM80UQC+uMm1neVV8DBVdbA3AfpY2S3icBAU= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1742249522; c=relaxed/simple; bh=JEdZ8g1iC7LvD0mCARyJ8WNae6kxKED75zVQlYCHfjg=; h=Date:From:To:Cc:Subject:Message-ID:MIME-Version:Content-Type: Content-Disposition:In-Reply-To; b=Gco8JM8dBqZFsiH8UdmX6YQwF9G1XVyqllBC5hFaqgvT+V7CXlPWzKQ+mPo2+tsgzKgP6P5BzKHuYwp3gmoENT/nDavk3qmAn4xyl8IBRNYutrdxDvkaUZ/x7XzpT6s3JJT1fJ1fcvVEDQAXs9ON1VtYXQzTkvdT1tvAEK1/cAk= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=Zk11OSnm; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="Zk11OSnm" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 8CCC2C4CEE3; Mon, 17 Mar 2025 22:12:01 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1742249521; bh=JEdZ8g1iC7LvD0mCARyJ8WNae6kxKED75zVQlYCHfjg=; h=Date:From:To:Cc:Subject:In-Reply-To:From; b=Zk11OSnmRIHe1gd+qJ9NrAMljHh+/a/ZiBfI7D7Qjd6Sb4UVTyl5AqAOgX1u+yorY 0lP7AxucUDDi9vYR52g1jtp+qhrCJVTD/n3duivvfyuNR4zi4V3ZyatQU67KtkHOwa yM+8fi1hSeGTfUVr2qKob17ces1ehwWrTIfDeQrJN7i8zratTI1tvK0h0+rE0gMplU TwvOB5I4p3GnjJkUotzsf0MNR1tweERRcH+lekiTra0tGkNv0tRDf5M/rQm4ImEpT3 lNzNHfAwZBEHqMXBg7Mvy5LN3lWqD5VkwwCqkR0imtuhLCB+j7ST32TmPizFtD8i6l MVVyg/Le2JilA== Date: Mon, 17 Mar 2025 17:12:00 -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?= , Richard Zhu , Lucas Stach , Shawn Guo , Sascha Hauer , Fabio Estevam , Niklas Cassel , Pengutronix Kernel Team , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-pci@vger.kernel.org, linux-arm-kernel@lists.infradead.org, imx@lists.linux.dev, Bjorn Helgaas Subject: Re: [PATCH v12 00/13] PCI: Use device bus range info to cleanup RC Host/EP pci_fixup_addr() Message-ID: <20250317221200.GA975949@bhelgaas> Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20250315201548.858189-1-helgaas@kernel.org> On Sat, Mar 15, 2025 at 03:15:35PM -0500, Bjorn Helgaas wrote: > From: Bjorn Helgaas > > This is a v12 based on Frank's v11 series. > > v11 https://lore.kernel.org/r/20250313-pci_fixup_addr-v11-0-01d2313502ab@nxp.com > > Changes from v11: > - Call devm_pci_alloc_host_bridge() early in dw_pcie_host_init(), before > any devicetree-related code > - Call devm_pci_epc_create() early in dw_pcie_ep_init(), before any > devicetree-related code > - Consolidate devicetree-related code in dw_pcie_host_get_resources() and > dw_pcie_ep_get_resources() > - Integrate dw_pcie_cfg0_setup() into dw_pcie_host_get_resources() > - Convert dw_pcie_init_parent_bus_offset() to dw_pcie_parent_bus_offset() > which returns the offset rather than setting it internally > - Split the debug comparison of devicetree info with .cpu_addr_fixup() to > separate patch so we can easily revert it later > - Drop "cpu_addr_fixup() usage detected" warning since we always warn > about something in that case anyway > > Any comments welcome. > > > Bjorn Helgaas (3): > PCI: dwc: Consolidate devicetree handling in > dw_pcie_host_get_resources() > PCI: dwc: ep: Call epc_create() early in dw_pcie_ep_init() > PCI: dwc: ep: Consolidate devicetree handling in > dw_pcie_ep_get_resources() > > Frank Li (10): > PCI: dwc: Use resource start as iomap() input in > dw_pcie_pme_turn_off() > PCI: dwc: Rename cpu_addr to parent_bus_addr for ATU configuration > PCI: dwc: Call devm_pci_alloc_host_bridge() early in > dw_pcie_host_init() > PCI: dwc: Add dw_pcie_parent_bus_offset() > PCI: dwc: Add dw_pcie_parent_bus_offset() checking and debug > PCI: dwc: Use devicetree 'reg[config]' to derive CPU -> ATU addr > offset > PCI: dwc: ep: Use devicetree 'reg[addr_space]' to derive CPU -> ATU > addr offset > PCI: dwc: ep: Ensure proper iteration over outbound map windows > PCI: dwc: Use parent_bus_offset to remove need for .cpu_addr_fixup() > PCI: imx6: Remove cpu_addr_fixup() > > drivers/pci/controller/dwc/pci-imx6.c | 18 +--- > .../pci/controller/dwc/pcie-designware-ep.c | 74 +++++++++++------ > .../pci/controller/dwc/pcie-designware-host.c | 57 ++++++++----- > drivers/pci/controller/dwc/pcie-designware.c | 82 ++++++++++++++----- > drivers/pci/controller/dwc/pcie-designware.h | 24 +++++- > 5 files changed, 171 insertions(+), 84 deletions(-) Applied to pci/controller/dwc-cpu-addr-fixup for v6.15. Thank you very much, Frank, for all your hard work on driving this series! I think this is a major improvement.