Hi Tatasi, Mark Many drivers are selecting strings "playback" / "capture" by own handling, but we have snd_pcm_direction_name() function for it. This patch use it. One note is that snd_pcm_direction_name() will select "Playback" and "Capture", instead of "playback" / "capture". Almost all drivers are using it as dev_dbg() or dev_err() so no problem. But some other drivers are using it as other purpose. It might be issue (?). For example ASoC debugfs dir name will be changed by this patch. Kuninori Morimoto (12): ALSA: pci: pcxhr: use snd_pcm_direction_name() ALSA: pci: rme9652: use snd_pcm_direction_name() ALSA: trace: use snd_pcm_direction_name() ALSA: aloop: use snd_pcm_direction_name() ALSA: pcm_timer: use snd_pcm_direction_name() ASoC: stm: use snd_pcm_direction_name() ASoC: sof: pcm: use snd_pcm_direction_name() ASoC: sof: intel: use snd_pcm_direction_name() ASoC: fsl: lpc3xxx-i2s: use snd_pcm_direction_name() ASoC: tegra: use snd_pcm_direction_name() ASoC: soc-pcm: use snd_pcm_direction_name() ASoC: soc-dapm: use snd_pcm_direction_name() include/trace/events/asoc.h | 3 ++- sound/core/pcm_timer.c | 3 +-- sound/drivers/aloop.c | 3 +-- sound/pci/pcxhr/pcxhr_mix22.c | 2 +- sound/pci/rme9652/hdspm.c | 6 ++---- sound/soc/fsl/lpc3xxx-i2s.c | 3 +-- sound/soc/soc-dapm.c | 3 +-- sound/soc/soc-pcm.c | 30 +++++++++++++++--------------- sound/soc/sof/intel/hda-stream.c | 4 +--- sound/soc/sof/pcm.c | 2 +- sound/soc/stm/stm32_i2s.c | 4 ++-- sound/soc/tegra/tegra210_i2s.c | 16 ++++++++-------- 12 files changed, 36 insertions(+), 43 deletions(-) -- 2.43.0