mbox series

[0/6] Small fixes

Message ID 20190827141712.21015-1-amadeuszx.slawinski@linux.intel.com (mailing list archive)
Headers show
Series Small fixes | expand

Message

Amadeusz Sławiński Aug. 27, 2019, 2:17 p.m. UTC 
Series of small fixes:
* fixes few issues found during checking code with static checkers
* fix few prints
* define function in header, like it should be
* release topology when done with it

Amadeusz Sławiński (6):
  ASoC: Intel: Skylake: Use correct function to access iomem space
  ASoC: Intel: Fix use of potentially uninitialized variable
  ASoC: dapm: Expose snd_soc_dapm_new_control_unlocked properly
  ASoC: Intel: Skylake: Print module type instead of id
  ASoC: Intel: Skylake: Release topology when we are done with it
  ASoC: Intel: NHLT: Fix debug print format

 include/sound/soc-dapm.h               |  3 +++
 sound/soc/intel/common/sst-ipc.c       |  2 ++
 sound/soc/intel/skylake/skl-debug.c    |  2 +-
 sound/soc/intel/skylake/skl-messages.c |  5 +++--
 sound/soc/intel/skylake/skl-nhlt.c     |  2 +-
 sound/soc/intel/skylake/skl-topology.c | 20 ++++++++++----------
 sound/soc/intel/skylake/skl.h          |  1 -
 sound/soc/soc-topology.c               |  6 ------
 8 files changed, 20 insertions(+), 21 deletions(-)

Comments

Pierre-Louis Bossart Aug. 27, 2019, 2:58 p.m. UTC | #1 
On 8/27/19 9:17 AM, Amadeusz Sławiński wrote:
> Series of small fixes:
> * fixes few issues found during checking code with static checkers
> * fix few prints
> * define function in header, like it should be
> * release topology when done with it
> 
> Amadeusz Sławiński (6):
>    ASoC: Intel: Skylake: Use correct function to access iomem space
>    ASoC: Intel: Fix use of potentially uninitialized variable
>    ASoC: dapm: Expose snd_soc_dapm_new_control_unlocked properly
>    ASoC: Intel: Skylake: Print module type instead of id
>    ASoC: Intel: Skylake: Release topology when we are done with it
>    ASoC: Intel: NHLT: Fix debug print format

LGTM

Reviewed-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>

> 
>   include/sound/soc-dapm.h               |  3 +++
>   sound/soc/intel/common/sst-ipc.c       |  2 ++
>   sound/soc/intel/skylake/skl-debug.c    |  2 +-
>   sound/soc/intel/skylake/skl-messages.c |  5 +++--
>   sound/soc/intel/skylake/skl-nhlt.c     |  2 +-
>   sound/soc/intel/skylake/skl-topology.c | 20 ++++++++++----------
>   sound/soc/intel/skylake/skl.h          |  1 -
>   sound/soc/soc-topology.c               |  6 ------
>   8 files changed, 20 insertions(+), 21 deletions(-)
>