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 C01A2C369A1 for ; Mon, 7 Apr 2025 18:52:28 +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-Transfer-Encoding:Content-Type:MIME-Version:Message-ID:Subject:Cc:To: From:Date:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:References:List-Owner; bh=wkTAESEKeKUtdyXTDTV8eheg+YrwZqve/99BaU7jXd0=; b=CNypLqO3wvBLq5GJhD5j319Vpm M9MMvj3R9RC8qneN9CcUeJOWZU1TyWQ21gULZ/m5QxSm5k05ZA1KpJlv7LUrKBv7KH9eO1aG//gCb E1pPW6/wDlCWBZZrjFHLxaBljmNtKOWLfM/4WzHDmLKYXS9Hud+fe7m8rDZosD9RO8QwGiR6CX0lX 8qD7MYer2shJx8gV9hkdRMMN4fg8k7pSy9n4sVNBbic9CyH88vmlWasKZPMyrUE2WedKbOJMfBgnP wncZ6EmZgD6boeIqzBJ58WibpLCgH1KjMqkc/tXMPBvyf9jMLYv+JXHRUyWb+XbO4MsQfqa0aY7zG 3DlveK+w==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.1 #2 (Red Hat Linux)) id 1u1rZy-00000001frs-0fyd; Mon, 07 Apr 2025 18:52:18 +0000 Received: from tor.source.kernel.org ([172.105.4.254]) by bombadil.infradead.org with esmtps (Exim 4.98.1 #2 (Red Hat Linux)) id 1u1qoy-00000001Siq-0WrI; Mon, 07 Apr 2025 18:03:44 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by tor.source.kernel.org (Postfix) with ESMTP id 066DB61164; Mon, 7 Apr 2025 18:03:35 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id F0CDAC4CEDD; Mon, 7 Apr 2025 18:03:42 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1744049023; bh=YHIy5Y1koKmkECfrSnEDyoWYCat4HjXEskfIpv3fwuc=; h=Date:From:To:Cc:Subject:In-Reply-To:From; b=RttCvuN6FYjIuyMSpBvHJlb4NGyWZ/3GBZXejCnlcffyyRb6qBm7entEeT5evaQPe LF0Y0672+3ZAnwDw6WTDneKn3v8souc6WjSgylaiDpxYunKnwcZj2aCfx1rd5invPL tBhB17TFw4gFjshVKtaDfsnXWzrIe8GsvSp0EKT1tgW4qdDsoptaI4oabflYTQKqah 0S0QDF0bxdh3qiwItJEEY2RlOOnJ8Yd1d4s+7BKH0GEopNDPQU+7M4phafo7qggkpP qmvHquthg0WYzrN/WkyrxyW/lHo+3k6cCJK2Ffia4l+TYXQIuehc971AnL7F8FZX1I x1p5l2w16/ZBw== Date: Mon, 7 Apr 2025 13:03:41 -0500 From: Bjorn Helgaas To: "Jiri Slaby (SUSE)" Cc: tglx@linutronix.de, maz@kernel.org, linux-kernel@vger.kernel.org, Jingoo Han , Manivannan Sadhasivam , Lorenzo Pieralisi , Krzysztof =?utf-8?Q?Wilczy=C5=84ski?= , Rob Herring , Bjorn Helgaas , Karthikeyan Mitran , Hou Zhiqiang , Toan Le , Joyce Ooi , Jim Quinlan , Nicolas Saenz Julienne , Florian Fainelli , Broadcom internal kernel review list , Ray Jui , Scott Branden , Ryder Lee , Jianjun Wang , Michal Simek , Daire McNamara , linux-pci@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-rpi-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org Subject: Re: [PATCH v2 07/57] irqdomain: pci: Switch to of_fwnode_handle() Message-ID: <20250407180341.GA189772@bhelgaas> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20250319092951.37667-8-jirislaby@kernel.org> 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 Wed, Mar 19, 2025 at 10:29:00AM +0100, Jiri Slaby (SUSE) wrote: > of_node_to_fwnode() is irqdomain's reimplementation of the "officially" > defined of_fwnode_handle(). The former is in the process of being > removed, so use the latter instead. > > Signed-off-by: Jiri Slaby (SUSE) > Cc: Jingoo Han > Cc: Manivannan Sadhasivam > Cc: Lorenzo Pieralisi > Cc: "Krzysztof Wilczyński" > Cc: Rob Herring > Cc: Bjorn Helgaas > Cc: Karthikeyan Mitran > Cc: Hou Zhiqiang > Cc: Toan Le > Cc: Joyce Ooi > Cc: Jim Quinlan > Cc: Nicolas Saenz Julienne > Cc: Florian Fainelli > Cc: Broadcom internal kernel review list > Cc: Ray Jui > Cc: Scott Branden > Cc: Ryder Lee > Cc: Jianjun Wang > Cc: Michal Simek > Cc: Daire McNamara > Cc: linux-pci@vger.kernel.org > Cc: linux-arm-kernel@lists.infradead.org > Cc: linux-rpi-kernel@lists.infradead.org > Cc: linux-mediatek@lists.infradead.org Applied to pci/irq for v6.16, thanks! > --- > drivers/pci/controller/dwc/pcie-designware-host.c | 2 +- > drivers/pci/controller/mobiveil/pcie-mobiveil-host.c | 2 +- > drivers/pci/controller/pci-xgene-msi.c | 2 +- > drivers/pci/controller/pcie-altera-msi.c | 2 +- > drivers/pci/controller/pcie-brcmstb.c | 2 +- > drivers/pci/controller/pcie-iproc-msi.c | 2 +- > drivers/pci/controller/pcie-mediatek.c | 2 +- > drivers/pci/controller/pcie-xilinx-dma-pl.c | 2 +- > drivers/pci/controller/pcie-xilinx-nwl.c | 2 +- > drivers/pci/controller/plda/pcie-plda-host.c | 2 +- > 10 files changed, 10 insertions(+), 10 deletions(-) > > diff --git a/drivers/pci/controller/dwc/pcie-designware-host.c b/drivers/pci/controller/dwc/pcie-designware-host.c > index ecc33f6789e3..d1cd48efad43 100644 > --- a/drivers/pci/controller/dwc/pcie-designware-host.c > +++ b/drivers/pci/controller/dwc/pcie-designware-host.c > @@ -227,7 +227,7 @@ static const struct irq_domain_ops dw_pcie_msi_domain_ops = { > int dw_pcie_allocate_domains(struct dw_pcie_rp *pp) > { > struct dw_pcie *pci = to_dw_pcie_from_pp(pp); > - struct fwnode_handle *fwnode = of_node_to_fwnode(pci->dev->of_node); > + struct fwnode_handle *fwnode = of_fwnode_handle(pci->dev->of_node); > > pp->irq_domain = irq_domain_create_linear(fwnode, pp->num_vectors, > &dw_pcie_msi_domain_ops, pp); > diff --git a/drivers/pci/controller/mobiveil/pcie-mobiveil-host.c b/drivers/pci/controller/mobiveil/pcie-mobiveil-host.c > index 0e088e74155d..6628eed9d26e 100644 > --- a/drivers/pci/controller/mobiveil/pcie-mobiveil-host.c > +++ b/drivers/pci/controller/mobiveil/pcie-mobiveil-host.c > @@ -435,7 +435,7 @@ static const struct irq_domain_ops msi_domain_ops = { > static int mobiveil_allocate_msi_domains(struct mobiveil_pcie *pcie) > { > struct device *dev = &pcie->pdev->dev; > - struct fwnode_handle *fwnode = of_node_to_fwnode(dev->of_node); > + struct fwnode_handle *fwnode = of_fwnode_handle(dev->of_node); > struct mobiveil_msi *msi = &pcie->rp.msi; > > mutex_init(&msi->lock); > diff --git a/drivers/pci/controller/pci-xgene-msi.c b/drivers/pci/controller/pci-xgene-msi.c > index 7bce327897c9..69a9c0a87639 100644 > --- a/drivers/pci/controller/pci-xgene-msi.c > +++ b/drivers/pci/controller/pci-xgene-msi.c > @@ -247,7 +247,7 @@ static int xgene_allocate_domains(struct xgene_msi *msi) > if (!msi->inner_domain) > return -ENOMEM; > > - msi->msi_domain = pci_msi_create_irq_domain(of_node_to_fwnode(msi->node), > + msi->msi_domain = pci_msi_create_irq_domain(of_fwnode_handle(msi->node), > &xgene_msi_domain_info, > msi->inner_domain); > > diff --git a/drivers/pci/controller/pcie-altera-msi.c b/drivers/pci/controller/pcie-altera-msi.c > index e1cee3c0575f..5fb3a2e0017e 100644 > --- a/drivers/pci/controller/pcie-altera-msi.c > +++ b/drivers/pci/controller/pcie-altera-msi.c > @@ -164,7 +164,7 @@ static const struct irq_domain_ops msi_domain_ops = { > > static int altera_allocate_domains(struct altera_msi *msi) > { > - struct fwnode_handle *fwnode = of_node_to_fwnode(msi->pdev->dev.of_node); > + struct fwnode_handle *fwnode = of_fwnode_handle(msi->pdev->dev.of_node); > > msi->inner_domain = irq_domain_add_linear(NULL, msi->num_of_vectors, > &msi_domain_ops, msi); > diff --git a/drivers/pci/controller/pcie-brcmstb.c b/drivers/pci/controller/pcie-brcmstb.c > index 8b2b099e81eb..1f356fca07a2 100644 > --- a/drivers/pci/controller/pcie-brcmstb.c > +++ b/drivers/pci/controller/pcie-brcmstb.c > @@ -581,7 +581,7 @@ static const struct irq_domain_ops msi_domain_ops = { > > static int brcm_allocate_domains(struct brcm_msi *msi) > { > - struct fwnode_handle *fwnode = of_node_to_fwnode(msi->np); > + struct fwnode_handle *fwnode = of_fwnode_handle(msi->np); > struct device *dev = msi->dev; > > msi->inner_domain = irq_domain_add_linear(NULL, msi->nr, &msi_domain_ops, msi); > diff --git a/drivers/pci/controller/pcie-iproc-msi.c b/drivers/pci/controller/pcie-iproc-msi.c > index 649fcb449f34..804b3a5787c5 100644 > --- a/drivers/pci/controller/pcie-iproc-msi.c > +++ b/drivers/pci/controller/pcie-iproc-msi.c > @@ -451,7 +451,7 @@ static int iproc_msi_alloc_domains(struct device_node *node, > if (!msi->inner_domain) > return -ENOMEM; > > - msi->msi_domain = pci_msi_create_irq_domain(of_node_to_fwnode(node), > + msi->msi_domain = pci_msi_create_irq_domain(of_fwnode_handle(node), > &iproc_msi_domain_info, > msi->inner_domain); > if (!msi->msi_domain) { > diff --git a/drivers/pci/controller/pcie-mediatek.c b/drivers/pci/controller/pcie-mediatek.c > index 811a8b4acd50..efcc4a7c17be 100644 > --- a/drivers/pci/controller/pcie-mediatek.c > +++ b/drivers/pci/controller/pcie-mediatek.c > @@ -485,7 +485,7 @@ static struct msi_domain_info mtk_msi_domain_info = { > > static int mtk_pcie_allocate_msi_domains(struct mtk_pcie_port *port) > { > - struct fwnode_handle *fwnode = of_node_to_fwnode(port->pcie->dev->of_node); > + struct fwnode_handle *fwnode = of_fwnode_handle(port->pcie->dev->of_node); > > mutex_init(&port->lock); > > diff --git a/drivers/pci/controller/pcie-xilinx-dma-pl.c b/drivers/pci/controller/pcie-xilinx-dma-pl.c > index dd117f07fc95..71cf13ae51c7 100644 > --- a/drivers/pci/controller/pcie-xilinx-dma-pl.c > +++ b/drivers/pci/controller/pcie-xilinx-dma-pl.c > @@ -470,7 +470,7 @@ static int xilinx_pl_dma_pcie_init_msi_irq_domain(struct pl_dma_pcie *port) > struct device *dev = port->dev; > struct xilinx_msi *msi = &port->msi; > int size = BITS_TO_LONGS(XILINX_NUM_MSI_IRQS) * sizeof(long); > - struct fwnode_handle *fwnode = of_node_to_fwnode(port->dev->of_node); > + struct fwnode_handle *fwnode = of_fwnode_handle(port->dev->of_node); > > msi->dev_domain = irq_domain_add_linear(NULL, XILINX_NUM_MSI_IRQS, > &dev_msi_domain_ops, port); > diff --git a/drivers/pci/controller/pcie-xilinx-nwl.c b/drivers/pci/controller/pcie-xilinx-nwl.c > index 8d6e2a89b067..9cf8a96f7bc4 100644 > --- a/drivers/pci/controller/pcie-xilinx-nwl.c > +++ b/drivers/pci/controller/pcie-xilinx-nwl.c > @@ -495,7 +495,7 @@ static int nwl_pcie_init_msi_irq_domain(struct nwl_pcie *pcie) > { > #ifdef CONFIG_PCI_MSI > struct device *dev = pcie->dev; > - struct fwnode_handle *fwnode = of_node_to_fwnode(dev->of_node); > + struct fwnode_handle *fwnode = of_fwnode_handle(dev->of_node); > struct nwl_msi *msi = &pcie->msi; > > msi->dev_domain = irq_domain_add_linear(NULL, INT_PCI_MSI_NR, > diff --git a/drivers/pci/controller/plda/pcie-plda-host.c b/drivers/pci/controller/plda/pcie-plda-host.c > index 4153214ca410..4c7a9fa311e3 100644 > --- a/drivers/pci/controller/plda/pcie-plda-host.c > +++ b/drivers/pci/controller/plda/pcie-plda-host.c > @@ -150,7 +150,7 @@ static struct msi_domain_info plda_msi_domain_info = { > static int plda_allocate_msi_domains(struct plda_pcie_rp *port) > { > struct device *dev = port->dev; > - struct fwnode_handle *fwnode = of_node_to_fwnode(dev->of_node); > + struct fwnode_handle *fwnode = of_fwnode_handle(dev->of_node); > struct plda_msi *msi = &port->msi; > > mutex_init(&port->msi.lock); > -- > 2.49.0 >