[alsa-devel] [RFCv2][PATCH 19/38] axfer: add an option to dump available hardware parameters

[Takashi Sakamoto]

In ALSA PCM interface, before configuring hardware actually, applications
can request available set of hardware parameters for runtime of PCM
substream. The set of parameters are represented and delivered by a
structure.

In alsa-lib PCM API, the above design is abstracted by a series of
snd_pcm_hw_params_xxx() functions. The structure is hidden from
applications.

In aplay, '--dump-hw-params' option is for this purpose. With this option,
the command output available set of the hardware parameters.

This commit adds support for the option. Unlike aplay, this commit takes
this program to finish after dumping the parameters for simplicity of
usage.

I note that all of combinations in the set are not necessarily available
when the PCM substream includes dependencies of parameters described by
constraints and rules.

Signed-off-by: Takashi Sakamoto <o-takashi at sakamocchi.jp>
---
 axfer/options.c         |  9 +++++++++
 axfer/options.h         |  3 +++
 axfer/subcmd-transfer.c |  2 ++
 axfer/xfer-libasound.c  | 11 +++++++++++
 4 files changed, 25 insertions(+)

diff --git a/axfer/options.c b/axfer/options.c
index a094ccb5..00b088bf 100644
--- a/axfer/options.c
+++ b/axfer/options.c
@@ -12,6 +12,11 @@
 #include <getopt.h>
 #include <math.h>
 
+enum no_short_opts {
+	/* 128 belongs to non us-ascii character set. */
+	OPT_DUMP_HW_PARAMS = 128,
+};
+
 static const char *const allowed_duplication[] = {
 	"/dev/null",
 	"/dev/zero",
@@ -278,6 +283,8 @@ int context_options_init(struct context_options *opts, int argc,
 		{"format",		1, 0, 'f'},
 		{"channels",		1, 0, 'c'},
 		{"rate",		1, 0, 'r'},
+		/* For debugging. */
+		{"dump-hw-params",	0, 0, OPT_DUMP_HW_PARAMS},
 		{NULL,			0, 0, 0},
 	};
 	const char *cntr_format_literal = NULL;
@@ -310,6 +317,8 @@ int context_options_init(struct context_options *opts, int argc,
 			opts->samples_per_frame = parse_l(optarg, &err);
 		else if (c == 'r')
 			opts->frames_per_second = parse_l(optarg, &err);
+		else if (c == OPT_DUMP_HW_PARAMS)
+			opts->dump_hw_params = true;
 		else
 			continue;
 
diff --git a/axfer/options.h b/axfer/options.h
index 9c9b46d7..743279af 100644
--- a/axfer/options.h
+++ b/axfer/options.h
@@ -31,6 +31,9 @@ struct context_options {
 	snd_pcm_format_t sample_format;
 	unsigned int samples_per_frame;
 	unsigned int frames_per_second;
+
+	/* For debugging. */
+	bool dump_hw_params;
 };
 
 int context_options_init(struct context_options *opts, int argc,
diff --git a/axfer/subcmd-transfer.c b/axfer/subcmd-transfer.c
index da11f43f..3410fa30 100644
--- a/axfer/subcmd-transfer.c
+++ b/axfer/subcmd-transfer.c
@@ -476,6 +476,8 @@ int subcmd_transfer(int argc, char *const *argv, snd_pcm_stream_t direction)
 	err = context_init(&ctx, direction);
 	if (err < 0)
 		goto end;
+	if (ctx.opts.dump_hw_params)
+		goto end;
 
 	err = context_pre_process(&ctx, direction, &expected_frame_count);
 	if (err < 0)
diff --git a/axfer/xfer-libasound.c b/axfer/xfer-libasound.c
index f4a1f51a..4ad01bbb 100644
--- a/axfer/xfer-libasound.c
+++ b/axfer/xfer-libasound.c
@@ -68,6 +68,17 @@ static int xfer_libasound_init(struct xfer_context *xfer,
 
 	/* TODO: Applying NO_PERIOD_WAKEUP should be done here. */
 
+	if (opts->dump_hw_params) {
+		logging(state, _("Available HW Params of node: %s\n"),
+			snd_pcm_name(state->handle));
+		snd_pcm_hw_params_dump(state->hw_params, state->log);
+		/*
+		 * TODO: there're more parameters which are not dumped by
+		 * alsa-lib.
+		 */
+		return 0;
+	}
+
 	return set_access_hw_param(state->handle, state->hw_params, opts);
 }
 
-- 
2.11.0