[pulseaudio-discuss] [PATCH] alsa: Print dB values instead of percentages in debug messages.
Tanu Kaskinen
tanu.kaskinen at digia.com
Wed Nov 17 05:50:28 PST 2010
---
src/modules/alsa/alsa-sink.c | 29 +++++++++++++++++++----------
src/modules/alsa/alsa-source.c | 23 ++++++++++++++++-------
2 files changed, 35 insertions(+), 17 deletions(-)
diff --git a/src/modules/alsa/alsa-sink.c b/src/modules/alsa/alsa-sink.c
index b740d53..a5aa2ec 100644
--- a/src/modules/alsa/alsa-sink.c
+++ b/src/modules/alsa/alsa-sink.c
@@ -1166,7 +1166,6 @@ static int io_mixer_callback(snd_mixer_elem_t *elem, unsigned int mask) {
static void sink_get_volume_cb(pa_sink *s) {
struct userdata *u = s->userdata;
pa_cvolume r;
- char t[PA_CVOLUME_SNPRINT_MAX];
pa_assert(u);
pa_assert(u->mixer_path);
@@ -1178,7 +1177,15 @@ static void sink_get_volume_cb(pa_sink *s) {
/* Shift down by the base volume, so that 0dB becomes maximum volume */
pa_sw_cvolume_multiply_scalar(&r, &r, s->base_volume);
- pa_log_debug("Read hardware volume: %s", pa_cvolume_snprint(t, sizeof(t), &r));
+ if (u->mixer_path->has_dB) {
+ char vol_str[PA_SW_CVOLUME_SNPRINT_DB_MAX];
+
+ pa_log_debug("Read hardware volume: %s", pa_sw_cvolume_snprint_dB(vol_str, sizeof(vol_str), &r));
+ } else {
+ char vol_str[PA_CVOLUME_SNPRINT_MAX];
+
+ pa_log_debug("Read hardware volume: %s", pa_cvolume_snprint(vol_str, sizeof(vol_str), &r));
+ }
if (pa_cvolume_equal(&u->hardware_volume, &r))
return;
@@ -1193,7 +1200,6 @@ static void sink_get_volume_cb(pa_sink *s) {
static void sink_set_volume_cb(pa_sink *s) {
struct userdata *u = s->userdata;
pa_cvolume r;
- char t[PA_CVOLUME_SNPRINT_MAX];
pa_bool_t write_to_hw = (s->flags & PA_SINK_SYNC_VOLUME) ? FALSE : TRUE;
pa_assert(u);
@@ -1214,6 +1220,7 @@ static void sink_set_volume_cb(pa_sink *s) {
if (u->mixer_path->has_dB) {
pa_cvolume new_soft_volume;
pa_bool_t accurate_enough;
+ char vol_str[PA_SW_CVOLUME_SNPRINT_DB_MAX];
/* Match exactly what the user requested by software */
pa_sw_cvolume_divide(&new_soft_volume, &s->real_volume, &u->hardware_volume);
@@ -1225,16 +1232,18 @@ static void sink_set_volume_cb(pa_sink *s) {
(pa_cvolume_min(&new_soft_volume) >= (PA_VOLUME_NORM - VOLUME_ACCURACY)) &&
(pa_cvolume_max(&new_soft_volume) <= (PA_VOLUME_NORM + VOLUME_ACCURACY));
- pa_log_debug("Requested volume: %s", pa_cvolume_snprint(t, sizeof(t), &s->real_volume));
- pa_log_debug("Got hardware volume: %s", pa_cvolume_snprint(t, sizeof(t), &u->hardware_volume));
- pa_log_debug("Calculated software volume: %s (accurate-enough=%s)", pa_cvolume_snprint(t, sizeof(t), &new_soft_volume),
+ pa_log_debug("Requested volume: %s", pa_sw_cvolume_snprint_dB(vol_str, sizeof(vol_str), &s->real_volume));
+ pa_log_debug("Got hardware volume: %s", pa_sw_cvolume_snprint_dB(vol_str, sizeof(vol_str), &u->hardware_volume));
+ pa_log_debug("Calculated software volume: %s (accurate-enough=%s)", pa_sw_cvolume_snprint_dB(vol_str, sizeof(vol_str), &new_soft_volume),
pa_yes_no(accurate_enough));
if (!accurate_enough)
s->soft_volume = new_soft_volume;
} else {
- pa_log_debug("Wrote hardware volume: %s", pa_cvolume_snprint(t, sizeof(t), &r));
+ char vol_str[PA_CVOLUME_SNPRINT_MAX];
+
+ pa_log_debug("Wrote hardware volume: %s", pa_cvolume_snprint(vol_str, sizeof(vol_str), &r));
/* We can't match exactly what the user requested, hence let's
* at least tell the user about it */
@@ -1262,10 +1271,10 @@ static void sink_write_volume_cb(pa_sink *s) {
(pa_cvolume_min(&tmp_vol) >= (PA_VOLUME_NORM - VOLUME_ACCURACY)) &&
(pa_cvolume_max(&tmp_vol) <= (PA_VOLUME_NORM + VOLUME_ACCURACY));
if (!accurate_enough) {
- char t[PA_CVOLUME_SNPRINT_MAX];
+ char vol_str[PA_SW_CVOLUME_SNPRINT_DB_MAX];
pa_log_debug("Written HW volume did not match with the request %s != %s",
- pa_cvolume_snprint(t, sizeof(t), &s->thread_info.current_hw_volume),
- pa_cvolume_snprint(t, sizeof(t), &hw_vol));
+ pa_sw_cvolume_snprint_dB(vol_str, sizeof(vol_str), &s->thread_info.current_hw_volume),
+ pa_sw_cvolume_snprint_dB(vol_str, sizeof(vol_str), &hw_vol));
}
}
}
diff --git a/src/modules/alsa/alsa-source.c b/src/modules/alsa/alsa-source.c
index 9795136..4aea41c 100644
--- a/src/modules/alsa/alsa-source.c
+++ b/src/modules/alsa/alsa-source.c
@@ -1072,7 +1072,6 @@ static int mixer_callback(snd_mixer_elem_t *elem, unsigned int mask) {
static void source_get_volume_cb(pa_source *s) {
struct userdata *u = s->userdata;
pa_cvolume r;
- char t[PA_CVOLUME_SNPRINT_MAX];
pa_assert(u);
pa_assert(u->mixer_path);
@@ -1084,7 +1083,15 @@ static void source_get_volume_cb(pa_source *s) {
/* Shift down by the base volume, so that 0dB becomes maximum volume */
pa_sw_cvolume_multiply_scalar(&r, &r, s->base_volume);
- pa_log_debug("Read hardware volume: %s", pa_cvolume_snprint(t, sizeof(t), &r));
+ if (u->mixer_path->has_dB) {
+ char vol_str[PA_SW_CVOLUME_SNPRINT_DB_MAX];
+
+ pa_log_debug("Read hardware volume: %s", pa_sw_cvolume_snprint_dB(vol_str, sizeof(vol_str), &r));
+ } else {
+ char vol_str[PA_CVOLUME_SNPRINT_MAX];
+
+ pa_log_debug("Read hardware volume: %s", pa_cvolume_snprint(vol_str, sizeof(vol_str), &r));
+ }
if (pa_cvolume_equal(&u->hardware_volume, &r))
return;
@@ -1099,7 +1106,6 @@ static void source_get_volume_cb(pa_source *s) {
static void source_set_volume_cb(pa_source *s) {
struct userdata *u = s->userdata;
pa_cvolume r;
- char t[PA_CVOLUME_SNPRINT_MAX];
pa_assert(u);
pa_assert(u->mixer_path);
@@ -1119,6 +1125,7 @@ static void source_set_volume_cb(pa_source *s) {
if (u->mixer_path->has_dB) {
pa_cvolume new_soft_volume;
pa_bool_t accurate_enough;
+ char vol_str[PA_SW_CVOLUME_SNPRINT_DB_MAX];
/* Match exactly what the user requested by software */
pa_sw_cvolume_divide(&new_soft_volume, &s->volume, &u->hardware_volume);
@@ -1130,16 +1137,18 @@ static void source_set_volume_cb(pa_source *s) {
(pa_cvolume_min(&new_soft_volume) >= (PA_VOLUME_NORM - VOLUME_ACCURACY)) &&
(pa_cvolume_max(&new_soft_volume) <= (PA_VOLUME_NORM + VOLUME_ACCURACY));
- pa_log_debug("Requested volume: %s", pa_cvolume_snprint(t, sizeof(t), &s->volume));
- pa_log_debug("Got hardware volume: %s", pa_cvolume_snprint(t, sizeof(t), &u->hardware_volume));
- pa_log_debug("Calculated software volume: %s (accurate-enough=%s)", pa_cvolume_snprint(t, sizeof(t), &new_soft_volume),
+ pa_log_debug("Requested volume: %s", pa_sw_cvolume_snprint_dB(vol_str, sizeof(vol_str), &s->volume));
+ pa_log_debug("Got hardware volume: %s", pa_sw_cvolume_snprint_dB(vol_str, sizeof(vol_str), &u->hardware_volume));
+ pa_log_debug("Calculated software volume: %s (accurate-enough=%s)", pa_sw_cvolume_snprint_dB(vol_str, sizeof(vol_str), &new_soft_volume),
pa_yes_no(accurate_enough));
if (!accurate_enough)
s->soft_volume = new_soft_volume;
} else {
- pa_log_debug("Wrote hardware volume: %s", pa_cvolume_snprint(t, sizeof(t), &r));
+ char vol_str[PA_CVOLUME_SNPRINT_MAX];
+
+ pa_log_debug("Wrote hardware volume: %s", pa_cvolume_snprint(vol_str, sizeof(vol_str), &r));
/* We can't match exactly what the user requested, hence let's
* at least tell the user about it */
--
1.7.3.2
More information about the pulseaudio-discuss
mailing list