From: Hui Wang <hui.wang@xxxxxxxxxxxxx> Add a user defined parameter lfe-crossover-freq for the lfe-filter, to pass this parameter to the lfe-filter, we need to change the pa_resampler_new() API as well. Signed-off-by: Hui Wang <hui.wang at canonical.com> --- man/pulse-daemon.conf.5.xml.in | 5 +++++ src/daemon/daemon-conf.c | 3 +++ src/daemon/daemon-conf.h | 1 + src/daemon/daemon.conf.in | 1 + src/daemon/main.c | 1 + src/modules/module-virtual-surround-sink.c | 2 +- src/pulsecore/core.c | 1 + src/pulsecore/core.h | 1 + src/pulsecore/resampler.c | 6 +++--- src/pulsecore/resampler.h | 1 + src/pulsecore/sink-input.c | 2 ++ src/pulsecore/source-output.c | 2 ++ src/tests/remix-test.c | 3 ++- src/tests/resampler-test.c | 7 ++++--- 14 files changed, 28 insertions(+), 8 deletions(-) diff --git a/man/pulse-daemon.conf.5.xml.in b/man/pulse-daemon.conf.5.xml.in index 42c3f73..bc91a5a 100644 --- a/man/pulse-daemon.conf.5.xml.in +++ b/man/pulse-daemon.conf.5.xml.in @@ -129,6 +129,11 @@ License along with PulseAudio; if not, see <http://www.gnu.org/licenses/>. </option> <option> + <p><opt>lfe-crossover-freq=</opt> The crossover frequency (in Hz) for the + LFE filter. Defaults to 120 Hz.</p> + </option> + + <option> <p><opt>use-pid-file=</opt> Create a PID file in the runtime directory (<file>$XDG_RUNTIME_DIR/pulse/pid</file>). If this is enabled you may use commands like <opt>--kill</opt> or <opt>--check</opt>. If diff --git a/src/daemon/daemon-conf.c b/src/daemon/daemon-conf.c index 61cf630..21a8edb 100644 --- a/src/daemon/daemon-conf.c +++ b/src/daemon/daemon-conf.c @@ -83,6 +83,7 @@ static const pa_daemon_conf default_conf = { .resample_method = PA_RESAMPLER_AUTO, .disable_remixing = false, .disable_lfe_remixing = false, + .lfe_crossover_freq = 120, .config_file = NULL, .use_pid_file = true, .system_instance = false, @@ -553,6 +554,7 @@ int pa_daemon_conf_load(pa_daemon_conf *c, const char *filename) { { "enable-remixing", pa_config_parse_not_bool, &c->disable_remixing, NULL }, { "disable-lfe-remixing", pa_config_parse_bool, &c->disable_lfe_remixing, NULL }, { "enable-lfe-remixing", pa_config_parse_not_bool, &c->disable_lfe_remixing, NULL }, + { "lfe-crossover-freq", pa_config_parse_unsigned, &c->lfe_crossover_freq, NULL }, { "load-default-script-file", pa_config_parse_bool, &c->load_default_script_file, NULL }, { "shm-size-bytes", pa_config_parse_size, &c->shm_size, NULL }, { "log-meta", pa_config_parse_bool, &c->log_meta, NULL }, @@ -745,6 +747,7 @@ char *pa_daemon_conf_dump(pa_daemon_conf *c) { pa_strbuf_printf(s, "resample-method = %s\n", pa_resample_method_to_string(c->resample_method)); pa_strbuf_printf(s, "enable-remixing = %s\n", pa_yes_no(!c->disable_remixing)); pa_strbuf_printf(s, "enable-lfe-remixing = %s\n", pa_yes_no(!c->disable_lfe_remixing)); + pa_strbuf_printf(s, "lfe-crossover-freq = %u\n", c->lfe_crossover_freq); pa_strbuf_printf(s, "default-sample-format = %s\n", pa_sample_format_to_string(c->default_sample_spec.format)); pa_strbuf_printf(s, "default-sample-rate = %u\n", c->default_sample_spec.rate); pa_strbuf_printf(s, "alternate-sample-rate = %u\n", c->alternate_sample_rate); diff --git a/src/daemon/daemon-conf.h b/src/daemon/daemon-conf.h index 62587b2..458784c 100644 --- a/src/daemon/daemon-conf.h +++ b/src/daemon/daemon-conf.h @@ -127,6 +127,7 @@ typedef struct pa_daemon_conf { unsigned default_n_fragments, default_fragment_size_msec; unsigned deferred_volume_safety_margin_usec; int deferred_volume_extra_delay_usec; + unsigned lfe_crossover_freq; pa_sample_spec default_sample_spec; uint32_t alternate_sample_rate; pa_channel_map default_channel_map; diff --git a/src/daemon/daemon.conf.in b/src/daemon/daemon.conf.in index 26908df..b48afa2 100644 --- a/src/daemon/daemon.conf.in +++ b/src/daemon/daemon.conf.in @@ -55,6 +55,7 @@ ifelse(@HAVE_DBUS@, 1, [dnl ; resample-method = speex-float-1 ; enable-remixing = yes ; enable-lfe-remixing = yes +; lfe-crossover-freq = 120 ; flat-volumes = yes diff --git a/src/daemon/main.c b/src/daemon/main.c index 68ed383..deb1e43 100644 --- a/src/daemon/main.c +++ b/src/daemon/main.c @@ -1029,6 +1029,7 @@ int main(int argc, char *argv[]) { c->default_fragment_size_msec = conf->default_fragment_size_msec; c->deferred_volume_safety_margin_usec = conf->deferred_volume_safety_margin_usec; c->deferred_volume_extra_delay_usec = conf->deferred_volume_extra_delay_usec; + c->lfe_crossover_freq = conf->lfe_crossover_freq; c->exit_idle_time = conf->exit_idle_time; c->scache_idle_time = conf->scache_idle_time; c->resample_method = conf->resample_method; diff --git a/src/modules/module-virtual-surround-sink.c b/src/modules/module-virtual-surround-sink.c index 8f05f53..6c7120a 100644 --- a/src/modules/module-virtual-surround-sink.c +++ b/src/modules/module-virtual-surround-sink.c @@ -743,7 +743,7 @@ int pa__init(pa_module*m) { pa_memblock_unref(silence.memblock); /* resample hrir */ - resampler = pa_resampler_new(u->sink->core->mempool, &hrir_temp_ss, &hrir_map, &hrir_ss, &hrir_map, + resampler = pa_resampler_new(u->sink->core->mempool, &hrir_temp_ss, &hrir_map, &hrir_ss, &hrir_map, u->sink->core->lfe_crossover_freq, PA_RESAMPLER_SRC_SINC_BEST_QUALITY, PA_RESAMPLER_NO_REMAP); u->hrir_samples = hrir_temp_chunk.length / pa_frame_size(&hrir_temp_ss) * hrir_ss.rate / hrir_temp_ss.rate; diff --git a/src/pulsecore/core.c b/src/pulsecore/core.c index 0e67005..0e63bac 100644 --- a/src/pulsecore/core.c +++ b/src/pulsecore/core.c @@ -144,6 +144,7 @@ pa_core* pa_core_new(pa_mainloop_api *m, bool shared, size_t shm_size) { c->realtime_priority = 5; c->disable_remixing = false; c->disable_lfe_remixing = false; + c->lfe_crossover_freq = 120; c->deferred_volume = true; c->resample_method = PA_RESAMPLER_SPEEX_FLOAT_BASE + 1; diff --git a/src/pulsecore/core.h b/src/pulsecore/core.h index 7d896bb..8570d47 100644 --- a/src/pulsecore/core.h +++ b/src/pulsecore/core.h @@ -160,6 +160,7 @@ struct pa_core { unsigned default_n_fragments, default_fragment_size_msec; unsigned deferred_volume_safety_margin_usec; int deferred_volume_extra_delay_usec; + unsigned lfe_crossover_freq; pa_defer_event *module_defer_unload_event; pa_hashmap *modules_pending_unload; /* pa_module -> pa_module (hashmap-as-a-set) */ diff --git a/src/pulsecore/resampler.c b/src/pulsecore/resampler.c index 157671d..13b272f 100644 --- a/src/pulsecore/resampler.c +++ b/src/pulsecore/resampler.c @@ -320,6 +320,7 @@ pa_resampler* pa_resampler_new( const pa_channel_map *am, const pa_sample_spec *b, const pa_channel_map *bm, + unsigned crossover_freq, pa_resample_method_t method, pa_resample_flags_t flags) { @@ -418,9 +419,8 @@ pa_resampler* pa_resampler_new( if (lfe_filter_required) { pa_sample_spec wss = r->o_ss; wss.format = r->work_format; - /* TODO: Temporary code that sets crossover freq to 120 Hz. This should be a parameter */ - r->lfe_filter = pa_lfe_filter_new(&wss, &r->o_cm, 120.0f); - pa_log_debug(" lfe filter activated (LR4 type)"); + r->lfe_filter = pa_lfe_filter_new(&wss, &r->o_cm, (float)crossover_freq); + pa_log_debug(" lfe filter activated (LR4 type), the crossover_freq = %uHz", crossover_freq); } /* initialize implementation */ diff --git a/src/pulsecore/resampler.h b/src/pulsecore/resampler.h index 45ad75a..3bc1054 100644 --- a/src/pulsecore/resampler.h +++ b/src/pulsecore/resampler.h @@ -118,6 +118,7 @@ pa_resampler* pa_resampler_new( const pa_channel_map *am, const pa_sample_spec *b, const pa_channel_map *bm, + unsigned crossover_freq, pa_resample_method_t resample_method, pa_resample_flags_t flags); diff --git a/src/pulsecore/sink-input.c b/src/pulsecore/sink-input.c index d95a93e..7d2202e 100644 --- a/src/pulsecore/sink-input.c +++ b/src/pulsecore/sink-input.c @@ -451,6 +451,7 @@ int pa_sink_input_new( core->mempool, &data->sample_spec, &data->channel_map, &data->sink->sample_spec, &data->sink->channel_map, + core->lfe_crossover_freq, data->resample_method, ((data->flags & PA_SINK_INPUT_VARIABLE_RATE) ? PA_RESAMPLER_VARIABLE_RATE : 0) | ((data->flags & PA_SINK_INPUT_NO_REMAP) ? PA_RESAMPLER_NO_REMAP : 0) | @@ -2168,6 +2169,7 @@ int pa_sink_input_update_rate(pa_sink_input *i) { new_resampler = pa_resampler_new(i->core->mempool, &i->sample_spec, &i->channel_map, &i->sink->sample_spec, &i->sink->channel_map, + i->core->lfe_crossover_freq, i->requested_resample_method, ((i->flags & PA_SINK_INPUT_VARIABLE_RATE) ? PA_RESAMPLER_VARIABLE_RATE : 0) | ((i->flags & PA_SINK_INPUT_NO_REMAP) ? PA_RESAMPLER_NO_REMAP : 0) | diff --git a/src/pulsecore/source-output.c b/src/pulsecore/source-output.c index c8090f7..4cc1053 100644 --- a/src/pulsecore/source-output.c +++ b/src/pulsecore/source-output.c @@ -396,6 +396,7 @@ int pa_source_output_new( core->mempool, &data->source->sample_spec, &data->source->channel_map, &data->sample_spec, &data->channel_map, + core->lfe_crossover_freq, data->resample_method, ((data->flags & PA_SOURCE_OUTPUT_VARIABLE_RATE) ? PA_RESAMPLER_VARIABLE_RATE : 0) | ((data->flags & PA_SOURCE_OUTPUT_NO_REMAP) ? PA_RESAMPLER_NO_REMAP : 0) | @@ -1625,6 +1626,7 @@ int pa_source_output_update_rate(pa_source_output *o) { new_resampler = pa_resampler_new(o->core->mempool, &o->source->sample_spec, &o->source->channel_map, &o->sample_spec, &o->channel_map, + o->core->lfe_crossover_freq, o->requested_resample_method, ((o->flags & PA_SOURCE_OUTPUT_VARIABLE_RATE) ? PA_RESAMPLER_VARIABLE_RATE : 0) | ((o->flags & PA_SOURCE_OUTPUT_NO_REMAP) ? PA_RESAMPLER_NO_REMAP : 0) | diff --git a/src/tests/remix-test.c b/src/tests/remix-test.c index 5193187..6feb8dc 100644 --- a/src/tests/remix-test.c +++ b/src/tests/remix-test.c @@ -47,6 +47,7 @@ int main(int argc, char *argv[]) { unsigned i, j; pa_mempool *pool; + unsigned crossover_freq = 120; pa_log_set_level(PA_LOG_DEBUG); @@ -66,7 +67,7 @@ int main(int argc, char *argv[]) { ss1.rate = ss2.rate = 44100; ss1.format = ss2.format = PA_SAMPLE_S16NE; - r = pa_resampler_new(pool, &ss1, &maps[i], &ss2, &maps[j], PA_RESAMPLER_AUTO, 0); + r = pa_resampler_new(pool, &ss1, &maps[i], &ss2, &maps[j], crossover_freq, PA_RESAMPLER_AUTO, 0); /* We don't really care for the resampler. We just want to * see the remixing debug output. */ diff --git a/src/tests/resampler-test.c b/src/tests/resampler-test.c index 96110fb..751d2dc 100644 --- a/src/tests/resampler-test.c +++ b/src/tests/resampler-test.c @@ -303,6 +303,7 @@ int main(int argc, char *argv[]) { bool all_formats = true; pa_resample_method_t method; int seconds; + unsigned crossover_freq = 120; static const struct option long_options[] = { {"help", 0, NULL, 'h'}, @@ -419,7 +420,7 @@ int main(int argc, char *argv[]) { b.rate, b.channels, pa_sample_format_to_string(b.format)); ts = pa_rtclock_now(); - pa_assert_se(resampler = pa_resampler_new(pool, &a, NULL, &b, NULL, method, 0)); + pa_assert_se(resampler = pa_resampler_new(pool, &a, NULL, &b, NULL, crossover_freq, method, 0)); pa_log_info("init: %llu", (long long unsigned)(pa_rtclock_now() - ts)); i.memblock = pa_memblock_new(pool, pa_usec_to_bytes(1*PA_USEC_PER_SEC, &a)); @@ -450,8 +451,8 @@ int main(int argc, char *argv[]) { pa_sample_format_to_string(b.format), pa_sample_format_to_string(a.format)); - pa_assert_se(forth = pa_resampler_new(pool, &a, NULL, &b, NULL, method, 0)); - pa_assert_se(back = pa_resampler_new(pool, &b, NULL, &a, NULL, method, 0)); + pa_assert_se(forth = pa_resampler_new(pool, &a, NULL, &b, NULL, crossover_freq, method, 0)); + pa_assert_se(back = pa_resampler_new(pool, &b, NULL, &a, NULL, crossover_freq, method, 0)); i.memblock = generate_block(pool, &a); i.length = pa_memblock_get_length(i.memblock); -- 1.9.1
