[v4,0/3] ALSA: hda: New NHLT functions and cleanup

Message

Cezary Rojewski Nov. 26, 2021, 1:47 p.m. UTC

Changes add two crucial functions: endpoint presence-check and
retrieval of endpoint's BLOB (hardware configuration) to NHLT API.

Outside of that, "is DMIC in NHLT?" check is being re-visited. No need
to check for channel count or anything DMIC-configuration related, just
straight up verify link_type presence.

Changes in v4:
- patch renaming nhlt_acpi_table struct to acpi_nhlt_table has been
  dropped due to naming conflicts with other parts of the kernel
- revoked previously appended tags from Mark and Pierre as every patch
  required an update due to above

Changes in v3:
- no code changes
- appended Mark's Acked-by tag for patch 4/4
- appended Pierre's Reviewed-by tag for all patches

Changes in v2:
- patch "ALSA hda: Drop device-argument in NHLT functions" has been
  dropped
- updated newly added declarations in intel-nhlt.h so warning:
  "no-previous-prototype-for-function" and error:
  "use-of-undeclared-identifier" are no longer observed when
  CONFIG_SND_INTEL_NHLT is not enabled
- added Mark's tag to the last patch of the series

Amadeusz Sławiński (3):
  ALSA: hda: Fill gaps in NHLT endpoint-interface
  ALSA: hda: Simplify DMIC-in-NHLT check
  ASoC: Intel: Skylake: Use NHLT API to search for blob

 include/sound/intel-nhlt.h             |  37 +++++++--
 sound/hda/intel-dsp-config.c           |   2 +-
 sound/hda/intel-nhlt.c                 | 102 +++++++++++++++++++++++++
 sound/soc/intel/skylake/skl-nhlt.c     | 102 -------------------------
 sound/soc/intel/skylake/skl-pcm.c      |   3 +
 sound/soc/intel/skylake/skl-topology.c |  29 ++++---
 sound/soc/intel/skylake/skl-topology.h |   1 +
 sound/soc/intel/skylake/skl.h          |   4 -
 8 files changed, 153 insertions(+), 127 deletions(-)

Comments

Cezary Rojewski Nov. 26, 2021, 1:55 p.m. UTC | #1

On 2021-11-26 2:47 PM, Cezary Rojewski wrote:
> Changes add two crucial functions: endpoint presence-check and
> retrieval of endpoint's BLOB (hardware configuration) to NHLT API.
> 
> Outside of that, "is DMIC in NHLT?" check is being re-visited. No need
> to check for channel count or anything DMIC-configuration related, just
> straight up verify link_type presence.
> 
> Changes in v4:
> - patch renaming nhlt_acpi_table struct to acpi_nhlt_table has been
>    dropped due to naming conflicts with other parts of the kernel
> - revoked previously appended tags from Mark and Pierre as every patch
>    required an update due to above

I'm sorry, copy-paste caused the tags to be re-added. Ignore this 
series, I'll re-send as v5.


Czarek