On Tue, 2018-01-16 at 10:37 -0600, Pierre-Louis Bossart wrote:
M4 expands $10 as $1 followed by 0, so test .conf topology files contained mclk_freq "codec0" instead of the expected value (e.g. 19200000).
m4 keeps same naming syntax for args > 9, but bash needs the {}
Applied and modified the commit message a little.
Liam
Parenthesis are required when there are more than 9 arguments to a macro...
Reported-by: "Wang, Yan" yan.wang@intel.com Signed-off-by: Pierre-Louis Bossart pierre-louis.bossart@linux.intel.com
topology/test/tplg-build.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/topology/test/tplg-build.sh b/topology/test/tplg-build.sh index 2392135..d322ce7 100755 --- a/topology/test/tplg-build.sh +++ b/topology/test/tplg-build.sh @@ -39,7 +39,7 @@ function simple_test { -DTEST_SSP_FORMAT=$6 \ -DTEST_PIPE_FORMAT=$4 \ -DTEST_SSP_BCLK=$9 \
-DTEST_SSP_MCLK=$10 \
-DTEST_SSP_MCLK=${10} \ -DTEST_SSP_PHY_BITS=$7 \ -DTEST_SSP_DATA_BITS=$8 \ $i.m4 > ${TFILE}.conf