Re: [PATCH v6 0/3] Add support for different DMIC configurations

[Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>]

On 4/27/20 8:27 AM, Mateusz Gorski wrote:
> Set of patches to enable DMIC capture on different hardware
> configurations.
> Information about supported DMIC configuration is read from NHLT and
> correct pipeline configuration is selected automatically.
> Also, adding additional option for topology binary name which is
> based on used machine driver.
> 
> Reviewed-by: Cezary Rojewski <cezary.rojewski@intel.com>

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

> 
> Mateusz Gorski (3):
>    ASoC: Intel: Skylake: Add alternative topology binary name
>    ASoC: Intel: Multiple I/O PCM format support for pipe
>    ASoC: Intel: Skylake: Automatic DMIC format configuration according to
>      information from NHLT
> 
> Changes in v2:
> - removed patch 1/4 swapping machine device and platform device
>    registration order
> - alt_tplg_name creation now uses different field to read machine driver
>    name
> - including of <sound/soc-acpi.h> moved to different patch
> 
> Changes in v3:
> - cosmetic changes in skl-topology.c file
> 
> Changes in v4:
> - refactored patch 2/3 to use one common helper function to deal with both
>    set and get pipe configuration operations, as suggested by Pierre
> - adjusted patch 3/3 to also use this helper function for DMIC pipes
> - added comment in patch 3/3 explaining the change of access rights for
>    DMIC enums
> 
> Changes in v5:
> - fixed alignment issue in patch 2/3
> - addressed "checkpatch.pl --strict" warnings in patches
> 
> Changes in v6:
> - fixed commit message in patch 2/3
> 
>   include/uapi/sound/skl-tplg-interface.h |   2 +
>   sound/soc/intel/skylake/skl-topology.c  | 178 +++++++++++++++++++++++-
>   sound/soc/intel/skylake/skl-topology.h  |   1 +
>   3 files changed, 176 insertions(+), 5 deletions(-)
>