*** alsabat features and fixes ***

From: "Lu, Han" <[email protected]>

Add default name for the playback and capture devices, in case they were not set by user through '-D', '-P' or '-C' options. The patch was tested on Ubuntu and Chrome OS.

Signed-off-by Lu, Han <[email protected]>

bat/alsa.c +0 -12

index 5eaa25b..5775748 100644
@@ -377,12 +377,6 @@ void *playback_alsa(struct bat *bat)
 	retval_play = 0;
 	memset(&sndpcm, 0, sizeof(sndpcm));
 
-	if (bat->playback.device == NULL) {
-		fprintf(bat->err, _("No PCM device for playback: exit\n"));
-		retval_play = 1;
-		goto exit1;
-	}
-
 	err = snd_pcm_open(&sndpcm.handle, bat->playback.device,
 			SND_PCM_STREAM_PLAYBACK, 0);
 	if (err != 0) {

@@ -533,12 +527,6 @@ void *record_alsa(struct bat *bat)
 	retval_record = 0;
 	memset(&sndpcm, 0, sizeof(sndpcm));
 
-	if (bat->capture.device == NULL) {
-		fprintf(bat->err, _("No PCM device for capture: exit\n"));
-		retval_record = 1;
-		goto exit1;
-	}
-
 	err = snd_pcm_open(&sndpcm.handle, bat->capture.device,
 			SND_PCM_STREAM_CAPTURE, 0);
 	if (err != 0) {

bat/bat.c +0 -0

index ddb60b7..9c637f2 100644

-----Original Message-----
From: Takashi Iwai [mailto:[email protected]]
Sent: Saturday, March 5, 2016 4:57 AM
To: Lu, Han <[email protected]>
Cc: [email protected]; Gautier, Bernard <[email protected]>; Popescu, Edward C <[email protected]>; [email protected]
Subject: Re: [PATCH 01/10] alsabat: add default device name for playback and capture

Quoted 44 lines · On Wed, 02 Mar 2016 09:53:11 +0100, [email protected] wrote:

From: "Lu, Han" <[email protected]>

Add default name for the playback and capture devices, in case they were not set by user through '-D', '-P' or '-C' options. The patch was tested on Ubuntu and Chrome OS.

Signed-off-by Lu, Han <[email protected]>

bat/alsa.c +0 -12

index 5eaa25b..5775748 100644
@@ -377,12 +377,6 @@ void *playback_alsa(struct bat *bat)
 	retval_play = 0;
 	memset(&sndpcm, 0, sizeof(sndpcm));
 
-	if (bat->playback.device == NULL) {
-		fprintf(bat->err, _("No PCM device for playback: exit\n"));
-		retval_play = 1;
-		goto exit1;
-	}
-
 	err = snd_pcm_open(&sndpcm.handle, bat->playback.device,
 			SND_PCM_STREAM_PLAYBACK, 0);
 	if (err != 0) {

@@ -533,12 +527,6 @@ void *record_alsa(struct bat *bat)
 	retval_record = 0;
 	memset(&sndpcm, 0, sizeof(sndpcm));
 
-	if (bat->capture.device == NULL) {
-		fprintf(bat->err, _("No PCM device for capture: exit\n"));
-		retval_record = 1;
-		goto exit1;
-	}
-
 	err = snd_pcm_open(&sndpcm.handle, bat->capture.device,
 			SND_PCM_STREAM_CAPTURE, 0);
 	if (err != 0) {

bat/bat.c +0 -0

index ddb60b7..9c637f2 100644

bat/bat.c static int bat_init(struct bat *bat) · line 478

Quoted 6 lines

bat/bat.c +3 -0

@@ -473,6 +473,10 @@ static int bat_init(struct bat *bat)
 			return err;
 	}
 
+	/* Set default playback and capture devices */
+	if (bat->playback.device == NULL && bat->capture.device == NULL)
+		bat->playback.device = bat->capture.device =

DEFAULT_DEV_NAME;

What if only one of playback and capture is specified?

Takashi

Quoted 65 lines

-----Original Message-----
From: Takashi Iwai [mailto:[email protected]]
Sent: Saturday, March 5, 2016 4:57 AM
To: Lu, Han <[email protected]>
Cc: [email protected]; Gautier, Bernard <[email protected]>; Popescu, Edward C <[email protected]>; [email protected]
Subject: Re: [PATCH 01/10] alsabat: add default device name for playback and capture

Quoted 44 lines · On Wed, 02 Mar 2016 09:53:11 +0100, [email protected] wrote:

From: "Lu, Han" <[email protected]>

Add default name for the playback and capture devices, in case they were not set by user through '-D', '-P' or '-C' options. The patch was tested on Ubuntu and Chrome OS.

Signed-off-by Lu, Han <[email protected]>

bat/alsa.c +0 -12

index 5eaa25b..5775748 100644
@@ -377,12 +377,6 @@ void *playback_alsa(struct bat *bat)
 	retval_play = 0;
 	memset(&sndpcm, 0, sizeof(sndpcm));
 
-	if (bat->playback.device == NULL) {
-		fprintf(bat->err, _("No PCM device for playback: exit\n"));
-		retval_play = 1;
-		goto exit1;
-	}
-
 	err = snd_pcm_open(&sndpcm.handle, bat->playback.device,
 			SND_PCM_STREAM_PLAYBACK, 0);
 	if (err != 0) {

@@ -533,12 +527,6 @@ void *record_alsa(struct bat *bat)
 	retval_record = 0;
 	memset(&sndpcm, 0, sizeof(sndpcm));
 
-	if (bat->capture.device == NULL) {
-		fprintf(bat->err, _("No PCM device for capture: exit\n"));
-		retval_record = 1;
-		goto exit1;
-	}
-
 	err = snd_pcm_open(&sndpcm.handle, bat->capture.device,
 			SND_PCM_STREAM_CAPTURE, 0);
 	if (err != 0) {

bat/bat.c +0 -0

index ddb60b7..9c637f2 100644

bat/bat.c static int bat_init(struct bat *bat) · line 478

Quoted 6 lines

bat/bat.c +3 -0

@@ -473,6 +473,10 @@ static int bat_init(struct bat *bat)
 			return err;
 	}
 
+	/* Set default playback and capture devices */
+	if (bat->playback.device == NULL && bat->capture.device == NULL)
+		bat->playback.device = bat->capture.device =

DEFAULT_DEV_NAME;

Quoted 4 lines

What if only one of playback and capture is specified?

Takashi

If only one of playback and capture is specified, the alsabat shall work on single line mode, where only one thread (playback or capture) will be started. The testbat.sh covers the test cases of single mode:

	feature_test "-P $dev_playback" "single line mode, playback"
	feature_test "-C $dev_capture --standalone" "single line mode, capture"

Previously, if not specify any device, the alsabat will start playback and capture Threads, and then quit threads with error text. The patch is intend to make the alsabat works in case no device be specified, and the patch does not influence the cases of single line mode.

From: "Lu, Han" <[email protected]>

Add support for standalone mode where alsabat will run on a different machine to the one being tested.
In standalone mode, the sound data can be generated, playback and captured just like in normal mode, but will not be analyzed. The alsabat being built without libfftw3 support is always work in standalone mode.
The alsabat in normal mode can also bypass data analysis using option "--standalone".

Signed-off-by Lu, Han <[email protected]>

bat/Makefile.am +0 -1

index 8dfafa9..5646e9a 100644
@@ -6,7 +6,6 @@ EXTRA_DIST = alsabat.1
 alsabat_SOURCES = \
 	bat.c \
 	common.c \
-	analyze.c \
 	signal.c \
 	convert.c \
 	alsa.c

@@ -15,8 +14,12 @@ noinst_HEADERS = \
 	common.h \

-----Original Message-----
From: Takashi Iwai [mailto:[email protected]]
Sent: Tuesday, March 8, 2016 6:19 PM
To: Lu, Han <[email protected]>
Cc: [email protected]; Gautier, Bernard <[email protected]>; Popescu, Edward C <[email protected]>; [email protected]
Subject: Re: [PATCH 02/10] alsabat: add standalone mode

Quoted 27 lines · On Wed, 02 Mar 2016 09:53:12 +0100, [email protected] wrote:

From: "Lu, Han" <[email protected]>

Add support for standalone mode where alsabat will run on a different machine to the one being tested.
In standalone mode, the sound data can be generated, playback and captured just like in normal mode, but will not be analyzed. The alsabat being built without libfftw3 support is always work in standalone mode.
The alsabat in normal mode can also bypass data analysis using option "--standalone".

Signed-off-by Lu, Han <[email protected]>

bat/Makefile.am index 8dfafa9..5646e9a renamed from bat/Makefile.am +0 -0

100644

bat/Makefile.am +0 -1

@@ -6,7 +6,6 @@ EXTRA_DIST = alsabat.1  alsabat_SOURCES = \
 	bat.c \
 	common.c \
-	analyze.c \
 	signal.c \
 	convert.c \
 	alsa.c

@@ -15,8 +14,12 @@ noinst_HEADERS = \
 	common.h \

bat/Makefile.am noinst_HEADERS = \ · line 22

Quoted 10 lines

bat/Makefile.am +6 -2

 	bat-signal.h \
 	alsa.h \
-	convert.h \
-	analyze.h
+	convert.h
+
+if HAVE_LIBFFTW3
+alsabat_SOURCES += analyze.c
+noinst_HEADERS += analyze.h
+endif

I wonder how "make dist" works if running on a system without fftw3... Does it pack these conditional files, too?

Takashi

From: "Lu, Han" <[email protected]>

Add support for alsabat to work on tinyalsa library based platforms such as Android and some Embedded Linux devices.
Use option '-t' to select tinyalsa library instead of ALSA library. If '-t' is used while tinyalsa library is not installed in system, alsabat will print error message and quit.

Signed-off-by Lu, Han <[email protected]>

bat/Makefile.am +5 -0

index 5646e9a..6101681 100644
@@ -21,6 +21,11 @@ alsabat_SOURCES += analyze.c
 noinst_HEADERS += analyze.h
 endif
 
+if HAVE_LIBTINYALSA
+alsabat_SOURCES += tinyalsa.c
+noinst_HEADERS += tinyalsa.h
+endif
+
 AM_CPPFLAGS = \
 	      -Wall -I$(top_srcdir)/include
 

bat/alsabat.1 +3 -0

index 5f41669..126232e 100644
@@ -106,6 +106,9 @@ Valid range is (DC_THRESHOLD, 40% * Sampling rate).
 \fI\-p\fP
 Total number of periods to play or capture.
 .TP
+\fI\-t\fP
+If tinyalsa lib is installed, use tinyalsa lib instead of alsa lib.
+.TP
 \fI\-\-log=#\fP
 Write stderr and stdout output to this log file.
 .TP

bat/bat.c +62 -0

index 85ec5aa..8db16c0 100644
@@ -36,6 +36,9 @@
 #ifdef HAVE_LIBFFTW3
 #include "analyze.h"
 #endif
+#ifdef HAVE_LIBTINYALSA
+#include "tinyalsa.h"
+#endif
 
 static int get_duration(struct bat *bat)
 {

@@ -125,6 +128,35 @@ static void get_format(struct bat *bat, char *optarg)
 	}
 }
 
+static int get_tiny_format(struct bat *bat, char *alsa_device,
+		unsigned int *tiny_card, unsigned int *tiny_device)
+{
+	char *tmp1, *tmp2, *tmp3;
+
+	if (alsa_device == NULL)
+		goto fail;
+
+	tmp1 = strchr(alsa_device, ':');
+	if (tmp1 == NULL)
+		goto fail;
+
+	tmp3 = tmp1 + 1;
+	tmp2 = strchr(tmp3, ',');
+	if (tmp2 == NULL)
+		goto fail;
+
+	tmp1 = tmp2 + 1;
+	*tiny_device = atoi(tmp1);
+	*tmp2 = '\0';
+	*tiny_card = atoi(tmp3);
+	*tmp2 = ',';
+
+	return 0;
+fail:
+	fprintf(bat->err, _("Invalid tiny format!\n"));
+	return -EINVAL;
+}
+
 static inline int thread_wait_completion(struct bat *bat,
 		pthread_t id, int **val)
 {

@@ -287,6 +319,7 @@ _("Usage: alsabat [-options]...\n"
 "  -k                     parameter for frequency detecting threshold\n"
 "  -F                     target frequency\n"
 "  -p                     total number of periods to play/capture\n"
+"  -t                     use tinyalsa instead of alsa\n"
 "      --log=#            file that both stdout and strerr redirecting to\n"
 "      --file=#           file for playback\n"
 "      --saveplay=#       file that storing playback content, for debug\n"

@@ -330,6 +363,7 @@ static void set_defaults(struct bat *bat)
 	bat->period_is_limited = false;
 	bat->log = stdout;
 	bat->err = stderr;
+	bat->tinyalsa = false;
 }
 
 static void parse_arguments(struct bat *bat, int argc, char *argv[])

@@ -406,6 +440,16 @@ static void parse_arguments(struct bat *bat, int argc, char *argv[])
 			bat->periods_total = atoi(optarg);
 			bat->period_is_limited = true;
 			break;
+		case 't':
+#ifdef HAVE_LIBTINYALSA
+			bat->playback.fct = &playback_tinyalsa;
+			bat->capture.fct = &record_tinyalsa;
+			bat->tinyalsa = true;
+#else
+			fprintf(bat->err, _("tinyalsa lib is not installed\n"));
+			exit(EXIT_FAILURE);
+#endif
+			break;
 		case 'h':
 		default:
 			usage(bat);

@@ -484,6 +528,24 @@ static int bat_init(struct bat *bat)
 	if (bat->playback.device == NULL && bat->capture.device == NULL)
 		bat->playback.device = bat->capture.device = DEFAULT_DEV_NAME;
 
+	/* Determine tiny device if needed */
+	if (bat->tinyalsa == true) {
+		if (bat->playback.mode != MODE_SINGLE) {
+			err = get_tiny_format(bat, bat->capture.device,
+					&bat->capture.card_tiny,
+					&bat->capture.device_tiny);
+			if (err < 0)
+				return err;
+		}
+		if (bat->capture.mode != MODE_SINGLE) {
+			err = get_tiny_format(bat, bat->playback.device,
+					&bat->playback.card_tiny,
+					&bat->playback.device_tiny);
+			if (err < 0)
+				return err;
+		}
+	}
+
 	/* Determine capture file */
 	if (bat->local) {
 		bat->capture.file = bat->playback.file;

bat/common.c +89 -0

index 798b00b..bbf969e 100644
@@ -18,16 +18,39 @@
 #include <stdlib.h>
 #include <stdbool.h>
 #include <errno.h>
+#include <signal.h>
 
 #include "aconfig.h"
 #include "gettext.h"
 
 #include "common.h"
 #include "alsa.h"
+#include "bat-signal.h"
 
 int retval_play;
 int retval_record;
 
+int is_capturing = 1;
+int is_playing = 1;
+
+/**
+ * Handling of Ctrl-C for capture
+ */
+void sigint_handler(int sig)
+{
+	is_capturing = 0;
+}
+
+/**
+ * Handling of Ctrl-C for playback
+ */
+void stream_close(int sig)
+{
+	/* allow the stream to be closed gracefully */
+	signal(sig, SIG_IGN);
+	is_playing = 0;
+}
+
 /* update chunk_fmt data to bat */
 static int update_fmt_to_bat(struct bat *bat, struct chunk_fmt *fmt)
 {

@@ -196,3 +219,69 @@ int write_wav_header(FILE *fp, struct wav_container *wav, struct bat *bat)
 
 	return 0;
 }
+
+/* update wav header when data size changed */
+int update_wav_header(struct bat *bat, FILE *fp, int bytes)
+{
+	int err = 0;
+	struct wav_container wav;
+
+	prepare_wav_info(&wav, bat);
+	wav.chunk.length = bytes;
+	wav.header.length = (wav.chunk.length) + sizeof(wav.chunk)
+		+ sizeof(wav.format) + sizeof(wav.header) - 8;
+	rewind(fp);
+	err = write_wav_header(fp, &wav, bat);
+
+	return err;
+}
+
+/*
+ * Generate buffer to be played either from input file or from generated data
+ * Return value
+ * <0 error
+ * 0 ok
+ * >0 break
+ */
+int generate_input_data0(struct bat *bat, void *buffer, int bytes, int frames)
+{
+	int err;
+	static int load;
+
+	if (bat->playback.file != NULL) {
+		/* From input file */
+		load = 0;
+
+		while (1) {
+			err = fread(buffer + load, 1, bytes - load, bat->fp);
+			if (0 == err) {
+				if (feof(bat->fp)) {
+					fprintf(bat->log,
+							_("End of playing.\n"));
+					return 1;
+				}
+			} else if (err < bytes - load) {
+				if (ferror(bat->fp)) {
+					fprintf(bat->err, _("Read file error"));
+					fprintf(bat->err, _(": %d\n"), err);
+					return -EIO;
+				}
+				load += err;
+			} else {
+				break;
+			}
+		}
+	} else {
+		/* Generate sine wave */
+		if ((bat->sinus_duration) && (load > bat->sinus_duration))
+			return 1;
+
+		err = generate_sine_wave(bat, frames, buffer);
+		if (err != 0)
+			return err;
+
+		load += frames;
+	}
+
+	return 0;
+}

bat/common.h +11 -0

index 30e39fc..0d92a8d 100644
@@ -14,6 +14,9 @@
  */
 
 #include <alsa/asoundlib.h>
+#ifdef HAVE_LIBTINYALSA
+#include <tinyalsa/asoundlib.h>
+#endif
 
 #define TEMP_RECORD_FILE_NAME		"/tmp/bat.wav.XXXXXX"
 #define DEFAULT_DEV_NAME		"default"

@@ -119,6 +122,8 @@ enum _bat_op_mode {
 
 struct pcm {
 	char *device;
+	unsigned int card_tiny;
+	unsigned int device_tiny;
 	char *file;
 	enum _bat_op_mode mode;
 	void *(*fct)(struct bat *);

@@ -171,6 +176,8 @@ struct bat {
 	void *buf;			/* PCM Buffer */
 
 	bool local;			/* true for internal test */
+
+	bool tinyalsa;			/* true to use tinyalsa lib */
 };
 
 struct analyze {

@@ -180,6 +187,10 @@ struct analyze {
 	double *mag;
 };
 
+void sigint_handler(int);
+void stream_close(int);
 void prepare_wav_info(struct wav_container *, struct bat *);
 int read_wav_header(struct bat *, char *, FILE *, bool);
 int write_wav_header(FILE *, struct wav_container *, struct bat *);
+int update_wav_header(struct bat *, FILE *, int);
+int generate_input_data0(struct bat *, void *, int, int);

bat/tinyalsa.c new file +422 -0

new file mode 100644
index 0000000..ab11247
@@ -0,0 +1,422 @@
+/*
+ * Copyright (C) 2013-2015 Intel Corporation
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ */
+
+#include <stdio.h>
+#include <string.h>
+#include <stdbool.h>
+#include <signal.h>
+#include <pthread.h>
+#include <errno.h>
+
+#include <tinyalsa/asoundlib.h>
+
+#include "aconfig.h"
+#include "gettext.h"
+
+#include "common.h"
+#include "tinyalsa.h"
+
+struct format_map_table {
+	int sample_bytes;
+	enum pcm_format format;
+};
+
+static struct format_map_table map_tables[] = {
+	{ 2, PCM_FORMAT_S16_LE },
+	{ 4, PCM_FORMAT_S32_LE },
+	{}
+};
+
+/**
+ * Called when thread is finished
+ */
+static void close_handle(void *handle)
+{
+	struct pcm *pcm = handle;
+
+	if (NULL != pcm)
+		pcm_close(pcm);
+}
+
+static int format_convert(struct bat *bat, struct pcm_config *config)
+{
+	struct format_map_table *t = map_tables;
+
+	for (; t->sample_bytes; t++) {
+		if (t->sample_bytes == bat->sample_size) {
+			config->format = t->format;
+			return 0;
+		}
+	}
+
+	fprintf(bat->err, _("Invalid format!\n"));
+	return -EINVAL;
+}
+
+static int init_config(struct bat *bat, struct pcm_config *config)
+{
+	config->channels = bat->channels;
+	config->rate = bat->rate;
+	config->period_size = 1024;
+	config->period_count = 4;
+	config->start_threshold = 0;
+	config->stop_threshold = 0;
+	config->silence_threshold = 0;
+
+	return format_convert(bat, config);
+}
+
+/**
+ * Check that a parameter is inside bounds
+ */
+static int check_param(struct bat *bat, struct pcm_params *params,
+		unsigned int param, unsigned int value,
+		char *param_name, char *param_unit)
+{
+	unsigned int min;
+	unsigned int max;
+	int ret = 0;
+
+	min = pcm_params_get_min(params, param);
+	if (value < min) {
+		fprintf(bat->err,
+			_("%s is %u%s, device only supports >= %u%s!\n"),
+			param_name, value, param_unit, min, param_unit);
+		ret = -EINVAL;
+	}
+
+	max = pcm_params_get_max(params, param);
+	if (value > max) {
+		fprintf(bat->err,
+			_("%s is %u%s, device only supports <= %u%s!\n"),
+			param_name, value, param_unit, max, param_unit);
+		ret = -EINVAL;
+	}
+
+	return ret;
+}
+
+/**
+ * Check all parameters
+ */
+static int check_playback_params(struct bat *bat,
+		struct pcm_config *config)
+{
+	struct pcm_params *params;
+	unsigned int card = bat->playback.card_tiny;
+	unsigned int device = bat->playback.device_tiny;
+	int err = 0;
+
+	params = pcm_params_get(card, device, PCM_OUT);
+	if (params == NULL) {
+		fprintf(bat->err, _("Unable to open PCM device %u!\n"),
+				device);
+		return -EINVAL;
+	}
+
+	err = check_param(bat, params, PCM_PARAM_RATE,
+			config->rate, "Sample rate", "Hz");
+	if (err < 0)
+		goto exit;
+	err = check_param(bat, params, PCM_PARAM_CHANNELS,
+			config->channels, "Sample", " channels");
+	if (err < 0)
+		goto exit;
+	err = check_param(bat, params, PCM_PARAM_SAMPLE_BITS,
+			bat->sample_size * 8, "Bitrate", " bits");
+	if (err < 0)
+		goto exit;
+	err = check_param(bat, params, PCM_PARAM_PERIOD_SIZE,
+			config->period_size, "Period size", "Hz");
+	if (err < 0)
+		goto exit;
+	err = check_param(bat, params, PCM_PARAM_PERIODS,
+			config->period_count, "Period count", "Hz");
+	if (err < 0)
+		goto exit;
+
+exit:
+	pcm_params_free(params);
+
+	return err;
+}
+
+/**
+ * Play sample
+ */
+static int play_sample(struct bat *bat, struct pcm *pcm,
+		void *buffer, int bytes)
+{
+	int err = 0;
+	FILE *fp = NULL;
+	int frames = bytes / bat->frame_size;
+	int bytes_total = 0;
+
+	if (bat->debugplay) {
+		fp = fopen(bat->debugplay, "wb");
+		if (fp == NULL) {
+			fprintf(bat->err, _("Cannot open file for capture: "));
+			fprintf(bat->err, _("%s %d\n"), bat->debugplay, -errno);
+			return -errno;
+		}
+		/* leave space for file header */
+		if (fseek(fp, sizeof(struct wav_container), SEEK_SET) != 0) {
+			fclose(fp);
+			return -errno;
+		}
+	}
+
+	do {
+		err = generate_input_data0(bat, buffer, bytes, frames);
+		if (err != 0)
+			break;
+
+		if (bat->debugplay) {
+			if (fwrite(buffer, 1, bytes, fp) != bytes) {
+				update_wav_header(bat, fp, bytes_total);
+				fclose(fp);
+				return -EIO;
+			}
+			bytes_total += bytes;
+		}
+
+		bat->periods_played++;
+		if (bat->period_is_limited
+				&& bat->periods_played >= bat->periods_total)
+			break;
+
+		err = pcm_write(pcm, buffer, bytes);
+		if (err != 0) {
+			fprintf(bat->err, _("Write PCM device error: %d\n"),
+					err);
+			break;
+		}
+	} while (is_playing);
+
+	if (bat->debugplay) {
+		err = update_wav_header(bat, fp, bytes_total);
+		fclose(fp);
+	}
+	return err;
+}
+
+/**
+ * Play
+ */
+void *playback_tinyalsa(struct bat *bat)
+{
+	int err = 0;
+	struct pcm_config config;
+	struct pcm *pcm = NULL;
+	void *buffer = NULL;
+	int bufbytes;
+	unsigned int card = bat->playback.card_tiny;
+	unsigned int device = bat->playback.device_tiny;
+
+	fprintf(bat->log, _("Entering playback thread (tinyalsa).\n"));
+
+	retval_play = 0;
+
+	/* init config */
+	err = init_config(bat, &config);
+	if (err < 0) {
+		retval_play = err;
+		goto exit1;
+	}
+
+	/* check param before open device */
+	err = check_playback_params(bat, &config);
+	if (err < 0) {
+		retval_play = err;
+		goto exit1;
+	}
+
+	/* init device */
+	pcm = pcm_open(card, device, PCM_OUT, &config);
+	if (!pcm || !pcm_is_ready(pcm)) {
+		fprintf(bat->err, _("Unable to open PCM device %u (%s)!\n"),
+				device, pcm_get_error(pcm));
+		retval_play = -EINVAL;
+		goto exit1;
+	}
+
+	/* init buffer */
+	bufbytes = pcm_frames_to_bytes(pcm, pcm_get_buffer_size(pcm));
+	buffer = malloc(bufbytes);
+	if (!buffer) {
+		retval_play = -ENOMEM;
+		goto exit2;
+	}
+
+	/* init playback source */
+	if (bat->playback.file == NULL) {
+		fprintf(bat->log, _("Playing generated audio sine wave"));
+		bat->sinus_duration == 0 ?
+			fprintf(bat->log, _(" endlessly\n")) :
+			fprintf(bat->log, _("\n"));
+	} else {
+		fprintf(bat->log, _("Playing input audio file: %s\n"),
+				bat->playback.file);
+		bat->fp = fopen(bat->playback.file, "rb");
+		if (bat->fp == NULL) {
+			fprintf(bat->err, _("Cannot open file for playback: "));
+			fprintf(bat->err, _("%s %d\n"),
+					bat->playback.file, -errno);
+			retval_play = -errno;
+			goto exit3;
+		}
+		/* Skip header */
+		err = read_wav_header(bat, bat->playback.file, bat->fp, true);
+		if (err != 0) {
+			retval_play = err;
+			goto exit4;
+		}
+	}
+
+	/* catch ctrl-c to shutdown cleanly */
+	signal(SIGINT, stream_close);
+
+	err = play_sample(bat, pcm, buffer, bufbytes);
+	if (err < 0) {
+		retval_play = err;
+		goto exit4;
+	}
+
+exit4:
+	if (bat->playback.file)
+		fclose(bat->fp);
+exit3:
+	free(buffer);
+exit2:
+	pcm_close(pcm);
+exit1:
+	pthread_exit(&retval_play);
+}
+
+/**
+ * Capture sample
+ */
+static int capture_sample(struct bat *bat, struct pcm *pcm,
+		void *buffer, unsigned int bytes)
+{
+	int err = 0;
+	FILE *fp = NULL;
+	unsigned int bytes_read = 0;
+	unsigned int bytes_count = bat->frames * bat->frame_size;
+
+	remove(bat->capture.file);
+	fp = fopen(bat->capture.file, "wb");
+	if (fp == NULL) {
+		fprintf(bat->err, _("Cannot open file for capture: %s %d\n"),
+				bat->capture.file, -errno);
+		return -errno;
+	}
+	/* leave space for file header */
+	if (fseek(fp, sizeof(struct wav_container), SEEK_SET) != 0) {
+		fclose(fp);
+		return -errno;
+	}
+
+	while (bytes_read < bytes_count && is_capturing
+			&& !pcm_read(pcm, buffer, bytes)) {
+		if (fwrite(buffer, 1, bytes, fp) != bytes)
+			break;
+
+		bytes_read += bytes;
+
+		bat->periods_played++;
+
+		if (bat->period_is_limited
+				&& bat->periods_played >= bat->periods_total)
+			break;
+	}
+
+	err = update_wav_header(bat, fp, bytes_read);
+
+	fclose(fp);
+	return err;
+}
+
+/**
+ * Record
+ */
+void *record_tinyalsa(struct bat *bat)
+{
+	int err = 0;
+	struct pcm_config config;
+	struct pcm *pcm;
+	void *buffer;
+	unsigned int bufbytes;
+
+	pthread_setcancelstate(PTHREAD_CANCEL_DISABLE, NULL);
+
+	fprintf(bat->log, _("Entering capture thread (tinyalsa).\n"));
+
+	retval_record = 0;
+
+	/* init config */
+	err = init_config(bat, &config);
+	if (err < 0) {
+		retval_record = err;
+		goto exit1;
+	}
+
+	/* init device */
+	pcm = pcm_open(bat->capture.card_tiny, bat->capture.device_tiny,
+			PCM_IN, &config);
+	if (!pcm || !pcm_is_ready(pcm)) {
+		fprintf(bat->err, _("Unable to open PCM device (%s)!\n"),
+				pcm_get_error(pcm));
+		retval_record = -EINVAL;
+		goto exit1;
+	}
+
+	/* init buffer */
+	bufbytes = pcm_frames_to_bytes(pcm, pcm_get_buffer_size(pcm));
+	buffer = malloc(bufbytes);
+	if (!buffer) {
+		retval_record = -ENOMEM;
+		goto exit2;
+	}
+
+	/* install signal handler and begin capturing Ctrl-C */
+	signal(SIGINT, sigint_handler);
+
+	pthread_setcancelstate(PTHREAD_CANCEL_ENABLE, NULL);
+	pthread_setcanceltype(PTHREAD_CANCEL_DEFERRED, NULL);
+	pthread_cleanup_push(close_handle, pcm);
+	pthread_cleanup_push(free, buffer);
+
+	fprintf(bat->log, _("Recording ...\n"));
+	err = capture_sample(bat, pcm, buffer, bufbytes);
+	if (err != 0) {
+		retval_record = err;
+		goto exit3;
+	}
+
+	/* Normally we will never reach this part of code (unless error in
+	 * previous call) (before exit3) as this thread will be cancelled
+	 *  by end of play thread. Except in single line mode. */
+	pthread_cleanup_pop(0);
+	pthread_cleanup_pop(0);
+	pthread_exit(&retval_record);
+
+exit3:
+	free(buffer);
+exit2:
+	pcm_close(pcm);
+exit1:
+	pthread_exit(&retval_record);
+}

bat/tinyalsa.h new file +23 -0

new file mode 100644
index 0000000..70e4749
@@ -0,0 +1,23 @@
+/*
+ * Copyright (C) 2013-2015 Intel Corporation
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ */
+
+extern int retval_play;
+extern int retval_record;
+
+extern int is_capturing;
+extern int is_playing;
+
+void *playback_tinyalsa(struct bat *);
+void *record_tinyalsa(struct bat *);

configure.ac +4 -0

index f6f8103..87dd237 100644
@@ -73,6 +73,9 @@ if test x$bat = xtrue; then
   dnl Check for libfftw3
   have_libfftw3="yes"
   AC_CHECK_LIB([fftw3], [fftw_malloc], , [have_libfftw3="no"])
+  dnl Check for libtinyalsa
+  have_libtinyalsa="yes"
+  AC_CHECK_LIB([tinyalsa], [pcm_open], , [have_libtinyalsa="no"])
   AC_CHECK_LIB([m], [sqrtf], , [AC_MSG_ERROR([Error: Need sqrtf])])
   AC_CHECK_LIB([pthread], [pthread_create], , [AC_MSG_ERROR([Error: need PTHREAD library])])
   FFTW_CFLAGS="$CFLAGS"

@@ -86,6 +89,7 @@ if test x$bat = xtrue; then
 
 fi
 AM_CONDITIONAL(HAVE_LIBFFTW3, test "$have_libfftw3" = "yes")
+AM_CONDITIONAL(HAVE_LIBTINYALSA, test "$have_libtinyalsa" = "yes")
 
 dnl Check for librt
 LIBRT=""

Signature · 6 lines

2.5.0

_______________________________________________
Alsa-devel mailing list
[email protected]
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

-----Original Message-----
From: Takashi Iwai [mailto:[email protected]]
Sent: Friday, March 11, 2016 10:14 PM
To: Lu, Han <[email protected]>
Cc: [email protected]; Gautier, Bernard <[email protected]>; Popescu, Edward C <[email protected]>; [email protected]
Subject: Re: [alsa-devel] [PATCH 03/10] alsabat: add tinyalsa support

Quoted 10 lines · On Wed, 02 Mar 2016 09:53:13 +0100, [email protected] wrote:

From: "Lu, Han" <[email protected]>

Add support for alsabat to work on tinyalsa library based platforms such as Android and some Embedded Linux devices.
Use option '-t' to select tinyalsa library instead of ALSA library. If '-t' is used while tinyalsa library is not installed in system, alsabat will print error message and quit.

Signed-off-by Lu, Han <[email protected]>

Hrm, I don't think it makes so much sense to build a binary bound with both libraries. Usually it's mutual exclusive, the backend is selected / detected via configure script.

thanks,

Takashi

bat/alsabat.1 Valid range is (DC_THRESHOLD, 40% * Sampling

Quoted 21 lines

bat/Makefile.am index 5646e9a..6101681 renamed from bat/Makefile.am +0 -0

100644

bat/Makefile.am +0 -0

@@ -21,6 +21,11 @@ alsabat_SOURCES += analyze.c  noinst_HEADERS +=

analyze.h endif

bat/Makefile.am +5 -0

+if HAVE_LIBTINYALSA
+alsabat_SOURCES += tinyalsa.c
+noinst_HEADERS += tinyalsa.h
+endif
+
 AM_CPPFLAGS = \
 	      -Wall -I$(top_srcdir)/include

bat/alsabat.1 index 5f41669..126232e renamed from bat/alsabat.1 +0 -0

100644

bat/alsabat.1 +0 -0

@@ -106,6 +106,9 @@ Valid range is (DC_THRESHOLD, 40% * Sampling

rate).

bat/bat.c static void parse_arguments(struct bat *bat, int

Quoted 74 lines

bat/alsabat.1 +3 -0

 \fI\-p\fP
 Total number of periods to play or capture.
 .TP
+\fI\-t\fP
+If tinyalsa lib is installed, use tinyalsa lib instead of alsa lib.
+.TP
 \fI\-\-log=#\fP
 Write stderr and stdout output to this log file.
 .TP

bat/bat.c +3 -0

index 85ec5aa..8db16c0 100644
@@ -36,6 +36,9 @@
 #ifdef HAVE_LIBFFTW3
 #include "analyze.h"
 #endif
+#ifdef HAVE_LIBTINYALSA
+#include "tinyalsa.h"
+#endif
 
 static int get_duration(struct bat *bat)  { @@ -125,6 +128,35 @@
 static void get_format(struct bat *bat, char *optarg)

bat/bat.c +30 -0

 	}
 }
 
+static int get_tiny_format(struct bat *bat, char *alsa_device,
+		unsigned int *tiny_card, unsigned int *tiny_device) {
+	char *tmp1, *tmp2, *tmp3;
+
+	if (alsa_device == NULL)
+		goto fail;
+
+	tmp1 = strchr(alsa_device, ':');
+	if (tmp1 == NULL)
+		goto fail;
+
+	tmp3 = tmp1 + 1;
+	tmp2 = strchr(tmp3, ',');
+	if (tmp2 == NULL)
+		goto fail;
+
+	tmp1 = tmp2 + 1;
+	*tiny_device = atoi(tmp1);
+	*tmp2 = '\0';
+	*tiny_card = atoi(tmp3);
+	*tmp2 = ',';
+
+	return 0;
+fail:
+	fprintf(bat->err, _("Invalid tiny format!\n"));
+	return -EINVAL;
+}
+
 static inline int thread_wait_completion(struct bat *bat,
 		pthread_t id, int **val)
 {

@@ -287,6 +319,7 @@ _("Usage: alsabat [-options]...\n"
 "  -k                     parameter for frequency detecting threshold\n"
 "  -F                     target frequency\n"
 "  -p                     total number of periods to play/capture\n"
+"  -t                     use tinyalsa instead of alsa\n"
 "      --log=#            file that both stdout and strerr redirecting to\n"
 "      --file=#           file for playback\n"
 "      --saveplay=#       file that storing playback content, for debug\n"

@@ -330,6 +363,7 @@ static void set_defaults(struct bat *bat)
 	bat->period_is_limited = false;
 	bat->log = stdout;
 	bat->err = stderr;
+	bat->tinyalsa = false;
 }
 
 static void parse_arguments(struct bat *bat, int argc, char *argv[])

@@ -406,6 +440,16 @@ static void parse_arguments(struct bat *bat, int

argc, char *argv[])

Quoted 17 lines

bat/bat.c +10 -0

 			bat->periods_total = atoi(optarg);
 			bat->period_is_limited = true;
 			break;
+		case 't':
+#ifdef HAVE_LIBTINYALSA
+			bat->playback.fct = &playback_tinyalsa;
+			bat->capture.fct = &record_tinyalsa;
+			bat->tinyalsa = true;
+#else
+			fprintf(bat->err, _("tinyalsa lib is not installed\n"));
+			exit(EXIT_FAILURE);
+#endif
+			break;
 		case 'h':
 		default:
 			usage(bat);

bat/bat.c static int bat_init(struct bat *bat) · line 529

Quoted 2 lines

bat/bat.c +0 -0

@@ -484,6 +528,24 @@ static int bat_init(struct bat *bat)
 	if (bat->playback.device == NULL && bat->capture.device == NULL)
 		bat->playback.device = bat->capture.device =

DEFAULT_DEV_NAME;

Quoted 12 lines

bat/bat.c +11 -0

+	/* Determine tiny device if needed */
+	if (bat->tinyalsa == true) {
+		if (bat->playback.mode != MODE_SINGLE) {
+			err = get_tiny_format(bat, bat->capture.device,
+					&bat->capture.card_tiny,
+					&bat->capture.device_tiny);
+			if (err < 0)
+				return err;
+		}
+		if (bat->capture.mode != MODE_SINGLE) {
+			err = get_tiny_format(bat, bat->playback.device,

bat/bat.c static int bat_init(struct bat *bat)

Quoted 10 lines

bat/bat.c +7 -0

+					&bat->playback.card_tiny,
+					&bat->playback.device_tiny);
+			if (err < 0)
+				return err;
+		}
+	}
+
 	/* Determine capture file */
 	if (bat->local) {
 		bat->capture.file = bat->playback.file; diff --git

a/bat/common.c

Quoted 76 lines

b/bat/common.c index 798b00b..bbf969e 100644

bat/common.c +23 -0

@@ -18,16 +18,39 @@
 #include <stdlib.h>
 #include <stdbool.h>
 #include <errno.h>
+#include <signal.h>
 
 #include "aconfig.h"
 #include "gettext.h"
 
 #include "common.h"
 #include "alsa.h"
+#include "bat-signal.h"
 
 int retval_play;
 int retval_record;
 
+int is_capturing = 1;
+int is_playing = 1;
+
+/**
+ * Handling of Ctrl-C for capture
+ */
+void sigint_handler(int sig)
+{
+	is_capturing = 0;
+}
+
+/**
+ * Handling of Ctrl-C for playback
+ */
+void stream_close(int sig)
+{
+	/* allow the stream to be closed gracefully */
+	signal(sig, SIG_IGN);
+	is_playing = 0;
+}
+
 /* update chunk_fmt data to bat */
 static int update_fmt_to_bat(struct bat *bat, struct chunk_fmt *fmt)

{ @@ -196,3 +219,69 @@ int write_wav_header(FILE *fp, struct wav_container *wav, struct bat *bat)

bat/common.c +29 -0

 	return 0;
 }
+
+/* update wav header when data size changed */ int
+update_wav_header(struct bat *bat, FILE *fp, int bytes) {
+	int err = 0;
+	struct wav_container wav;
+
+	prepare_wav_info(&wav, bat);
+	wav.chunk.length = bytes;
+	wav.header.length = (wav.chunk.length) + sizeof(wav.chunk)
+		+ sizeof(wav.format) + sizeof(wav.header) - 8;
+	rewind(fp);
+	err = write_wav_header(fp, &wav, bat);
+
+	return err;
+}
+
+/*
+ * Generate buffer to be played either from input file or from
+generated data
+ * Return value
+ * <0 error
+ * 0 ok
+ * >0 break
+ */
+int generate_input_data0(struct bat *bat, void *buffer, int bytes,
+int frames) {
+	int err;
+	static int load;
+

bat/common.c int write_wav_header(FILE *fp, struct wav_container *wav, struct bat *bat)

Quoted 10 lines

bat/common.c +10 -0

+	if (bat->playback.file != NULL) {
+		/* From input file */
+		load = 0;
+
+		while (1) {
+			err = fread(buffer + load, 1, bytes - load, bat->fp);
+			if (0 == err) {
+				if (feof(bat->fp)) {
+					fprintf(bat->log,
+							_("End of

playing.\n"));

Quoted 233 lines

bat/common.c +27 -0

+					return 1;
+				}
+			} else if (err < bytes - load) {
+				if (ferror(bat->fp)) {
+					fprintf(bat->err, _("Read file error"));
+					fprintf(bat->err, _(": %d\n"), err);
+					return -EIO;
+				}
+				load += err;
+			} else {
+				break;
+			}
+		}
+	} else {
+		/* Generate sine wave */
+		if ((bat->sinus_duration) && (load > bat->sinus_duration))
+			return 1;
+
+		err = generate_sine_wave(bat, frames, buffer);
+		if (err != 0)
+			return err;
+
+		load += frames;
+	}
+
+	return 0;
+}

bat/common.h +9 -0

@@ -14,6 +14,9 @@
  */
 
 #include <alsa/asoundlib.h>
+#ifdef HAVE_LIBTINYALSA
+#include <tinyalsa/asoundlib.h>
+#endif
 
 #define TEMP_RECORD_FILE_NAME		"/tmp/bat.wav.XXXXXX"
 #define DEFAULT_DEV_NAME		"default"

@@ -119,6 +122,8 @@ enum _bat_op_mode {
 
 struct pcm {
 	char *device;
+	unsigned int card_tiny;
+	unsigned int device_tiny;
 	char *file;
 	enum _bat_op_mode mode;
 	void *(*fct)(struct bat *);

@@ -171,6 +176,8 @@ struct bat {
 	void *buf;			/* PCM Buffer */
 
 	bool local;			/* true for internal test */
+
+	bool tinyalsa;			/* true to use tinyalsa lib */
 };
 
 struct analyze {

@@ -180,6 +187,10 @@ struct analyze {
 	double *mag;
 };
 
+void sigint_handler(int);
+void stream_close(int);
 void prepare_wav_info(struct wav_container *, struct bat *);  int

read_wav_header(struct bat *, char *, FILE *, bool); int write_wav_header(FILE *, struct wav_container *, struct bat *);

bat/common.h +2 -0

+int update_wav_header(struct bat *, FILE *, int); int
+generate_input_data0(struct bat *, void *, int, int);

bat/tinyalsa.c new file +159 -0

index 0000000..ab11247
@@ -0,0 +1,422 @@
+/*
+ * Copyright (C) 2013-2015 Intel Corporation
+ *
+ * This program is free software; you can redistribute it and/or
+modify
+ * it under the terms of the GNU General Public License as published
+by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ */
+
+#include <stdio.h>
+#include <string.h>
+#include <stdbool.h>
+#include <signal.h>
+#include <pthread.h>
+#include <errno.h>
+
+#include <tinyalsa/asoundlib.h>
+
+#include "aconfig.h"
+#include "gettext.h"
+
+#include "common.h"
+#include "tinyalsa.h"
+
+struct format_map_table {
+	int sample_bytes;
+	enum pcm_format format;
+};
+
+static struct format_map_table map_tables[] = {
+	{ 2, PCM_FORMAT_S16_LE },
+	{ 4, PCM_FORMAT_S32_LE },
+	{}
+};
+
+/**
+ * Called when thread is finished
+ */
+static void close_handle(void *handle) {
+	struct pcm *pcm = handle;
+
+	if (NULL != pcm)
+		pcm_close(pcm);
+}
+
+static int format_convert(struct bat *bat, struct pcm_config *config)
+{
+	struct format_map_table *t = map_tables;
+
+	for (; t->sample_bytes; t++) {
+		if (t->sample_bytes == bat->sample_size) {
+			config->format = t->format;
+			return 0;
+		}
+	}
+
+	fprintf(bat->err, _("Invalid format!\n"));
+	return -EINVAL;
+}
+
+static int init_config(struct bat *bat, struct pcm_config *config) {
+	config->channels = bat->channels;
+	config->rate = bat->rate;
+	config->period_size = 1024;
+	config->period_count = 4;
+	config->start_threshold = 0;
+	config->stop_threshold = 0;
+	config->silence_threshold = 0;
+
+	return format_convert(bat, config);
+}
+
+/**
+ * Check that a parameter is inside bounds  */ static int
+check_param(struct bat *bat, struct pcm_params *params,
+		unsigned int param, unsigned int value,
+		char *param_name, char *param_unit) {
+	unsigned int min;
+	unsigned int max;
+	int ret = 0;
+
+	min = pcm_params_get_min(params, param);
+	if (value < min) {
+		fprintf(bat->err,
+			_("%s is %u%s, device only supports >= %u%s!\n"),
+			param_name, value, param_unit, min, param_unit);
+		ret = -EINVAL;
+	}
+
+	max = pcm_params_get_max(params, param);
+	if (value > max) {
+		fprintf(bat->err,
+			_("%s is %u%s, device only supports <= %u%s!\n"),
+			param_name, value, param_unit, max, param_unit);
+		ret = -EINVAL;
+	}
+
+	return ret;
+}
+
+/**
+ * Check all parameters
+ */
+static int check_playback_params(struct bat *bat,
+		struct pcm_config *config)
+{
+	struct pcm_params *params;
+	unsigned int card = bat->playback.card_tiny;
+	unsigned int device = bat->playback.device_tiny;
+	int err = 0;
+
+	params = pcm_params_get(card, device, PCM_OUT);
+	if (params == NULL) {
+		fprintf(bat->err, _("Unable to open PCM device %u!\n"),
+				device);
+		return -EINVAL;
+	}
+
+	err = check_param(bat, params, PCM_PARAM_RATE,
+			config->rate, "Sample rate", "Hz");
+	if (err < 0)
+		goto exit;
+	err = check_param(bat, params, PCM_PARAM_CHANNELS,
+			config->channels, "Sample", " channels");
+	if (err < 0)
+		goto exit;
+	err = check_param(bat, params, PCM_PARAM_SAMPLE_BITS,
+			bat->sample_size * 8, "Bitrate", " bits");
+	if (err < 0)
+		goto exit;
+	err = check_param(bat, params, PCM_PARAM_PERIOD_SIZE,
+			config->period_size, "Period size", "Hz");
+	if (err < 0)
+		goto exit;
+	err = check_param(bat, params, PCM_PARAM_PERIODS,
+			config->period_count, "Period count", "Hz");
+	if (err < 0)
+		goto exit;
+
+exit:
+	pcm_params_free(params);
+
+	return err;
+}
+
+/**
+ * Play sample
+ */
+static int play_sample(struct bat *bat, struct pcm *pcm,
+		void *buffer, int bytes)
+{

bat/tinyalsa.c line 169

Quoted 10 lines

bat/tinyalsa.c new file +10 -0

+	int err = 0;
+	FILE *fp = NULL;
+	int frames = bytes / bat->frame_size;
+	int bytes_total = 0;
+
+	if (bat->debugplay) {
+		fp = fopen(bat->debugplay, "wb");
+		if (fp == NULL) {
+			fprintf(bat->err, _("Cannot open file for capture: "));
+			fprintf(bat->err, _("%s %d\n"), bat->debugplay, -

errno);

Quoted 100 lines

bat/tinyalsa.c new file +26 -0

+			return -errno;
+		}
+		/* leave space for file header */
+		if (fseek(fp, sizeof(struct wav_container), SEEK_SET) != 0) {
+			fclose(fp);
+			return -errno;
+		}
+	}
+
+	do {
+		err = generate_input_data0(bat, buffer, bytes, frames);
+		if (err != 0)
+			break;
+
+		if (bat->debugplay) {
+			if (fwrite(buffer, 1, bytes, fp) != bytes) {
+				update_wav_header(bat, fp, bytes_total);
+				fclose(fp);
+				return -EIO;
+			}
+			bytes_total += bytes;
+		}
+
+		bat->periods_played++;
+		if (bat->period_is_limited
+				&& bat->periods_played >= bat-

periods_total)

bat/tinyalsa.c new file +73 -0

+			break;
+
+		err = pcm_write(pcm, buffer, bytes);
+		if (err != 0) {
+			fprintf(bat->err, _("Write PCM device error: %d\n"),
+					err);
+			break;
+		}
+	} while (is_playing);
+
+	if (bat->debugplay) {
+		err = update_wav_header(bat, fp, bytes_total);
+		fclose(fp);
+	}
+	return err;
+}
+
+/**
+ * Play
+ */
+void *playback_tinyalsa(struct bat *bat) {
+	int err = 0;
+	struct pcm_config config;
+	struct pcm *pcm = NULL;
+	void *buffer = NULL;
+	int bufbytes;
+	unsigned int card = bat->playback.card_tiny;
+	unsigned int device = bat->playback.device_tiny;
+
+	fprintf(bat->log, _("Entering playback thread (tinyalsa).\n"));
+
+	retval_play = 0;
+
+	/* init config */
+	err = init_config(bat, &config);
+	if (err < 0) {
+		retval_play = err;
+		goto exit1;
+	}
+
+	/* check param before open device */
+	err = check_playback_params(bat, &config);
+	if (err < 0) {
+		retval_play = err;
+		goto exit1;
+	}
+
+	/* init device */
+	pcm = pcm_open(card, device, PCM_OUT, &config);
+	if (!pcm || !pcm_is_ready(pcm)) {
+		fprintf(bat->err, _("Unable to open PCM device %u (%s)!\n"),
+				device, pcm_get_error(pcm));
+		retval_play = -EINVAL;
+		goto exit1;
+	}
+
+	/* init buffer */
+	bufbytes = pcm_frames_to_bytes(pcm, pcm_get_buffer_size(pcm));
+	buffer = malloc(bufbytes);
+	if (!buffer) {
+		retval_play = -ENOMEM;
+		goto exit2;
+	}
+
+	/* init playback source */
+	if (bat->playback.file == NULL) {
+		fprintf(bat->log, _("Playing generated audio sine wave"));
+		bat->sinus_duration == 0 ?
+			fprintf(bat->log, _(" endlessly\n")) :
+			fprintf(bat->log, _("\n"));
+	} else {
+		fprintf(bat->log, _("Playing input audio file: %s\n"),
+				bat->playback.file);

bat/tinyalsa.c

Quoted 10 lines

bat/tinyalsa.c new file +10 -0

+		bat->fp = fopen(bat->playback.file, "rb");
+		if (bat->fp == NULL) {
+			fprintf(bat->err, _("Cannot open file for playback: "));
+			fprintf(bat->err, _("%s %d\n"),
+					bat->playback.file, -errno);
+			retval_play = -errno;
+			goto exit3;
+		}
+		/* Skip header */
+		err = read_wav_header(bat, bat->playback.file, bat->fp,

true);

Quoted 175 lines

bat/tinyalsa.c new file +61 -0

+		if (err != 0) {
+			retval_play = err;
+			goto exit4;
+		}
+	}
+
+	/* catch ctrl-c to shutdown cleanly */
+	signal(SIGINT, stream_close);
+
+	err = play_sample(bat, pcm, buffer, bufbytes);
+	if (err < 0) {
+		retval_play = err;
+		goto exit4;
+	}
+
+exit4:
+	if (bat->playback.file)
+		fclose(bat->fp);
+exit3:
+	free(buffer);
+exit2:
+	pcm_close(pcm);
+exit1:
+	pthread_exit(&retval_play);
+}
+
+/**
+ * Capture sample
+ */
+static int capture_sample(struct bat *bat, struct pcm *pcm,
+		void *buffer, unsigned int bytes)
+{
+	int err = 0;
+	FILE *fp = NULL;
+	unsigned int bytes_read = 0;
+	unsigned int bytes_count = bat->frames * bat->frame_size;
+
+	remove(bat->capture.file);
+	fp = fopen(bat->capture.file, "wb");
+	if (fp == NULL) {
+		fprintf(bat->err, _("Cannot open file for capture: %s %d\n"),
+				bat->capture.file, -errno);
+		return -errno;
+	}
+	/* leave space for file header */
+	if (fseek(fp, sizeof(struct wav_container), SEEK_SET) != 0) {
+		fclose(fp);
+		return -errno;
+	}
+
+	while (bytes_read < bytes_count && is_capturing
+			&& !pcm_read(pcm, buffer, bytes)) {
+		if (fwrite(buffer, 1, bytes, fp) != bytes)
+			break;
+
+		bytes_read += bytes;
+
+		bat->periods_played++;
+
+		if (bat->period_is_limited
+				&& bat->periods_played >= bat-

periods_total)

bat/tinyalsa.c new file +79 -0

+			break;
+	}
+
+	err = update_wav_header(bat, fp, bytes_read);
+
+	fclose(fp);
+	return err;
+}
+
+/**
+ * Record
+ */
+void *record_tinyalsa(struct bat *bat) {
+	int err = 0;
+	struct pcm_config config;
+	struct pcm *pcm;
+	void *buffer;
+	unsigned int bufbytes;
+
+	pthread_setcancelstate(PTHREAD_CANCEL_DISABLE, NULL);
+
+	fprintf(bat->log, _("Entering capture thread (tinyalsa).\n"));
+
+	retval_record = 0;
+
+	/* init config */
+	err = init_config(bat, &config);
+	if (err < 0) {
+		retval_record = err;
+		goto exit1;
+	}
+
+	/* init device */
+	pcm = pcm_open(bat->capture.card_tiny, bat->capture.device_tiny,
+			PCM_IN, &config);
+	if (!pcm || !pcm_is_ready(pcm)) {
+		fprintf(bat->err, _("Unable to open PCM device (%s)!\n"),
+				pcm_get_error(pcm));
+		retval_record = -EINVAL;
+		goto exit1;
+	}
+
+	/* init buffer */
+	bufbytes = pcm_frames_to_bytes(pcm, pcm_get_buffer_size(pcm));
+	buffer = malloc(bufbytes);
+	if (!buffer) {
+		retval_record = -ENOMEM;
+		goto exit2;
+	}
+
+	/* install signal handler and begin capturing Ctrl-C */
+	signal(SIGINT, sigint_handler);
+
+	pthread_setcancelstate(PTHREAD_CANCEL_ENABLE, NULL);
+	pthread_setcanceltype(PTHREAD_CANCEL_DEFERRED, NULL);
+	pthread_cleanup_push(close_handle, pcm);
+	pthread_cleanup_push(free, buffer);
+
+	fprintf(bat->log, _("Recording ...\n"));
+	err = capture_sample(bat, pcm, buffer, bufbytes);
+	if (err != 0) {
+		retval_record = err;
+		goto exit3;
+	}
+
+	/* Normally we will never reach this part of code (unless error in
+	 * previous call) (before exit3) as this thread will be cancelled
+	 *  by end of play thread. Except in single line mode. */
+	pthread_cleanup_pop(0);
+	pthread_cleanup_pop(0);
+	pthread_exit(&retval_record);
+
+exit3:
+	free(buffer);
+exit2:
+	pcm_close(pcm);
+exit1:
+	pthread_exit(&retval_record);
+}

bat/tinyalsa.h new file +25 -0

index 0000000..70e4749
@@ -0,0 +1,23 @@
+/*
+ * Copyright (C) 2013-2015 Intel Corporation
+ *
+ * This program is free software; you can redistribute it and/or
+modify
+ * it under the terms of the GNU General Public License as published
+by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ */
+
+extern int retval_play;
+extern int retval_record;
+
+extern int is_capturing;
+extern int is_playing;
+
+void *playback_tinyalsa(struct bat *); void *record_tinyalsa(struct
+bat *);

configure.ac if test x$bat = xtrue; then · line 80

Quoted 8 lines

configure.ac +3 -0

@@ -73,6 +73,9 @@ if test x$bat = xtrue; then
   dnl Check for libfftw3
   have_libfftw3="yes"
   AC_CHECK_LIB([fftw3], [fftw_malloc], , [have_libfftw3="no"])
+  dnl Check for libtinyalsa
+  have_libtinyalsa="yes"
+  AC_CHECK_LIB([tinyalsa], [pcm_open], , [have_libtinyalsa="no"])
   AC_CHECK_LIB([m], [sqrtf], , [AC_MSG_ERROR([Error: Need sqrtf])])
   AC_CHECK_LIB([pthread], [pthread_create], , [AC_MSG_ERROR([Error:

need PTHREAD library])])

Quoted 17 lines

configure.ac +1 -1

   FFTW_CFLAGS="$CFLAGS"

@@ -86,6 +89,7 @@ if test x$bat = xtrue; then
 
 fi
 AM_CONDITIONAL(HAVE_LIBFFTW3, test "$have_libfftw3" = "yes")
+AM_CONDITIONAL(HAVE_LIBTINYALSA, test "$have_libtinyalsa" = "yes")
 
 dnl Check for librt
 LIBRT=""
--

2.5.0

_______________________________________________
Alsa-devel mailing list
[email protected]
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

From: "Lu, Han" <[email protected]>

Replace fixed "1" to variable for thread return value.

Signed-off-by Lu, Han <[email protected]>

bat/alsa.c +2 -2

index 0a5f899..189b0e9 100644
@@ -309,13 +309,13 @@ void *playback_alsa(struct bat *bat)
 	if (err != 0) {
 		fprintf(bat->err, _("Cannot open PCM playback device: "));
 		fprintf(bat->err, _("%s(%d)\n"), snd_strerror(err), err);
-		retval_play = 1;
+		retval_play = err;
 		goto exit1;
 	}
 
 	err = set_snd_pcm_params(bat, &sndpcm);
 	if (err != 0) {
-		retval_play = 1;
+		retval_play = err;
 		goto exit2;
 	}
 

-----Original Message-----
From: Takashi Iwai [mailto:[email protected]]
Sent: Friday, March 11, 2016 9:34 PM
To: Lu, Han <[email protected]>
Cc: [email protected]; Gautier, Bernard <[email protected]>; Popescu, Edward C <[email protected]>; [email protected]
Subject: Re: [alsa-devel] [PATCH 09/10] alsabat: use variable for thread return value

Quoted 27 lines · On Wed, 02 Mar 2016 09:53:19 +0100, [email protected] wrote:

From: "Lu, Han" <[email protected]>

Replace fixed "1" to variable for thread return value.

Signed-off-by Lu, Han <[email protected]>

bat/alsa.c +2 -2

index 0a5f899..189b0e9 100644
@@ -309,13 +309,13 @@ void *playback_alsa(struct bat *bat)
 	if (err != 0) {
 		fprintf(bat->err, _("Cannot open PCM playback device: "));
 		fprintf(bat->err, _("%s(%d)\n"), snd_strerror(err), err);
-		retval_play = 1;
+		retval_play = err;
 		goto exit1;
 	}
 
 	err = set_snd_pcm_params(bat, &sndpcm);
 	if (err != 0) {
-		retval_play = 1;
+		retval_play = err;
 		goto exit2;
 	}
 

bat/alsa.c void *playback_alsa(struct bat *bat) · line 335

Quoted 6 lines

bat/alsa.c +1 -1

@@ -332,13 +332,13 @@ void *playback_alsa(struct bat *bat)
 			fprintf(bat->err, _("Cannot open file for playback: "));
 			fprintf(bat->err, _("%s %d\n"),
 					bat->playback.file, -errno);
-			retval_play = 1;
+			retval_play = -errno;

Is the original errno still preserved at this point...?

Takashi

Hi Takashi,

Quoted 39 lines

-----Original Message-----
From: Takashi Iwai [mailto:[email protected]]
Sent: Friday, March 11, 2016 9:34 PM
To: Lu, Han <[email protected]>
Cc: [email protected]; Gautier, Bernard <[email protected]>; Popescu, Edward C <[email protected]>; [email protected]
Subject: Re: [alsa-devel] [PATCH 09/10] alsabat: use variable for thread return value

Quoted 27 lines · On Wed, 02 Mar 2016 09:53:19 +0100, [email protected] wrote:

From: "Lu, Han" <[email protected]>

Replace fixed "1" to variable for thread return value.

Signed-off-by Lu, Han <[email protected]>

bat/alsa.c +2 -2

index 0a5f899..189b0e9 100644
@@ -309,13 +309,13 @@ void *playback_alsa(struct bat *bat)
 	if (err != 0) {
 		fprintf(bat->err, _("Cannot open PCM playback device: "));
 		fprintf(bat->err, _("%s(%d)\n"), snd_strerror(err), err);
-		retval_play = 1;
+		retval_play = err;
 		goto exit1;
 	}
 
 	err = set_snd_pcm_params(bat, &sndpcm);
 	if (err != 0) {
-		retval_play = 1;
+		retval_play = err;
 		goto exit2;
 	}
 

Quoted 8 lines

bat/alsa.c void *playback_alsa(struct bat *bat) · line 335

Quoted 6 lines

bat/alsa.c +1 -1

@@ -332,13 +332,13 @@ void *playback_alsa(struct bat *bat)
 			fprintf(bat->err, _("Cannot open file for playback: "));
 			fprintf(bat->err, _("%s %d\n"),
 					bat->playback.file, -errno);
-			retval_play = 1;
+			retval_play = -errno;

Is the original errno still preserved at this point...?

[han] I think so, the complete code section here is

	...
	bat->fp = fopen(bat->playback.file, "rb");
	if (bat->fp == NULL) {
		fprintf(bat->err, _("Cannot open file for playback: "));
		fprintf(bat->err, _("%s %d\n"),
				bat->playback.file, -errno);

bat/alsa.c +1 -1

-		retval_play = 1;
+		retval_play = -errno;

		goto exit3;
	}
	...

So the use of errno should be safe.

-----Original Message-----
From: Takashi Iwai [mailto:[email protected]]
Sent: Monday, March 14, 2016 5:21 PM
To: Lu, Han <[email protected]>
Cc: [email protected]; Gautier, Bernard <[email protected]>; Popescu, Edward C <[email protected]>; [email protected]
Subject: Re: [alsa-devel] [PATCH 09/10] alsabat: use variable for thread return value

Quoted 62 lines · Han

Hi Takashi,

Quoted 38 lines

-----Original Message-----
From: Takashi Iwai [mailto:[email protected]]
Sent: Friday, March 11, 2016 9:34 PM
To: Lu, Han <[email protected]>
Cc: [email protected]; Gautier, Bernard <[email protected]>; Popescu, Edward C <[email protected]>; [email protected]
Subject: Re: [alsa-devel] [PATCH 09/10] alsabat: use variable for thread return value

Quoted 26 lines · On Wed, 02 Mar 2016 09:53:19 +0100, [email protected] wrote:

From: "Lu, Han" <[email protected]>

Replace fixed "1" to variable for thread return value.

Signed-off-by Lu, Han <[email protected]>

bat/alsa.c +2 -2

@@ -309,13 +309,13 @@ void *playback_alsa(struct bat *bat)
 	if (err != 0) {
 		fprintf(bat->err, _("Cannot open PCM playback device: "));
 		fprintf(bat->err, _("%s(%d)\n"), snd_strerror(err), err);
-		retval_play = 1;
+		retval_play = err;
 		goto exit1;
 	}
 
 	err = set_snd_pcm_params(bat, &sndpcm);
 	if (err != 0) {
-		retval_play = 1;
+		retval_play = err;
 		goto exit2;
 	}
 

Quoted 8 lines

bat/alsa.c void *playback_alsa(struct bat *bat) · line 335

Quoted 6 lines

bat/alsa.c +1 -1

@@ -332,13 +332,13 @@ void *playback_alsa(struct bat *bat)
 			fprintf(bat->err, _("Cannot open file for playback: "));
 			fprintf(bat->err, _("%s %d\n"),
 					bat->playback.file, -errno);
-			retval_play = 1;
+			retval_play = -errno;

Is the original errno still preserved at this point...?

[han] I think so, the complete code section here is

	...
	bat->fp = fopen(bat->playback.file, "rb");
	if (bat->fp == NULL) {
		fprintf(bat->err, _("Cannot open file for playback: "));
		fprintf(bat->err, _("%s %d\n"),
				bat->playback.file, -errno);

bat/alsa.c +1 -1

-		retval_play = 1;
+		retval_play = -errno;

		goto exit3;
	}
	...

Quoted 1 line

So the use of errno should be safe.

You call a bunch of functions between the fopen() error and the reference of errno. The errno reference should be done immediately after the error.

Takashi

Hi Takashi,

Quoted 80 lines

-----Original Message-----
From: Takashi Iwai [mailto:[email protected]]
Sent: Monday, March 14, 2016 5:21 PM
To: Lu, Han <[email protected]>
Cc: [email protected]; Gautier, Bernard <[email protected]>; Popescu, Edward C <[email protected]>; [email protected]
Subject: Re: [alsa-devel] [PATCH 09/10] alsabat: use variable for thread return value

Quoted 62 lines · Han

Hi Takashi,

Quoted 38 lines

-----Original Message-----
From: Takashi Iwai [mailto:[email protected]]
Sent: Friday, March 11, 2016 9:34 PM
To: Lu, Han <[email protected]>
Cc: [email protected]; Gautier, Bernard <[email protected]>; Popescu, Edward C <[email protected]>; [email protected]
Subject: Re: [alsa-devel] [PATCH 09/10] alsabat: use variable for thread return value

Quoted 26 lines · On Wed, 02 Mar 2016 09:53:19 +0100, [email protected] wrote:

From: "Lu, Han" <[email protected]>

Replace fixed "1" to variable for thread return value.

Signed-off-by Lu, Han <[email protected]>

bat/alsa.c +2 -2

@@ -309,13 +309,13 @@ void *playback_alsa(struct bat *bat)
 	if (err != 0) {
 		fprintf(bat->err, _("Cannot open PCM playback device: "));
 		fprintf(bat->err, _("%s(%d)\n"), snd_strerror(err), err);
-		retval_play = 1;
+		retval_play = err;
 		goto exit1;
 	}
 
 	err = set_snd_pcm_params(bat, &sndpcm);
 	if (err != 0) {
-		retval_play = 1;
+		retval_play = err;
 		goto exit2;
 	}
 

Quoted 8 lines

bat/alsa.c void *playback_alsa(struct bat *bat) · line 335

Quoted 6 lines

bat/alsa.c +1 -1

@@ -332,13 +332,13 @@ void *playback_alsa(struct bat *bat)
 			fprintf(bat->err, _("Cannot open file for playback: "));
 			fprintf(bat->err, _("%s %d\n"),
 					bat->playback.file, -errno);
-			retval_play = 1;
+			retval_play = -errno;

Is the original errno still preserved at this point...?

[han] I think so, the complete code section here is

	...
	bat->fp = fopen(bat->playback.file, "rb");
	if (bat->fp == NULL) {
		fprintf(bat->err, _("Cannot open file for playback: "));
		fprintf(bat->err, _("%s %d\n"),
				bat->playback.file, -errno);

bat/alsa.c +1 -1

-		retval_play = 1;
+		retval_play = -errno;

		goto exit3;
	}
	...

Quoted 1 line

So the use of errno should be safe.

You call a bunch of functions between the fopen() error and the reference of errno. The errno reference should be done immediately after the error.

Quoted 1 line

Takashi

[han] sorry! I ignored the errno could be changed by the printf(). I'll rewrite the patch and fix other similar errors.

-----Original Message-----
From: Takashi Iwai [mailto:[email protected]]
Sent: Monday, March 14, 2016 5:44 PM
To: Lu, Han <[email protected]>
Cc: [email protected]; Gautier, Bernard <[email protected]>; Popescu, Edward C <[email protected]>; [email protected]
Subject: Re: [alsa-devel] [PATCH 09/10] alsabat: use variable for thread return value

Quoted 36 lines · Han

Hi Takashi,

Quoted 33 lines

-----Original Message-----
From: Takashi Iwai [mailto:[email protected]]
Sent: Monday, March 14, 2016 5:21 PM
To: Lu, Han <[email protected]>
Cc: [email protected]; Gautier, Bernard <[email protected]>; Popescu, Edward C <[email protected]>; [email protected]
Subject: Re: [alsa-devel] [PATCH 09/10] alsabat: use variable for thread return value

Quoted 21 lines · Han

Hi Takashi,

Quoted 18 lines

-----Original Message-----
From: Takashi Iwai [mailto:[email protected]]
Sent: Friday, March 11, 2016 9:34 PM
To: Lu, Han <[email protected]>
Cc: [email protected]; Gautier, Bernard <[email protected]>; Popescu, Edward C <[email protected]>; [email protected]
Subject: Re: [alsa-devel] [PATCH 09/10] alsabat: use variable for thread return value

Quoted 7 lines · On Wed, 02 Mar 2016 09:53:19 +0100, [email protected] wrote:

From: "Lu, Han" <[email protected]>

Replace fixed "1" to variable for thread return value.

Signed-off-by Lu, Han <[email protected]>

Quoted 7 lines

Quoted 7 lines

Quoted 7 lines

Quoted 7 lines

bat/alsa.c void *playback_alsa(struct bat *bat) · line 310

Quoted 7 lines

bat/alsa.c index 0a5f899..189b0e9 renamed from bat/alsa.c +0 -0

100644

bat/alsa.c +0 -0

@@ -309,13 +309,13 @@ void *playback_alsa(struct bat *bat)
 	if (err != 0) {
 		fprintf(bat->err, _("Cannot open PCM playback

device: "));

Quoted 1 line

Quoted 1 line

Quoted 1 line

Quoted 1 line

bat/alsa.c void *playback_alsa(struct bat *bat) · line 311

Quoted 1 line

bat/alsa.c +0 -0

 		fprintf(bat->err, _("%s(%d)\n"), snd_strerror(err),

err);

Quoted 12 lines

Quoted 12 lines

Quoted 12 lines

Quoted 12 lines

Quoted 12 lines

bat/alsa.c +2 -2

-		retval_play = 1;
+		retval_play = err;
 		goto exit1;
 	}
 
 	err = set_snd_pcm_params(bat, &sndpcm);
 	if (err != 0) {
-		retval_play = 1;
+		retval_play = err;
 		goto exit2;
 	}
 

Quoted 2 lines

Quoted 2 lines

Quoted 2 lines

Quoted 2 lines

bat/alsa.c void *playback_alsa(struct bat *bat) · line 332

Quoted 2 lines

bat/alsa.c +0 -0

@@ -332,13 +332,13 @@ void *playback_alsa(struct bat *bat)
 			fprintf(bat->err, _("Cannot open file for

playback: "));

Quoted 7 lines

Quoted 7 lines

Quoted 7 lines

Quoted 6 lines

bat/alsa.c void *playback_alsa(struct bat *bat) · line 335

Quoted 4 lines

bat/alsa.c +1 -1

 			fprintf(bat->err, _("%s %d\n"),
 					bat->playback.file, -errno);
-			retval_play = 1;
+			retval_play = -errno;

Is the original errno still preserved at this point...?

Quoted 16 lines

Quoted 8 lines

Quoted 8 lines

[han] I think so, the complete code section here is

	...
	bat->fp = fopen(bat->playback.file, "rb");
	if (bat->fp == NULL) {
		fprintf(bat->err, _("Cannot open file for playback: "));
		fprintf(bat->err, _("%s %d\n"),
				bat->playback.file, -errno);

(quoted diff) +0 -1

-		retval_play = 1;

Quoted 8 lines

Quoted 5 lines

bat/alsa.c +1 -0

+		retval_play = -errno;

		goto exit3;
	}
	...

So the use of errno should be safe.

You call a bunch of functions between the fopen() error and the reference of errno. The errno reference should be done immediately

after the error.

Quoted 6 lines

Quoted 3 lines

Takashi

[han] sorry! I ignored the errno could be changed by the printf(). I'll rewrite the patch and fix other similar errors.

While we're at it: at the next respin, could you split the patchset, one for trivial fixes that can be applied immediately, and another patchset for extending the feature? Mixing up both fixes and enhancements made it difficult to pick up each patch. In that way, I'll be able to apply the fix patches more quickly while reviewing the enhancement patches more intensively.

thanks,

Takashi

From: "Lu, Han" <[email protected]>

Add bash script for alsabat feature test.
Examples for alsa tests:
./bat/testbat.sh
Examples for tinyalsa tests:

        ./bat/testbat.sh plughw:1,0 plughw:1,0 1

Signed-off-by Lu, Han <[email protected]>

bat/testbat.sh new file +150 -0

new file mode 100644
index 0000000..93b9f70
@@ -0,0 +1,160 @@
+#!/bin/bash
+
+exe="alsabat"
+file_sin="default.wav"
+file_sin_dual="default_dual.wav"
+logdir="tmp"
+# default devices
+dev_playback="default"
+dev_capture="default"
+# features passes vs. features all
+feature_pass=0
+feature_cnt=0
+
+commands="$exe -P $dev_playback -C $dev_capture"
+
+init_counter () {
+	feature_pass=0
+	feature_all=0
+}
+
+evaluate_result () {
+	feature_cnt=$((feature_cnt+1))
+	if [ $1 -eq 0 ]; then
+		feature_pass=$((feature_pass+1))
+		echo "pass"
+	else
+		echo "fail"
+	fi
+}
+
+print_result () {
+	echo "[$feature_pass/$feature_cnt] features passes."
+}
+
+feature_test () {
+	echo "============================================"
+	echo "$feature_cnt: ALSA $2"
+	echo "-------------------------------------------"
+	echo "$commands $1 --log=$logdir/$feature_cnt.log"
+	$commands $1 --log=$logdir/$feature_cnt.log
+	evaluate_result $?
+	echo "$commands $1" >> $logdir/$((feature_cnt-1)).log
+}
+
+# test items
+feature_list_test () {
+	commands="$exe"
+	feature_test "--saveplay ${file_sin}" \
+			"generate test file with default params"
+	sleep 5
+	feature_test "-P $dev_playback" "single line mode, playback"
+	feature_test "-C $dev_capture --standalone" "single line mode, capture"
+	commands="$exe -P $dev_playback -C $dev_capture"
+	feature_test "--file ${file_sin}" "play wav file and detect"
+	feature_test "" "generate sine wave and detect"
+	feature_test "-c1" "configurable channel number: 1"
+	feature_test "-c2 -F 19:16757" "configurable channel number: 2"
+	feature_test "-r44100" "configurable sample rate: 44100"
+	feature_test "-r48000" "configurable sample rate: 48000"
+	feature_test "-n16387" "configurable duration: in samples"
+	feature_test "-n2.5s" "configurable duration: in seconds"
+	feature_test "-f U8 --saveplay U8.wav" "configurable data depth: 8 bit"
+	feature_test "-f S16_LE --saveplay S16_LE.wav" \
+			"configurable data depth: 16 bit"
+	feature_test "-f S24_3LE --saveplay S24_3LE.wav" \
+			"configurable data depth: 24 bit"
+	feature_test "-f S32_LE --saveplay S32_LE.wav" \
+			"configurable data depth: 32 bit"
+	feature_test "-f cd --saveplay cd.wav" "configurable data depth: cd"
+	feature_test "-f dat --saveplay dat.wav" "configurable data depth: dat"
+	tmpfreq=17583
+	feature_test "-F $tmpfreq --standalone" \
+			"standalone mode: play and capture"
+	latestfile=`ls -t1 /tmp/bat.wav.* | head -n 1`
+	feature_test "--local -F $tmpfreq --file $latestfile" \
+			"local mode: analyze local file"
+
+	print_result
+}
+
+feature_test_tiny () {
+	echo "============================================"
+	echo "$feature_cnt: tinyalsa $2"
+	echo "-------------------------------------------"
+	echo "$commands -c2 -t $1 --log=$logdir/$feature_cnt.log"
+	$commands -c2 -t $1 --log=$logdir/$feature_cnt.log
+	evaluate_result $?
+	echo "$commands -c2 -t $1" >> $logdir/$((feature_cnt-1)).log
+}
+
+# tinyalsa test items; device may not support "default" name nor some formats
+feature_list_test_tiny () {
+	commands="$exe"
+	feature_test_tiny "-P $dev_playback" "single line mode, playback"
+	feature_test_tiny "-C $dev_capture --standalone" \
+			"single line mode, capture"
+	commands="$exe -P $dev_playback -C $dev_capture"
+	feature_test_tiny "--saveplay ${file_sin_dual}" \
+			"generate sine wave and detect"
+	feature_test_tiny "--file ${file_sin_dual}" "play wav file and detect"
+	feature_test_tiny "-F 19:16757" "configurable channel number: 2"
+	feature_test_tiny "-r44100" "configurable sample rate: 44100"
+	feature_test_tiny "-r48000" "configurable sample rate: 48000"
+	feature_test_tiny "-n16387" "configurable duration: in samples"
+	feature_test_tiny "-n2.5s" "configurable duration: in seconds"
+	feature_test_tiny "-f S16_LE --saveplay S16_LE.wav" \
+			"configurable data depth: 16 bit"
+	feature_test_tiny "-f S32_LE --saveplay S32_LE.wav" \
+			"configurable data depth: 32 bit"
+	feature_test_tiny "-f cd --saveplay cd.wav" \
+			"configurable data depth: cd"
+	feature_test_tiny "-f dat --saveplay dat.wav" \
+			"configurable data depth: dat"
+	tmpfreq=17583
+	feature_test_tiny "-F $tmpfreq --standalone" \
+			"standalone mode: play and capture"
+	latestfile=`ls -t1 /tmp/bat.wav.* | head -n 1`
+	feature_test_tiny "--local -F $tmpfreq --file $latestfile" \
+			"local mode: analyze local file"
+
+	print_result
+}
+
+echo "*******************************************"
+echo "                BAT Test                   "
+echo "-------------------------------------------"
+
+# get device
+echo "usage:"
+echo "  $0 <sound card>"
+echo "  $0 <device-playback> <device-capture> <1 for tinyalsa, blank for alsa>"
+
+use_tinyalsa="0"
+
+if [ $# -eq 3 ]; then
+	dev_playback=$1
+	dev_capture=$2
+	use_tinyalsa=$3
+elif [ $# -eq 2 ]; then
+	dev_playback=$1
+	dev_capture=$2
+elif [ $# -eq 1 ]; then
+	dev_playback=$1
+	dev_capture=$1
+fi
+
+echo "current setting:"
+echo "  $0 $dev_playback $dev_capture $3"
+
+# run