[PATCH v4 05/15] PCI: dwc: Silence set affinity failed warning

Marek Vasut marek.vasut+renesas at mailbox.org
Tue Jul 23 06:27:05 PDT 2024


Use newly introduced MSI_FLAG_NO_AFFINITY, which keeps .irq_set_affinity unset
and allows migrate_one_irq() code in cpuhotplug.c to exit right away, without
printing "IRQ...: set affinity failed(-22)" warning.

Remove .irq_set_affinity implementation which only return -EINVAL from this
controller driver.

Signed-off-by: Marek Vasut <marek.vasut+renesas at mailbox.org>
---
Cc: "Krzysztof Wilczyński" <kw at linux.com>
Cc: "Pali Rohár" <pali at kernel.org>
Cc: "Uwe Kleine-König" <u.kleine-koenig at pengutronix.de>
Cc: Aleksandr Mishin <amishin at t-argos.ru>
Cc: Anna-Maria Behnsen <anna-maria at linutronix.de>
Cc: Anup Patel <apatel at ventanamicro.com>
Cc: Bjorn Helgaas <bhelgaas at google.com>
Cc: Broadcom internal kernel review list <bcm-kernel-feedback-list at broadcom.com>
Cc: Daire McNamara <daire.mcnamara at microchip.com>
Cc: Damien Le Moal <dlemoal at kernel.org>
Cc: Florian Fainelli <florian.fainelli at broadcom.com>
Cc: Hou Zhiqiang <Zhiqiang.Hou at nxp.com>
Cc: Jianjun Wang <jianjun.wang at mediatek.com>
Cc: Jim Quinlan <jim2101024 at gmail.com>
Cc: Jingoo Han <jingoohan1 at gmail.com>
Cc: Jisheng Zhang <Jisheng.Zhang at synaptics.com>
Cc: Jon Hunter <jonathanh at nvidia.com>
Cc: Jonathan Derrick <jonathan.derrick at linux.dev>
Cc: Jonathan Hunter <jonathanh at nvidia.com>
Cc: Joyce Ooi <joyce.ooi at intel.com>
Cc: Karthikeyan Mitran <m.karthikeyan at mobiveil.co.in>
Cc: Kishon Vijay Abraham I <kishon at kernel.org>
Cc: Koichiro Den <den at valinux.co.jp>
Cc: Lorenzo Pieralisi <lpieralisi at kernel.org>
Cc: Manivannan Sadhasivam <manivannan.sadhasivam at linaro.org>
Cc: Marc Zyngier <maz at kernel.org>
Cc: Michal Simek <michal.simek at amd.com>
Cc: Nicolas Saenz Julienne <nsaenz at kernel.org>
Cc: Niklas Cassel <cassel at kernel.org>
Cc: Nipun Gupta <nipun.gupta at amd.com>
Cc: Nirmal Patel <nirmal.patel at linux.intel.com>
Cc: Rob Herring <robh at kernel.org>
Cc: Ryder Lee <ryder.lee at mediatek.com>
Cc: Shivamurthy Shastri <shivamurthy.shastri at linutronix.de>
Cc: Siddharth Vadapalli <s-vadapalli at ti.com>
Cc: Thierry Reding <thierry.reding at gmail.com>
Cc: Thomas Gleixner <tglx at linutronix.de>
Cc: Thomas Petazzoni <thomas.petazzoni at bootlin.com>
Cc: Yoshihiro Shimoda <yoshihiro.shimoda.uh at renesas.com>
Cc: linux-arm-kernel at lists.infradead.org
Cc: linux-mediatek at lists.infradead.org
Cc: linux-pci at vger.kernel.org
Cc: linux-renesas-soc at vger.kernel.org
Cc: linux-rpi-kernel at lists.infradead.org
Cc: linux-tegra at vger.kernel.org
---
V4: - New patch
---
 drivers/pci/controller/dwc/pci-keystone.c         |  7 -------
 drivers/pci/controller/dwc/pcie-designware-host.c | 12 +++---------
 2 files changed, 3 insertions(+), 16 deletions(-)

