[v4,06/15] PCI: mediatek-gen3: Silence set affinity failed warning

Commit Message

Marek Vasut July 23, 2024, 1:27 p.m. UTC

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 <[e-mail]>
---
Cc: "Krzysztof Wilczyński" <[e-mail]>
Cc: "Pali Rohár" <[e-mail]>
Cc: "Uwe Kleine-König" <[e-mail]>
Cc: Aleksandr Mishin <[e-mail]>
Cc: Anna-Maria Behnsen <[e-mail]>
Cc: Anup Patel <[e-mail]>
Cc: Bjorn Helgaas <[e-mail]>
Cc: Broadcom internal kernel review list <[e-mail]>
Cc: Daire McNamara <[e-mail]>
Cc: Damien Le Moal <[e-mail]>
Cc: Florian Fainelli <[e-mail]>
Cc: Hou Zhiqiang <[e-mail]>
Cc: Jianjun Wang <[e-mail]>
Cc: Jim Quinlan <[e-mail]>
Cc: Jingoo Han <[e-mail]>
Cc: Jisheng Zhang <[e-mail]>
Cc: Jon Hunter <[e-mail]>
Cc: Jonathan Derrick <[e-mail]>
Cc: Jonathan Hunter <[e-mail]>
Cc: Joyce Ooi <[e-mail]>
Cc: Karthikeyan Mitran <[e-mail]>
Cc: Kishon Vijay Abraham I <[e-mail]>
Cc: Koichiro Den <[e-mail]>
Cc: Lorenzo Pieralisi <[e-mail]>
Cc: Manivannan Sadhasivam <[e-mail]>
Cc: Marc Zyngier <[e-mail]>
Cc: Michal Simek <[e-mail]>
Cc: Nicolas Saenz Julienne <[e-mail]>
Cc: Niklas Cassel <[e-mail]>
Cc: Nipun Gupta <[e-mail]>
Cc: Nirmal Patel <[e-mail]>
Cc: Rob Herring <[e-mail]>
Cc: Ryder Lee <[e-mail]>
Cc: Shivamurthy Shastri <[e-mail]>
Cc: Siddharth Vadapalli <[e-mail]>
Cc: Thierry Reding <[e-mail]>
Cc: Thomas Gleixner <[e-mail]>
Cc: Thomas Petazzoni <[e-mail]>
Cc: Yoshihiro Shimoda <[e-mail]>
Cc: [e-mail]
Cc: [e-mail]
Cc: [e-mail]
Cc: [e-mail]
Cc: [e-mail]
Cc: [e-mail]
---
V4: - New patch
---
 drivers/pci/controller/pcie-mediatek-gen3.c | 13 +++----------
 1 file changed, 3 insertions(+), 10 deletions(-)

Comments

Jianjun Wang (王建军) July 24, 2024, 6:27 a.m. UTC | #1

On Tue, 2024-07-23 at 15:27 +0200, Marek Vasut wrote:
>  	 
> External email : Please do not click links or open attachments until
> you have verified the sender or the content.
>  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 <[e-mail]>

Acked-by: Jianjun Wang <[e-mail]>

Patch

diff --git a/drivers/pci/controller/pcie-mediatek-gen3.c b/drivers/pci/controller/pcie-mediatek-gen3.c
index b7e8e24f6a408..392b6ef3473f2 100644
--- a/drivers/pci/controller/pcie-mediatek-gen3.c
+++ b/drivers/pci/controller/pcie-mediatek-gen3.c
@@ -424,12 +424,6 @@  static int mtk_pcie_startup_port(struct mtk_gen3_pcie *pcie)
 	return 0;
 }
 
-static int mtk_pcie_set_affinity(struct irq_data *data,
-				 const struct cpumask *mask, bool force)
-{
-	return -EINVAL;
-}
-
 static void mtk_pcie_msi_irq_mask(struct irq_data *data)
 {
 	pci_msi_mask_irq(data);
@@ -450,8 +444,9 @@  static struct irq_chip mtk_msi_irq_chip = {
 };
 
 static struct msi_domain_info mtk_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	= &mtk_msi_irq_chip,
 };
 
@@ -517,7 +512,6 @@  static struct irq_chip mtk_msi_bottom_irq_chip = {
 	.irq_mask		= mtk_msi_bottom_irq_mask,
 	.irq_unmask		= mtk_msi_bottom_irq_unmask,
 	.irq_compose_msi_msg	= mtk_compose_msi_msg,
-	.irq_set_affinity	= mtk_pcie_set_affinity,
 	.name			= "MSI",
 };
 
@@ -618,7 +612,6 @@  static struct irq_chip mtk_intx_irq_chip = {
 	.irq_mask		= mtk_intx_mask,
 	.irq_unmask		= mtk_intx_unmask,
 	.irq_eoi		= mtk_intx_eoi,
-	.irq_set_affinity	= mtk_pcie_set_affinity,
 	.name			= "INTx",
 };