Merge tag 'sound-4.13-rc1' of git://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound

Pull sound updates from Takashi Iwai:
 "This development cycle resulted in a fair amount of changes in both
  core and driver sides. The most significant change in ALSA core is
  about PCM. Also the support of of-graph card and the new DAPM widget
  for DSP are noteworthy changes in ASoC core. And there're lots of
  small changes splat over the tree, as you can see in diffstat.

  Below are a few highlights:

  ALSA core:
   - Removal of set_fs() hackery from PCM core stuff, and the code
     reorganization / optimization thereafter
   - Improved support of PCM ack ops, and a new ABI for improved
     control/status mmap handling
   - Lots of constifications in various codes

  ASoC core:
   - The support of of-graph card, which may work as a better generic
     device for a replacement of simple-card
   - New widget types intended mainly for use with DSPs

  ASoC drivers:
   - New drivers for Allwinner V3s SoCs
   - Ensonic ES8316 codec support
   - More Intel SKL and KBL works
   - More device support for Intel SST Atom (mostly for cheap tablets
     and 2-in-1 devices)
   - Support for Rockchip PDM controllers
   - Support for STM32 I2S and S/PDIF controllers
   - Support for ZTE AUD96P22 codecs

  HD-audio:
   - Support of new Realtek codecs (ALC215/ALC285/ALC289), more quirks
     for HP and Dell machines
   - A few more fixes for i915 component binding"

* tag 'sound-4.13-rc1' of git://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound: (418 commits)
  ALSA: hda - Fix unbalance of i915 module refcount
  ASoC: Intel: Skylake: Remove driver debugfs exit
  ASoC: Intel: Skylake: explicitly add the headers sst-dsp.h
  ALSA: hda/realtek - Remove GPIO_MASK
  ALSA: hda/realtek - Fix typo of pincfg for Dell quirk
  ALSA: pcm: add a documentation for tracepoints
  ALSA: atmel: ac97c: fix error return code in atmel_ac97c_probe()
  ALSA: x86: fix error return code in hdmi_lpe_audio_probe()
  ASoC: Intel: Skylake: Add support to read firmware registers
  ASoC: Intel: Skylake: Add sram address to sst_addr structure
  ASoC: Intel: Skylake: Debugfs facility to dump module config
  ASoC: Intel: Skylake: Add debugfs support
  ASoC: fix semicolon.cocci warnings
  ASoC: rt5645: Add quirk override by module option
  ASoC: rsnd: make arrays path and cmd_case static const
  ASoC: audio-graph-card: add widgets and routing for external amplifier support
  ASoC: audio-graph-card: update bindings for amplifier support
  ASoC: rt5665: calibration should be done before jack detection
  ASoC: rsnd: constify dev_pm_ops structures.
  ASoC: nau8825: change crosstalk-bypass property to bool type
  ...

1 files changed, 39 insertions, 65 deletions

