1. 30 May, 2020 14 commits
  2. 29 May, 2020 16 commits
  3. 28 May, 2020 5 commits
  4. 27 May, 2020 5 commits
    • Mark Brown's avatar
      Merge series "ASoC: SOF: finalize Baytrail/CherryTrail support" from... · 93e0f69a
      Mark Brown authored
      Merge series "ASoC: SOF: finalize Baytrail/CherryTrail support" from Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>:
      
      This long-overdue patchset adds missing system suspend/resume support
      and hardens the IPC to solve module load/unload issues on specific
      devices such as Cyan Chromebook. With this series SOF is finally
      iso-feature with the legacy driver. Thanks to Ranjani and Keyon for
      the basic patches and Enric for testing.
      
      The last part needed for Baytrail/Cherrytrail is simplification of the
      driver/card names and turn-key UCM support (on-going work with
      Jaroslav).
      
      Keyon Jie (1):
        ASoC: SOF: Intel: BYT: harden IPC initialization and handling
      
      Pierre-Louis Bossart (3):
        ASoC: Intel: bytcr_rt5640/51: remove .ignore_suspend
        ASoC: Intel: byt/cht: add .pm_ops
        ASoC: SOF: Intel: BYT: mask BUSY or DONE interrupts in handler
      
      Ranjani Sridharan (4):
        ASoC: SOF: Intel: byt: Add PM callbacks
        ASoC: SOF: pm: handle resume on legacy Intel platforms
        ASoC: SOF: ipc: ignore DSP replies received when they are not expected
        ASoC: SOF: Intel: BYT: add .remove op
      
       sound/soc/intel/boards/bytcht_cx2072x.c      |   3 +
       sound/soc/intel/boards/bytcht_da7213.c       |   3 +
       sound/soc/intel/boards/bytcht_es8316.c       |   3 +
       sound/soc/intel/boards/bytcr_rt5640.c        |   7 +-
       sound/soc/intel/boards/bytcr_rt5651.c        |   6 +-
       sound/soc/intel/boards/cht_bsw_max98090_ti.c |   3 +
       sound/soc/intel/boards/cht_bsw_nau8824.c     |   3 +
       sound/soc/intel/boards/cht_bsw_rt5645.c      |   3 +
       sound/soc/intel/boards/cht_bsw_rt5672.c      |   3 +
       sound/soc/sof/intel/byt.c                    | 116 ++++++++++++++-----
       sound/soc/sof/ipc.c                          |   9 +-
       sound/soc/sof/pm.c                           |   8 +-
       sound/soc/sof/sof-priv.h                     |   2 +-
       13 files changed, 122 insertions(+), 47 deletions(-)
      
      base-commit: 0d71a5cf
      --
      2.20.1
      93e0f69a
    • Wei Yongjun's avatar
      ASoC: mmp-sspa: Fix return value check in asoc_mmp_sspa_probe() · 18545763
      Wei Yongjun authored
      In case of error, the function devm_ioremap() returns NULL pointer not
      ERR_PTR(). The IS_ERR() test in the return value check should be
      replaced with NULL test.
      Signed-off-by: default avatarWei Yongjun <weiyongjun1@huawei.com>
      Link: https://lore.kernel.org/r/20200527030210.124393-1-weiyongjun1@huawei.comSigned-off-by: default avatarMark Brown <broonie@kernel.org>
      18545763
    • Dan Murphy's avatar
      ASoC: tlv320adcx140: Fix warnings when using W=1 · 850ba84b
      Dan Murphy authored
      Fix the warnings when using the W=1 compiler flag.
      
      sound/soc/codecs/tlv320adcx140.c: In function ‘adcx140_reset’:
      sound/soc/codecs/tlv320adcx140.c:570:6: warning: variable ‘ret’ set but not used [-Wunused-but-set-variable]
        570 |  int ret = 0;
            |      ^~~
      
      This was set but only used in case where the reset GPIO is not defined.
      Have the function return the value of ret.
      
      sound/soc/codecs/tlv320adcx140.c: In function ‘adcx140_codec_probe’:
      sound/soc/codecs/tlv320adcx140.c:778:18: warning: comparison of unsigned expression < 0 is always false [-Wtype-limits]
        778 |  if (bias_source < ADCX140_MIC_BIAS_VAL_VREF ||
            |                  ^
      sound/soc/codecs/tlv320adcx140.c:789:18: warning: comparison of unsigned expression < 0 is always false [-Wtype-limits]
        789 |  if (vref_source < ADCX140_MIC_BIAS_VREF_275V ||
      
      This condition will not occur since if the dt property is not set then
      the *_source variable is set to the default value.  So there is no way
      that *_source can be less then 0.  Which is what each #define is set to.
      The code just needs to make sure that the dt property is not out of the
      upper bounds.
      Reported-by: default avatarkbuild test robot <lkp@intel.com>
      Signed-off-by: default avatarDan Murphy <dmurphy@ti.com>
      Link: https://lore.kernel.org/r/20200526175247.15309-1-dmurphy@ti.comSigned-off-by: default avatarMark Brown <broonie@kernel.org>
      850ba84b
    • Dinghao Liu's avatar
      ASoC: wm8962: Fix runtime PM imbalance on error · 920bef64
      Dinghao Liu authored
      pm_runtime_get_sync() increments the runtime PM usage counter even
      the call returns an error code. Thus a pairing decrement is needed
      on the error handling path to keep the counter balanced.
      Signed-off-by: default avatarDinghao Liu <dinghao.liu@zju.edu.cn>
      Acked-by: default avatarCharles Keepax <ckeepax@opensource.cirrus.com>
      Link: https://lore.kernel.org/r/20200527024625.9937-1-dinghao.liu@zju.edu.cnSigned-off-by: default avatarMark Brown <broonie@kernel.org>
      920bef64
    • Dinghao Liu's avatar
      ASoC: img-spdif-in: Fix runtime PM imbalance on error · 53865b32
      Dinghao Liu authored
      pm_runtime_get_sync() increments the runtime PM usage counter even
      the call returns an error code. Thus a pairing decrement is needed
      on the error handling path to keep the counter balanced.
      Signed-off-by: default avatarDinghao Liu <dinghao.liu@zju.edu.cn>
      Link: https://lore.kernel.org/r/20200527084326.4131-1-dinghao.liu@zju.edu.cnSigned-off-by: default avatarMark Brown <broonie@kernel.org>
      53865b32