Skip to content
  1. Mar 25, 2021
  2. Mar 24, 2021
  3. Mar 22, 2021
  4. Mar 20, 2021
    • Mark Brown's avatar
      Merge series "ASoC: tidyup error message timing" from Kuninori Morimoto... · 3722e4ec
      Mark Brown authored
      Merge series "ASoC: tidyup error message timing" from Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>:
      
      Hi Mark
      
      Indicating error message when failed case is very useful for debuging.
      In many case, it uses below style.
      
      	int function(...)
      	{
      		...
      		return ret;
      	}
      
      	int caller(...)
      	{
      		...
      		ret = function(...);
      		if (ret < 0)
      			dev_err(...)
      		...
      	}
      
      This is not so bad, but in this style *each caller* needs to indicate
      duplicate same error message, and some caller is forgetting to do it.
      And caller can't indicate detail function() error information.
      
      I know many people have many opinion, but if function() indicates error
      message, we can get same and detail information without forgot, and it is better.
      This patch-set tidyup to do it.
      
      	int function(...)
      	{
      		...
      		if (ret < 0)
      			dev_err(...)
      
      		return ret;
      	}
      
      	int caller(...)
      	{
      		...
      		ret = function(...);
      		...
      	}
      
      Kuninori Morimoto (14):
        ASoC: soc-pcm: indicate error message at soc_pcm_open()
        ASoC: soc-pcm: indicate error message at soc_pcm_hw_params()
        ASoC: soc-pcm: indicate error message at soc_pcm_prepare()
        ASoC: soc-pcm: indicate error message at dpcm_path_get()
        ASoC: soc-pcm: indicate error message at dpcm_be_dai_trigger()
        ASoC: soc-pcm: indicate error message at dpcm_apply_symmetry()
        ASoC: soc-pcm: indicate error message at dpcm_run_update_startup/shutdown()
        ASoC: soc-pcm: indicate error message at dpcm_fe/be_dai_startup()
        ASoC: soc-pcm: indicate error message at dpcm_fe/be_dai_hw_params()
        ASoC: soc-pcm: indicate error message at dpcm_fe/be_dai_prepare()
        ASoC: soc-pcm: don't indicate error message for soc_pcm_hw_free()
        ASoC: soc-pcm: don't indicate error message for dpcm_be_dai_hw_free()
        ASoC: don't indicate error message for snd_soc_[pcm_]dai_xxx()
        ASoC: don't indicate error message for snd_soc_[pcm_]component_xxx()
      
       include/sound/soc-dpcm.h |   2 +-
       sound/soc/soc-compress.c |   9 +-
       sound/soc/soc-core.c     |  22 +----
       sound/soc/soc-dapm.c     |  24 ++---
       sound/soc/soc-pcm.c      | 197 +++++++++++++++++++--------------------
       5 files changed, 108 insertions(+), 146 deletions(-)
      
      --
      2.25.1
      3722e4ec
  5. Mar 19, 2021
    • Wei Yongjun's avatar
      ASoC: rt1019: make symbol 'rt1019_i2c_driver' static · e6d8af66
      Wei Yongjun authored
      The sparse tool complains as follows:
      
      sound/soc/codecs/rt1019.c:927:19: warning:
       symbol 'rt1019_i2c_driver' was not declared. Should it be static?
      
      This symbol is not used outside of rt1019.c, so this
      commit marks it static.
      
      Fixes: 7ec79d38
      
       ("ASoC: rt1019: add rt1019 amplifier driver")
      Reported-by: default avatarHulk Robot <hulkci@huawei.com>
      Signed-off-by: default avatarWei Yongjun <weiyongjun1@huawei.com>
      Link: https://lore.kernel.org/r/20210319094102.4185096-1-weiyongjun1@huawei.com
      
      
      Signed-off-by: default avatarMark Brown <broonie@kernel.org>
      e6d8af66
    • Kuninori Morimoto's avatar
      ASoC: don't indicate error message for snd_soc_[pcm_]component_xxx() · 60adbd8f
      Kuninori Morimoto authored
      
      
      All snd_soc_component_xxx() and snd_soc_pcm_component_xxx() itself
      indicate error message if failed.
      Its caller doesn't need to indicate duplicated error message.
      This patch removes it.
      
      All snd_soc_component_xxx() indicate error message if failed.
      Its caller doesn't need to indicate duplicated error message.
      This patch removes it.
      
      Signed-off-by: default avatarKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
      Link: https://lore.kernel.org/r/878s6puta6.wl-kuninori.morimoto.gx@renesas.com
      
      
      Signed-off-by: default avatarMark Brown <broonie@kernel.org>
      60adbd8f
    • Kuninori Morimoto's avatar
      ASoC: don't indicate error message for snd_soc_[pcm_]dai_xxx() · 62462e01
      Kuninori Morimoto authored
      
      
      All snd_soc_dai_xxx() and snd_soc_pcm_dai_xxx() itself
      indicate error message if failed.
      Its caller doesn't need to indicate duplicated error message.
      This patch removes it.
      
      Signed-off-by: default avatarKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
      Link: https://lore.kernel.org/r/87a6r5utaa.wl-kuninori.morimoto.gx@renesas.com
      
      
      Signed-off-by: default avatarMark Brown <broonie@kernel.org>
      62462e01
    • Kuninori Morimoto's avatar
    • Kuninori Morimoto's avatar
      ASoC: soc-pcm: don't indicate error message for soc_pcm_hw_free() · e20c9c4f
      Kuninori Morimoto authored
      
      
      soc_pcm_hw_free() never fail, error message is not needed.
      We can't use void function for it, because it is used
      part of struct snd_pcm_ops :: hw_free.
      
      Signed-off-by: default avatarKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
      Link: https://lore.kernel.org/r/87czw1utaj.wl-kuninori.morimoto.gx@renesas.com
      
      
      Signed-off-by: default avatarMark Brown <broonie@kernel.org>
      e20c9c4f
    • Kuninori Morimoto's avatar
      ASoC: soc-pcm: indicate error message at dpcm_fe/be_dai_prepare() · 273db971
      Kuninori Morimoto authored
      
      
      Indicating error message when failed case is very useful for debuging.
      In many case, its style is like below.
      
      	int function(...)
      	{
      		...
      		return ret;
      	}
      
      	int caller(...)
      	{
      		...
      		ret = function(...);
      		if (ret < 0)
      			dev_err(...)
      		...
      	}
      
      This is not so bad, but in this style *each caller* needs to indicate
      duplicate same error message, and some caller is forgetting to do it.
      And caller can't indicate detail function() error information.
      
      If function() indicates error message, we can get same and
      detail information without forgot.
      
      	int function(...)
      	{
      		...
      		if (ret < 0)
      			dev_err(...)
      
      		return ret;
      	}
      
      	int caller(...)
      	{
      		...
      		ret = function(...);
      		...
      	}
      
      This patch follow above style at dpcm_fe/be_dai_prepare()
      
      Signed-off-by: default avatarKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
      Link: https://lore.kernel.org/r/87eeghutap.wl-kuninori.morimoto.gx@renesas.com
      
      
      Signed-off-by: default avatarMark Brown <broonie@kernel.org>
      273db971
    • Kuninori Morimoto's avatar
      ASoC: soc-pcm: indicate error message at dpcm_fe/be_dai_hw_params() · 33b6b94f
      Kuninori Morimoto authored
      
      
      Indicating error message when failed case is very useful for debuging.
      In many case, its style is like below.
      
      	int function(...)
      	{
      		...
      		return ret;
      	}
      
      	int caller(...)
      	{
      		...
      		ret = function(...);
      		if (ret < 0)
      			dev_err(...)
      		...
      	}
      
      This is not so bad, but in this style *each caller* needs to indicate
      duplicate same error message, and some caller is forgetting to do it.
      And caller can't indicate detail function() error information.
      
      If function() indicates error message, we can get same and
      detail information without forgot.
      
      	int function(...)
      	{
      		...
      		if (ret < 0)
      			dev_err(...)
      
      		return ret;
      	}
      
      	int caller(...)
      	{
      		...
      		ret = function(...);
      		...
      	}
      
      This patch follow above style at dpcm_fe/be_dai_hw_params()
      
      Signed-off-by: default avatarKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
      Link: https://lore.kernel.org/r/87ft0xutat.wl-kuninori.morimoto.gx@renesas.com
      
      
      Signed-off-by: default avatarMark Brown <broonie@kernel.org>
      33b6b94f
    • Kuninori Morimoto's avatar
      ASoC: soc-pcm: indicate error message at dpcm_fe/be_dai_startup() · 06aaeb87
      Kuninori Morimoto authored
      
      
      Indicating error message when failed case is very useful for debuging.
      In many case, its style is like below.
      
      	int function(...)
      	{
      		...
      		return ret;
      	}
      
      	int caller(...)
      	{
      		...
      		ret = function(...);
      		if (ret < 0)
      			dev_err(...)
      		...
      	}
      
      This is not so bad, but in this style *each caller* needs to indicate
      duplicate same error message, and some caller is forgetting to do it.
      And caller can't indicate detail function() error information.
      
      If function() indicates error message, we can get same and
      detail information without forgot.
      
      	int function(...)
      	{
      		...
      		if (ret < 0)
      			dev_err(...)
      
      		return ret;
      	}
      
      	int caller(...)
      	{
      		...
      		ret = function(...);
      		...
      	}
      
      This patch follow above style at dpcm_fe/be_dai_startup().
      
      Signed-off-by: default avatarKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
      Link: https://lore.kernel.org/r/87h7ldutay.wl-kuninori.morimoto.gx@renesas.com
      
      
      Signed-off-by: default avatarMark Brown <broonie@kernel.org>
      06aaeb87
    • Kuninori Morimoto's avatar
      ASoC: soc-pcm: indicate error message at dpcm_run_update_startup/shutdown() · 81c82a9e
      Kuninori Morimoto authored
      
      
      Indicating error message when failed case is very useful for debuging.
      In many case, its style is like below.
      
      	int function(...)
      	{
      		...
      		return ret;
      	}
      
      	int caller(...)
      	{
      		...
      		ret = function(...);
      		if (ret < 0)
      			dev_err(...)
      		...
      	}
      
      This is not so bad, but in this style *each caller* needs to indicate
      duplicate same error message, and some caller is forgetting to do it.
      And caller can't indicate detail function() error information.
      
      If function() indicates error message, we can get same and
      detail information without forgot.
      
      	int function(...)
      	{
      		...
      		if (ret < 0)
      			dev_err(...)
      
      		return ret;
      	}
      
      	int caller(...)
      	{
      		...
      		ret = function(...);
      		...
      	}
      
      This patch also
      do below to dpcm_run_update_startup()
      	1) remove duplicated ret = -EINVAL
      	2) remove blank line
      do below to dpcm_run_update_shutdown()
      	1) remove unused ret
      
      Signed-off-by: default avatarKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
      Link: https://lore.kernel.org/r/87im5tutb3.wl-kuninori.morimoto.gx@renesas.com
      
      
      Signed-off-by: default avatarMark Brown <broonie@kernel.org>
      81c82a9e
    • Kuninori Morimoto's avatar
      ASoC: soc-pcm: indicate error message at dpcm_apply_symmetry() · bbd2bac8
      Kuninori Morimoto authored
      
      
      Indicating error message when failed case is very useful for debuging.
      In many case, its style is like below.
      
      	int function(...)
      	{
      		...
      		return ret;
      	}
      
      	int caller(...)
      	{
      		...
      		ret = function(...);
      		if (ret < 0)
      			dev_err(...)
      		...
      	}
      
      This is not so bad, but in this style *each caller* needs to indicate
      duplicate same error message, and some caller is forgetting to do it.
      And caller can't indicate detail function() error information.
      
      If function() indicates error message, we can get same and
      detail information without forgot.
      
      	int function(...)
      	{
      		...
      		if (ret < 0)
      			dev_err(...)
      
      		return ret;
      	}
      
      	int caller(...)
      	{
      		...
      		ret = function(...);
      		...
      	}
      
      This patch follow above style at dpcm_apply_symmetry(...)
      
      Signed-off-by: default avatarKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
      Link: https://lore.kernel.org/r/87k0q9utb9.wl-kuninori.morimoto.gx@renesas.com
      
      
      Signed-off-by: default avatarMark Brown <broonie@kernel.org>
      bbd2bac8
    • Kuninori Morimoto's avatar
      ASoC: soc-pcm: indicate error message at dpcm_be_dai_trigger() · db3aa39c
      Kuninori Morimoto authored
      
      
      Indicating error message when failed case is very useful for debuging.
      In many case, its style is like below.
      
      	int function(...)
      	{
      		...
      		return ret;
      	}
      
      	int caller(...)
      	{
      		...
      		ret = function(...);
      		if (ret < 0)
      			dev_err(...)
      		...
      	}
      
      This is not so bad, but in this style *each caller* needs to indicate
      duplicate same error message, and some caller is forgetting to do it.
      And caller can't indicate detail function() error information.
      
      If function() indicates error message, we can get same and
      detail information without forgot.
      
      	int function(...)
      	{
      		...
      		if (ret < 0)
      			dev_err(...)
      
      		return ret;
      	}
      
      	int caller(...)
      	{
      		...
      		ret = function(...);
      		...
      	}
      
      Now, dpcm_be_dai_trigger() user uses it like below.
      
      	err = dpcm_be_dai_trigger(...);
      	if (err < 0)
      		dev_err(..., "ASoC: trigger FE failed %d\n", err);
      
      But we can get more detail information if dpcm_be_dai_trigger() itself
      had dev_err(). And above error message is confusable,
      failed is *BE*, not *FE*.
      
      This patch indicates error message at dpcm_be_dai_trigger().
      
      Signed-off-by: default avatarKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
      Link: https://lore.kernel.org/r/87lfaputbe.wl-kuninori.morimoto.gx@renesas.com
      
      
      Signed-off-by: default avatarMark Brown <broonie@kernel.org>
      db3aa39c
    • Kuninori Morimoto's avatar
      ASoC: soc-pcm: indicate error message at dpcm_path_get() · d479f00b
      Kuninori Morimoto authored
      
      
      Indicating error message when failed case is very useful for debuging.
      In many case, its style is like below.
      
      	int function(...)
      	{
      		...
      		return ret;
      	}
      
      	int caller(...)
      	{
      		...
      		ret = function(...);
      		if (ret < 0)
      			dev_err(...)
      		...
      	}
      
      This is not so bad, but in this style *each caller* needs to indicate
      duplicate same error message, and some caller is forgetting to do it.
      And caller can't indicate detail function() error information.
      
      If function() indicates error message, we can get same and
      detail information without forgot.
      
      	int function(...)
      	{
      		...
      		if (ret < 0)
      			dev_err(...)
      
      		return ret;
      	}
      
      	int caller(...)
      	{
      		...
      		ret = function(...);
      		...
      	}
      
      Now, many place uses dpcm_path_get() like below
      
      	ret = dpcm_path_get(...);
      	if (ret < 0)
      		goto error;
      (A)	else if (ret == 0)
      		dev_dbg(...)
      
      But here, (A) part can be indicated at dpcm_path_get() not caller.
      It is simple and readable code.
      
      This patch do it.
      Small detail behaviors will be exchanged by this patch.
      
      	1) indicates debug info (= path numbers) if path > 0 case only
      	   (It was *always* indicated).
      	2) soc_dpcm_fe_runtime_update() is indicating error message
      	   for paths < 0 case, but it is already done at dpcm_path_get().
      	   Thus just remove it. but dev_dbg() vs dev_warn() is exchanged.
      
      Signed-off-by: default avatarKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
      Link: https://lore.kernel.org/r/87mtv5utbj.wl-kuninori.morimoto.gx@renesas.com
      
      
      Signed-off-by: default avatarMark Brown <broonie@kernel.org>
      d479f00b
    • Kuninori Morimoto's avatar
      ASoC: soc-pcm: indicate error message at soc_pcm_prepare() · dab7eeb4
      Kuninori Morimoto authored
      
      
      Indicating error message when failed case is very useful for debuging.
      In many case, its style is like below.
      
      	int function(...)
      	{
      		...
      		return ret;
      	}
      
      	int caller(...)
      	{
      		...
      		ret = function(...);
      		if (ret < 0)
      			dev_err(...)
      		...
      	}
      
      This is not so bad, but in this style *each caller* needs to indicate
      duplicate same error message, and some caller is forgetting to do it.
      And caller can't indicate detail function() error information.
      
      If function() indicates error message, we can get same and
      detail information without forgot.
      
      	int function(...)
      	{
      		...
      		if (ret < 0)
      			dev_err(...)
      
      		return ret;
      	}
      
      	int caller(...)
      	{
      		...
      		ret = function(...);
      		...
      	}
      
      This patch follow above style at soc_pcm_prepare().
      
      By this patch, dpcm_fe/be_dai_prepare(...)
      temporary lacks FE/BE error info, but it will reborn soon.
      
      Signed-off-by: default avatarKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
      Link: https://lore.kernel.org/r/87o8flutbn.wl-kuninori.morimoto.gx@renesas.com
      
      
      Signed-off-by: default avatarMark Brown <broonie@kernel.org>
      dab7eeb4
    • Kuninori Morimoto's avatar
      ASoC: soc-pcm: indicate error message at soc_pcm_hw_params() · cb11f79b
      Kuninori Morimoto authored
      
      
      Indicating error message when failed case is very useful for debuging.
      In many case, its style is like below.
      
      	int function(...)
      	{
      		...
      		return ret;
      	}
      
      	int caller(...)
      	{
      		...
      		ret = function(...);
      		if (ret < 0)
      			dev_err(...)
      		...
      	}
      
      This is not so bad, but in this style *each caller* needs to indicate
      duplicate same error message, and some caller is forgetting to do it.
      And caller can't indicate detail function() error information.
      
      If function() indicates error message, we can get same and
      detail information without forgot.
      
      	int function(...)
      	{
      		...
      		if (ret < 0)
      			dev_err(...)
      
      		return ret;
      	}
      
      	int caller(...)
      	{
      		...
      		ret = function(...);
      		...
      	}
      
      This patch follow above style at soc_pcm_hw_params().
      
      By this patch, dpcm_fe/be_dai_hw_params(...)
      temporary lacks FE/BE error info, but it will reborn soon.
      
      Signed-off-by: default avatarKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
      Link: https://lore.kernel.org/r/87pn01utbt.wl-kuninori.morimoto.gx@renesas.com
      
      
      Signed-off-by: default avatarMark Brown <broonie@kernel.org>
      cb11f79b
    • Kuninori Morimoto's avatar
      ASoC: soc-pcm: indicate error message at soc_pcm_open() · e4b044f4
      Kuninori Morimoto authored
      
      
      Indicating error message when failed case is very useful for debuging.
      In many case, its style is like below.
      
      	int function(...)
      	{
      		...
      		return ret;
      	}
      
      	int caller(...)
      	{
      		...
      		ret = function(...);
      		if (ret < 0)
      			dev_err(...)
      		...
      	}
      
      This is not so bad, but in this style *each caller* needs to indicate
      duplicate same error message, and some caller is forgetting to do it.
      And caller can't indicate detail function() error information.
      
      If function() indicates error message, we can get same and
      detail information without forgot.
      
      	int function(...)
      	{
      		...
      		if (ret < 0)
      			dev_err(...)
      
      		return ret;
      	}
      
      	int caller(...)
      	{
      		...
      		ret = function(...);
      		...
      	}
      
      This patch follow above style at soc_pcm_open().
      
      By this patch, dpcm_fe/be_dai_startup(...)
      temporary lacks FE/BE error info, but it will reborn soon.
      
      Signed-off-by: default avatarKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
      Link: https://lore.kernel.org/r/87r1khutby.wl-kuninori.morimoto.gx@renesas.com
      
      
      Signed-off-by: default avatarMark Brown <broonie@kernel.org>
      e4b044f4
    • Mark Brown's avatar
      Merge series "ASoC: codecs: remove cppcheck warnings" from Pierre-Louis... · bb9faecc
      Mark Brown authored
      Merge series "ASoC: codecs: remove cppcheck warnings" from Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>:
      
      Lots of small fixes in various codec drivers that should have no
      functional impact.
      
      Pierre-Louis Bossart (23):
        ASoC: ab8500-codec: remove useless structure
        ASoC: ad1836: remove useless return
        ASoC: adau1977: remove useless return
        ASoC: cros_ec_codec: remove null pointer dereference warning
        ASoC: cx2070x: remove useless assignment
        ASoC: cx2070x: remove duplicate else branch
        ASoC: da7219-aad: remove useless initialization
        ASoC: hdac_hdmi: remove useless initializations
        ASoC: hdac_hdmi: align function arguments
        ASoC: hdmi-codec: remove useless initialization
        ASoC: hdmi-codec: remove unused spk_mask member
        ASoC: max98090: remove useless assignment
        ASoC: mt6358: remove useless initializations
        ASoC: mt6359: remove useless assignment
        ASoC: nau8825: remove useless assignment
        ASoC: pcm1681: remove useless assignment
        ASoC: sigmadsp: align function prototype
        ASoC: sti-sas: remove unused struct members
        ASoC: tas2562: remove useless assignment
        ASoC: tas2562: remove warning on return value
        ASoC: tas2770: remove useless initialization
        ASoC: tlv320dac33: clarify expression
        ASoC: tscs454: remove useless test on PLL disable
      
       sound/soc/codecs/ab8500-codec.c  |  7 -------
       sound/soc/codecs/ad1836.c        |  2 --
       sound/soc/codecs/adau1977.c      |  2 --
       sound/soc/codecs/cros_ec_codec.c |  2 +-
       sound/soc/codecs/cx2072x.c       | 11 ++++-------
       sound/soc/codecs/da7219-aad.c    |  2 +-
       sound/soc/codecs/hdac_hdmi.c     | 14 +++++++-------
       sound/soc/codecs/hdac_hdmi.h     |  2 +-
       sound/soc/codecs/hdmi-codec.c    |  3 +--
       sound/soc/codecs/max98090.c      |  2 +-
       sound/soc/codecs/mt6358.c        |  4 ++--
       sound/soc/codecs/mt6359.c        |  4 ++--
       sound/soc/codecs/nau8825.c       |  2 +-
       sound/soc/codecs/pcm1681.c       |  2 +-
       sound/soc/codecs/sigmadsp.h      |  2 +-
       sound/soc/codecs/sti-sas.c       |  3 ---
       sound/soc/codecs/tas2562.c       |  3 +--
       sound/soc/codecs/tas2770.c       |  2 +-
       sound/soc/codecs/tlv320dac33.c   |  2 +-
       sound/soc/codecs/tscs454.c       |  7 ++++++-
       20 files changed, 32 insertions(+), 46 deletions(-)
      
      --
      2.25.1
      bb9faecc
    • Mark Brown's avatar
      Merge series "Fix reset controls and RPM of NVIDIA Tegra ASoC drivers" from... · cdd23ae6
      Mark Brown authored
      Merge series "Fix reset controls and RPM of NVIDIA Tegra ASoC drivers" from Dmitry Osipenko <digetx@gmail.com>:
      
      Hi,
      
      This series adds missing hardware reset controls to I2S and AC97 drivers,
      corrects runtime PM usage and drivers probe/remove order. Currently drivers
      happen to work properly because reset is implicitly deasserted by tegra-clk
      driver, but clk driver shouldn't touch the resets and we need to fix it
      because this breaks other Tegra drivers. Previously we fixed the resets of
      the AHUB and HDMI codec drivers, but turned out that we missed the I2C and
      AC97 drivers.
      
      Thanks to Paul Fertser for testing the pending clk patches and finding
      that audio got broken on Tegra20 AC100 netbook because of the missing I2S
      reset.
      
      Changelog:
      
      v5: - After taking another look at the drivers I noticed couple more
            things that could be improved. These new patches correct runtime PM
            and probe/remove order of the drivers:
      
              ASoC: tegra20: spdif: Correct driver removal order
              ASoC: tegra20: spdif: Remove handing of disabled runtime PM
              ASoC: tegra20: i2s: Add system level suspend-resume callbacks
              ASoC: tegra20: i2s: Correct driver removal order
              ASoC: tegra20: i2s: Use devm_clk_get()
              ASoC: tegra20: i2s: Remove handing of disabled runtime PM
              ASoC: tegra30: i2s: Correct driver removal order
              ASoC: tegra30: i2s: Use devm_clk_get()
              ASoC: tegra30: i2s: Remove handing of disabled runtime PM
              ASoC: tegra30: ahub: Reset global variable
              ASoC: tegra30: ahub: Correct suspend-resume callbacks
              ASoC: tegra30: ahub: Remove handing of disabled runtime PM
      
      v4: - Added missing prototype for reset_control_bulk_put().
      
      v3: - Fixed reset stubs for !CONFIG_RESET_CONTROLLER.
      
      v2: - After some more testing I found that I2S control logic doesn't require
            I2S clock to be enabled for resetting. Hence it's fine to have I2S to
            be reset by parent AHUB driver, so I dropped "tegra30: i2s: Add reset
            control" patch.
      
          - While I was double-checking resets on Tegra30, I found that that
            Tegra30 I2S driver has a broken runtime PM which doesn't restore
            hardware state on resume and it's lost after AHUB RPM-resume.
            Thus, added this new patch "tegra30: i2s: Restore hardware state
            on runtime PM resume".
      
          - Added new patches which switch AHUB driver to use reset-bulk API.
            I took the RFC patch from Philipp Zabel, fixed it and added
            devm_reset_control_bulk_optional_get_exclusive_released() that
            will be useful for further Tegra GPU patches. This is a minor
            improvement which makes code cleaner.
      
      Dmitry Osipenko (16):
        ASoC: tegra20: ac97: Add reset control
        ASoC: tegra20: i2s: Add reset control
        ASoC: tegra30: i2s: Restore hardware state on runtime PM resume
        ASoC: tegra30: ahub: Switch to use reset-bulk API
        ASoC: tegra20: spdif: Correct driver removal order
        ASoC: tegra20: spdif: Remove handing of disabled runtime PM
        ASoC: tegra20: i2s: Add system level suspend-resume callbacks
        ASoC: tegra20: i2s: Correct driver removal order
        ASoC: tegra20: i2s: Use devm_clk_get()
        ASoC: tegra20: i2s: Remove handing of disabled runtime PM
        ASoC: tegra30: i2s: Correct driver removal order
        ASoC: tegra30: i2s: Use devm_clk_get()
        ASoC: tegra30: i2s: Remove handing of disabled runtime PM
        ASoC: tegra30: ahub: Reset global variable
        ASoC: tegra30: ahub: Correct suspend-resume callbacks
        ASoC: tegra30: ahub: Remove handing of disabled runtime PM
      
      Philipp Zabel (1):
        reset: Add reset_control_bulk API
      
       drivers/reset/core.c            | 215 ++++++++++++++++++++++
       include/linux/reset.h           | 315 ++++++++++++++++++++++++++++++++
       sound/soc/tegra/tegra20_ac97.c  |  21 +++
       sound/soc/tegra/tegra20_ac97.h  |   1 +
       sound/soc/tegra/tegra20_i2s.c   |  60 +++---
       sound/soc/tegra/tegra20_i2s.h   |   1 +
       sound/soc/tegra/tegra20_spdif.c |  16 +-
       sound/soc/tegra/tegra30_ahub.c  | 168 ++++++-----------
       sound/soc/tegra/tegra30_ahub.h  |   5 +-
       sound/soc/tegra/tegra30_i2s.c   |  65 ++-----
       10 files changed, 667 insertions(+), 200 deletions(-)
      
      --
      2.30.2
      
      base-commit: a38fd874
      cdd23ae6