Re: [PATCH 13/20] ASoC: Intel: replace dpcm_playback/capture to playback/capture_only

["Amadeusz Sławiński" <amadeuszx.slawinski@linux.intel.com>]

On 5/18/2023 7:48 AM, Kuninori Morimoto wrote:
> soc_get_playback_capture() is now handling DPCM and normal comprehensively
> for playback/capture stream. We can use playback/capture_only flag
> instead of using dpcm_playback/capture. This patch replace these.
> 
> NOTE
> Some Intel code was complex. Please check this patch deeply.
> 
> Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
> ---
>   sound/soc/intel/avs/boards/da7219.c           |  2 -
>   sound/soc/intel/avs/boards/dmic.c             |  4 +-
>   sound/soc/intel/avs/boards/hdaudio.c          |  4 --
>   sound/soc/intel/avs/boards/i2s_test.c         |  2 -
>   sound/soc/intel/avs/boards/max98357a.c        |  2 +-
>   sound/soc/intel/avs/boards/max98373.c         |  2 -
>   sound/soc/intel/avs/boards/max98927.c         |  2 -
>   sound/soc/intel/avs/boards/nau8825.c          |  2 -
>   sound/soc/intel/avs/boards/rt274.c            |  2 -
>   sound/soc/intel/avs/boards/rt286.c            |  2 -
>   sound/soc/intel/avs/boards/rt298.c            |  2 -
>   sound/soc/intel/avs/boards/rt5682.c           |  2 -
>   sound/soc/intel/avs/boards/ssm4567.c          |  2 -
>   sound/soc/intel/boards/bdw-rt5650.c           |  4 --
>   sound/soc/intel/boards/bdw-rt5677.c           |  4 --
>   sound/soc/intel/boards/bdw_rt286.c            | 10 ++--
>   sound/soc/intel/boards/bxt_da7219_max98357a.c | 32 ++++++-----
>   sound/soc/intel/boards/bxt_rt298.c            | 26 +++++----
>   sound/soc/intel/boards/bytcht_cx2072x.c       |  6 +--
>   sound/soc/intel/boards/bytcht_da7213.c        |  6 +--
>   sound/soc/intel/boards/bytcht_es8316.c        |  6 +--
>   sound/soc/intel/boards/bytcht_nocodec.c       |  6 +--
>   sound/soc/intel/boards/bytcr_rt5640.c         |  6 +--
>   sound/soc/intel/boards/bytcr_rt5651.c         |  6 +--
>   sound/soc/intel/boards/bytcr_wm5102.c         |  6 +--
>   sound/soc/intel/boards/cht_bsw_max98090_ti.c  |  6 +--
>   sound/soc/intel/boards/cht_bsw_nau8824.c      |  6 +--
>   sound/soc/intel/boards/cht_bsw_rt5645.c       |  6 +--
>   sound/soc/intel/boards/cht_bsw_rt5672.c       |  6 +--
>   sound/soc/intel/boards/cml_rt1011_rt5682.c    | 14 ++---
>   sound/soc/intel/boards/ehl_rt5660.c           | 14 +++--
>   sound/soc/intel/boards/glk_rt5682_max98357a.c | 30 +++++------
>   sound/soc/intel/boards/hsw_rt5640.c           | 10 ++--
>   sound/soc/intel/boards/kbl_da7219_max98357a.c | 26 +++++----
>   sound/soc/intel/boards/kbl_da7219_max98927.c  | 54 +++++++++----------
>   sound/soc/intel/boards/kbl_rt5660.c           | 18 +++----
>   sound/soc/intel/boards/kbl_rt5663_max98927.c  | 44 +++++++--------
>   .../intel/boards/kbl_rt5663_rt5514_max98927.c | 22 ++++----
>   sound/soc/intel/boards/skl_hda_dsp_common.c   | 14 ++---
>   .../soc/intel/boards/skl_nau88l25_max98357a.c | 26 +++++----
>   sound/soc/intel/boards/skl_nau88l25_ssm4567.c | 26 ++++-----
>   sound/soc/intel/boards/skl_rt286.c            | 26 +++++----
>   sound/soc/intel/boards/sof_cs42l42.c          | 12 +----
>   sound/soc/intel/boards/sof_da7219_max98373.c  | 16 +++---
>   sound/soc/intel/boards/sof_es8336.c           |  8 ++-
>   sound/soc/intel/boards/sof_nau8825.c          | 12 +----
>   sound/soc/intel/boards/sof_pcm512x.c          |  8 +--
>   sound/soc/intel/boards/sof_rt5682.c           | 12 +----
>   sound/soc/intel/boards/sof_sdw.c              |  4 +-
>   sound/soc/intel/boards/sof_ssp_amp.c          | 11 ++--
>   sound/soc/intel/boards/sof_wm8804.c           |  2 -
>   51 files changed, 209 insertions(+), 372 deletions(-)
> 

I've looked twice at avs part and skimmed through rest of boards, 
overall seems good, but please split sound/soc/intel/avs/boards/ and 
sound/soc/intel/boards/ into two separate patches.