ASoC: snd_soc_component_driver has pmdown_time
authorKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Wed, 11 Oct 2017 01:38:08 +0000 (01:38 +0000)
committerMark Brown <broonie@kernel.org>
Mon, 23 Oct 2017 09:28:02 +0000 (11:28 +0200)
Current snd_soc_runtime_ignore_pmdown_time() tallys all Codec and
CPU's "ignore_pmdown_time". Now, CPU (= via compoent)
ignore_pmdown_time is fixed as "true". Codec's one is copied from Codec
driver. This means Codec side default is "false".

Current all Codec driver will be replaced into Component, thus, we can
use for_each_rtdcom() for this totalization. This patch adds new
"pmdown_time" on Component driver. Its inverted value will be used
for this "ignore" totalizaton.

Of course all existing Component driver doesn't have its settings now,
thus, all existing "pmdown_time" is "false". This means all
Components will ignore pmdown time. This is current CPU behavior.
To keep compatibility, snd_soc_runtime_ignore_pmdown_time() totalize
Component's inverted "pmdown_time" (= total will be true) and
Codec's "ignore_pmdown_time" (= depends on Codec driver settings).
Because It is using AND operation, its result is based on Codec driver
settings only.
This means this operation can keep compatibility and doesn't have
nonconformity.

When we replace Codec to Component, the driver which has
".ignore_pmdown_time = true" will be just removed,
and the driver which doesn't have it will have new
".pmdown_time = true".

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
include/sound/soc.h
sound/soc/soc-pcm.c

index ae0a27fdc01684ba3e0c3d35242c576e29940b10..44fab951b686b57ca45582568a1d970f11169f9a 100644 (file)
@@ -836,6 +836,7 @@ struct snd_soc_component_driver {
        /* bits */
        unsigned int idle_bias_on:1;
        unsigned int suspend_bias_off:1;
        /* bits */
        unsigned int idle_bias_on:1;
        unsigned int suspend_bias_off:1;
+       unsigned int pmdown_time:1; /* care pmdown_time at stop */
 };
 
 struct snd_soc_component {
 };
 
 struct snd_soc_component {
index daaa670ee9b767bca5e305a1f85232912b55a951..8075856668c23b10e0f16eebb78cb59b8c7a233a 100644 (file)
@@ -133,16 +133,25 @@ void snd_soc_runtime_deactivate(struct snd_soc_pcm_runtime *rtd, int stream)
  */
 bool snd_soc_runtime_ignore_pmdown_time(struct snd_soc_pcm_runtime *rtd)
 {
  */
 bool snd_soc_runtime_ignore_pmdown_time(struct snd_soc_pcm_runtime *rtd)
 {
+       struct snd_soc_rtdcom_list *rtdcom;
+       struct snd_soc_component *component;
        int i;
        bool ignore = true;
 
        if (!rtd->pmdown_time || rtd->dai_link->ignore_pmdown_time)
                return true;
 
        int i;
        bool ignore = true;
 
        if (!rtd->pmdown_time || rtd->dai_link->ignore_pmdown_time)
                return true;
 
+       for_each_rtdcom(rtd, rtdcom) {
+               component = rtdcom->component;
+
+               ignore &= !component->driver->pmdown_time;
+       }
+
+       /* this will be removed */
        for (i = 0; i < rtd->num_codecs; i++)
                ignore &= rtd->codec_dais[i]->component->ignore_pmdown_time;
 
        for (i = 0; i < rtd->num_codecs; i++)
                ignore &= rtd->codec_dais[i]->component->ignore_pmdown_time;
 
-       return rtd->cpu_dai->component->ignore_pmdown_time && ignore;
+       return ignore;
 }
 
 /**
 }
 
 /**