diff --git a/sound/pci/hda/patch_hdmi.c b/sound/pci/hda/patch_hdmi.c
index 90e4ff87445e..76c85f08bea6 100644
--- a/sound/pci/hda/patch_hdmi.c
+++ b/sound/pci/hda/patch_hdmi.c
@@ -174,7 +174,6 @@ struct hdmi_spec {
         /* i915/powerwell (Haswell+/Valleyview+) specific */
         bool use_acomp_notifier; /* use i915 eld_notify callback for hotplug */
         struct i915_audio_component_audio_ops i915_audio_ops;
-        bool i915_bound; /* was i915 bound in this driver? */
 
         struct hdac_chmap chmap;
         hda_nid_t vendor_nid;
@@ -2234,8 +2233,6 @@ static void generic_spec_free(struct hda_codec *codec)
         struct hdmi_spec *spec = codec->spec;
 
         if (spec) {
-                if (spec->i915_bound)
-                        snd_hdac_i915_exit(&codec->bus->core);
                 hdmi_array_free(spec);
                 kfree(spec);
                 codec->spec = NULL;
@@ -2506,19 +2503,41 @@ static void i915_pin_cvt_fixup(struct hda_codec *codec,
         }
 }
 
-/* Intel Haswell and onwards; audio component with eld notifier */
-static int intel_hsw_common_init(struct hda_codec *codec, hda_nid_t vendor_nid)
+/* precondition and allocation for Intel codecs */
+static int alloc_intel_hdmi(struct hda_codec *codec)
 {
-        struct hdmi_spec *spec;
-        int err;
-
-        /* HSW+ requires i915 binding */
+        /* requires i915 binding */
         if (!codec->bus->core.audio_component) {
                 codec_info(codec, "No i915 binding for Intel HDMI/DP codec\n");
                 return -ENODEV;
         }
 
-        err = alloc_generic_hdmi(codec);
+        return alloc_generic_hdmi(codec);
+}
+
+/* parse and post-process for Intel codecs */
+static int parse_intel_hdmi(struct hda_codec *codec)
+{
+        int err;
+
+        err = hdmi_parse_codec(codec);
+        if (err < 0) {
+                generic_spec_free(codec);
+                return err;
+        }
+
+        generic_hdmi_init_per_pins(codec);
+        register_i915_notifier(codec);
+        return 0;
+}
+
+/* Intel Haswell and onwards; audio component with eld notifier */
+static int intel_hsw_common_init(struct hda_codec *codec, hda_nid_t vendor_nid)
+{
+        struct hdmi_spec *spec;
+        int err;
+
+        err = alloc_intel_hdmi(codec);
         if (err < 0)
                 return err;
         spec = codec->spec;
@@ -2542,15 +2561,7 @@ static int intel_hsw_common_init(struct hda_codec *codec, hda_nid_t vendor_nid)
         spec->ops.setup_stream = i915_hsw_setup_stream;
         spec->ops.pin_cvt_fixup = i915_pin_cvt_fixup;
 
-        err = hdmi_parse_codec(codec);
-        if (err < 0) {
-                generic_spec_free(codec);
-                return err;
-        }
-
-        generic_hdmi_init_per_pins(codec);
-        register_i915_notifier(codec);
-        return 0;
+        return parse_intel_hdmi(codec);
 }
 
 static int patch_i915_hsw_hdmi(struct hda_codec *codec)
@@ -2569,13 +2580,7 @@ static int patch_i915_byt_hdmi(struct hda_codec *codec)
         struct hdmi_spec *spec;
         int err;
 
-        /* requires i915 binding */
-        if (!codec->bus->core.audio_component) {
-                codec_info(codec, "No i915 binding for Intel HDMI/DP codec\n");
-                return -ENODEV;
-        }
-
-        err = alloc_generic_hdmi(codec);
+        err = alloc_intel_hdmi(codec);
         if (err < 0)
                 return err;
         spec = codec->spec;
@@ -2590,49 +2595,18 @@ static int patch_i915_byt_hdmi(struct hda_codec *codec)
 
         spec->ops.pin_cvt_fixup = i915_pin_cvt_fixup;
 
-        err = hdmi_parse_codec(codec);
-        if (err < 0) {
-                generic_spec_free(codec);
-                return err;
-        }
-
-        generic_hdmi_init_per_pins(codec);
-        register_i915_notifier(codec);
-        return 0;
+        return parse_intel_hdmi(codec);
 }
 
 /* Intel IronLake, SandyBridge and IvyBridge; with eld notifier */
 static int patch_i915_cpt_hdmi(struct hda_codec *codec)
 {
-        struct hdmi_spec *spec;
         int err;
 
-        /* no i915 component should have been bound before this */
-        if (WARN_ON(codec->bus->core.audio_component))
-                return -EBUSY;
-
-        err = alloc_generic_hdmi(codec);
+        err = alloc_intel_hdmi(codec);
         if (err < 0)
                 return err;
-        spec = codec->spec;
-
-        /* Try to bind with i915 now */
-        err = snd_hdac_i915_init(&codec->bus->core);
-        if (err < 0)
-                goto error;
-        spec->i915_bound = true;
-
-        err = hdmi_parse_codec(codec);
-        if (err < 0)
-                goto error;
-
-        generic_hdmi_init_per_pins(codec);
-        register_i915_notifier(codec);
-        return 0;
-
- error:
-        generic_spec_free(codec);
-        return err;
+        return parse_intel_hdmi(codec);
 }
 
 /*
@@ -2782,21 +2756,21 @@ static int nvhdmi_7x_init_8ch(struct hda_codec *codec)
         return 0;
 }
 
-static unsigned int channels_2_6_8[] = {
+static const unsigned int channels_2_6_8[] = {
         2, 6, 8
 };
 
-static unsigned int channels_2_8[] = {
+static const unsigned int channels_2_8[] = {
         2, 8
 };
 
-static struct snd_pcm_hw_constraint_list hw_constraints_2_6_8_channels = {
+static const struct snd_pcm_hw_constraint_list hw_constraints_2_6_8_channels = {
         .count = ARRAY_SIZE(channels_2_6_8),
         .list = channels_2_6_8,
         .mask = 0,
 };
 
-static struct snd_pcm_hw_constraint_list hw_constraints_2_8_channels = {
+static const struct snd_pcm_hw_constraint_list hw_constraints_2_8_channels = {
         .count = ARRAY_SIZE(channels_2_8),
         .list = channels_2_8,
         .mask = 0,
@@ -2807,7 +2781,7 @@ static int simple_playback_pcm_open(struct hda_pcm_stream *hinfo,
                                     struct snd_pcm_substream *substream)
 {
         struct hdmi_spec *spec = codec->spec;
-        struct snd_pcm_hw_constraint_list *hw_constraints_channels = NULL;
+        const struct snd_pcm_hw_constraint_list *hw_constraints_channels = NULL;
 
         switch (codec->preset->vendor_id) {
         case 0x10de0002: