mirror of
				https://gitlab.freedesktop.org/pulseaudio/pulseaudio.git
				synced 2025-11-03 09:01:50 -05:00 
			
		
		
		
	simplify pa_alsa_init_proplist_pcm() a bit and include resolution bits in alsa device props
This commit is contained in:
		
							parent
							
								
									c9c63c295f
								
							
						
					
					
						commit
						ff58fa8870
					
				
					 4 changed files with 28 additions and 20 deletions
				
			
		| 
						 | 
				
			
			@ -1236,14 +1236,11 @@ pa_sink *pa_alsa_sink_new(pa_module *m, pa_modargs *ma, const char*driver, pa_ca
 | 
			
		|||
    uint32_t nfrags, hwbuf_size, frag_size, tsched_size, tsched_watermark;
 | 
			
		||||
    snd_pcm_uframes_t period_frames, tsched_frames;
 | 
			
		||||
    size_t frame_size;
 | 
			
		||||
    snd_pcm_info_t *pcm_info = NULL;
 | 
			
		||||
    int err;
 | 
			
		||||
    pa_bool_t use_mmap = TRUE, b, use_tsched = TRUE, d, ignore_dB = FALSE;
 | 
			
		||||
    pa_usec_t usec;
 | 
			
		||||
    pa_sink_new_data data;
 | 
			
		||||
 | 
			
		||||
    snd_pcm_info_alloca(&pcm_info);
 | 
			
		||||
 | 
			
		||||
    pa_assert(m);
 | 
			
		||||
    pa_assert(ma);
 | 
			
		||||
 | 
			
		||||
| 
						 | 
				
			
			@ -1378,11 +1375,6 @@ pa_sink *pa_alsa_sink_new(pa_module *m, pa_modargs *ma, const char*driver, pa_ca
 | 
			
		|||
    if (u->use_tsched)
 | 
			
		||||
        pa_log_info("Successfully enabled timer-based scheduling mode.");
 | 
			
		||||
 | 
			
		||||
    if ((err = snd_pcm_info(u->pcm_handle, pcm_info)) < 0) {
 | 
			
		||||
        pa_log("Error fetching PCM info: %s", snd_strerror(err));
 | 
			
		||||
        goto fail;
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
    /* ALSA might tweak the sample spec, so recalculate the frame size */
 | 
			
		||||
    frame_size = pa_frame_size(&ss);
 | 
			
		||||
 | 
			
		||||
| 
						 | 
				
			
			@ -1396,7 +1388,7 @@ pa_sink *pa_alsa_sink_new(pa_module *m, pa_modargs *ma, const char*driver, pa_ca
 | 
			
		|||
    pa_sink_new_data_set_sample_spec(&data, &ss);
 | 
			
		||||
    pa_sink_new_data_set_channel_map(&data, &map);
 | 
			
		||||
 | 
			
		||||
    pa_alsa_init_proplist_pcm(m->core, data.proplist, pcm_info);
 | 
			
		||||
    pa_alsa_init_proplist_pcm(m->core, data.proplist, u->pcm_handle);
 | 
			
		||||
    pa_proplist_sets(data.proplist, PA_PROP_DEVICE_STRING, u->device_name);
 | 
			
		||||
    pa_proplist_setf(data.proplist, PA_PROP_DEVICE_BUFFERING_BUFFER_SIZE, "%lu", (unsigned long) (period_frames * frame_size * nfrags));
 | 
			
		||||
    pa_proplist_setf(data.proplist, PA_PROP_DEVICE_BUFFERING_FRAGMENT_SIZE, "%lu", (unsigned long) (period_frames * frame_size));
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -1071,13 +1071,10 @@ pa_source *pa_alsa_source_new(pa_module *m, pa_modargs *ma, const char*driver, p
 | 
			
		|||
    uint32_t nfrags, hwbuf_size, frag_size, tsched_size, tsched_watermark;
 | 
			
		||||
    snd_pcm_uframes_t period_frames, tsched_frames;
 | 
			
		||||
    size_t frame_size;
 | 
			
		||||
    snd_pcm_info_t *pcm_info = NULL;
 | 
			
		||||
    int err;
 | 
			
		||||
    pa_bool_t use_mmap = TRUE, b, use_tsched = TRUE, d, ignore_dB = FALSE;
 | 
			
		||||
    pa_source_new_data data;
 | 
			
		||||
 | 
			
		||||
    snd_pcm_info_alloca(&pcm_info);
 | 
			
		||||
 | 
			
		||||
    pa_assert(m);
 | 
			
		||||
 | 
			
		||||
    ss = m->core->default_sample_spec;
 | 
			
		||||
| 
						 | 
				
			
			@ -1203,11 +1200,6 @@ pa_source *pa_alsa_source_new(pa_module *m, pa_modargs *ma, const char*driver, p
 | 
			
		|||
    if (u->use_tsched)
 | 
			
		||||
        pa_log_info("Successfully enabled timer-based scheduling mode.");
 | 
			
		||||
 | 
			
		||||
    if ((err = snd_pcm_info(u->pcm_handle, pcm_info)) < 0) {
 | 
			
		||||
        pa_log("Error fetching PCM info: %s", snd_strerror(err));
 | 
			
		||||
        goto fail;
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
    /* ALSA might tweak the sample spec, so recalculate the frame size */
 | 
			
		||||
    frame_size = pa_frame_size(&ss);
 | 
			
		||||
 | 
			
		||||
| 
						 | 
				
			
			@ -1221,7 +1213,7 @@ pa_source *pa_alsa_source_new(pa_module *m, pa_modargs *ma, const char*driver, p
 | 
			
		|||
    pa_source_new_data_set_sample_spec(&data, &ss);
 | 
			
		||||
    pa_source_new_data_set_channel_map(&data, &map);
 | 
			
		||||
 | 
			
		||||
    pa_alsa_init_proplist_pcm(m->core, data.proplist, pcm_info);
 | 
			
		||||
    pa_alsa_init_proplist_pcm(m->core, data.proplist, u->pcm_handle);
 | 
			
		||||
    pa_proplist_sets(data.proplist, PA_PROP_DEVICE_STRING, u->device_name);
 | 
			
		||||
    pa_proplist_setf(data.proplist, PA_PROP_DEVICE_BUFFERING_BUFFER_SIZE, "%lu", (unsigned long) (period_frames * frame_size * nfrags));
 | 
			
		||||
    pa_proplist_setf(data.proplist, PA_PROP_DEVICE_BUFFERING_FRAGMENT_SIZE, "%lu", (unsigned long) (period_frames * frame_size));
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -37,6 +37,7 @@
 | 
			
		|||
#include <pulsecore/macro.h>
 | 
			
		||||
#include <pulsecore/core-util.h>
 | 
			
		||||
#include <pulsecore/atomic.h>
 | 
			
		||||
#include <pulsecore/core-error.h>
 | 
			
		||||
 | 
			
		||||
#include "alsa-util.h"
 | 
			
		||||
 | 
			
		||||
| 
						 | 
				
			
			@ -1361,7 +1362,7 @@ void pa_alsa_init_proplist_card(pa_core *c, pa_proplist *p, int card) {
 | 
			
		|||
#endif
 | 
			
		||||
}
 | 
			
		||||
 | 
			
		||||
void pa_alsa_init_proplist_pcm(pa_core *c, pa_proplist *p, snd_pcm_info_t *pcm_info) {
 | 
			
		||||
void pa_alsa_init_proplist_pcm_info(pa_core *c, pa_proplist *p, snd_pcm_info_t *pcm_info) {
 | 
			
		||||
 | 
			
		||||
    static const char * const alsa_class_table[SND_PCM_CLASS_LAST+1] = {
 | 
			
		||||
        [SND_PCM_CLASS_GENERIC] = "generic",
 | 
			
		||||
| 
						 | 
				
			
			@ -1427,6 +1428,28 @@ void pa_alsa_init_proplist_pcm(pa_core *c, pa_proplist *p, snd_pcm_info_t *pcm_i
 | 
			
		|||
        pa_proplist_sets(p, PA_PROP_DEVICE_DESCRIPTION, n);
 | 
			
		||||
}
 | 
			
		||||
 | 
			
		||||
void pa_alsa_init_proplist_pcm(pa_core *c, pa_proplist *p, snd_pcm_t *pcm) {
 | 
			
		||||
    snd_pcm_hw_params_t *hwparams;
 | 
			
		||||
    snd_pcm_info_t *info;
 | 
			
		||||
    int bits, err;
 | 
			
		||||
 | 
			
		||||
    snd_pcm_hw_params_alloca(&hwparams);
 | 
			
		||||
    snd_pcm_info_alloca(&info);
 | 
			
		||||
 | 
			
		||||
    if ((err = snd_pcm_hw_params_current(pcm, hwparams)) < 0)
 | 
			
		||||
        pa_log_warn("Error fetching hardware parameter info: %s", snd_strerror(err));
 | 
			
		||||
    else {
 | 
			
		||||
 | 
			
		||||
        if ((bits = snd_pcm_hw_params_get_sbits(hwparams)) >= 0)
 | 
			
		||||
            pa_proplist_setf(p, "alsa.resolution_bits", "%i", bits);
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
    if ((err = snd_pcm_info(pcm, info)) < 0)
 | 
			
		||||
        pa_log_warn("Error fetching PCM info: %s", snd_strerror(err));
 | 
			
		||||
    else
 | 
			
		||||
        pa_alsa_init_proplist_pcm_info(c, p, info);
 | 
			
		||||
}
 | 
			
		||||
 | 
			
		||||
int pa_alsa_recover_from_poll(snd_pcm_t *pcm, int revents) {
 | 
			
		||||
    snd_pcm_state_t state;
 | 
			
		||||
    int err;
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -120,8 +120,9 @@ void pa_alsa_dump_status(snd_pcm_t *pcm);
 | 
			
		|||
void pa_alsa_redirect_errors_inc(void);
 | 
			
		||||
void pa_alsa_redirect_errors_dec(void);
 | 
			
		||||
 | 
			
		||||
void pa_alsa_init_proplist_pcm(pa_core *c, pa_proplist *p, snd_pcm_info_t *pcm_info);
 | 
			
		||||
void pa_alsa_init_proplist_pcm_info(pa_core *c, pa_proplist *p, snd_pcm_info_t *pcm_info);
 | 
			
		||||
void pa_alsa_init_proplist_card(pa_core *c, pa_proplist *p, int card);
 | 
			
		||||
void pa_alsa_init_proplist_pcm(pa_core *c, pa_proplist *p, snd_pcm_t *pcm);
 | 
			
		||||
 | 
			
		||||
int pa_alsa_recover_from_poll(snd_pcm_t *pcm, int revents);
 | 
			
		||||
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
	Add table
		Add a link
		
	
		Reference in a new issue