diff --git a/drivers/pci/controller/dwc/pci-keystone.c b/drivers/pci/controller/dwc/pci-keystone.c
index 52c6420ae2003..ce9d9e0a52609 100644
--- a/drivers/pci/controller/dwc/pci-keystone.c
+++ b/drivers/pci/controller/dwc/pci-keystone.c
@@ -189,12 +189,6 @@ static void ks_pcie_compose_msi_msg(struct irq_data *data, struct msi_msg *msg)
 		(int)data->hwirq, msg->address_hi, msg->address_lo);
 }
 
-static int ks_pcie_msi_set_affinity(struct irq_data *irq_data,
-				    const struct cpumask *mask, bool force)
-{
-	return -EINVAL;
-}
-
 static void ks_pcie_msi_mask(struct irq_data *data)
 {
 	struct dw_pcie_rp *pp = irq_data_get_irq_chip_data(data);
@@ -247,7 +241,6 @@ static struct irq_chip ks_pcie_msi_irq_chip = {
 	.name = "KEYSTONE-PCI-MSI",
 	.irq_ack = ks_pcie_msi_irq_ack,
 	.irq_compose_msi_msg = ks_pcie_compose_msi_msg,
-	.irq_set_affinity = ks_pcie_msi_set_affinity,
 	.irq_mask = ks_pcie_msi_mask,
 	.irq_unmask = ks_pcie_msi_unmask,
 };
diff --git a/drivers/pci/controller/dwc/pcie-designware-host.c b/drivers/pci/controller/dwc/pcie-designware-host.c
index a0822d5371bc5..3e41865c72904 100644
--- a/drivers/pci/controller/dwc/pcie-designware-host.c
+++ b/drivers/pci/controller/dwc/pcie-designware-host.c
@@ -48,8 +48,9 @@ static struct irq_chip dw_pcie_msi_irq_chip = {
 };
 
 static struct msi_domain_info dw_pcie_msi_domain_info = {
-	.flags	= (MSI_FLAG_USE_DEF_DOM_OPS | MSI_FLAG_USE_DEF_CHIP_OPS |
-		   MSI_FLAG_PCI_MSIX | MSI_FLAG_MULTI_PCI_MSI),
+	.flags	= MSI_FLAG_USE_DEF_DOM_OPS | MSI_FLAG_USE_DEF_CHIP_OPS |
+		  MSI_FLAG_NO_AFFINITY | MSI_FLAG_PCI_MSIX |
+		  MSI_FLAG_MULTI_PCI_MSI,
 	.chip	= &dw_pcie_msi_irq_chip,
 };
 
@@ -116,12 +117,6 @@ static void dw_pci_setup_msi_msg(struct irq_data *d, struct msi_msg *msg)
 		(int)d->hwirq, msg->address_hi, msg->address_lo);
 }
 
-static int dw_pci_msi_set_affinity(struct irq_data *d,
-				   const struct cpumask *mask, bool force)
-{
-	return -EINVAL;
-}
-
 static void dw_pci_bottom_mask(struct irq_data *d)
 {
 	struct dw_pcie_rp *pp = irq_data_get_irq_chip_data(d);
@@ -177,7 +172,6 @@ static struct irq_chip dw_pci_msi_bottom_irq_chip = {
 	.name = "DWPCI-MSI",
 	.irq_ack = dw_pci_bottom_ack,
 	.irq_compose_msi_msg = dw_pci_setup_msi_msg,
-	.irq_set_affinity = dw_pci_msi_set_affinity,
 	.irq_mask = dw_pci_bottom_mask,
 	.irq_unmask = dw_pci_bottom_unmask,
 };
-- 
2.43.0




More information about the linux-arm-kernel mailing list