mbox series

[v3,0/5] ALSA/ASoC: Conditionally skip i915 init and cleanups

Message ID 20240226124432.1203798-1-cezary.rojewski@intel.com
Headers show
Series ALSA/ASoC: Conditionally skip i915 init and cleanups | expand

Message

Cezary Rojewski Feb. 26, 2024, 12:44 p.m. UTC
A small set of changes to improve initialization of the audio stack on
HDAudio devices and pair of cleanups.

As the first change is the most important one here, following is the
technical background for it:

Commit 78f613ba1efb ("drm/i915: finish removal of CNL") and its friends
removed support for i915 for all CNL-based platforms. HDAudio library,
however, still treats such platforms as valid candidates for i915
binding. Update query mechanism to reflect changes made in drm tree.

At the same time, i915 support for LKF-based platforms has not been
provided so remove them from valid binding candidates.

The snd_soc_hda change is a follow up for the above and the cleanup
patches do not bring any functional changes.

Changes in v3:
- snd_soc_hda_codec now returns -ENODEV on attach() if i915 is not
  present
- denylist now const
- added new patch for the avs-driver to address -ENODEV during
  probe_codec()
- note: retained reviewed-by for patch 1/4 as changes are minimal

Changes in v2:
- list of problematic VGA devices is now declared locally, no more
  touching drm stuff

Cezary Rojewski (5):
  ALSA: hda: Skip i915 initialization on CNL/LKF-based platforms
  ASoC: codecs: hda: Skip HDMI/DP registration if i915 is missing
  ASoC: Intel: avs: Ignore codecs with no suppoting driver
  ASoC: codecs: hda: Cleanup error messages
  ALSA: hda: Reuse for_each_pcm_streams()

 sound/hda/hdac_i915.c      | 32 +++++++++++++++++++++++++++++---
 sound/pci/hda/hda_codec.c  |  2 +-
 sound/soc/codecs/hda.c     | 15 ++++++++++-----
 sound/soc/intel/avs/core.c |  9 +++++----
 4 files changed, 45 insertions(+), 13 deletions(-)

Comments

Takashi Iwai March 4, 2024, 8:18 a.m. UTC | #1
On Mon, 26 Feb 2024 13:44:27 +0100,
Cezary Rojewski wrote:
> 
> A small set of changes to improve initialization of the audio stack on
> HDAudio devices and pair of cleanups.
> 
> As the first change is the most important one here, following is the
> technical background for it:
> 
> Commit 78f613ba1efb ("drm/i915: finish removal of CNL") and its friends
> removed support for i915 for all CNL-based platforms. HDAudio library,
> however, still treats such platforms as valid candidates for i915
> binding. Update query mechanism to reflect changes made in drm tree.
> 
> At the same time, i915 support for LKF-based platforms has not been
> provided so remove them from valid binding candidates.
> 
> The snd_soc_hda change is a follow up for the above and the cleanup
> patches do not bring any functional changes.
> 
> Changes in v3:
> - snd_soc_hda_codec now returns -ENODEV on attach() if i915 is not
>   present
> - denylist now const
> - added new patch for the avs-driver to address -ENODEV during
>   probe_codec()
> - note: retained reviewed-by for patch 1/4 as changes are minimal
> 
> Changes in v2:
> - list of problematic VGA devices is now declared locally, no more
>   touching drm stuff
> 
> Cezary Rojewski (5):
>   ALSA: hda: Skip i915 initialization on CNL/LKF-based platforms
>   ASoC: codecs: hda: Skip HDMI/DP registration if i915 is missing
>   ASoC: Intel: avs: Ignore codecs with no suppoting driver
>   ASoC: codecs: hda: Cleanup error messages
>   ALSA: hda: Reuse for_each_pcm_streams()

Applied to for-next branch now.  Thanks.


Takashi