diff mbox

[22/39] fireworks: Add proc interface for debugging purpose

Message ID 1393558072-25926-23-git-send-email-o-takashi@sakamocchi.jp (mailing list archive)
State Changes Requested
Delegated to: Takashi Iwai
Headers show

Commit Message

Takashi Sakamoto Feb. 28, 2014, 3:27 a.m. UTC
This commit adds proc interface to output infomation for debugging.
 - firmware information
 - sampling rate and clock source
 - physical metering (linear value)

Signed-off-by: Takashi Sakamoto <o-takashi@sakamocchi.jp>
---
 sound/firewire/fireworks/Makefile         |   2 +-
 sound/firewire/fireworks/fireworks.c      |  14 ++-
 sound/firewire/fireworks/fireworks.h      |  11 ++
 sound/firewire/fireworks/fireworks_proc.c | 189 ++++++++++++++++++++++++++++++
 4 files changed, 213 insertions(+), 3 deletions(-)
 create mode 100644 sound/firewire/fireworks/fireworks_proc.c

Comments

Takashi Iwai Feb. 28, 2014, 6:54 a.m. UTC | #1
At Fri, 28 Feb 2014 12:27:35 +0900,
Takashi Sakamoto wrote:
> 
> This commit adds proc interface to output infomation for debugging.
>  - firmware information
>  - sampling rate and clock source
>  - physical metering (linear value)
> 
> Signed-off-by: Takashi Sakamoto <o-takashi@sakamocchi.jp>
> ---
>  sound/firewire/fireworks/Makefile         |   2 +-
>  sound/firewire/fireworks/fireworks.c      |  14 ++-
>  sound/firewire/fireworks/fireworks.h      |  11 ++
>  sound/firewire/fireworks/fireworks_proc.c | 189 ++++++++++++++++++++++++++++++
>  4 files changed, 213 insertions(+), 3 deletions(-)
>  create mode 100644 sound/firewire/fireworks/fireworks_proc.c
> 
> diff --git a/sound/firewire/fireworks/Makefile b/sound/firewire/fireworks/Makefile
> index 1bccb65..52bd15e 100644
> --- a/sound/firewire/fireworks/Makefile
> +++ b/sound/firewire/fireworks/Makefile
> @@ -1,3 +1,3 @@
>  snd-fireworks-objs := fireworks_transaction.o fireworks_command.o \
> -		      fireworks_stream.o fireworks.o
> +		      fireworks_stream.o fireworks_proc.o fireworks.o
>  obj-m += snd-fireworks.o
> diff --git a/sound/firewire/fireworks/fireworks.c b/sound/firewire/fireworks/fireworks.c
> index 6d7de19..e370204 100644
> --- a/sound/firewire/fireworks/fireworks.c
> +++ b/sound/firewire/fireworks/fireworks.c
> @@ -130,6 +130,16 @@ get_hardware_info(struct snd_efw *efw)
>  	efw->pcm_playback_channels[0] = hwinfo->amdtp_rx_pcm_channels;
>  	efw->pcm_playback_channels[1] = hwinfo->amdtp_rx_pcm_channels_2x;
>  	efw->pcm_playback_channels[2] = hwinfo->amdtp_rx_pcm_channels_4x;
> +
> +	/* hardware metering */
> +	efw->phys_out = hwinfo->phys_out;
> +	efw->phys_in = hwinfo->phys_in;
> +	efw->phys_out_grp_count = hwinfo->phys_out_grp_count;
> +	efw->phys_in_grp_count = hwinfo->phys_in_grp_count;
> +	memcpy(&efw->phys_out_grps, hwinfo->phys_out_grps,
> +	       sizeof(struct snd_efw_phys_grp) * HWINFO_MAX_CAPS_GROUPS);
> +	memcpy(&efw->phys_in_grps, hwinfo->phys_in_grps,
> +	       sizeof(struct snd_efw_phys_grp) * HWINFO_MAX_CAPS_GROUPS);
>  end:
>  	kfree(hwinfo);
>  	return err;
> @@ -189,6 +199,8 @@ efw_probe(struct fw_unit *unit,
>  	if (err < 0)
>  		goto error;
>  
> +	snd_efw_proc_init(efw);
> +
>  	err = snd_card_register(card);
>  	if (err < 0)
>  		goto error;
> @@ -211,8 +223,6 @@ static void efw_update(struct fw_unit *unit)
>  
>  	snd_efw_transaction_bus_reset(efw->unit);
>  	snd_efw_stream_update_duplex(efw);
> -
> -	return;
>  }
>  
>  static void efw_remove(struct fw_unit *unit)
> diff --git a/sound/firewire/fireworks/fireworks.h b/sound/firewire/fireworks/fireworks.h
> index c022c71..4d69c0d 100644
> --- a/sound/firewire/fireworks/fireworks.h
> +++ b/sound/firewire/fireworks/fireworks.h
> @@ -21,6 +21,7 @@
>  #include <sound/core.h>
>  #include <sound/initval.h>
>  #include <sound/pcm.h>
> +#include <sound/info.h>
>  
>  #include "../packets-buffer.h"
>  #include "../iso-resources.h"
> @@ -65,6 +66,14 @@ struct snd_efw {
>  	struct cmp_connection in_conn;
>  	unsigned int capture_substreams;
>  	unsigned int playback_substreams;
> +
> +	/* hardware metering parameters */
> +	unsigned int phys_out;
> +	unsigned int phys_in;
> +	unsigned int phys_out_grp_count;
> +	unsigned int phys_in_grp_count;
> +	struct snd_efw_phys_grp phys_out_grps[HWINFO_MAX_CAPS_GROUPS];
> +	struct snd_efw_phys_grp phys_in_grps[HWINFO_MAX_CAPS_GROUPS];
>  };
>  
>  struct snd_efw_transaction {
> @@ -178,6 +187,8 @@ int snd_efw_stream_stop_duplex(struct snd_efw *efw);
>  void snd_efw_stream_update_duplex(struct snd_efw *efw);
>  void snd_efw_stream_destroy_duplex(struct snd_efw *efw);
>  
> +void snd_efw_proc_init(struct snd_efw *efw);
> +
>  #define SND_EFW_DEV_ENTRY(vendor, model) \
>  { \
>  	.match_flags	= IEEE1394_MATCH_VENDOR_ID | \
> diff --git a/sound/firewire/fireworks/fireworks_proc.c b/sound/firewire/fireworks/fireworks_proc.c
> new file mode 100644
> index 0000000..62f758b
> --- /dev/null
> +++ b/sound/firewire/fireworks/fireworks_proc.c
> @@ -0,0 +1,189 @@
> +/*
> + * fireworks_proc.c - a part of driver for Fireworks based devices
> + *
> + * Copyright (c) 2009-2010 Clemens Ladisch
> + * Copyright (c) 2013 Takashi Sakamoto
> + *
> + * Licensed under the terms of the GNU General Public License, version 2.
> + */
> +
> +#include "./fireworks.h"
> +
> +static inline const char*
> +get_phys_name(struct snd_efw_phys_grp *grp)
> +{
> +	const char *ch_type[] = {
> +		"Analog", "S/PDIF", "ADAT", "S/PDIF or ADAT",
> +		"Mirroring", "Headphones", "I2S", "Guitar",
> +		"Pirzo Guitar", "Guitar String", "Virtual", "Dummy"
> +	};
> +
> +	if (grp->type < 10)
> +		return ch_type[grp->type];
> +	else if (grp->type == 0x10000)
> +		return ch_type[10];
> +	else
> +		return ch_type[11];
> +}
> +
> +static void
> +proc_read_hwinfo(struct snd_info_entry *entry, struct snd_info_buffer *buffer)
> +{
> +	struct snd_efw *efw = entry->private_data;
> +	unsigned short i;
> +	struct snd_efw_hwinfo *hwinfo;
> +
> +	hwinfo = kmalloc(sizeof(struct snd_efw_hwinfo), GFP_KERNEL);
> +	if (hwinfo == NULL)
> +		return;
> +
> +	if (snd_efw_command_get_hwinfo(efw, hwinfo) < 0)
> +		goto end;
> +
> +	snd_iprintf(buffer, "guid_hi: 0x%X\n", hwinfo->guid_hi);
> +	snd_iprintf(buffer, "guid_lo: 0x%X\n", hwinfo->guid_lo);
> +	snd_iprintf(buffer, "type: 0x%X\n", hwinfo->type);
> +	snd_iprintf(buffer, "version: 0x%X\n", hwinfo->version);
> +	snd_iprintf(buffer, "vendor_name: %s\n", hwinfo->vendor_name);
> +	snd_iprintf(buffer, "model_name: %s\n", hwinfo->model_name);
> +
> +	snd_iprintf(buffer, "dsp_version: 0x%X\n", hwinfo->dsp_version);
> +	snd_iprintf(buffer, "arm_version: 0x%X\n", hwinfo->arm_version);
> +	snd_iprintf(buffer, "fpga_version: 0x%X\n", hwinfo->fpga_version);
> +
> +	snd_iprintf(buffer, "flags: 0x%X\n", hwinfo->flags);
> +
> +	snd_iprintf(buffer, "max_sample_rate: 0x%X\n", hwinfo->max_sample_rate);
> +	snd_iprintf(buffer, "min_sample_rate: 0x%X\n", hwinfo->min_sample_rate);
> +	snd_iprintf(buffer, "supported_clock: 0x%X\n",
> +		    hwinfo->supported_clocks);
> +
> +	snd_iprintf(buffer, "phys out: 0x%X\n", hwinfo->phys_out);
> +	snd_iprintf(buffer, "phys in: 0x%X\n", hwinfo->phys_in);
> +
> +	snd_iprintf(buffer, "phys in grps: 0x%X\n",
> +		    hwinfo->phys_in_grp_count);
> +	for (i = 0; i < hwinfo->phys_in_grp_count; i++) {
> +		snd_iprintf(buffer,
> +			    "phys in grp[0x%d]: type 0x%d, count 0x%d\n",
> +			    i, hwinfo->phys_out_grps[i].type,
> +			    hwinfo->phys_out_grps[i].count);
> +	}
> +
> +	snd_iprintf(buffer, "phys out grps: 0x%X\n",
> +		    hwinfo->phys_out_grp_count);
> +	for (i = 0; i < hwinfo->phys_out_grp_count; i++) {
> +		snd_iprintf(buffer,
> +			    "phys out grps[0x%d]: type 0x%d, count 0x%d\n",
> +			    i, hwinfo->phys_out_grps[i].type,
> +			    hwinfo->phys_out_grps[i].count);
> +	}
> +
> +	snd_iprintf(buffer, "amdtp rx pcm channels 1x: 0x%X\n",
> +		    hwinfo->amdtp_rx_pcm_channels);
> +	snd_iprintf(buffer, "amdtp tx pcm channels 1x: 0x%X\n",
> +		    hwinfo->amdtp_tx_pcm_channels);
> +	snd_iprintf(buffer, "amdtp rx pcm channels 2x: 0x%X\n",
> +		    hwinfo->amdtp_rx_pcm_channels_2x);
> +	snd_iprintf(buffer, "amdtp tx pcm channels 2x: 0x%X\n",
> +		    hwinfo->amdtp_tx_pcm_channels_2x);
> +	snd_iprintf(buffer, "amdtp rx pcm channels 4x: 0x%X\n",
> +		    hwinfo->amdtp_rx_pcm_channels_4x);
> +	snd_iprintf(buffer, "amdtp tx pcm channels 4x: 0x%X\n",
> +		    hwinfo->amdtp_tx_pcm_channels_4x);
> +
> +	snd_iprintf(buffer, "midi out ports: 0x%X\n", hwinfo->midi_out_ports);
> +	snd_iprintf(buffer, "midi in ports: 0x%X\n", hwinfo->midi_in_ports);
> +
> +	snd_iprintf(buffer, "mixer playback channels: 0x%X\n",
> +		    hwinfo->mixer_playback_channels);
> +	snd_iprintf(buffer, "mixer capture channels: 0x%X\n",
> +		    hwinfo->mixer_capture_channels);
> +end:
> +	kfree(hwinfo);
> +}
> +
> +static void
> +proc_read_clock(struct snd_info_entry *entry, struct snd_info_buffer *buffer)
> +{
> +	struct snd_efw *efw = entry->private_data;
> +	enum snd_efw_clock_source clock_source;
> +	unsigned int sampling_rate;
> +
> +	if (snd_efw_command_get_clock_source(efw, &clock_source) < 0)
> +		return;
> +
> +	if (snd_efw_command_get_sampling_rate(efw, &sampling_rate) < 0)
> +		return;
> +
> +	snd_iprintf(buffer, "Clock Source: %d\n", clock_source);
> +	snd_iprintf(buffer, "Sampling Rate: %d\n", sampling_rate);
> +}
> +
> +/*
> + * NOTE:
> + *  dB = 20 * log10(linear / 0x01000000)
> + *  -144.0 dB when linear is 0
> + */
> +static void
> +proc_read_phys_meters(struct snd_info_entry *entry,
> +		      struct snd_info_buffer *buffer)
> +{
> +	struct snd_efw *efw = entry->private_data;
> +	struct snd_efw_phys_meters *meters;
> +	unsigned int g, c, m, max, size;
> +	const char *name;
> +	u32 *linear;
> +	int err;
> +
> +	size = sizeof(struct snd_efw_phys_meters) +
> +	       (efw->phys_in + efw->phys_out) * sizeof(u32);
> +	meters = kzalloc(size, GFP_KERNEL);
> +	if (meters == NULL)
> +		return;
> +
> +	err = snd_efw_command_get_phys_meters(efw, meters, size);
> +	if (err < 0)
> +		goto end;
> +
> +	snd_iprintf(buffer, "Physical Meters:\n");
> +
> +	m = 0;
> +	max = min(efw->phys_out, meters->out_meters);
> +	linear = meters->values;
> +	snd_iprintf(buffer, " %d Outputs:\n", max);
> +	for (g = 0; g < efw->phys_out_grp_count; g++) {
> +		name = get_phys_name(&efw->phys_out_grps[g]);
> +		for (c = 0; c < efw->phys_out_grps[g].count; c++) {
> +			if (m < max)
> +				snd_iprintf(buffer, "\t%s [%d]: %d\n",
> +					    name, c, linear[m++]);
> +		}
> +	}
> +
> +	m = 0;
> +	max = min(efw->phys_in, meters->in_meters);
> +	linear = meters->values + meters->out_meters;
> +	snd_iprintf(buffer, " %d Inputs:\n", max);
> +	for (g = 0; g < efw->phys_in_grp_count; g++) {
> +		name = get_phys_name(&efw->phys_in_grps[g]);
> +		for (c = 0; c < efw->phys_in_grps[g].count; c++)
> +			if (m < max)
> +				snd_iprintf(buffer, "\t%s [%d]: %d\n",
> +					    name, c, linear[m++]);
> +	}
> +end:
> +	kfree(meters);
> +}
> +
> +void snd_efw_proc_init(struct snd_efw *efw)
> +{
> +	struct snd_info_entry *entry;
> +
> +	if (!snd_card_proc_new(efw->card, "#firmware", &entry))
> +		snd_info_set_text_ops(entry, efw, proc_read_hwinfo);
> +	if (!snd_card_proc_new(efw->card, "#clock", &entry))
> +		snd_info_set_text_ops(entry, efw, proc_read_clock);
> +	if (!snd_card_proc_new(efw->card, "#meters", &entry))
> +		snd_info_set_text_ops(entry, efw, proc_read_phys_meters);

Why '#' is needed for each file name?


Takashi
Takashi Sakamoto March 1, 2014, 1:17 p.m. UTC | #2
Hi,

(Feb 28 2014 15:54), Takashi Iwai wrote:
> Why '#' is needed for each file name?

It's for my convinience to ask testers gathering output from procfs 
nodes. With this '#', I can just tell them '/proc/asound/cardX/#*' to 
get enough information.

Is it rude?


Thanks

Takashi Sakamoto
o-takashi@sakamocchi.jp
Takashi Iwai March 3, 2014, 9:01 a.m. UTC | #3
At Sat, 01 Mar 2014 22:17:58 +0900,
Takashi Sakamoto wrote:
> 
> Hi,
> 
> (Feb 28 2014 15:54), Takashi Iwai wrote:
> > Why '#' is needed for each file name?
> 
> It's for my convinience to ask testers gathering output from procfs 
> nodes. With this '#', I can just tell them '/proc/asound/cardX/#*' to 
> get enough information.
> 
> Is it rude?

You can use another prefix like "firewire-xxx", "firewire#xxx", etc,
which can be better understood.  Or, create a proc subdirectory
instead, and put all firewire-specific things there.


thanks,

Takashi
Takashi Sakamoto March 4, 2014, 2:10 p.m. UTC | #4
Iwai-san,

(Mar 3 2014 18:01), Takashi Iwai wrote:
> You can use another prefix like "firewire-xxx", "firewire#xxx", etc,
> which can be better understood.  Or, create a proc subdirectory
> instead, and put all firewire-specific things there.

OK. I decide to add subdirectory named "firewire" and push these nodes 
to it.


Thanks

Takashi Sakamoto
o-takashi@sakamocchi.jp
diff mbox

Patch

diff --git a/sound/firewire/fireworks/Makefile b/sound/firewire/fireworks/Makefile
index 1bccb65..52bd15e 100644
--- a/sound/firewire/fireworks/Makefile
+++ b/sound/firewire/fireworks/Makefile
@@ -1,3 +1,3 @@ 
 snd-fireworks-objs := fireworks_transaction.o fireworks_command.o \
-		      fireworks_stream.o fireworks.o
+		      fireworks_stream.o fireworks_proc.o fireworks.o
 obj-m += snd-fireworks.o
diff --git a/sound/firewire/fireworks/fireworks.c b/sound/firewire/fireworks/fireworks.c
index 6d7de19..e370204 100644
--- a/sound/firewire/fireworks/fireworks.c
+++ b/sound/firewire/fireworks/fireworks.c
@@ -130,6 +130,16 @@  get_hardware_info(struct snd_efw *efw)
 	efw->pcm_playback_channels[0] = hwinfo->amdtp_rx_pcm_channels;
 	efw->pcm_playback_channels[1] = hwinfo->amdtp_rx_pcm_channels_2x;
 	efw->pcm_playback_channels[2] = hwinfo->amdtp_rx_pcm_channels_4x;
+
+	/* hardware metering */
+	efw->phys_out = hwinfo->phys_out;
+	efw->phys_in = hwinfo->phys_in;
+	efw->phys_out_grp_count = hwinfo->phys_out_grp_count;
+	efw->phys_in_grp_count = hwinfo->phys_in_grp_count;
+	memcpy(&efw->phys_out_grps, hwinfo->phys_out_grps,
+	       sizeof(struct snd_efw_phys_grp) * HWINFO_MAX_CAPS_GROUPS);
+	memcpy(&efw->phys_in_grps, hwinfo->phys_in_grps,
+	       sizeof(struct snd_efw_phys_grp) * HWINFO_MAX_CAPS_GROUPS);
 end:
 	kfree(hwinfo);
 	return err;
@@ -189,6 +199,8 @@  efw_probe(struct fw_unit *unit,
 	if (err < 0)
 		goto error;
 
+	snd_efw_proc_init(efw);
+
 	err = snd_card_register(card);
 	if (err < 0)
 		goto error;
@@ -211,8 +223,6 @@  static void efw_update(struct fw_unit *unit)
 
 	snd_efw_transaction_bus_reset(efw->unit);
 	snd_efw_stream_update_duplex(efw);
-
-	return;
 }
 
 static void efw_remove(struct fw_unit *unit)
diff --git a/sound/firewire/fireworks/fireworks.h b/sound/firewire/fireworks/fireworks.h
index c022c71..4d69c0d 100644
--- a/sound/firewire/fireworks/fireworks.h
+++ b/sound/firewire/fireworks/fireworks.h
@@ -21,6 +21,7 @@ 
 #include <sound/core.h>
 #include <sound/initval.h>
 #include <sound/pcm.h>
+#include <sound/info.h>
 
 #include "../packets-buffer.h"
 #include "../iso-resources.h"
@@ -65,6 +66,14 @@  struct snd_efw {
 	struct cmp_connection in_conn;
 	unsigned int capture_substreams;
 	unsigned int playback_substreams;
+
+	/* hardware metering parameters */
+	unsigned int phys_out;
+	unsigned int phys_in;
+	unsigned int phys_out_grp_count;
+	unsigned int phys_in_grp_count;
+	struct snd_efw_phys_grp phys_out_grps[HWINFO_MAX_CAPS_GROUPS];
+	struct snd_efw_phys_grp phys_in_grps[HWINFO_MAX_CAPS_GROUPS];
 };
 
 struct snd_efw_transaction {
@@ -178,6 +187,8 @@  int snd_efw_stream_stop_duplex(struct snd_efw *efw);
 void snd_efw_stream_update_duplex(struct snd_efw *efw);
 void snd_efw_stream_destroy_duplex(struct snd_efw *efw);
 
+void snd_efw_proc_init(struct snd_efw *efw);
+
 #define SND_EFW_DEV_ENTRY(vendor, model) \
 { \
 	.match_flags	= IEEE1394_MATCH_VENDOR_ID | \
diff --git a/sound/firewire/fireworks/fireworks_proc.c b/sound/firewire/fireworks/fireworks_proc.c
new file mode 100644
index 0000000..62f758b
--- /dev/null
+++ b/sound/firewire/fireworks/fireworks_proc.c
@@ -0,0 +1,189 @@ 
+/*
+ * fireworks_proc.c - a part of driver for Fireworks based devices
+ *
+ * Copyright (c) 2009-2010 Clemens Ladisch
+ * Copyright (c) 2013 Takashi Sakamoto
+ *
+ * Licensed under the terms of the GNU General Public License, version 2.
+ */
+
+#include "./fireworks.h"
+
+static inline const char*
+get_phys_name(struct snd_efw_phys_grp *grp)
+{
+	const char *ch_type[] = {
+		"Analog", "S/PDIF", "ADAT", "S/PDIF or ADAT",
+		"Mirroring", "Headphones", "I2S", "Guitar",
+		"Pirzo Guitar", "Guitar String", "Virtual", "Dummy"
+	};
+
+	if (grp->type < 10)
+		return ch_type[grp->type];
+	else if (grp->type == 0x10000)
+		return ch_type[10];
+	else
+		return ch_type[11];
+}
+
+static void
+proc_read_hwinfo(struct snd_info_entry *entry, struct snd_info_buffer *buffer)
+{
+	struct snd_efw *efw = entry->private_data;
+	unsigned short i;
+	struct snd_efw_hwinfo *hwinfo;
+
+	hwinfo = kmalloc(sizeof(struct snd_efw_hwinfo), GFP_KERNEL);
+	if (hwinfo == NULL)
+		return;
+
+	if (snd_efw_command_get_hwinfo(efw, hwinfo) < 0)
+		goto end;
+
+	snd_iprintf(buffer, "guid_hi: 0x%X\n", hwinfo->guid_hi);
+	snd_iprintf(buffer, "guid_lo: 0x%X\n", hwinfo->guid_lo);
+	snd_iprintf(buffer, "type: 0x%X\n", hwinfo->type);
+	snd_iprintf(buffer, "version: 0x%X\n", hwinfo->version);
+	snd_iprintf(buffer, "vendor_name: %s\n", hwinfo->vendor_name);
+	snd_iprintf(buffer, "model_name: %s\n", hwinfo->model_name);
+
+	snd_iprintf(buffer, "dsp_version: 0x%X\n", hwinfo->dsp_version);
+	snd_iprintf(buffer, "arm_version: 0x%X\n", hwinfo->arm_version);
+	snd_iprintf(buffer, "fpga_version: 0x%X\n", hwinfo->fpga_version);
+
+	snd_iprintf(buffer, "flags: 0x%X\n", hwinfo->flags);
+
+	snd_iprintf(buffer, "max_sample_rate: 0x%X\n", hwinfo->max_sample_rate);
+	snd_iprintf(buffer, "min_sample_rate: 0x%X\n", hwinfo->min_sample_rate);
+	snd_iprintf(buffer, "supported_clock: 0x%X\n",
+		    hwinfo->supported_clocks);
+
+	snd_iprintf(buffer, "phys out: 0x%X\n", hwinfo->phys_out);
+	snd_iprintf(buffer, "phys in: 0x%X\n", hwinfo->phys_in);
+
+	snd_iprintf(buffer, "phys in grps: 0x%X\n",
+		    hwinfo->phys_in_grp_count);
+	for (i = 0; i < hwinfo->phys_in_grp_count; i++) {
+		snd_iprintf(buffer,
+			    "phys in grp[0x%d]: type 0x%d, count 0x%d\n",
+			    i, hwinfo->phys_out_grps[i].type,
+			    hwinfo->phys_out_grps[i].count);
+	}
+
+	snd_iprintf(buffer, "phys out grps: 0x%X\n",
+		    hwinfo->phys_out_grp_count);
+	for (i = 0; i < hwinfo->phys_out_grp_count; i++) {
+		snd_iprintf(buffer,
+			    "phys out grps[0x%d]: type 0x%d, count 0x%d\n",
+			    i, hwinfo->phys_out_grps[i].type,
+			    hwinfo->phys_out_grps[i].count);
+	}
+
+	snd_iprintf(buffer, "amdtp rx pcm channels 1x: 0x%X\n",
+		    hwinfo->amdtp_rx_pcm_channels);
+	snd_iprintf(buffer, "amdtp tx pcm channels 1x: 0x%X\n",
+		    hwinfo->amdtp_tx_pcm_channels);
+	snd_iprintf(buffer, "amdtp rx pcm channels 2x: 0x%X\n",
+		    hwinfo->amdtp_rx_pcm_channels_2x);
+	snd_iprintf(buffer, "amdtp tx pcm channels 2x: 0x%X\n",
+		    hwinfo->amdtp_tx_pcm_channels_2x);
+	snd_iprintf(buffer, "amdtp rx pcm channels 4x: 0x%X\n",
+		    hwinfo->amdtp_rx_pcm_channels_4x);
+	snd_iprintf(buffer, "amdtp tx pcm channels 4x: 0x%X\n",
+		    hwinfo->amdtp_tx_pcm_channels_4x);
+
+	snd_iprintf(buffer, "midi out ports: 0x%X\n", hwinfo->midi_out_ports);
+	snd_iprintf(buffer, "midi in ports: 0x%X\n", hwinfo->midi_in_ports);
+
+	snd_iprintf(buffer, "mixer playback channels: 0x%X\n",
+		    hwinfo->mixer_playback_channels);
+	snd_iprintf(buffer, "mixer capture channels: 0x%X\n",
+		    hwinfo->mixer_capture_channels);
+end:
+	kfree(hwinfo);
+}
+
+static void
+proc_read_clock(struct snd_info_entry *entry, struct snd_info_buffer *buffer)
+{
+	struct snd_efw *efw = entry->private_data;
+	enum snd_efw_clock_source clock_source;
+	unsigned int sampling_rate;
+
+	if (snd_efw_command_get_clock_source(efw, &clock_source) < 0)
+		return;
+
+	if (snd_efw_command_get_sampling_rate(efw, &sampling_rate) < 0)
+		return;
+
+	snd_iprintf(buffer, "Clock Source: %d\n", clock_source);
+	snd_iprintf(buffer, "Sampling Rate: %d\n", sampling_rate);
+}
+
+/*
+ * NOTE:
+ *  dB = 20 * log10(linear / 0x01000000)
+ *  -144.0 dB when linear is 0
+ */
+static void
+proc_read_phys_meters(struct snd_info_entry *entry,
+		      struct snd_info_buffer *buffer)
+{
+	struct snd_efw *efw = entry->private_data;
+	struct snd_efw_phys_meters *meters;
+	unsigned int g, c, m, max, size;
+	const char *name;
+	u32 *linear;
+	int err;
+
+	size = sizeof(struct snd_efw_phys_meters) +
+	       (efw->phys_in + efw->phys_out) * sizeof(u32);
+	meters = kzalloc(size, GFP_KERNEL);
+	if (meters == NULL)
+		return;
+
+	err = snd_efw_command_get_phys_meters(efw, meters, size);
+	if (err < 0)
+		goto end;
+
+	snd_iprintf(buffer, "Physical Meters:\n");
+
+	m = 0;
+	max = min(efw->phys_out, meters->out_meters);
+	linear = meters->values;
+	snd_iprintf(buffer, " %d Outputs:\n", max);
+	for (g = 0; g < efw->phys_out_grp_count; g++) {
+		name = get_phys_name(&efw->phys_out_grps[g]);
+		for (c = 0; c < efw->phys_out_grps[g].count; c++) {
+			if (m < max)
+				snd_iprintf(buffer, "\t%s [%d]: %d\n",
+					    name, c, linear[m++]);
+		}
+	}
+
+	m = 0;
+	max = min(efw->phys_in, meters->in_meters);
+	linear = meters->values + meters->out_meters;
+	snd_iprintf(buffer, " %d Inputs:\n", max);
+	for (g = 0; g < efw->phys_in_grp_count; g++) {
+		name = get_phys_name(&efw->phys_in_grps[g]);
+		for (c = 0; c < efw->phys_in_grps[g].count; c++)
+			if (m < max)
+				snd_iprintf(buffer, "\t%s [%d]: %d\n",
+					    name, c, linear[m++]);
+	}
+end:
+	kfree(meters);
+}
+
+void snd_efw_proc_init(struct snd_efw *efw)
+{
+	struct snd_info_entry *entry;
+
+	if (!snd_card_proc_new(efw->card, "#firmware", &entry))
+		snd_info_set_text_ops(entry, efw, proc_read_hwinfo);
+	if (!snd_card_proc_new(efw->card, "#clock", &entry))
+		snd_info_set_text_ops(entry, efw, proc_read_clock);
+	if (!snd_card_proc_new(efw->card, "#meters", &entry))
+		snd_info_set_text_ops(entry, efw, proc_read_phys_meters);
+}