ASoC: SDCA: Add sdca_irq_cleanup_late()

The SDCA IRQs are split into two groups, those registered at bus probe
time (basically just FDL) and those registered at component time.
There currently exists only a single cleanup function, if the FDL IRQ
is freed at component time, then nothing would re-register it if the
component is probed again. But the IRQs depending on a component need
to be freed if the card is destroyed so they can't use stale
components.

Split the clean up into two functions one for the component level and
one for the bus level.

Signed-off-by: Charles Keepax <ckeepax@opensource.cirrus.com>
Link: https://patch.msgid.link/20260721143636.361814-3-ckeepax@opensource.cirrus.com
Signed-off-by: Mark Brown <broonie@kernel.org>

diff --git a/include/sound/sdca_interrupts.h b/include/sound/sdca_interrupts.h
index 28fd44eb933433d9ea86ba9b8a99b1283fa51cf6..38c6c58c2cc78eb78ab05ad7d3724784e197c415 100644 (file)
--- a/include/sound/sdca_interrupts.h
+++ b/include/sound/sdca_interrupts.h
@@ -33,6 +33,7 @@ struct sdca_function_data;
  * @priv: Pointer to private data for use by the handler.
  * @irq: IRQ number allocated to this interrupt, also used internally to track
  * the IRQ being assigned.
+ * @early_request: Flag to indicate this IRQ was requested at bus probe time.
  */
 struct sdca_interrupt {
        const char *name;
@@ -48,6 +49,7 @@ struct sdca_interrupt {
        void *priv;
 
        int irq;
+       bool early_request;
 };
 
 /**
@@ -86,6 +88,9 @@ int sdca_irq_populate(struct sdca_function_data *function,
 void sdca_irq_cleanup(struct device *dev,
                      struct sdca_function_data *function,
                      struct sdca_interrupt_info *info);
+void sdca_irq_cleanup_late(struct device *dev,
+                          struct sdca_function_data *function,
+                          struct sdca_interrupt_info *info);
 
 struct sdca_interrupt_info *devm_sdca_irq_allocate(struct device *dev,
                                                   struct regmap *regmap, int irq);

diff --git a/sound/soc/sdca/sdca_class_function.c b/sound/soc/sdca/sdca_class_function.c
index 1496a15f7d2ac7d5e22ab3605be9af6012931c10..5ae6c727c796d3460d18ff8d75664de0d2d1c5e9 100644 (file)
--- a/sound/soc/sdca/sdca_class_function.c
+++ b/sound/soc/sdca/sdca_class_function.c
@@ -408,7 +408,7 @@ static void class_function_remove(struct auxiliary_device *auxdev)
 {
        struct class_function_drv *drv = auxiliary_get_drvdata(auxdev);
 
-       sdca_irq_cleanup(drv->dev, drv->function, drv->core->irq_info);
+       sdca_irq_cleanup_late(drv->dev, drv->function, drv->core->irq_info);
 }
 
 static int class_function_runtime_suspend(struct device *dev)

diff --git a/sound/soc/sdca/sdca_interrupts.c b/sound/soc/sdca/sdca_interrupts.c
index 1e4efc0609d93f128206587a7e95476202238889..d86884a89c4059d92ecdfe5931d360ce141b67f2 100644 (file)
--- a/sound/soc/sdca/sdca_interrupts.c
+++ b/sound/soc/sdca/sdca_interrupts.c
@@ -456,6 +456,8 @@ int sdca_irq_populate_early(struct device *dev, struct regmap *regmap,
                                if (ret)
                                        return ret;
 
+                               interrupt->early_request = true;
+
                                ret = sdca_fdl_alloc_state(interrupt);
                                if (ret)
                                        return ret;
@@ -562,17 +564,10 @@ int sdca_irq_populate(struct sdca_function_data *function,
 }
 EXPORT_SYMBOL_NS_GPL(sdca_irq_populate, "SND_SOC_SDCA");
 
-/**
- * sdca_irq_cleanup - Free all the individual IRQs for an SDCA Function
- * @dev: Device pointer against which the sdca_interrupt_info was allocated.
- * @function: Pointer to the SDCA Function.
- * @info: Pointer to the SDCA interrupt info for this device.
- *
- * Typically this would be called from the driver for a single SDCA Function.
- */
-void sdca_irq_cleanup(struct device *dev,
-                     struct sdca_function_data *function,
-                     struct sdca_interrupt_info *info)
+static void sdca_irq_cleanup_flags(struct device *dev,
+                                  struct sdca_function_data *function,
+                                  struct sdca_interrupt_info *info,
+                                  bool late_cleanup)
 {
        int i;
 
@@ -584,13 +579,49 @@ void sdca_irq_cleanup(struct device *dev,
                if (interrupt->function != function || !interrupt->irq)
                        continue;
 
+               if (interrupt->early_request && !late_cleanup)
+                       continue;
+
                sdca_irq_free_locked(dev, info, i, interrupt->name, interrupt);
 
                kfree(interrupt->name);
        }
 }
+
+/**
+ * sdca_irq_cleanup - Free the regular IRQs for an SDCA Function
+ * @dev: Device pointer against which the sdca_interrupt_info was allocated.
+ * @function: Pointer to the SDCA Function.
+ * @info: Pointer to the SDCA interrupt info for this device.
+ *
+ * Typically this would be called from the driver for a single SDCA Function
+ * from component remove.
+ */
+void sdca_irq_cleanup(struct device *dev,
+                     struct sdca_function_data *function,
+                     struct sdca_interrupt_info *info)
+{
+       sdca_irq_cleanup_flags(dev, function, info, false);
+}
 EXPORT_SYMBOL_NS_GPL(sdca_irq_cleanup, "SND_SOC_SDCA");
 
+/**
+ * sdca_irq_cleanup_late - Free the early IRQs for an SDCA Function
+ * @dev: Device pointer against which the sdca_interrupt_info was allocated.
+ * @function: Pointer to the SDCA Function.
+ * @info: Pointer to the SDCA interrupt info for this device.
+ *
+ * Typically this would be called from the driver for a single SDCA Function
+ * from bus remove.
+ */
+void sdca_irq_cleanup_late(struct device *dev,
+                          struct sdca_function_data *function,
+                          struct sdca_interrupt_info *info)
+{
+       sdca_irq_cleanup_flags(dev, function, info, true);
+}
+EXPORT_SYMBOL_NS_GPL(sdca_irq_cleanup_late, "SND_SOC_SDCA");
+
 /**
  * devm_sdca_irq_allocate - allocate an SDCA interrupt structure for a device
  * @sdev: Device pointer against which things should be allocated.