]> www.infradead.org Git - users/willy/xarray.git/commitdiff
ASoC: remove unused substream in macro soc_component_mark_pop
authorHongbo Li <lihongbo22@huawei.com>
Wed, 21 Aug 2024 07:08:12 +0000 (15:08 +0800)
committerMark Brown <broonie@kernel.org>
Sun, 29 Sep 2024 23:09:52 +0000 (01:09 +0200)
The soc_component_mark_pop don't need substream, and also
substream is not used in this macro, so we can remove it.
This is detected by macro_checker.py script.

Signed-off-by: Hongbo Li <lihongbo22@huawei.com>
Link: https://patch.msgid.link/20240821070815.2326534-3-lihongbo22@huawei.com
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/soc-component.c

index b3d7bb91e29497f7a756d0f98c068d8f13e631b5..b67ef78f405c3be7f118e4c355e8fcf990e19e6c 100644 (file)
@@ -58,7 +58,7 @@ static inline int soc_component_field_shift(struct snd_soc_component *component,
  * In such case, we can update these macros.
  */
 #define soc_component_mark_push(component, substream, tgt)     ((component)->mark_##tgt = substream)
-#define soc_component_mark_pop(component, substream, tgt)      ((component)->mark_##tgt = NULL)
+#define soc_component_mark_pop(component, tgt) ((component)->mark_##tgt = NULL)
 #define soc_component_mark_match(component, substream, tgt)    ((component)->mark_##tgt == substream)
 
 void snd_soc_component_set_aux(struct snd_soc_component *component,
@@ -339,7 +339,7 @@ void snd_soc_component_module_put(struct snd_soc_component *component,
                module_put(component->dev->driver->owner);
 
        /* remove the mark from module */
-       soc_component_mark_pop(component, mark, module);
+       soc_component_mark_pop(component, module);
 }
 
 int snd_soc_component_open(struct snd_soc_component *component,
@@ -370,7 +370,7 @@ int snd_soc_component_close(struct snd_soc_component *component,
                ret = component->driver->close(component, substream);
 
        /* remove marked substream */
-       soc_component_mark_pop(component, substream, open);
+       soc_component_mark_pop(component, open);
 
        return soc_component_ret(component, ret);
 }
@@ -515,7 +515,7 @@ void snd_soc_component_compr_free(struct snd_soc_component *component,
                component->driver->compress_ops->free(component, cstream);
 
        /* remove marked substream */
-       soc_component_mark_pop(component, cstream, compr_open);
+       soc_component_mark_pop(component, compr_open);
 }
 EXPORT_SYMBOL_GPL(snd_soc_component_compr_free);
 
@@ -1210,7 +1210,7 @@ void snd_soc_pcm_component_hw_free(struct snd_pcm_substream *substream,
                }
 
                /* remove marked substream */
-               soc_component_mark_pop(component, substream, hw_params);
+               soc_component_mark_pop(component, hw_params);
        }
 }
 
@@ -1254,7 +1254,7 @@ int snd_soc_pcm_component_trigger(struct snd_pcm_substream *substream,
                        r = soc_component_trigger(component, substream, cmd);
                        if (r < 0)
                                ret = r; /* use last ret */
-                       soc_component_mark_pop(component, substream, trigger);
+                       soc_component_mark_pop(component, trigger);
                }
        }
 
@@ -1294,7 +1294,7 @@ void snd_soc_pcm_component_pm_runtime_put(struct snd_soc_pcm_runtime *rtd,
                pm_runtime_put_autosuspend(component->dev);
 
                /* remove marked stream */
-               soc_component_mark_pop(component, stream, pm);
+               soc_component_mark_pop(component, pm);
        }
 }