[0/7,v2] ASoC: soc-core cleanup step8

Message

Kuninori Morimoto Dec. 18, 2019, 2:44 a.m. UTC

Hi Mark

These are v2 of soc-core cleanup step8.
There is no relationship between each patches.

 - [6/7] renamed function name
 - [7/7] new patch which cares SND_SOC_DAPM_STREAM_STOP

Kuninori Morimoto (7):
  1) ASoC: soc-core: remove snd_soc_rtdcom_list
  2) ASoC: soc-core: care .ignore_suspend for Component suspend
  3) ASoC: soc-core: remove duplicate pinctrl operation when suspend
  4) ASoC: soc-core: do pinctrl_pm_select_xxx() as component
  5) ASoC: soc-core: add snd_soc_close_delayed_work()
  6) ASoC: soc-dapm: add snd_soc_dapm_stream_stop()
  7) ASoC: use snd_soc_dapm_stream_stop() for SND_SOC_DAPM_STREAM_STOP

 include/sound/soc-dapm.h  |   1 +
 include/sound/soc.h       |  19 +++---
 sound/soc/soc-component.c |  33 +++++-----
 sound/soc/soc-compress.c  | 112 +++++++++------------------------
 sound/soc/soc-core.c      | 157 +++++++++++++++++++++++-----------------------
 sound/soc/soc-dapm.c      |  23 +++++++
 sound/soc/soc-pcm.c       | 115 ++++++++-------------------------
 7 files changed, 181 insertions(+), 279 deletions(-)

Comments

Ranjani Sridharan Dec. 18, 2019, 6:06 a.m. UTC | #1

On Wed, 2019-12-18 at 11:44 +0900, Kuninori Morimoto wrote:
> Hi Mark
> 
> These are v2 of soc-core cleanup step8.
> There is no relationship between each patches.
> 
>  - [6/7] renamed function name
>  - [7/7] new patch which cares SND_SOC_DAPM_STREAM_STOP
> 
> Kuninori Morimoto (7):
>   1) ASoC: soc-core: remove snd_soc_rtdcom_list
>   2) ASoC: soc-core: care .ignore_suspend for Component suspend
>   3) ASoC: soc-core: remove duplicate pinctrl operation when suspend
>   4) ASoC: soc-core: do pinctrl_pm_select_xxx() as component
>   5) ASoC: soc-core: add snd_soc_close_delayed_work()
>   6) ASoC: soc-dapm: add snd_soc_dapm_stream_stop()
>   7) ASoC: use snd_soc_dapm_stream_stop() for
Thanks, Morimoto-San. Other than the Patch 2 in the series, all others
look good. The patch itself looks OK I'm just not sure if the direction
is correct.

Reviewed-By: Ranjani Sridharan <[e-mail]>

Kuninori Morimoto Dec. 19, 2019, 5:57 a.m. UTC | #2

Hi Mark

I noticed that this patchset is not enough for latest for-5.6 branch.
I will post v3 for latest for-5.6.

> These are v2 of soc-core cleanup step8.
> There is no relationship between each patches.
> 
>  - [6/7] renamed function name
>  - [7/7] new patch which cares SND_SOC_DAPM_STREAM_STOP
> 
> Kuninori Morimoto (7):
>   1) ASoC: soc-core: remove snd_soc_rtdcom_list
>   2) ASoC: soc-core: care .ignore_suspend for Component suspend
>   3) ASoC: soc-core: remove duplicate pinctrl operation when suspend
>   4) ASoC: soc-core: do pinctrl_pm_select_xxx() as component
>   5) ASoC: soc-core: add snd_soc_close_delayed_work()
>   6) ASoC: soc-dapm: add snd_soc_dapm_stream_stop()
>   7) ASoC: use snd_soc_dapm_stream_stop() for SND_SOC_DAPM_STREAM_STOP
> 
>  include/sound/soc-dapm.h  |   1 +
>  include/sound/soc.h       |  19 +++---
>  sound/soc/soc-component.c |  33 +++++-----
>  sound/soc/soc-compress.c  | 112 +++++++++------------------------
>  sound/soc/soc-core.c      | 157 +++++++++++++++++++++++-----------------------
>  sound/soc/soc-dapm.c      |  23 +++++++
>  sound/soc/soc-pcm.c       | 115 ++++++++-------------------------
>  7 files changed, 181 insertions(+), 279 deletions(-)
> 
> -- 
> 2.7.4
>