TinyAlsa: Add support for testing using tinycap and tinyplay..

The parameter has been added to the lib.sh library,
allowing the selection of tinyalsa as the testing tool.

Signed-off-by: Arkadiusz Cholewinski <arkadiuszx.cholewinski@intel.com>

Author: gbarkadiusz

README.md

@@ -53,11 +53,13 @@ Usage: ./check-playback.sh [OPTION]
     ...
 ```
 
-Some tests support SOF_ALSA_OPTS, SOF_APLAY_OPTS and SOF_ARECORD_OPTS,
+Some tests support SOF_ALSA_TOOL, SOF_ALSA_OPTS, SOF_APLAY_OPTS and SOF_ARECORD_OPTS,
 work in progress. Where supported, optional parameters in SOF_APLAY_OPTS
 and SOF_ARECORD_OPTS are passed to all aplay and arecord
 invocations. SOF_ALSA_OPTS parameters are passed to both aplay and
-arecord. Warning these environments variables do NOT support parameters
+arecord. SOF_ALSA_TOOL is used to select tool for testing.
+Set SOF_ALSA_TOOL to 'alsa' or 'tinyalsa' to select tool.
+Warning these environments variables do NOT support parameters
 with whitespace or globbing characters, in other words this does NOT
 work:
 

case-lib/lib.sh

@@ -752,17 +752,80 @@ func_lib_check_pa()
 # However, 1. arrays would complicate the user interface 2. ALSA does not
 # seem to need arguments with whitespace or globbing characters.
 
+# SOF_ALSA_TOOL:
+# This option is used for selecting tool for testing,
+# So far, supported tools are 'alsa' and 'tinyalsa'
+# To select appropriate tool, set SOF_ALSA_TOOL to one of above
+# before using 'aplay_opts' or 'arecord_opts' function.
+# (e.g., SOF_ALSA_TOOL=alsa)
+
+
+# Function to extract the card number and device number from $dev option (e.g., hw:0,10)
+parse_audio_device() {
+    # Extract the card number (e.g., "0" from hw:0,10)
+    card_nr=$(printf '%s' "$1" | cut -d ':' -f2 | cut -d ',' -f1)
+
+    # Extract the device number (e.g., "10" from hw:0,10)
+    dev_nr=$(printf '%s' "$1" | cut -d ',' -f2)
+}
+
+# Function to extract the numeric format value from the $fmt_elem option
+extract_format_number() {
+    # (e.g., extracting '16' from 'S16_LE')
+    format=$(printf '%s' "$1" | grep '[0-9]\+' -o)
+}
+
+# Initialize the parameters using for audio testing.
+# shellcheck disable=SC2034
+initialize_audio_params()
+{
+    local idx="$1"
+
+    channel=$(func_pipeline_parse_value "$idx" channel)
+    rate=$(func_pipeline_parse_value "$idx" rate)
+    fmts=$(func_pipeline_parse_value "$idx" fmt)
+    dev=$(func_pipeline_parse_value "$idx" dev)
+    pcm=$(func_pipeline_parse_value "$idx" pcm)
+    type=$(func_pipeline_parse_value "$idx" type)
+    snd=$(func_pipeline_parse_value "$idx" snd)
+
+    : "${SOF_ALSA_TOOL:="alsa"}"
+    if [[ "$SOF_ALSA_TOOL" = "tinyalsa" ]]; then
+       parse_audio_device "$dev"
+    fi
+}
+
 aplay_opts()
 {
-    dlogc "aplay $SOF_ALSA_OPTS $SOF_APLAY_OPTS $*"
-    # shellcheck disable=SC2086
-    aplay $SOF_ALSA_OPTS $SOF_APLAY_OPTS "$@"
+    if [[ "$SOF_ALSA_TOOL" = "tinyalsa" ]]; then
+        dlogc "tinyplay $*"
+        # shellcheck disable=SC2154
+        sox -n -r "$rate" -c "$channel" noise.wav synth "$duration" white
+        tinyplay -D "$card_nr" -d "$dev_nr"  -i wav noise.wav
+    elif [[ "$SOF_ALSA_TOOL" = "alsa" ]]; then
+        dlogc "aplay $SOF_ALSA_OPTS $SOF_APLAY_OPTS $*"
+        # shellcheck disable=SC2086
+        aplay $SOF_ALSA_OPTS $SOF_APLAY_OPTS "$@"
+    else
+        printf '%s' "Unknown alsa tool: $SOF_ALSA_TOOL "
+    fi
 }
+
 arecord_opts()
 {
-    dlogc "arecord $SOF_ALSA_OPTS $SOF_ARECORD_OPTS $*"
-    # shellcheck disable=SC2086
-    arecord $SOF_ALSA_OPTS $SOF_ARECORD_OPTS "$@"
+
+    if [[ "$SOF_ALSA_TOOL" = "tinyalsa" ]]; then
+        dlogc "tinycap $*"
+        # shellcheck disable=SC2154
+        extract_format_number "$fmt_elem"
+        tinycap "$file" -D "$card_nr" -d "$dev_nr" -c "$channel" -t "$duration" -r "$rate" -b "$format"
+    elif [[ "$SOF_ALSA_TOOL" = "alsa" ]]; then
+        dlogc "arecord $SOF_ALSA_OPTS $SOF_ARECORD_OPTS $*"
+        # shellcheck disable=SC2086
+        arecord $SOF_ALSA_OPTS $SOF_ARECORD_OPTS "$@"
+    else
+        printf '%s' "Unknown alsa tool: $SOF_ALSA_TOOL "
+    fi
 }
 
 die()

test-case/check-capture-tinyalsa.sh

@@ -0,0 +1,10 @@
+#!/bin/bash
+
+set -e
+
+export SOF_ALSA_TOOL=tinyalsa
+
+TESTDIR=$(cd "$(dirname "${BASH_SOURCE[0]}")/.." && pwd)
+
+# Need "exec" otherwise it believes to be a Sub-Test
+exec "$TESTDIR"/test-case/check-capture.sh "$@"

test-case/check-capture.sh

@@ -68,19 +68,14 @@ for round in $(seq 1 $round_cnt)
 do
     for idx in $(seq 0 $((PIPELINE_COUNT - 1)))
     do
-        channel=$(func_pipeline_parse_value "$idx" channel)
-        rate=$(func_pipeline_parse_value "$idx" rate)
-        fmt=$(func_pipeline_parse_value "$idx" fmt)
-        dev=$(func_pipeline_parse_value "$idx" dev)
-        pcm=$(func_pipeline_parse_value "$idx" pcm)
-        type=$(func_pipeline_parse_value "$idx" type)
-        snd=$(func_pipeline_parse_value "$idx" snd)
+
+        initialize_audio_params "$idx"
 
         if [ ${OPT_VAL['F']} = '1' ]; then
-            fmt=$(func_pipeline_parse_value "$idx" fmts)
+            fmts=$(func_pipeline_parse_value "$idx" fmts)
         fi
 
-        for fmt_elem in $fmt
+        for fmt_elem in $fmts
         do
             for i in $(seq 1 $loop_cnt)
             do

test-case/check-playback-tinyalsa.sh

@@ -0,0 +1,10 @@
+#!/bin/bash
+
+set -e
+
+export SOF_ALSA_TOOL=tinyalsa
+
+TESTDIR=$(cd "$(dirname "${BASH_SOURCE[0]}")/.." && pwd)
+
+# Need "exec" otherwise it believes to be a Sub-Test
+exec "$TESTDIR"/test-case/check-playback.sh "$@"

test-case/check-playback.sh

@@ -75,13 +75,8 @@ for round in $(seq 1 $round_cnt)
 do
     for idx in $(seq 0 $((PIPELINE_COUNT - 1)))
     do
-        channel=$(func_pipeline_parse_value "$idx" channel)
-        rate=$(func_pipeline_parse_value "$idx" rate)
-        fmts=$(func_pipeline_parse_value "$idx" fmt)
-        dev=$(func_pipeline_parse_value "$idx" dev)
-        pcm=$(func_pipeline_parse_value "$idx" pcm)
-        type=$(func_pipeline_parse_value "$idx" type)
-        snd=$(func_pipeline_parse_value "$idx" snd)
+
+        initialize_audio_params "$idx"
 
         if [ ${OPT_VAL['F']} = '1' ]; then
             fmts=$(func_pipeline_parse_value "$idx" fmts)