genirq/msi: Remove msi_post_free()

covanam · maddy-kerneldev · commit 91daac8a6893 · 2025-09-23T14:29:51.000+05:30
The only user of msi_post_free() - powerpc/pseries - has been changed to use msi_teardown(). Remove this unused callback. Signed-off-by: Nam Cao <namcao@linutronix.de> Reviewed-by: Thomas Gleixner <tglx@linutronix.de> Signed-off-by: Madhavan Srinivasan <maddy@linux.ibm.com> Link: https://patch.msgid.link/20250916061007.964005-1-namcao@linutronix.de
diff --git a/include/linux/msi.h b/include/linux/msi.h
@@ -431,8 +431,6 @@ struct msi_domain_info;
  *			function.
  * @domain_free_irqs:	Optional function to override the default free
  *			function.
- * @msi_post_free:	Optional function which is invoked after freeing
- *			all interrupts.
  * @msi_translate:	Optional translate callback to support the odd wire to
  *			MSI bridges, e.g. MBIGEN
  *
@@ -473,8 +471,6 @@ struct msi_domain_ops {
 					     struct device *dev, int nvec);
 	void		(*domain_free_irqs)(struct irq_domain *domain,
 					    struct device *dev);
-	void		(*msi_post_free)(struct irq_domain *domain,
-					 struct device *dev);
 	int		(*msi_translate)(struct irq_domain *domain, struct irq_fwspec *fwspec,
 					 irq_hw_number_t *hwirq, unsigned int *type);
 };
diff --git a/kernel/irq/msi.c b/kernel/irq/msi.c
@@ -1644,9 +1644,6 @@ static void msi_domain_free_locked(struct device *dev, struct msi_ctrl *ctrl)
 	else
 		__msi_domain_free_irqs(dev, domain, ctrl);
 
-	if (ops->msi_post_free)
-		ops->msi_post_free(domain, dev);
-
 	if (info->flags & MSI_FLAG_FREE_MSI_DESCS)
 		msi_domain_free_descs(dev, ctrl);
 }

Original file line number	Diff line number	Diff line change
`@@ -1644,9 +1644,6 @@ static void msi_domain_free_locked(struct device dev, struct msi_ctrl ctrl)`
`1644`	`1644`	`else`
`1645`	`1645`	`__msi_domain_free_irqs(dev, domain, ctrl);`
`1646`	`1646`
`1647`		`- if (ops->msi_post_free)`
`1648`		`- ops->msi_post_free(domain, dev);`
`1649`		`-`
`1650`	`1647`	`if (info->flags & MSI_FLAG_FREE_MSI_DESCS)`
`1651`	`1648`	`msi_domain_free_descs(dev, ctrl);`
`1652`	`1649`	`}`